Skip to content

[Flynn] Week 5 #447

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 5 commits into from
Sep 14, 2024
Merged

[Flynn] Week 5 #447

merged 5 commits into from
Sep 14, 2024

Conversation

obzva
Copy link
Contributor

@obzva obzva commented Sep 9, 2024

답안 제출 문제

체크 리스트

  • PR을 프로젝트에 추가하고 Week를 현재 주차로 설정해주세요.
  • 바로 앞에 PR을 열어주신 분을 코드 검토자로 지정해주세요.
  • 문제를 모두 푸시면 프로젝트에서 Status를 In Review로 설정해주세요.
  • 코드 검토자 1분 이상으로부터 승인을 받으셨다면 PR을 병합해주세요.

@github-actions github-actions bot added the py label Sep 9, 2024
@obzva obzva changed the title Solution: Best Time to Buy and Sell Stock [Flynn] Week 5 Sep 9, 2024
@obzva obzva requested a review from taekwon-dev September 9, 2024 14:43
@DaleSeo
Copy link
Member

DaleSeo commented Sep 9, 2024

@obzva 옷 😮 줄곧 C++을 쓰시다가 어떤 심경의 변화가 있으셨나요?

@obzva
Copy link
Contributor Author

obzva commented Sep 10, 2024

다른 언어로 적은 코드를 알아보기 어렵다는 후기가 있어서 파이썬으로 제출하려고 합니다 ㅎㅎ

@DaleSeo
Copy link
Member

DaleSeo commented Sep 10, 2024

@obzva 그렇다고 굳이 프로그래밍 언어까지 바꾸실 필요가 있을까요? 저는 다양성이 좋은데 ㅎㅎ 요즘 언어 하나 해서는 먹고 살기 힘든데 리뷰 하시는 분들께도 분명 도움이 될 거라 생각합니다! 아무튼 It's up to you 입니다 :) 그냥 궁금해서 여쭤봤습니다.

@obzva
Copy link
Contributor Author

obzva commented Sep 10, 2024

저는 두 개 다 편해서, 상관 없습니다 ㅎㅎㅎ
이번 주만 파이썬으로 진행할게요 그럼 ㅋㅋㅋㅋ

@obzva obzva marked this pull request as ready for review September 13, 2024 04:02
@obzva obzva requested a review from a team as a code owner September 13, 2024 04:02
@DaleSeo
Copy link
Member

DaleSeo commented Sep 13, 2024

  • 문제를 모두 푸시면 프로젝트에서 Status를 In Review로 설정해주세요.

Shot 2024-09-13 at 17 35 13@2x

@obzva 다 푸신 거 맞죠? 체크 리스트와 프로젝트의 상태가 맞지 않아서 여쭤봅니당~

@obzva
Copy link
Contributor Author

obzva commented Sep 14, 2024

제가 상태를 안 넘겼네요., 뭔가 호다닥 하다가 놓쳤나 봅니다 죄송합니다

@obzva obzva requested a review from DaleSeo September 14, 2024 06:54
@obzva
Copy link
Contributor Author

obzva commented Sep 14, 2024

@taekwon-dev 안녕하세요 리뷰 좀 부탁드립니다~

Copy link
Member

@DaleSeo DaleSeo left a comment

Choose a reason for hiding this comment

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

수고 많으셨습니다!

return False
parent = parent[c]

return "word" in parent and parent["word"] == word
Copy link
Member

@DaleSeo DaleSeo Sep 14, 2024

Choose a reason for hiding this comment

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

값이 크게 중요한가요? 그냥 키가 존재하는지만 체크해도 충분하지 않나 하는 생각이 들었습니다.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

달레님 말씀이 맞습니다 ㅎㅎ
그냥 여부만 체크해도 충분하겠네요

@obzva obzva merged commit b69d13e into DaleStudy:main Sep 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.

2 participants