Skip to content

[Jay-Mo-99] Week 1 #686

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

Merged
merged 15 commits into from
Dec 14, 2024
Merged

[Jay-Mo-99] Week 1 #686

merged 15 commits into from
Dec 14, 2024

Conversation

Jay-Mo-99
Copy link
Contributor

@Jay-Mo-99 Jay-Mo-99 commented Dec 12, 2024

답안 제출 문제

체크 리스트

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

@Jay-Mo-99 Jay-Mo-99 added the py label Dec 12, 2024
@Jay-Mo-99 Jay-Mo-99 requested a review from a team as a code owner December 12, 2024 06:21
Copy link
Contributor

@TonyKim9401 TonyKim9401 left a comment

Choose a reason for hiding this comment

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

깔끔하고 가독성 좋은 풀이 방법에 감탄합니다 :)

Comment on lines 21 to 24
if len(list(set(nums))) == len(nums):
return False
else:
return True
Copy link
Contributor

Choose a reason for hiding this comment

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

if 문을 활용해서 true/false를 구분한다면 if문 조건식을 그대로 return을 하면 어떨까요?

Copy link
Contributor

Choose a reason for hiding this comment

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

else 를 안쓰고 return True 해도 괜찮을 것 같네요!

Comment on lines 28 to 31
if temp == temp[::-1]:
return True
else:
return False
Copy link
Contributor

Choose a reason for hiding this comment

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

위에 문제와 마찬가지로 조건식을 그대로 return하면 어떨까 합니다!

@TonyKim9401
Copy link
Contributor

안녕하세요 @Jay-Mo-99 님!
혹시 계속해서 워크플로우가 실패하신다면 아래의 링크를 확인해 주세요!
#664 (comment)

@Jay-Mo-99
Copy link
Contributor Author

Jay-Mo-99 commented Dec 13, 2024

안녕하세요,@TonyKim9401 님. 해당 방법으로 rebase문제를 해결 했습니다. 감사합니다. 덧붙여 PR에 대해서 추가 질문이 있습니다. 1. 만약 리뷰가 달렸으면 이제는 문제를 풀고 commit을 추가하지 못하나요? 2. 만약 그렇다면 PR은 최대한 모든 문제를 풀고 그 다음 pull request를 하는게 지향되나요? 아니면 문제를 하나씩 풀고 그때마다 파일 및 커밋 추가 후 push가 더 지향되나요? 3. 한국시간으로 12월 14일까지는 문제를 계속 풀어볼 수 있나요?

@Jay-Mo-99 Jay-Mo-99 removed their assignment Dec 13, 2024
@Jay-Mo-99 Jay-Mo-99 requested a review from jungsiroo December 13, 2024 06:57
@Jay-Mo-99 Jay-Mo-99 assigned Jay-Mo-99 and unassigned jungsiroo Dec 13, 2024
@obzva
Copy link
Contributor

obzva commented Dec 13, 2024

@Jay-Mo-99 안녕하세요 코치 Flynn입니다
대신 답변 드리자면

  1. 아닙니다 계속 커밋 달아주셔도 됩니다. (새로운 문제 풀이 추가, 리뷰 코멘트를 반영한 수정사항 등등)
  2. 아뇨 오히려 반대로 첫 문제 풀자마자 PR을 열어주시는게 더 좋습니다. 리뷰를 일찍 시작할 수 있으니까요.
  3. 네 14일까지 문제를 풀어보실 수 있지만, 그럼 다른 분들이 리뷰해드리기 촉박하겠죠? (+ Jay Mo 님께서도 다른 분 PR 리뷰를 해주셔야하는데 시간이 부족하실 겁니다) 그래서 목요일까지 목표하는 바까지의 풀이를 완료하길 권장 드립니다 ㅎㅎ

@Jay-Mo-99
Copy link
Contributor Author

답변 감사합니다. 참고해서 이번주 과제들 완료하도록 하겠습니다.

Copy link
Contributor

@jungsiroo jungsiroo left a comment

Choose a reason for hiding this comment

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

5문제 푸시느라 고생하셨습니다~!


#removes all non alpha numeric(only accept alphabet and number) items from the s
temp = lower(s)
temp = " ".join(re.split("[^a-zA-Z0-9]*",temp)).replace(" ","")
Copy link
Contributor

Choose a reason for hiding this comment

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

temp = ''.join 을 한다면 뒤에 replace를 안해도 될 것 같습니다!

→ temp = "".join(re.split("[^a-zA-Z0-9]*",temp))

@Jay-Mo-99 Jay-Mo-99 merged commit b937d2c into DaleStudy:main Dec 14, 2024
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
No open projects
Status: Completed
Development

Successfully merging this pull request may close these issues.

4 participants