코딩 열나게 연습중인데 한번 봐주셨으면 합니다

식사하는철학자 2022-06-07 10:37:13 4 0
1440, 0.00...258처럼 코드에서 별도의 설명없이 하드코딩된 숫자들을 매직넘버라고 부르는데 보는 입장에서 의미를 모르면 가독성을 크게 떨어뜨리기 때문에 상수 선언이나 주석을 통해 의미를 명시해주는 게 좋습니다.
추천 답글 신고

추천완료
식사하는철학자 2022-06-07 10:42:29 1 0
두번째의 60, 24, 365도 일종의 매직넘버이고 그리 잘한 건 아니지만 문맥상 60, 24, 365의 의미를 짐작하는 것과 1440, 0.00000190258이 어디서 왔는지를 추측하는 건 난이도가 다르다고 생각합니다.
추천 신고

추천완료
식사하는철학자 2022-06-07 10:51:34 1 0
상기한 점이 제가 코드를 한 눈에 봤을 때 보이는 가장 큰 단점이고, 그 외에도 불명확한 변수명의 의미, 변수 선언 위치, 불필요해보이는 float과 double의 혼용(하지말라는 건 아닌데 코드에서는 굳이 구분할 이유가 없어 보임) 등이 보입니다.
추천 신고

추천완료
박창이 2022-06-07 10:57:46 1 0
아 되게 불친절한 코드였군요.. 변수명은 뭐로 바꾸는게 좋을까요? 날하고 년을 분으로 계산하는 코드짜는 과제였어서 days하고 year이 들어간거거든요 제딴애는 구분한다고 써놓은거라..ㅋㅋ
추천 신고

추천완료
식사하는철학자 2022-06-07 11:04:05 1 0
입력한 숫자에는 결국 분이 들어가니깐 days나 years보다는 minutes, 또는 입력값이라는 걸 명시하기위해 두번째처럼 minutesEntered가 적절해 보입니다. days나 years에는 그 입력값을 이용해서 날짜나 년으로 환산한 값이 들어가는 게 좋을 것 같아요.
추천 신고

추천완료
박창이 2022-06-08 09:28:46 2 0
그렇군요 감사합니다! 현역분의 분석을 들어보고싶었습니다 ㅋㅋ
추천 신고

추천완료
식사하는철학자 2022-06-08 15:10:58 2 0
넵 충분히 잘하고 계시니 계속 정진하시기를 응원하겠습니다! 입문할 때부터 자기 코드를 복기하고 개선하는 방법을 고민하는 건 아주 좋은 습관이라고 생각합니다.
추천 신고

추천완료
다민이아빠 2022-07-16 16:43:35 0 0
흐름이 없네요. 코드를 조각조각을 내야 수정할때 가독성, 짧은 변경점, 재사용성을 높일 것 같네요. 다른 개발자가 수정을 하게 될때 짜증이 안나게 만든다는 느낌으로 하면 좋을 것 같네요.
추천 답글 신고

추천완료

코딩 열나게 연습중인데 한번 봐주셨으면 합니다 [8]