[Yu-Won] Week 2 solutions

[Yu-Won]

답안 제출 문제

• Valid Anagram #218
• Climbing Stairs #230
• Product of Array Except Self #239
• 3Sum #241
• Validate Binary Search Tree #251

작성자 체크 리스트

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

검토자 체크 리스트

Important

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

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

[SamTheKorean]

@DaleStudy 리뷰해줘

[dalestudy]

전체적으로 깔끔하고 체계적인 문제 해결 방식이 돋보입니다. 특히, 각 문제에 대한 설명과 요구사항이 명확하게 주어졌으며, 함수 구현도 직관적입니다. 다만, 몇 가지 개선점을 제안드리고 싶습니다.

1. 시간/공간 복잡도 명시 요청: 각 함수에 대한 TC, SC를 주석으로 추가하는 것을 권장합니다. 예를 들어, threeSum은 TC: O(n^2), SC: O(n)으로 표기하면 더 좋겠습니다.

2. 복잡도 평가: 현재 구현된 알고리즘은 대부분 적절하며, threeSum의 경우 정렬 후 이중 포인터로 중복 제거와 탐색을 수행하여 효율적입니다. climbingStairs는 DP 방식으로 최적이며, productOfArrayExceptSelf는 왼쪽 곱셈과 오른쪽 곱셈을 이용한 표준 솔루션입니다. 이들 모두 적절한 접근법입니다.

3. 가독성 및 스타일: 변수 네이밍이 직관적이고, 코드 전개도 명확합니다. 다만, 일부 변수명(left, right, current)에 대해 간단한 주석을 덧붙이면 더 좋겠습니다.

4. 추가 제안:

   • isValidBST 함수에서 root 파라미터가 선언되지 않았습니다. 함수 선언에서 nums 대신 root로 변경하거나, 매개변수 이름 일치시켜 주세요.
   • validAnagram 함수에서 count 객체를 사용하는 방식은 효율적이며 좋습니다. 하지만, 문자열이 크거나 문자 집합이 넓을 경우를 대비해 Map 사용도 고려할 수 있습니다.

격려와 함께, 이미 훌륭하게 구현하셨으니 다음 단계로 복잡도 표기와 일부 변수명/매개변수 명확화에 신경 써보시면 더 깔끔한 코드가 될 것 같습니다. 계속 좋은 작업 기대합니다!

[SamTheKorean]

5문제 모두 푸시느라 고생많으셨습니다!