Donggyu bugfix

[donggyu412]

• 분석하기 그래프 수정

• 일간 :현재 날짜로 부터 시작해서 5일 전 까지의 점수만 가지고 그래프 그림

• 주간 :현재 날짜로 부터 5주 전까지의 점수로 그래프를 그림

• 월간: 현재 날짜로 부터 6개월 전까지의 점수를 그래프를 그림

• 무료 식단 추천 버그 픽스

[github-actions]

Code review by ChatGPT

[github-actions]

코드 검토의 주요 사항

1. 데이터 로드 후 정렬: 데이터가 로드된 후에 유효한 데이터에 대한 날짜순으로 정렬하는 것은 좋지만, 정렬 방식이 바뀌면 정렬 기준이 명확하지 않아질 수 있습니다. 이 부분의 로직을 명확히 정의하거나 관련 주석을 더욱 구체화하는 것이 좋습니다.

2. 로컬 날짜 문자열 변환 헬퍼: getLocalDateString 함수가 지역 시간을 기준으로 날짜 형식을 변환하는데, 이를 사용하는 다른 날짜 데이터와의 차이가 생길 수 있는 점이 우려됩니다. 모든 날짜에서 같은 표준을 사용하도록 보장하는 것이 필요합니다.

3. 유효성 검사 반복: 여러 곳에서 isNaN과 같은 유효성 검사 로직이 반복되고 있습니다. 이 로직을 유틸리티 함수로 분리하면 코드의 일관성과 가독성이 높아질 것입니다.

4. 안전하지 않은 코드: trendData에서 인덱스를 사용하여 값을 가져올 때, 해당 인덱스에 데이터가 없을 경우 문제가 발생할 수 있습니다. trendData.length > 0의 경우를 모두 파악한 다음 마지막 아이템을 가져오는 방법을 명확하게 하는 것이 좋습니다.

5. 주간 데이터 처리의 정확성: 주간 데이터 처리 부분에서 기본적으로 주의 깊게 평균 또는 최신 점수를 만들어야 할 필요가 있습니다. 현재는 가장 최신의 데이터만 선택하는 방식입니다. 필요 시 평균을 내는 로직이 추가될 수 있습니다.

6. UI 개선 제안: 그래프가 데이터가 전혀 없는 경우를 더 잘 처리하기 위해 명확한 메시지를 추가하면 유용합니다. 현재는 0으로 설정되어 있음에도 그래프가 표시되는 부분이 사용자를 혼란스럽게 만들 수 있습니다.

7. 불필요한 코드: 사용하지 않거나 중복된 변수를 정리하여 불필요한 코드의 복잡성을 줄일 수 있습니다.

이러한 사항들은 코드의 안정성과 가독성을 개선하는 데 도움이 될 수 있습니다. 검토 후 주의 깊게 수정하면 좋을 것 같습니다.

[github-actions]

코드 변경 사항에 대한 리뷰입니다.

1. 상태 관리: selectedFoods 상태를 Set으로 구현한 것은 좋지만, 이를 사용할 때 불변성을 유지하는 방법에 대한 주의가 필요합니다. React의 useState는 값을 직접 수정하기보다는 새로운 객체를 반환하는 것이 좋습니다. 현재 형태는 적절하지만, 다른 부분에서는 불변성이 깨질 위험이 있습니다.

2. 에러 핸들링: userPreferencesAPI.addPreferences 호출에서 에러 발생 시에도 프로세스를 계속 진행한다고 명시했지만, 에러 발생 상황에 대한 사용자 피드백이 부족합니다. 사용자가 차후에 오류를 인지할 수 있을까요? 오류 발생 시 사용자에게 알림을 제공하거나 로그에 남기는 방법을 고려할 필요가 있습니다.

3. 비동기 코드: 두 개의 비동기 API 호출을 순차적으로 진행하고 있으나, 첫 번째 호출이 실패할 경우 두 번째 호출은 무시됩니다. 이는 비즈니스 로직에 따라 의도된 것일 수 있지만, 명확한 코멘트나 문서화가 필요합니다.

4. 스타일 관리: 새로운 스타일이 추가되었습니다. 이 스타일이 기존 UI와 일관성을 유지하고 있는지 확인해야 합니다. 특히 반응형 디자인에 영향을 줄 수 있습니다.

5. 의미 있는 키: meal.foods.map((food, fIdx) => {...}에서 fIdx를 통해 고유한 키를 생성하고 있습니다. 배열 내 데이터가 변할 경우, 이는 렌더링 최적화에 좋지 않을 수 있습니다. food.id와 같은 고유 식별자를 사용하는 것이 바람직합니다.

기타 정리 및 주석 사항을 통해 코드의 가독성을 높일 수 있습니다.

[github-actions]

코드 변경 사항을 살펴보았을 때 몇 가지 우려 사항이 있습니다. 1. 멤버십 전환과 관련된 'handleToggleMembership' 함수 내에서 authAPI.toggleMembership 호출 결과에 대한 error 처리가 부족해 보입니다. 이 호출이 실패할 경우 사용자에게 적절한 피드백을 제공하지 않으면 좋지 않을 수 있습니다. 2. 카카오 사용자 확인을 위한 'handleDeleteAccount' 함수 내에서 profileData의 null 확인이 필수적입니다. 현재 이 변수가 null일 경우 접근 시 오류가 발생할 수 있어, 이에 대한 방어 코드를 추가하는 것이 좋습니다. 3. 다른 네비게이션 관련 부분에서도 null 체크가 필요할 수 있습니다. 4. 'handleLogoutPress' 함수와 같은 몇몇 비동기 함수가 중첩될 때 사용자의 인터페이스에 성능 저하를 유발할 수 있습니다. 가능하다면 이부분에서도 비동기 방식으로 진행되는 것이 좋습니다. 5. 전반적으로 AsyncStorage와 API 호출 간의 에러 핸들링을 강력하게 하는 것이 필요합니다. 코드를 합치기 전에 이러한 점을 보완하는 것이 좋겠습니다.