Skip to content

[Helena] Week 11 solutions #181

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

Merged
merged 1 commit into from
Jul 17, 2024
Merged

[Helena] Week 11 solutions #181

merged 1 commit into from
Jul 17, 2024

Conversation

yolophg
Copy link
Contributor

@yolophg yolophg commented Jul 13, 2024

  • Palindromic Substrings
  • Decode Ways
  • Coin Change
  • Maximum Product Subarray
  • Word Break

Copy link
Contributor

@SamTheKorean SamTheKorean left a comment

Choose a reason for hiding this comment

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

고생하셨습니다!

@SamTheKorean
Copy link
Contributor

이건 일반적인 질문인데, 보통 보기에 분명한 코드에도 주석을 달아놓는게 유리할까요?

@DaleSeo
Copy link
Member

DaleSeo commented Jul 16, 2024

이건 일반적인 질문인데, 보통 보기에 분명한 코드에도 주석을 달아놓는게 유리할까요?

@SamTheKorean 좋은 질문이라서 제가 의견 드립니다 :)
코드가 주석없이 이해가 잘 되는 상황에서 주석을 추가하면 유지보수에 득보다는 해가 됩니다. 유지보수하면서 코드가 바뀔 때 마다 주석도 그와 일치하도록 맞춰줘야 하는데 개발자들이 참 놓치기 어려운 부분이기 때문입니다. 혼선만 일으키고 도움이 안 되는 소위, 죽은 주석이 되기 쉽습니다.

@SamTheKorean
Copy link
Contributor

이건 일반적인 질문인데, 보통 보기에 분명한 코드에도 주석을 달아놓는게 유리할까요?

@SamTheKorean 좋은 질문이라서 제가 의견 드립니다 :)

코드가 주석없이 이해가 잘 되는 상황에서 주석을 추가하면 유지보수에 득보다는 해가 됩니다. 유지보수하면서 코드가 바뀔 때 마다 주석도 그와 일치하도록 맞춰줘야 하는데 개발자들이 참 놓치기 어려운 부분이기 때문입니다. 혼선만 일으키고 도움이 안 되는 소위, 죽은 주석이 되기 쉽습니다.

넵 설명감사합니다! 전에 제 다른 코드에 달레님이 관련해서 피드백 주신 적이 있어서 언급했습니다! 그래도 코딩테스트에서는 혹시나 유리한 점이 될 수 있을까 여쭤봤습니다.

@DaleSeo
Copy link
Member

DaleSeo commented Jul 17, 2024

그래도 코딩테스트에서는 혹시나 유리한 점이 될 수 있을까 여쭤봤습니다.

코딩테스트에서는 입으로 잘 터는 게 더 유리하지 않을까요? ㅋㅋ

@DaleSeo DaleSeo merged commit 381a61a into DaleStudy:main Jul 17, 2024
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
No open projects
Status: Done
Development

Successfully merging this pull request may close these issues.

3 participants