Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

[23주차] 손지민 #315

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
jewan100 merged 6 commits into main from jimin
Feb 23, 2025
Merged

[23주차] 손지민 #315

jewan100 merged 6 commits into main from jimin
Feb 23, 2025

Conversation

Copy link
Contributor

@jmxx219 jmxx219 commented Feb 18, 2025

No description provided.

Comment on lines +84 to +87
Copy link
Contributor

Choose a reason for hiding this comment

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

저는 상어가 움직이는거랑 냄새 처리를 따로 처리해줬는데, 움직일 때 바로 해당 구역의 냄새 유지 시간을 K + 1로 초기화하면 나중에 이미 냄새나는 구역과 나누어서 생각하지 않아서 더 편하고, 효율적인 것 같네요!!

jewan100 and yeongleej reacted with thumbs up emoji
Copy link
Contributor

@yeongleej yeongleej Feb 20, 2025

Choose a reason for hiding this comment

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

오 완전 좋은 로직이네요!!

Copy link
Contributor

@jewan100 jewan100 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 +7 to +9
Copy link
Contributor

Choose a reason for hiding this comment

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

저도.. 25퍼에서 한번 틀렸서요...

Copy link
Contributor

Choose a reason for hiding this comment

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

저도....

jewan100 reacted with laugh emoji
Copy link
Contributor

Choose a reason for hiding this comment

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

오호.. 상어의 움직임을 다 처리하고 이동을 해주셨군요!! 전 처음에 이부분을 잘못처리했던거같은데, 두 부분을 함수로 분리해주니까 좋네요!!

@jewan100 jewan100 merged commit 6fae217 into main Feb 23, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@KodaHye KodaHye KodaHye approved these changes

@yeongleej yeongleej yeongleej left review comments

@baexxbin baexxbin baexxbin approved these changes

+1 more reviewer

@jewan100 jewan100 jewan100 approved these changes

Reviewers whose approvals may not affect merge requirements
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

AltStyle によって変換されたページ (->オリジナル) /