인프런 커뮤니티 질문&답변

류재준님 안녕하세요. 🙂
제보해 주신 부분에 else 문이 들어가야 하지만 else문이 생략되고 if 문만 사용된 것을 확인했습니다.

확인해보니 다행히 코드를 설명하는 동영상과 배포된 pdf에는 else 문이 정상적으로 사용되어 있어서 큰 이슈는 없지만 말씀해 주신 실습 동영상과 github에는 else문이 누락되어 있습니다.

실행 결과에는 영향을 주지 않지만 표현력과 일관성 측면에서 오해를 불러일으킬 수 있어 github의 코드를수정했습니다.

다만 동영상 자료는 세밀한 수정이 필요해서 일단 현재 상태를 유지하고 시간이 날 때 다시 업로드하도록 하겠습니다.

강의 자세히 들어주시고 세세한 부분까지 확인해 주셔서 정말 감사합니다!

안녕하세요, 인프런 AI 인턴입니다.

류재준님, 2-4-예제. 절차에서 객체로 강의에서 발견하신 코드 관련하여 제보해 주셔서 감사합니다. 제보해 주신 부분은 코드의 논리 흐름에서 혼란을 야기할 수 있는 곳이니, 확인 후 수정이 필요할 수도 있을 것 같습니다.

강의에서의 코드 문제:

해당 if-else 문의 구조에서 중복으로 if문이 사용되어 예상치 않은 조건 확인 로직이 구현되었을 수 있습니다. 제보해 주신 코드 문제를 DiscountCondition 클래스의 isSatisfiedBy() 메서드에서 확인하셨다고 하였는데, 이는 코드의 논리적 오류가 발생할 가능성이 있습니다. 강사님께서 이 부분을 고려하여 수정할 수 있도록 제보해주신 점에 감사드립니다.

이미 유사한 문제 논의

과거에 비슷한 코드 논리 오류가 다른 강의에서 논의된 사례가 있으니, 이를 참조하시어 이해를 돕고 강사님께 추가 문의하실 수 있습니다. 예를 들어, “ 객체지향 설계에서의 코드 리팩토링 사례” 에서 강사님께서 코드의 논리적 오류나 개선점을 다루었던 다른 사례를 확인해보실 수 있습니다.

해당 상황은 강사님께 전달되어 필요한 경우 코드 리포지토리가 업데이트되면, 수강하시는 모든 분들이 더욱더 원활하게 학습할 수 있도록 개선될 것입니다.

저는 질문자님의 이해를 돕고자 지식공유자님의 기존 답변을 참고하여 유사한 답변 링크를 추천해드리고 있습니다.
현재 베타 기능으로 답변이 만족스럽지 않을 수 있는 점 양해 부탁드립니다. 🙏
추가적으로 궁금한 점이 있으시면, 이어서 질문해 주세요. 곧 지식공유자께서 답변해 주실 것입니다.