Skip to content

Conversation

Invidam
Copy link
Contributor

@Invidam Invidam commented May 25, 2024

Caution

시간 투자를 많이 못했어서, 리뷰 (꼼꼼히) 안해주셔도 괜찮아요!

4주차 풀이 올립니다. 졸업작품 마감 기간이라 시간을 많이 못 쏟았어요. 꼼꼼히 신경 못 쓴 부분이 많을 것이고 리뷰해주시는 데 불편함이 있을까봐 리뷰 안해주셔도 저는 괜찮습니다. (이번 주차 의의는 바쁜 와중에도 루틴을 지키자 였어서요.)

요즘 신경 못 쓴 만큼 다음 주 부터는 더더욱 열심히 하겠습니다!

Copy link
Contributor

Choose a reason for hiding this comment

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

Go 는 약간 간단한 라이브러리가 없어서 어려운것 같아요 😱 근데 또 쓰시는 분들은 그맛 에 쓰시더라구요

어떻게 해야 bit으로 처리할 수 있을지 궁금했는데 덕분에 궁금증이 해소되었네요...!

Copy link
Contributor Author

Choose a reason for hiding this comment

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

이미 간단한 라이브러리가 없는 상황에서, 간단한 라이브러리를 추가한다는 게 생각보다 단순한 일이 아니라는 걸 배웠어요.

이미 대부분의 사람들이 간단한 라이브러리를 구현해서 사용하고 있는 와중에 추가된다면 어려움이 있겠다. 하위호환성 문제가 있겠다를 느꼈어요

Copy link
Contributor

Choose a reason for hiding this comment

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

어떻게 O(n)으로 이 문제를 해결할 수 있을지 궁금하긴한데
코드 양이 헉소리가 나오군요 ; )

시간날 때 구경해봐야겠습니다.

Copy link
Contributor Author

@Invidam Invidam May 26, 2024

Choose a reason for hiding this comment

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

헉.. 실수로 주석에다가 시간복잡도를 변경했어요
정렬비용이 있어 $O(n)$이 아닙니다..ㅠ

앞으론 꼼꼼히 보겠습니다! ㅋㅋㅋㅋㅋ

@DaleSeo
Copy link
Member

DaleSeo commented May 26, 2024

@Invidam 님, 다음 PR부터는 Iteration 표시해주시면 감사하겠습니다! (#71 참고)
이 번 PR에는 제가 대신 해드리겠습니다 :)

Comment on lines +34 to +36
# Today I Learned
- 다른 풀이로 풀 때, `abs()`를 이용해서 해결하려고 하였는데 해당 라이브러리가 존재하지 않았다. 왜 그런지 찾아봤는데 너무 간단한 라이브러리는 철학에 안맞는다고 안넣어준다 하더라.
- 참고: https://go.dev/doc/faq#x_in_std No newline at end of file
Copy link
Member

Choose a reason for hiding this comment

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

Go의 철학이 돋보이는 대목이네요.
그말은 즉슨, min(_:_:)이나 max(_:_:)와 같은 메서드도 존재하지 않겠군요..?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

다행히? 작년(1.21) 부터는 생겼네요!

https://tip.golang.org/doc/go1.21

@SamTheKorean SamTheKorean merged commit 8d0f2c5 into DaleStudy:main May 26, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
No open projects
Status: Done
Development

Successfully merging this pull request may close these issues.

5 participants