[seungseung88] WEEK 02 solutions

[seungseung88]

답안 제출 문제

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

작성자 체크 리스트

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

검토자 체크 리스트

Important

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

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

[Copilot]

Copilot reviewed 1 out of 1 changed files in this pull request and generated no comments.

Comments suppressed due to low confidence (1)

valid-anagram/seungseung88.js:19

• [nitpick] Consider renaming the variable 'i' to a more descriptive name (e.g., 'freq') to better convey that it represents character frequency differences.

for (const i of Object.values(count)) {

[Copilot]

Copilot reviewed 1 out of 1 changed files in this pull request and generated no comments.

Comments suppressed due to low confidence (1)

valid-anagram/seungseung88.js:19

• [nitpick] Consider renaming the variable 'i' to a more descriptive name like 'countValue' or 'frequency' to improve code readability.

for (const i of Object.values(count)) {

[Copilot]

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

Comments suppressed due to low confidence (2)

valid-anagram/seungseung88.js:19

• [nitpick] Consider renaming the variable 'i' to a more descriptive name, such as 'frequency', to improve code clarity.

for (const i of Object.values(count)) {

climbing-stairs/seungseung88.js:8

• [nitpick] Consider renaming 'arr' to a more descriptive variable like 'dp' or 'steps' to clarify its purpose as a dynamic programming array.

const arr = [1, 2];

[grapefruitgreentealoe]

시간, 공간복잡도 효율을 생각하며 잘 작성해주신 거 같습니다 LGTM!

[grapefruitgreentealoe]

for in 연산으로 바꾸셔서 Object.values 메소드 배열 생성으로 인한 비용을 줄여서 작성하시면 더 효율적일 거 같습니다!