[실습] 연습문제 week1/02 제출합니다

[minsoo-web]

Describe your changes

사실 처음 문제를 만났을 때, 이게 뭐가 문제이지..? 라고 생각하긴 했습니다.
그래서 가장 먼저 한 생각이, reduce 쓰면 되지 않을까? 라는 생각이었고 for문을 reduce로 바꿔서 내장 메소드로 한 번 추상화를 했습니다.
그리고 reduce의 콜백함수를 선언하기 보다는 별도의 순수 함수를 따로 만들어서 추상화를 한 번 더 했습니다.

💬 질문 사항이에요

• sum 함수를 한 번 더 선언하는게 투머치인가?!, 순수함수를 따로 선언함으로써 계산을 분리한건데, accumulate 함수 자체도 계산이라, 계산에서 계산을 분리할 필요가 있을까 라는 고민이 들었습니다.
• 다른 분들은 이 함수를 어떻게 리팩토링 해보셨을지 너무 궁금합니다,,
• initialValue를 받을 수 있도록 개선해봤는데, 이 함수의 요구사항이 뭔진 잘 모르지만 배열의 합계를 계산하는 함수라면, initialValue가 필요할까? 라는 고민도 해봤습니다.
• 함수 이름에 배열 관련된 내용을 추가할까 고민했었는데,, 어떻게 생각하시는지 궁금합니다.

🤷‍♂️ 확인 받고 싶은 부분이에요

🔥 이건 꼭 확인해주세요

[tooooo1]

코드 너무 깔끔해서 따로 코멘트 드릴게... 음... LGTM!!!!!!!

질문 답변

sum 함수의 추가 선언에 대한 고민: sum 함수를 따로 선언하는 것은 가독성 고민일 것 같아요. 투머치라고 보기보다는 선택의 문제라고 생각합니다.

함수 이름에 대한 고민: accumulate도 좋은데요? 음... 배열의 합계만을 계산하는 경우 sumArray, totalArray와 같은 이름이 있을 것 같네요

[minsoo-web]

충일님 리뷰 감사합니다 !!

sum 함수의 추가 선언에 대한 고민: sum 함수를 따로 선언하는 것은 가독성 고민일 것 같아요. 투머치라고 보기보다는 선택의 문제라고 생각합니다.

이 부분이 어제 테오님께 문의 드린 방향이랑 비슷한 것 같아요,
처음부터 분리하는 것보다는 요구사항 변동이 왔을 때 유연하게 설계할 수 있는 기반만 만들어놓고 분리를 해야 할 때 분리를 하는 것이 맞는 것 같단 생각이 어제 정리가 된 것 같습니다.

함수 이름에 대한 고민: accumulate도 좋은데요? 음... 배열의 합계만을 계산하는 경우 sumArray, totalArray와 같은 이름이 있을 것 같네요

지금 생각해보니, accumulate 자체는 뭔가 암묵적 출력을 해줄 것만 같은 함수 이름인 것 같단 생각이 들어서 더 고민을 했던 것 같아요 ㅋㅋ큐ㅠ