Skip to content

[김한기] Week 5 Solutions #106

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 3 commits into from
Jun 4, 2024
Merged

[김한기] Week 5 Solutions #106

merged 3 commits into from
Jun 4, 2024

Conversation

meoooh
Copy link
Contributor

@meoooh meoooh commented May 31, 2024

No description provided.

@meoooh meoooh marked this pull request as draft May 31, 2024 02:04
@meoooh meoooh changed the title [김한기] Week 5 Solutions - draft [김한기] Week 5 Solutions May 31, 2024
@DaleSeo
Copy link
Member

DaleSeo commented May 31, 2024

Iteration 설정 부탁드립니다!

@meoooh
Copy link
Contributor Author

meoooh commented Jun 1, 2024

아이고 3문제 밖에 못풀었네요...

@meoooh meoooh marked this pull request as ready for review June 1, 2024 16:05
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.

아이고 3문제 밖에 못풀었네요...

하나도 안 푸신 것 보다는 낫죠! 😂

Copy link
Member

Choose a reason for hiding this comment

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

이 문제는 Elixir로 해결이 어렵나 보죠?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

�아 그건아닌데 어쩌다보니 파이썬으로 풀게되었습니다 ㅎㅎ 근데 다른분들 코드를 보니 제가 특별히 복잡하게 푼것같네요 ㅎㅎ

Comment on lines +5 to +8
Enum.frequencies |>
Enum.sort_by(fn {_, v} -> v end, :desc) |>
Enum.take(k) |>
Enum.map(fn {k, _} -> k end)
Copy link
Member

Choose a reason for hiding this comment

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

Elixir스러운 직관적인 FP 코드네요! 이 문제는 Elixir가 아니더라도 다른 언어를 쓰시는 분들도 이렇게 많이 짜시더라고요 :)

@leokim0922
Copy link
Contributor

한기님 남은 문제 해당 PR에서 마저 처리하실 예정이신가요 아님 다음 PR로 넘겨서 푸실 예정이신가요 :)?

@meoooh
Copy link
Contributor Author

meoooh commented Jun 4, 2024 via email

@DaleSeo DaleSeo merged commit 3c65a62 into DaleStudy:main Jun 4, 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.

3 participants