Skip to content

에러 고침 #125

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
minseo23232 merged 1 commit into from
Dec 14, 2025
Merged

에러 고침 #125

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
108 changes: 69 additions & 39 deletions src/screens/exercise/ExerciseScreen.tsx
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -477,13 +477,12 @@ const ExerciseScreen = ({ navigation }: any) => {
const [monthlyProgress, setMonthlyProgress] = useState<
DailyProgressWeekItem[]
>([]);
<<<<<<< HEAD
const [todayProgress, setTodayProgress] =
useState<DailyProgressWeekItem | null>(null);
=======
// 달력에 표시할 칼로리 데이터 (날짜별)
const [calendarCalories, setCalendarCalories] = useState<Record<string, number>>({});
>>>>>>> main
const [calendarCalories, setCalendarCalories] = useState<
Record<string, number>
>({});
const [showMonthView, setShowMonthView] = useState(false);
const [isModalOpen, setIsModalOpen] = useState(false);
const [isIntroVisible, setIsIntroVisible] = useState(false);
Expand Down Expand Up @@ -2206,34 +2205,55 @@ const ExerciseScreen = ({ navigation }: any) => {
// 달력에 표시할 날짜들의 칼로리 데이터 로드
const loadCalendarCalories = async (dates: string[]) => {
try {
console.log('📅 [운동 화면] 달력 칼로리 데이터 로드 시작:', dates.length, '일');

console.log(
"📅 [운동 화면] 달력 칼로리 데이터 로드 시작:",
dates.length,
"일"
);

// 각 날짜에 대해 영양성분 요약 조회 (병렬 처리)
const nutritionPromises = dates.map(async (date, index) => {
try {
console.log(`📡 [운동 화면] ${index + 1}/${dates.length} - ${date} 영양성분 조회 중...`);
console.log(
`📡 [운동 화면] ${index + 1}/${
dates.length
} - ${date} 영양성분 조회 중...`
);
const summary = await mealAPI.getNutritionSummary(date);
const calories = summary.calories || 0;
console.log(`✅ [운동 화면] ${index + 1}/${dates.length} - ${date} 칼로리: ${calories}kcal`);
console.log(
`✅ [운동 화면] ${index + 1}/${
dates.length
} - ${date} 칼로리: ${calories}kcal`
);
return { date, calories };
} catch (error) {
console.error(`❌ [운동 화면] ${index + 1}/${dates.length} - ${date} 영양성분 조회 실패:`, error);
console.error(
`❌ [운동 화면] ${index + 1}/${
dates.length
} - ${date} 영양성분 조회 실패:`,
error
);
return { date, calories: 0 };
}
});

const nutritionResults = await Promise.all(nutritionPromises);
console.log('📅 [운동 화면] 달력 칼로리 데이터 조회 완료:', nutritionResults.length, '일');
console.log(
"📅 [운동 화면] 달력 칼로리 데이터 조회 완료:",
nutritionResults.length,
"일"
);

// 상태 업데이트
const caloriesMap: Record<string, number> = {};
nutritionResults.forEach(({ date, calories }) => {
caloriesMap[date] = calories;
});
setCalendarCalories(prev => ({ ...prev, ...caloriesMap }));

setCalendarCalories((prev) => ({ ...prev, ...caloriesMap }));
} catch (error) {
console.error('❌ [운동 화면] 달력 칼로리 데이터 로드 실패:', error);
console.error("❌ [운동 화면] 달력 칼로리 데이터 로드 실패:", error);
}
};

Expand All @@ -2253,13 +2273,15 @@ const ExerciseScreen = ({ navigation }: any) => {
React.useEffect(() => {
if (showMonthView) {
loadMonthlyProgress(monthBase.getFullYear(), monthBase.getMonth());

// 해당 월의 모든 날짜에 대해 칼로리 데이터 로드
const year = monthBase.getFullYear();
const month = monthBase.getMonth();
const firstOfMonth = new Date(year, month, 1);
const nextMonth = new Date(year, month + 1, 1);
const daysInMonth = Math.round((nextMonth.getTime() - firstOfMonth.getTime()) / (1000 * 60 * 60 * 24));
const daysInMonth = Math.round(
(nextMonth.getTime() - firstOfMonth.getTime()) / (1000 * 60 * 60 * 24)
);
const monthDates = Array.from({ length: daysInMonth }).map((_, i) => {
const d = new Date(year, month, i + 1);
return formatDateToString(d);
Expand All @@ -2274,13 +2296,15 @@ const ExerciseScreen = ({ navigation }: any) => {
if (showMonthView) {
// 달력을 펼칠 때 monthBase의 달 데이터 가져오기
loadMonthlyProgress(monthBase.getFullYear(), monthBase.getMonth());

// 해당 월의 모든 날짜에 대해 칼로리 데이터 로드
const year = monthBase.getFullYear();
const month = monthBase.getMonth();
const firstOfMonth = new Date(year, month, 1);
const nextMonth = new Date(year, month + 1, 1);
const daysInMonth = Math.round((nextMonth.getTime() - firstOfMonth.getTime()) / (1000 * 60 * 60 * 24));
const daysInMonth = Math.round(
(nextMonth.getTime() - firstOfMonth.getTime()) / (1000 * 60 * 60 * 24)
);
const monthDates = Array.from({ length: daysInMonth }).map((_, i) => {
const d = new Date(year, month, i + 1);
return formatDateToString(d);
Expand All @@ -2289,7 +2313,7 @@ const ExerciseScreen = ({ navigation }: any) => {
} else {
// 달력을 접을 때 선택된 날짜의 달 데이터 가져오기 (주간 달력 표시 시)
loadMonthlyProgress(dateToFetch.getFullYear(), dateToFetch.getMonth());

// 이번 주의 날짜 범위 계산 (일~토)
const getStartOfWeek = (d: Date) => {
const n = new Date(d.getFullYear(), d.getMonth(), d.getDate());
Expand All @@ -2314,7 +2338,7 @@ const ExerciseScreen = ({ navigation }: any) => {
React.useEffect(() => {
const dateToFetch = selectedDate || new Date();
loadMonthlyProgress(dateToFetch.getFullYear(), dateToFetch.getMonth());

// 주간 달력인 경우 이번 주 7일의 칼로리 데이터 로드
if (!showMonthView) {
const getStartOfWeek = (d: Date) => {
Expand Down Expand Up @@ -2342,37 +2366,31 @@ const ExerciseScreen = ({ navigation }: any) => {
useFocusEffect(
React.useCallback(() => {
if (!userIdLoaded) return;

// StatsScreen에서 날짜 처리를 하므로 여기서는 날짜를 변경하지 않음
// 단지 현재 선택된 날짜를 사용하여 데이터 로드
const dateToFetch = selectedDate || new Date();
setMonthBase(new Date(dateToFetch.getFullYear(), dateToFetch.getMonth(), 1));

setMonthBase(
new Date(dateToFetch.getFullYear(), dateToFetch.getMonth(), 1)
);

loadGoalData();
loadWeeklyCalories();
loadTodayProgress(); // 오늘 진행률 로드 (게이지 업데이트)
// 해당 달의 월별 데이터 가져오기
<<<<<<< HEAD
loadMonthlyProgress(today.getFullYear(), today.getMonth());
}, [
userIdLoaded,
loadGoalData,
loadWeeklyCalories,
loadTodayProgress,
setSelectedDate,
])
=======
loadMonthlyProgress(dateToFetch.getFullYear(), dateToFetch.getMonth());

// 달력 칼로리 데이터 새로고침
const dateToUse = selectedDate || today;
const dateToUse = selectedDate || new Date();
if (showMonthView) {
// 월간 달력인 경우 해당 월의 모든 날짜
const year = dateToUse.getFullYear();
const month = dateToUse.getMonth();
const firstOfMonth = new Date(year, month, 1);
const nextMonth = new Date(year, month + 1, 1);
const daysInMonth = Math.round((nextMonth.getTime() - firstOfMonth.getTime()) / (1000 * 60 * 60 * 24));
const daysInMonth = Math.round(
(nextMonth.getTime() - firstOfMonth.getTime()) / (1000 * 60 * 60 * 24)
);
const monthDates = Array.from({ length: daysInMonth }).map((_, i) => {
const d = new Date(year, month, i + 1);
return formatDateToString(d);
Expand All @@ -2397,8 +2415,14 @@ const ExerciseScreen = ({ navigation }: any) => {
});
loadCalendarCalories(weekDates);
}
}, [userIdLoaded, loadGoalData, loadWeeklyCalories, showMonthView, selectedDate])
>>>>>>> main
}, [
userIdLoaded,
loadGoalData,
loadWeeklyCalories,
loadTodayProgress,
showMonthView,
selectedDate,
])
);

// 완료 횟수 저장 helper
Expand Down Expand Up @@ -3708,7 +3732,10 @@ const ExerciseScreen = ({ navigation }: any) => {
const dayProgress = getDayProgress(d);
const dateStr = formatDateToString(d);
// 달력 칼로리 데이터 우선 사용, 없으면 진행률 데이터 사용
const calories = calendarCalories[dateStr] ?? dayProgress?.totalCalorie ?? 0;
const calories =
calendarCalories[dateStr] ??
dayProgress?.totalCalorie ??
0;
const rate = dayProgress?.exerciseRate || 0;
return (
<>
Expand Down Expand Up @@ -3799,7 +3826,10 @@ const ExerciseScreen = ({ navigation }: any) => {
const dayProgress = getDayProgress(d);
const dateStr = formatDateToString(d);
// 달력 칼로리 데이터 우선 사용, 없으면 진행률 데이터 사용
const calories = calendarCalories[dateStr] ?? dayProgress?.totalCalorie ?? 0;
const calories =
calendarCalories[dateStr] ??
dayProgress?.totalCalorie ??
0;
const rate = dayProgress?.exerciseRate || 0;
return (
<>
Copy link

@github-actions github-actions bot Dec 14, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

잠재적 버그 및 위험 요소

  1. 병렬 처리에서의 오류 처리: loadCalendarCalories 함수 내에서 각 날짜에 대한 API 호출 오류가 발생할 경우, 이미 반환된 결과가 누락될 수 있습니다. 만약 API 응답이 성공햇지만 calories 필드가 자주 없으면, 최악의 경우 해당 날짜의 칼로리가 항상 0으로 설정됩니다. 이는 실제 데이터 손실로 이어질 수 있습니다.

  2. 메모리 누수 가능성: useEffect에 의존성 배열이 loadTodayProgress를 포함하고 있습니다. 이 함수가 비동기적으로 작동하는 경우, 컴포넌트가 언마운트된 후에도 상태 업데이트를 시도할 수 있습니다. 이는 메모리 누수를 초래할 수 있습니다.

  3. 상태 업데이트 및 렌더링: setCalendarCalories 호출에서 prev를 통해 상태를 업데이트하고 있는데, 이 방법은 적절하긴 하지만, 상태 변경이 자주 일어날 경우 렌더링 성능에 영향을 줄 수 있습니다.

개선 제안

  1. 오류 로깅 개선: 각 날짜의 API 호출 성공 여부를 더 효과적으로 체크하고, 발생한 모든 오류를 로그로 기록하여 디버깅을 용이하게 해야 합니다.

  2. 불필요한 렌더링 최소화: setCalendarCalories와 같은 상태 업데이트 호출이 자주 발생하면, 이로 인해 불필요한 렌더링이 발생할 수 있습니다. 그러므로 배치 업데이트를 고려해보세요.

  3. 최적화된 API 호출: 여러 날짜에 대한 API 호출을 병렬 처리하고 있는데, API 호출의 수가 많아지면 최악의 경우 성능 저하가 발생할 수 있습니다. 이를 위해 효율적으로 배치할 수 있는 방법을 고려하는 것이 좋습니다.

  4. 의존성 배열 명확화: useEffect의 의존성 배열에 setSelectedDate를 포함하는 것을 다시 고려하세요. 이 구조는 의도하지 않은 업데이트를 초래할 수 있습니다.

Expand Down