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.

Sign up for GitHub

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

Jump to bottom

[Wan] Week 5 #859

Merged
merged 4 commits into from
Jan 12, 2025
Merged

[Wan] Week 5 #859

merged 4 commits into from
Jan 12, 2025

Conversation

taewanseoul
Copy link
Contributor

@taewanseoul taewanseoul commented Jan 6, 2025 •
edited
Loading

답안 제출 문제

체크 리스트

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

@taewanseoul taewanseoul requested a review from a team as a code owner January 6, 2025 13:18
@github-actions github-actions bot added the ts label Jan 6, 2025
@taewanseoul taewanseoul marked this pull request as draft January 6, 2025 13:19
@taewanseoul taewanseoul marked this pull request as ready for review January 6, 2025 13:31
@taewanseoul taewanseoul requested a review from Jeehay28 January 6, 2025 13:33
Jeehay28
Jeehay28 reviewed Jan 6, 2025
View reviewed changes
best-time-to-buy-and-sell-stock/taewanseoul.ts Outdated
*/

// O(n) time
// O(n) space
Copy link
Contributor

Choose a reason for hiding this comment

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

사용하는 변수는 highestProfit, left, right 3개로, 입력값의 크기와 관계없이 메모리 사용량이 일정하므로
공간 복잡도는 O(1)이 되지 않을까요?

Copy link
Contributor Author

@taewanseoul taewanseoul Jan 9, 2025 •
edited
Loading

Choose a reason for hiding this comment

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

아 그렇네요. 확인 감사합니다~

@taewanseoul taewanseoul changed the title [Wan] WEEK 05 [Wan] Week 5 Jan 7, 2025
@Jeehay28 Jeehay28 self-requested a review January 12, 2025 10:44
@TonyKim9401 TonyKim9401 merged commit 2c43c5b into DaleStudy:main Jan 12, 2025
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Jeehay28 Jeehay28 Jeehay28 approved these changes

Assignees

@taewanseoul taewanseoul

Labels
ts
Projects
리트코드 스터디 3기
Status: Completed
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@taewanseoul @Jeehay28 @TonyKim9401