Skip to content
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.

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

[실습] 연습문제 week1/03 제출합니다 #14

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

minsoo-web
Copy link
Member

Describe your changes

커밋내역을 보시면 아시겠지만 ㅋㅋ큐ㅠㅠ 문제를 완전 잘못 이해하고 풀어버려서
다시 풀었습니다.

다시 풀었을 때 첫 접근 방법은 2중 reduce문을 사용해서 변수명을 직관적으로 지으면서 가독성을 높혔었는데
그럼 기존의 2중 for 문이랑 크게 다르지 않다고 생각해서 n^2 시간 복잡도를 개선해봤습니다.
flatMap을 활용해서, 대각선 이하의 숫자들만 잘라내고 그 값들을 더했습니다.

💬 질문 사항이에요

  • 이게 함수형이랑 관계가 있는 리펙토링인가..?
  • 가독성은 오히려 낮아진 건 아닐까?

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

🔥 이건 꼭 확인해주세요

@minsoo-web minsoo-web self-assigned this Aug 2, 2023
@minsoo-web minsoo-web added the 실습 문제 제출 실습 문제가 제출되었을 때 사용되는 라벨입니다. label Aug 2, 2023
@minsoo-web minsoo-web requested a review from a team August 2, 2023 03:42
Comment on lines +12 to +13
expect(multiDimensionalAccumulate(multiDimensionalArr)).toBe(86)
expect(multiDimensionalAccumulate(multiDimensionalArr)).toBe(86)
Copy link
Member

Choose a reason for hiding this comment

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

혹시 왜 두줄로 하셨는지 궁금해요

Copy link
Member Author

Choose a reason for hiding this comment

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

순수함수니까 실행 횟수와 관계없이, 같은 input에서 같은 output이 나오는 것을 테스트하고자 했습니다!

Copy link
Member

Choose a reason for hiding this comment

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

아하! 그러면 함수를 두 번 호출하는 것이 2번의 기준으로 strict mode를 의미하는 것으로 추측하였는데 이유를 들을 수 있을까요? 테스트 횟수에 대한 기준이 있는 것인지 궁금해요. 테스트를 접한지 얼마되지 않아서 실수로 느껴졌었거든요.

Copy link
Member Author

@minsoo-web minsoo-web Aug 11, 2023

Choose a reason for hiding this comment

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

아 저도 저런 식으로 똑같이 적는 방법은 이번이 처음이어서 2번이냐 몇 번이냐는 크게 중요하진 않았던 것 같고,
multiDimensionalArr 을 암묵적 출력으로 변경시키지 않는다는 걸 보여주고 싶어서 일부러 변수로 선언하고 두 번 호출해봤던 것 같습니다.

지금 생각해보면 실행 후의 multiDimensionalArr 이 실행 전과 동일하다라는 걸 테스트 하는 게 좀 더 명확했겠네요

Copy link
Member

@tooooo1 tooooo1 left a comment

Choose a reason for hiding this comment

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

아 저도 저런 식으로 똑같이 적는 방법은 이번이 처음이어서 2번이냐 몇 번이냐는 크게 중요하진 않았던 것 같고, multiDimensionalArr 을 암묵적 출력으로 변경시키지 않는다는 걸 보여주고 싶어서 일부러 변수로 선언하고 두 번 호출해봤던 것 같습니다.

지금 생각해보면 실행 후의 multiDimensionalArr 이 실행 전과 동일하다라는 걸 테스트 하는 게 좀 더 명확했겠네요

그렇군요. 테스트 속에 테스트가 있었던 거였군용. 마지막 줄 와닿았습니다 ㅎㅎ :)
순수함수 체크할 때는 별도의 테스트로 확인해봐야겠어요

궁금한 점 해결해주셔서 감사해요!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
실습 문제 제출 실습 문제가 제출되었을 때 사용되는 라벨입니다.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants