Skip to content

인바디 에러 표시 수정 #140

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 15, 2025
Merged

인바디 에러 표시 수정 #140

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
15 changes: 14 additions & 1 deletion src/screens/inbody/InBodyScreen.tsx
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -732,7 +732,7 @@ const InBodyScreen = ({ navigation, route }: any) => {
0
);

// 숫자 타입 검증 및 유효성 확인 (데이터가 없어도 API 호출 시도)
// 숫자 타입 검증 및 유효성 확인
const finalWeight =
weight !== undefined && !isNaN(weight) && isFinite(weight) ? weight : 0;
const finalBodyFatPercentage =
Expand All @@ -749,6 +749,19 @@ const InBodyScreen = ({ navigation, route }: any) => {
: 0;
const finalDateStr = dateStr || todayStr;

// ✅ 인바디 입력값이 전혀 없으면 API 호출하지 않고 코멘트도 비움
if (
!inBodyData ||
(!finalWeight && !finalBodyFatPercentage && !finalSkeletalMuscleMass)
) {
console.log(
"[INBODY][COMMENT] 인바디 입력값이 없어 코멘트 API 호출 생략"
);
setAiComment("");
setAiCommentLoading(false);
return;
}

console.log(
"[INBODY][COMMENT] API 호출 데이터 (인바디 데이터 없을 수 있음):",
{
Copy link

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

Choose a reason for hiding this comment

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

코드 패치에 대해 몇 가지 우려 사항이 있습니다:

  1. 데이터 검증 로직: finalWeight, finalBodyFatPercentage, finalSkeletalMuscleMass의 유효성을 검사하는 것에 대한 조건은 적절하지만, finalSkeletalMuscleMass의 값을 확인하는 코드가 없으니 여기에 대한 확인이 필요합니다. 사용자가 해당 값의 тип видуけ, API 호출 시 오류를 방지할 수 있습니다.

  2. 부적합한 조건 처리: finalWeight, finalBodyFatPercentage, finalSkeletalMuscleMass는 모두 0으로 초기화될 수 있습니다. 따라서 이 코드에서 이는 유효 응답이 아닐 경우 0인지 확인하는 조건을 다시 생각해 볼 필요가 있습니다.

  3. 비어있는 범위: inBodyData가 널이거나, finalWeight, finalBodyFatPercentage, finalSkeletalMuscleMass가 모두 0인 경우에만 API 호출을 하지 않도록 하고 하지만 하드코딩된 값인 0에 대한 설명이 필요할 수 있습니다. 잠재적으로 다른 상황에서 이들 값이 0인 것이 유효한 경우인지에 대한 논의가 필요합니다.

  4. 로깅 메시지: console.log는 디버깅을 위한 좋은 도구지만, 프로덕션 환경에서는 로그 레벨을 조정하거나 더 정교한 로깅 시스템을 사용하는 것을 고려할 수 있습니다. 이는 개발, 테스트 및 운영 환경을 구분해야 할 필요가 있을 수 있습니다.

  5. 타입 안전성: route, navigation 등의 타입을 any로 지정하는 것은 프로그램에서 타입 안전성을 떨어뜨릴 수 있습니다. 명확한 인터페이스를 정의하여 작업을 더 안전하게 만들 수 있습니다.

이러한 점들을 고려할 때, 현재 상태에서는 코드가 문제가 발생할 가능성이 있으므로 머지하기 전에 수정하는 것이 좋습니다.

Expand Down