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.

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

[정수론] 9월 6일 #3

Open
wants to merge 4 commits into
base: main
Choose a base branch
from
Open

[정수론] 9월 6일 #3

wants to merge 4 commits into from

Conversation

godori1012
Copy link
Collaborator

학번: 2071093
이름: 고은서

과제 제출
기존 제출: 1735, 6588
추가 제출:

@kimhj010502
Copy link

kimhj010502 commented Sep 9, 2023

은서님 안녕하세요! 정수론 구현 문제 2840번 코드가 제출되지 않았네요 확인 부탁드립니다~!

+) Pull Request에 2주차 코드도 포함되어있네요 과제 제출 방법을 다시 한 번 확인해주세요!

Copy link

@kwakrhkr59 kwakrhkr59 left a comment

Choose a reason for hiding this comment

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

[정수론 알고리즘 코드 리뷰 완료]
1735(P2), 6588(P3) 안녕하세요 은서님~ 정수론 필수 알고리즘 문제 코드리뷰 완료되셨습니다! 다만 제출해주신 코드를 확인해보니 시간초과의 위험이 매우 커 보이네요...! 6588번의 경우는 시간초과로 인해 백준에서 맞았습니다가 뜨지 않고 있습니다😥 코드를 제출해주셔도 백준에서 통과가 되지 않을 경우 출석으로 인정이 안되니 꼭 확인하시고 수정해서 올려주시면 감사하겠습니다!

gcd = i;
break;
}
}

Choose a reason for hiding this comment

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

P2. a와 b의 공통된 약수인지 확인해주는 것도 좋지만 이 경우 시간복잡도는 O(N)이죠! 어려운 문제의 경우 이 방법으로 풀었을 때 시간 초과가 날 수 있어요😥 따라서 더 효율적인 유클리드 호제법으로 구현해주시면 좋을 것 같습니다!

// 소수
detection = 0;
}
}

Choose a reason for hiding this comment

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

P1. 소수를 판별해주실 때 에라토스테네스의 체를 활용하는 게 아니라 반복문으로 일일이 체크해주고 계시네요! 숫자 하나에 대해서만 판별하는 거라면 괜찮지만, 지금은 여러 개의 소수를 판단해야 하기 때문에 이 부분으로 인해 시간초과가 나고 있네요😥 에라토스테네스의 체로 입력 범위 내의 숫자들에 대해 미리 소수인지를 판단해주고, 그 다음에 두 소수를 찾아주시는 식으로 풀어주시면 좋을 것 같습니다!

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.

3 participants