Skip to content

운동 로직 수정 #126

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 2 commits into from
Dec 14, 2025
Merged

운동 로직 수정 #126

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
daeb5e4
인바디 정보 바 중복 해결
minseo23232 Dec 14, 2025
bb97814
운동 진행 수정
minseo23232 Dec 14, 2025
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
39 changes: 35 additions & 4 deletions src/components/modals/ExerciseModal.tsx
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -599,6 +599,8 @@ const ExerciseModal: React.FC<ExerciseModalProps> = ({
const hasPrevSequence = hasSequenceControls && sequenceIndex! > 0;
const hasNextSequence =
hasSequenceControls && sequenceIndex! < sequenceLength - 1;
const isLastSequence = hasSequenceControls && sequenceIndex === sequenceLength - 1;
const canFinishWorkout = isLastSequence && allSetsCompleted;

const isExerciseSelected = (exercise: any) => {
const key = getExerciseKey(exercise);
Expand Down Expand Up @@ -2292,10 +2294,21 @@ const getExerciseDisplayName = React.useCallback(
</View>
)}
<TouchableOpacity
style={styles.endWorkoutBtn}
style={[
styles.endWorkoutBtn,
!canFinishWorkout && styles.endWorkoutBtnDisabled,
]}
onPress={handleSave}
disabled={!canFinishWorkout}
>
<Text style={styles.endWorkoutBtnText}>운동 끝내기</Text>
<Text
style={[
styles.endWorkoutBtnText,
!canFinishWorkout && styles.endWorkoutBtnTextDisabled,
]}
>
운동 끝내기
</Text>
</TouchableOpacity>
</View>
</View>
Expand Down Expand Up @@ -2796,10 +2809,21 @@ const getExerciseDisplayName = React.useCallback(

<View style={styles.footer}>
<TouchableOpacity
style={styles.endWorkoutBtn}
style={[
styles.endWorkoutBtn,
!canFinishWorkout && styles.endWorkoutBtnDisabled,
]}
onPress={handleSave}
disabled={!canFinishWorkout}
>
<Text style={styles.endWorkoutBtnText}>운동 끝내기</Text>
<Text
style={[
styles.endWorkoutBtnText,
!canFinishWorkout && styles.endWorkoutBtnTextDisabled,
]}
>
운동 끝내기
</Text>
</TouchableOpacity>
</View>
</View>
Expand Down Expand Up @@ -3368,11 +3392,18 @@ const styles = StyleSheet.create({
borderRadius: 12,
alignItems: "center",
},
endWorkoutBtnDisabled: {
backgroundColor: "#2a2a2a",
opacity: 0.5,
},
endWorkoutBtnText: {
color: "#ffffff",
fontSize: 16,
fontWeight: "600",
},
endWorkoutBtnTextDisabled: {
color: "#888888",
},
instructionScroll: {
maxHeight: 260,
},
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. sequenceIndex! 사용: sequenceIndex!를 사용하고 있는데, 이는 sequenceIndex가 undefined일 수 있는 상황을 무시하는 것입니다. 이로 인해 런타임 에러가 발생할 가능성이 있습니다. ! 연산자는 타입스크립트에서 null 또는 undefined임을 명시적으로 방지하려 하지만, 실제 값이 존재하지 않을 경우 문제가 발생할 수 있습니다. 따라서, 이 부분에서 더 안전한 null 체크를 적용하는 것을 고려해야 합니다.

  2. 의존성 경고: canFinishWorkout의 값이 allSetsCompleted와 같은 다른 상태들에 의존하고 있습니다. 만약 이러한 상태들이 변경될 경우, ExerciseModal가 올바르게 업데이트되지 않을 수 있습니다. 이 부분에 대한 의존성을 명확히 할 필요가 있습니다.

  3. 성능 문제: style 배열에서 canFinishWorkout 값에 따라 동적으로 스타일을 변경하고 있습니다. 다수의 요소에서 동일한 방식으로 스타일을 설정할 경우 렌더링 성능에 영향을 줄 수 있습니다. 스타일을 미리 계산해두고 조건부로 적용하는 방법을 고려해볼 수 있습니다.

개선 제안

  • sequenceIndex에 대한 보다 강력한 타입 검사를 추가하세요. 예를 들어, TypeScript로 검사할 때, 타입을 정의하여 undefined인 경우에 대해 명확한 핸들링을 추가하는 것이 필요합니다.
  • 버튼 및 텍스트 비활성 상태에 대한 시각적 피드백을 더 강하게 주는 것도 고려해보세요. 예를 들어, 비활성 상태일 때 색상 외에도 버튼에 테두리를 추가할 수 있습니다.
  • 코드의 유지보수를 용이하게 하기 위해, isLastSequence 및 canFinishWorkout 연산 로직을 별도의 함수로 추출하여 가독성을 높이는 것도 좋습니다.

Expand Down
4 changes: 2 additions & 2 deletions src/screens/exercise/ExerciseScreen.tsx
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -605,12 +605,12 @@ const ExerciseScreen = ({ navigation }: any) => {
typeof goalData.weeklyCalorieGoal === "number" &&
goalData.weeklyCalorieGoal > 0
) {
parts.push(`${goalData.weeklyCalorieGoal}kcal`);
parts.push(`${goalData.weeklyCalorieGoal.toLocaleString()}kcal`);
}
if (parts.length === 0) {
return "목표치가 아직 설정되지 않았습니다";
}
return `목표치 | ${parts.join(" · ")}`;
return `목표치 | ${parts.join(", ")}`;
}, [goalData]);

const trimmedCompletionTitle = completionSummaryTitle.trim();
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. 세미콜론 관련 이슈: 이 코드 블록에 세미콜론이 일관성 있게 사용되지 않았습니다. JS에서 세미콜론은 선택적이지만, 일관된 스타일을 유지하는 것이 좋습니다.

  2. toLocaleString 사용: toLocaleString()을 사용하여 숫자를 문자열로 변환하는 것은 가능하지만, 이 함수는 로케일에 따라 다르게 동작할 수 있습니다. 코드가 의도한 대로 동작하는지 확인해야 합니다. 만약 영어권을 기준으로 한다면 특별한 옵션을 지정해줘야의 적절한 포맷을 보장할 수 있습니다. 예를 들어, goalData.weeklyCalorieGoal.toLocaleString('en-US')와 같이 사용하면 더 안전합니다.

  3. 문구 변경: 원래의 구분자 ·에서 ,로 변경한 것이 정말 의도한 것인지 확인해야 합니다. 두 구분자가 의미가 다를 수 있기 때문입니다. 의미가 바뀌는 것에 대한 설명이 필요합니다.

따라서 이 코드를 머지하기 전에 자세한 테스트를 진행해보고, 의도한 형식과 구분자가 맞는지 확인하는 것이 좋습니다.

Expand Down
12 changes: 1 addition & 11 deletions src/screens/inbody/InBodyScreen.tsx
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -22,7 +22,6 @@ import {
} from "../../utils/inbodyApi";
import AsyncStorage from "@react-native-async-storage/async-storage";
import { eventBus } from "../../utils/eventBus";
import InbodyDateNavigator from "../../components/common/InbodyDateNavigator";
import InBodyCalendarModal from "../../components/common/InBodyCalendarModal";
import { ROUTES } from "../../constants/routes";
import { ACCESS_TOKEN_KEY } from "../../services/apiConfig";
Expand Down Expand Up @@ -1212,7 +1211,7 @@ const InBodyScreen = ({ navigation, route }: any) => {
</View>
)}

{/* 날짜 네비게이터 */}
{/* 날짜 선택 바 (캘린더 버튼만) */}
{inBodyData && selectedDate && (
<View style={styles.dateNavigatorContainer}>
<TouchableOpacity
Expand All @@ -1227,15 +1226,6 @@ const InBodyScreen = ({ navigation, route }: any) => {
</Text>
<Icon name="chevron-down" size={16} color="#E3FF7C" />
</TouchableOpacity>
{availableDates.length > 0 && (
<View style={styles.dateNavigatorWrapper}>
<InbodyDateNavigator
dates={availableDates}
onChange={handleDateNavigatorChange}
selectedDate={selectedDate}
/>
</View>
)}
</View>
)}

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. 버그 가능성: InbodyDateNavigator 컴포넌트가 삭제되었습니다. 이를 대체할 만한 기능이 다른 곳에서 처리되지 않는 경우, 애플리케이션의 날짜 선택 기능이 정상적으로 작동하지 않을 수 있습니다. 사용자가 날짜를 선택할 수 있는 다른 방법이 제공되는지 확인하세요.

  2. 코드 가독성: 주석을 수정한 것은 좋지만, 전체 기능의 문맥을 고려하여 주석이 실제 기능과 잘 맞는지 확인하는 것이 중요합니다. 주석은 사용자가 코드를 이해하는 데 도움을 줘야 하지만, 삭제된 코드의 기능을 대체하는 것이 없다면 혼란을 초래할 수 있습니다.

  3. 로직 검토 필요: availableDates가 비어있는 경우도 고려해야 합니다. 현재 코드는 이를 처리하지 않으며, 이에 대한 검증이 필요한 경우 여전히 유효한지 확인해야 합니다.

  4. 테스트 부족 가능성: 삭제된 요소가 누락된 부분에 대한 충분한 테스트가 이루어지지 않았을 수 있습니다. 코드 변경 후에는 해당 기능의 정상 작동을 테스트하기 위한 단위 테스트 또는 통합 테스트를 작성하는 것이 좋습니다.

  5. UI/UX 측면: 캘린더 버튼만 사용자가 날짜를 선택하는 데 적합한지 다시 한번 검토해야 합니다. 사용자가 직관적으로 날짜 선택을 할 수 있는지 확인하는 것이 중요합니다.

Expand Down
12 changes: 12 additions & 0 deletions src/screens/main/HomeScreen.tsx
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -649,6 +649,18 @@ const HomeScreen = ({ navigation }: any) => {
};
}, []);

