Skip to content

Conversation

jiji-hoon96
Copy link
Contributor

@jiji-hoon96 jiji-hoon96 commented Mar 30, 2025

답안 제출 문제

작성자 체크 리스트

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

검토자 체크 리스트

Important

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

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

@jiji-hoon96 jiji-hoon96 moved this from Solving to In Review in 리트코드 스터디 4기 Apr 3, 2025
Copy link
Member

@wisdom08 wisdom08 left a comment

Choose a reason for hiding this comment

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

주석이 잘 작성되어 있다고 느껴집니다.
수고하셨습니다👍


function longestConsecutive(nums: number[]): number {
if (nums.length === 0) return 0;
const sortNum = Array.from(new Set(nums)).sort((a, b) => a - b);
Copy link
Member

Choose a reason for hiding this comment

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

sortedNum이 더 명확한 변수명일 것 같아요!


if (sortNum.length === 1) return 1;

const resultArray : number[] = []
Copy link
Member

Choose a reason for hiding this comment

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

최댓값만 필요하니까 resultArray 제거하고 maxCount 변수 하나로도 추적할 수 있지 않을까요?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

오! 피드백 감사합니다
개선해보았어요!

for(const [key] of sortObject){
if(k>0){
result.push(Number(key))
k--;
Copy link
Member

Choose a reason for hiding this comment

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

k를 직접 감소시키는 방법도 있지만 slice를 쓸 수도 있을 것 같아요!

Copy link
Contributor Author

Choose a reason for hiding this comment

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

sortObject.slice(0, k).map(([key]) => Number(key))

코드가 더 간결해졌군요!

@jiji-hoon96 jiji-hoon96 moved this from In Review to Completed in 리트코드 스터디 4기 Apr 6, 2025
@jiji-hoon96 jiji-hoon96 merged commit 3189d1f into DaleStudy:main Apr 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.

2 participants