Skip to content

[정수론] 9월 23일#5

Open
aqswa wants to merge 1 commit intomainfrom
정수론-및-조합론

Hidden character warning

The head ref may contain hidden characters: "\uc815\uc218\ub860-\ubc0f-\uc870\ud569\ub860"
Open

[정수론] 9월 23일#5
aqswa wants to merge 1 commit intomainfrom
정수론-및-조합론

Conversation

@aqswa
Copy link
Collaborator

@aqswa aqswa commented Sep 23, 2021

No description provided.

Copy link
Member

@HyunJiLim0406 HyunJiLim0406 left a comment

Choose a reason for hiding this comment

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

p3
수고하셨습니다! 4문제 다 잘 푸셨네요~
근데 소소하게 리팩토링 할 부분들이 보여요!
조금만 수정하고 리뷰어로 저 호출해주세요

Comment on lines 10 to 19
Copy link
Member

Choose a reason for hiding this comment

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

이 문제는 간단해서 이렇게 풀어도 괜찮지만, 사실 이건 하드코딩에 가깝습니다!
이항계수 공식이나 파스칼의 삼각형을 이용해서 코드를 정리하는게 좋아요

Comment on lines 28 to 29
Copy link
Member

Choose a reason for hiding this comment

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

이 2개는 합칠 수 있겠네요!

Comment on lines 17 to 22
Copy link
Member

Choose a reason for hiding this comment

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

이 두 조건도 같은 일을 하고 있으니 합칠 수 있구요!

Comment on lines 10 to 11
Copy link
Member

Choose a reason for hiding this comment

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

숫자를 바로 쓰는 것보단 상수로 선언해서 쓰는게 좋아요!

Comment on lines 12 to 19
Copy link
Member

Choose a reason for hiding this comment

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

이 부분은 에라토스테네스의 체 연산 부분이니 함수로 만들어주세요

Comment on lines 27 to 28
Copy link
Member

Choose a reason for hiding this comment

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

n/2까지만 돌아도 된다는걸 캐치하셨네요! 3부터 시작해도 된다는 것두요!
근데 연산을 줄일 수 있는 방법이 하나 더 있어요

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants