Skip to content

Conversation

delight010
Copy link
Contributor

@delight010 delight010 commented Sep 3, 2025

답안 제출 문제

작성자 체크 리스트

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

검토자 체크 리스트

Important

본인 답안 제출 뿐만 아니라 다른 분 PR 하나 이상을 반드시 검토를 해주셔야 합니다!

  • 바로 이전에 올라온 PR에 본인을 코드 리뷰어로 추가해주세요.
  • 본인이 검토해야하는 PR의 답안 코드에 피드백을 주세요.
  • 토요일 전까지 PR을 병합할 수 있도록 승인해주세요.

@@ -0,0 +1,61 @@
class Solution {
Copy link
Contributor

Choose a reason for hiding this comment

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

저와는 다른 접근법으로 푸셔서 많이 배워갑니다..!!! 공간 복잡도나 시간 복잡도를 올려주셔도 좋을 것 같아요!

Copy link
Contributor Author

Choose a reason for hiding this comment

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

추가하였습니다!!

@delight010 delight010 moved this from Solving to In Review in 리트코드 스터디 5기 Sep 5, 2025
@@ -0,0 +1,26 @@
class Solution {
// Time complexity O(N)
// Space complexity O(min(m,n))
Copy link
Contributor

Choose a reason for hiding this comment

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

let charArray = Array(s)를 하고 있기 때문에 공간 복잡도에서 무조건 String의 길이는 사용이 될거라 O(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.

다시 보니 그러네요!! 아직 복잡도 계산하는 데 익숙치않아서 클로드의 도움을 받았는 데, 한 번 더 유심히 봐야겠어요. 감사합니다

Copy link
Contributor

@TonyKim9401 TonyKim9401 left a comment

Choose a reason for hiding this comment

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

7주차 문제 풀이 고생하셨습니다!
코드 리뷰도 진행 잘 되어있고 너무 감사합니다!

@delight010 delight010 moved this from In Review to Completed in 리트코드 스터디 5기 Sep 6, 2025
@delight010 delight010 merged commit 672aeba into DaleStudy:main Sep 6, 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.

4 participants