Skip to content

Conversation

Jay-Mo-99
Copy link
Contributor

@Jay-Mo-99 Jay-Mo-99 commented Jan 14, 2025

답안 제출 문제

체크 리스트

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

@Jay-Mo-99 Jay-Mo-99 requested a review from a team as a code owner January 14, 2025 00:34
@github-actions github-actions bot added the py label Jan 14, 2025
@Jay-Mo-99 Jay-Mo-99 requested a review from Chaedie January 14, 2025 00:35
Comment on lines +46 to +49
if (len(temp) == 0) :
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.

안녕하세요, @Jay-Mo-99 님,
개인 선호 차이인것 같습니다만, 아래와 같이 마무리해도 좋을것 같습니다. 😀

Suggested change
if (len(temp) == 0) :
return True
else:
return False
return len(temp) == 0

또는

Suggested change
if (len(temp) == 0) :
return True
else:
return False
return not temp

Copy link
Contributor Author

Choose a reason for hiding this comment

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

return에 바로 조건문을 입력하면 코드 수를 줄일수 있군요. if~else 구조 대신에 자주 사용해보도록 해봐야 겠어요. 피드백 감사합니다.

@Jay-Mo-99 Jay-Mo-99 closed this Jan 17, 2025
@Jay-Mo-99 Jay-Mo-99 reopened this Jan 17, 2025
Comment on lines +14 to +19
#Time Complexity: O(N)
#- while : s와 e가 만날때까지 최대 N번 반복된다. 각 반복에서의 연산들은 O(1)에 해당된다. -> O(N)


#Space Complexity: O(1)
#- s,e,max_area: 변수는 상수로 작용된다 -> O(1)
Copy link
Contributor

Choose a reason for hiding this comment

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

👍 이번 한주도 수고하셨습니다..!

@SamTheKorean SamTheKorean merged commit d82761b into DaleStudy:main Jan 19, 2025
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.

3 participants