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 문제 풀이 #317

Merged
merged 7 commits into from
Aug 18, 2024
Merged

Conversation

taekwon-dev
Copy link
Contributor

@taekwon-dev taekwon-dev commented Aug 12, 2024

@yolophg
Copy link
Contributor

yolophg commented Aug 12, 2024

안녕하세요, 코치 Helena 입니다.
아래 사진과 같이 Projects 에서 iteration 설정도 같이 부탁드립니다!
이번엔 제가 해두었는데, 원할한 스터디 그룹 진도 관리와 통계를 위해서 중요한 부분이니 다음 이터레이션부터는 해주시면 감사드리겠습니다 :)
Screenshot 2024-08-12 at 11 41 08

* - 데이터 순서 보장 필요 없음
*/
public boolean containsDuplicate(int[] nums) {
HashSet<Integer> set = new HashSet<>();
Copy link
Contributor

Choose a reason for hiding this comment

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

알고리즘과 관련없는 이야기인데

변수 선언을

Set<Integer> set = new HashSet<>();

으로 하는게 자바 컨벤션상 더 좋지 않을까 생각이 들었습니다.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@dev-jonghoonpark 생각해보니까 interface 로 선언해두는 것이 나중에 다른 구현체로 갈아끼울 때도 더 유연하게 대응할 수 있을 것 같아서 더 좋을 것 같네요 ㅎㅎ 코멘트 감사합니다!

@taekwon-dev taekwon-dev marked this pull request as ready for review August 17, 2024 01:20
@taekwon-dev
Copy link
Contributor Author

taekwon-dev commented Aug 17, 2024

Note: 1주차 간단 후기

  • 확실히 생각보다 일과 병행하기 쉽지 않다.
  • Ruby, Elixir 사용하다가 Java 오랜만에 쓰니까 괄호와 ; 를 생략한다 😂
  • 예전에 공부했던 내용들이 반갑기도 하지만 낮설다.
  • 시,공간 복잡도를 고민하는 과정이 의미가 있는 것 같다.
  • BST 문제는 순회 방식 관련해서 도움을 받아서 풀었다. (이번 기회에 순회 방식 다시 한 번 내 머리속에 리마인드)
  • Palindromic Substrings는 고민만 하다가 결국 못 풀었다. (생각해볼 케이스가 점점 늘어나서, 이걸 도대체 어떻게 풀어야 하는거지..? )

Comment on lines +22 to +28
List<Map.Entry<Integer, Integer>> list = new ArrayList<>(map.entrySet());
list.sort(new Comparator<Map.Entry<Integer, Integer>>() {
@Override
public int compare(Map.Entry<Integer, Integer> e1, Map.Entry<Integer, Integer> e2) {
return e2.getValue().compareTo(e1.getValue());
}
});
Copy link
Contributor

Choose a reason for hiding this comment

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

익명 클래스를 람다식으로 바꿔 보시는것도 좋을것 같아요 :)

@TonyKim9401
Copy link
Contributor

TonyKim9401 commented Aug 17, 2024

Note: 1주차 간단 후기

  • 확실히 생각보다 일과 병행하기 쉽지 않다.
  • Ruby, Elixir 사용하다가 Java 오랜만에 쓰니까 괄호와 ; 를 생략한다 😂
  • 예전에 공부했던 내용들이 반갑기도 하지만 낮설다.
  • 시,공간 복잡도를 고민하는 과정이 의미가 있는 것 같다.
  • BST 문제는 순회 방식 관련해서 도움을 받아서 풀었다. (이번 기회에 순회 방식 다시 한 번 내 머리속에 리마인드)
  • Palindromic Substrings는 고민만 하다가 결국 못 풀었다. (생각해볼 케이스가 점점 늘어나서, 이걸 도대체 어떻게 풀어야 하는거지..? )

마지막 문제였던 Palindromic Substring은 문자열을 각각 잘라서 앞,뒤를 비교하는 식으로 하면 어떨까요!
https://github.com/DaleStudy/leetcode-study/pull/299/files
가독성과 효율은 좋지 않을 수 있지만 제 풀이 공유드립니다!

Copy link
Contributor

@DaleSeo DaleSeo left a comment

Choose a reason for hiding this comment

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

수고 많으셨습니다. 4문제만 푸셔도 충분히 잘 하신 거에요. 모임 전까지 PR 병합 부탁드리겠습니다.


return -1;
}
}
Copy link
Contributor

Choose a reason for hiding this comment

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

마지막 라인 브레이크가 누락된 파일이 디폴트 브랜치로 병합되지 않도록 각별히 주의 바랍니다. (참고: #334)

@taekwon-dev taekwon-dev merged commit 0748714 into DaleStudy:main Aug 18, 2024
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
No open projects
Status: Completed
Development

Successfully merging this pull request may close these issues.

6 participants