Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

P174. 풀이 2 반환형 #32

Closed
WonJongWoon opened this issue Sep 12, 2020 · 3 comments
Closed

P174. 풀이 2 반환형 #32

WonJongWoon opened this issue Sep 12, 2020 · 3 comments

Comments

@WonJongWoon
Copy link

WonJongWoon commented Sep 12, 2020

P173의 "두 수의 합" 문제의 리트코드에서 제시하는 함수의 반환형은 List[int] 입니다.

하지만 해당 문제의 풀이2, 풀이3, 풀이 5들은 다음과 같이 Tuple[int] 를 반환하고 있습니다.

# 풀이 2
return nums.index(n), nums[i+1:].index(complement) + (i+1)
# 풀이 3
return nums.index(num), nums_map[target - num]
# 풀이 5
return left, right

풀이 1, 풀이4의 경우는 리트코드에서 제시하는 바와 같이 List[int] 을 반환하고 있습니다.

# 풀이1
return [i, j]
# 풀이 4
return [nums_map[target - num], i]

이전에 올라왔던 이슈 중 그룹 애너그램 문제과 동일한 이슈인 것 같습니다.

이 문제 말고도 일부 문제에서 List를 요구하는데 Tuple을 반환하는 경우가 빈번한 것 같습니다.

큰 문제는 아니지만, 리트코드가 제시하는 타입에 맞추는 것이 책의 전체 일관성이나,

제출에도 좋은 것 같아서 이슈를 작성하여 봅니다.

@likejazz
Copy link
Collaborator

네, 좋은 지적입니다.
모두 타입 힌트에서 지정한대로 List[int] 형태로 변경하는게 맞겠습니다.
코드와 책의 본문 모두 정오표에 반영하도록 하겠습니다.
감사합니다.

@daeungdaeung
Copy link

good comments, good guys

@likejazz likejazz closed this as completed Oct 7, 2020
@ubless607
Copy link

ubless607 commented Jan 11, 2022

@likejazz 174번 return 값을 [i, nums.index(complement)]로 작성하면 더 간결한 답이 될 수 있다고 생각합니다. 혹시 제가 놓친 부분이 있을까요?

edit) 생각을 해보니 리스트 안에 같은 숫자가 있을 경우에 대한 예외를 처리하지 못할 것 같습니다. [i, lst[i + 1 :].index(complement) + (i + 1)]으로 수정합니다.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants