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

[mallayon] Week 6 #899

Merged
merged 6 commits into from
Jan 17, 2025
Merged

[mallayon] Week 6 #899

merged 6 commits into from
Jan 17, 2025

Conversation

mmyeon
Copy link
Contributor

@mmyeon mmyeon commented Jan 14, 2025

답안 제출 문제

체크 리스트

  • 우측 메뉴에서 PR을 Projects에 추가해주세요.
  • Projects의 오른쪽 버튼(▼)을 눌러 확장한 뒤, Week를 현재 주차로 설정해주세요.
  • 바로 앞에 PR을 열어주신 분을 코드 검토자로 지정해주세요.
  • 문제를 모두 푸시면 프로젝트에서 StatusIn Review로 설정해주세요.
  • 코드 검토자 1분 이상으로부터 승인을 받으셨다면 PR을 병합해주세요.

@mmyeon mmyeon self-assigned this Jan 14, 2025
@mmyeon mmyeon requested a review from a team as a code owner January 14, 2025 05:26
@github-actions github-actions bot added the ts label Jan 14, 2025
@mmyeon mmyeon requested a review from Jay-Mo-99 January 14, 2025 05:26
Copy link
Contributor

@HC-kang HC-kang left a comment

Choose a reason for hiding this comment

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

@mmyeon 님~ 항상 좋은 풀이 남겨주셔서 감사합니다!!
덕분에 리뷰하면서 정말 많이 배우고 있어요.
6주차도 고생 많으셨고, 15주차까지 열심히 응원하겠습니다~

@@ -0,0 +1,33 @@
/**
*@link https://leetcode.com/problems/container-with-most-water/description/
Copy link
Contributor

Choose a reason for hiding this comment

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

감사합니다 👍 리뷰할 때 문제 보기 너무 좋아요!

*
* 공간복잡도 : O(k)
* - pairs 객체 고정된 크기로 저장 O(1)
* - stack에 열린 괄호 개수만큼 담기니까 O(k)
Copy link
Contributor

Choose a reason for hiding this comment

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

전체 길이 n 중, 열린 괄호의 수를 k(<n)로 표현하신거죠?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

넵 맞습니다 !
k에 대한 설명이 빠졌네요! 주석 좀 더 자세하게 변경했습니다 ! 감사합니다 😄

* - m x n 행렬의 모든 숫자 방문하니까 O(m x n)
*
* 공간복잡도 : O(n)
* - 숫자 길이만큼 배열에 담음 *
Copy link
Contributor

Choose a reason for hiding this comment

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

숫자 길이가 혹시 행렬의 모든 숫자 말씀이실까요?
그렇다면 시간복잡도와 같이 O(m x n)이 되지 않을까 싶어서요!

Copy link
Contributor Author

Choose a reason for hiding this comment

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

네 맞습니다!
생각해보니 숫자 길이 n만큼만 반복하니까 O(n)으로 작성하는게 더 이해하기 좋을 것 같아요!
복잡도 모두 O(n)으로 수정했습니다~
꼼꼼히 봐주셔서 감사합니다 !!

};

while (top <= bottom && left <= right) {
moveRight();
Copy link
Contributor

Choose a reason for hiding this comment

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

정말 깔끔하네요 👍

@mmyeon
Copy link
Contributor Author

mmyeon commented Jan 17, 2025

@mmyeon 님~ 항상 좋은 풀이 남겨주셔서 감사합니다!! 덕분에 리뷰하면서 정말 많이 배우고 있어요. 6주차도 고생 많으셨고, 15주차까지 열심히 응원하겠습니다~

@HC-kang 님,
복잡도 관련 질문 해주셔서 한번 더 고민해볼 수 있어서 좋았습니다.
매주 꼼꼼한 리뷰와 응원해주셔서 정말 감사합니다 😃
남은 기간도 잘 부탁드립니다 !!

@mmyeon mmyeon merged commit e7df407 into DaleStudy:main Jan 17, 2025
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
Status: Completed
Development

Successfully merging this pull request may close these issues.

2 participants