// 인바디 업데이트 이벤트 리스너
useEffect(() => {
const unsubscribe = eventBus.on("inbodyUpdated", () => {
console.log("[HOME] 인바디 업데이트 이벤트 수신, 인바디 데이터 새로고침");
loadInBodyData();
});

return () => {
unsubscribe?.();
};
}, []);

const handleCalendarClick = () => {
navigation.navigate("Calendar");
};
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. 메모리 누수 위험: unsubscribe 함수가 정의된 후에만 호출되는 것처럼 보이지만, eventBus.on이 항상 unsubscribe를 반환하는지 확인해야 합니다. 그렇지 않으면 메모리 누수의 위험이 있을 수 있습니다.

  2. 이벤트 처리 제한: inbodyUpdated 이벤트가 발생할 때마다 loadInBodyData를 호출하는 것이 성능에 악영향을 줄 수 있습니다. 데이터 로딩에 대한 제한 조건을 두거나, 중복 호출을 방지하는 로직이 필요할 수 있습니다.

  3. 타입 안전성: eventBus와 loadInBodyData의 타입이 명시되어 있지 않습니다. 타입스크립트를 사용하는 경우, 매개변수와 반환 값의 타입을 명시하여 코드의 안정성을 높이는 것이 좋습니다.

  4. 의존성 배열: useEffect의 의존성 배열이 비어 있습니다. 이를 통해 부작용이 특정 상태 변화에 따라 실행되도록 할 수 있습니다. 현재의 경우 적절한 의존성을 추가하여 필요한 경우에만 이펙트를 재실행하도록 해야 합니다.

전반적으로 이 코드는 주의가 필요하며, 위의 사항들을 고려하여 개선할 필요가 있습니다.

Expand Down