[YuuuuuuYu] WEEK 01 solutions

[YuuuuuuYu]

답안 제출 문제

• Contains Duplicate #217
• Two Sum #219
• Top K Frequent Elements #237
• Longest Consecutive Sequence #240
• House Robber #264

작성자 체크 리스트

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

검토자 체크 리스트

Important

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

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

[doh6077]

안녕하세요!
map.get(pair) != i 조건은 이 구현에서는 항상 참이기 때문에 생해도 될 것 같아요.
for 문이 인덱스를 0부터 순서대로 돌고, map.put(nums[i], i)가 if 문 뒤에서 호출되기 때문에, if가 실행될 때 map에는 항상 현재 인덱스 i보다 작은 인덱스들만 들어 있습니다. map.get(pair)가 i와 같아질 수 없어서, 해당 조건은 생략하고 if (map.containsKey(pair))만 사용해서 코드를 단순화할 수 있을 것 같아요!

[YuuuuuuYu]

안녕하세요! map.get(pair) != i 조건은 이 구현에서는 항상 참이기 때문에 생해도 될 것 같아요. for 문이 인덱스를 0부터 순서대로 돌고, map.put(nums[i], i)가 if 문 뒤에서 호출되기 때문에, if가 실행될 때 map에는 항상 현재 인덱스 i보다 작은 인덱스들만 들어 있습니다. map.get(pair)가 i와 같아질 수 없어서, 해당 조건은 생략하고 if (map.containsKey(pair))만 사용해서 코드를 단순화할 수 있을 것 같아요!

그렇네요! 저 부분을 많이 봤었는데 containsKey가 앞에 선언되면서
의미 없어진 조건인데 확인을 못했네요
피드백 감사합니다!