[dohyeon2] WEEK 02 solutions#2396
Conversation
Add time and space complexity comments to climbStairs method
Refactor comments in 3Sum solution for clarity and consistency
first approach second approach children solution update comments 개행문자 삽입 Update comments in validate-binary-search-tree solution for clarity Enhance comment clarity in validate-binary-search-tree solution
|
@DaleStudy 리뷰해줘 |
|
좋은 작업입니다! 여러 문제를 깔끔하게 해결했고, 특히 시간/공간 복잡도에 대한 주석도 명확하게 달아두셨네요. 몇 가지 개선 제안을 드리자면:
전반적으로 매우 잘 구성되어 있고, 주석도 명확하며, 깔끔한 스타일입니다. 앞으로도 다양한 문제를 이렇게 체계적으로 해결한다면 더욱 실력이 향상될 거예요! 계속해서 좋은 코드 기대하겠습니다. |
jiji-hoon96
left a comment
jiji-hoon96
left a comment
There was a problem hiding this comment.
고생많으셨습니다!
알고리즘 해결 방법이 정석적이고, 시간복잡도 최적화를 위해 많이 고민하신 것 같네요
코드 구조도 깔끔하고 너무 좋습니다!
| } | ||
|
|
||
| return Arrays.stream(nums).map((n) -> { | ||
| return production / n; |
There was a problem hiding this comment.
division 제약 조건인걸로 알고있습니다!!
한번 확인해보시면 좋을 것 같아요
There was a problem hiding this comment.
...without using the division operation.
역시 사람 말은 끝까지 들어야하네요.
알려주셔서 감사합니다!
다시 올바른 방법으로 풀어볼게요
3sum/dohyeon2.java
Outdated
| if (sum < 0) { | ||
| left++; | ||
| } | ||
| if (sum > 0) { | ||
| right--; |
There was a problem hiding this comment.
sum == 0 도 처리하고있어보여요.
if/else 문으로 바꾸면 좋겠습니다~
There was a problem hiding this comment.
sum == 0 이후에도 계속해서 조건 체크가 일어난다는 말씀이시죠?
해당 부분 개선해보겠습니다~
… in 3Sum solution Refine space complexity comment in 3Sum solution for accuracy
Add missing newline at the end of the file in dohyeon2.java
3 participants
답안 제출 문제
#230 문제풀이
작성자 체크 리스트
In Review로 설정해주세요.검토자 체크 리스트
Important
본인 답안 제출 뿐만 아니라 다른 분 PR 하나 이상을 반드시 검토를 해주셔야 합니다!