Conversation
| int main() | ||
| { | ||
| //변수 선언 | ||
| int N, K, sum=0; |
There was a problem hiding this comment.
P2. 알튜비튜에서 변수 이름은 스네이크 표기법으로 작성하도록 합니다! 클린코드 가이드를 확인해주세요~!
|
|
||
| for(int i=0; i<K; i++){ | ||
| cin>>w[i].n>>w[i].s; | ||
| sum+=w[i].n; |
| string s=""; | ||
| }; | ||
|
|
||
| string wheelF(vector<wheel> &w, vector<string> &ans,int N, int K){//출력을 간단히 하기 위해 ans 벡터를 인자로 가져옴. |
| using namespace std; | ||
|
|
||
| struct wheel{ | ||
| int n; |
There was a problem hiding this comment.
P2. 스네이크 표기법을 활용하여 main 함수의 N과 K를 소문자 n, k로 정의하게 되면 여기서의 n과 겹치게 되겠네요! 다른 변수를 사용해주세요!
|
|
||
| struct wheel{ | ||
| int n; | ||
| string s=""; |
There was a problem hiding this comment.
알파벳 하나가 입력으로 주어지기 때문에 string 자료형이 아닌 char형으로 정의해줘도 돼요!
| if(ans[temp] != ""&&ans[temp]!=w[i].s){ | ||
| return "!"; | ||
| } | ||
| auto it=find(ans.begin(), ans.end(), w[i].s);//중복 제거를 안해주면 틀린다. |
There was a problem hiding this comment.
P2. find() 함수를 사용하여 알파벳 중복 체크를 해줬네요! 이 방법도 괜찮지만 find 연산의 시간복잡도는 O(n)으로 n이 커지게 되면 시간복잡도 측면에서 비효율적일 수 있어요! 알파벳 중복 확인 배열을 활용하는 방법도 있으니 참고해주세요~
| string wheelF(vector<wheel> &w, vector<string> &ans,int N, int K){//출력을 간단히 하기 위해 ans 벡터를 인자로 가져옴. | ||
| int temp=0; | ||
| for(int i=0; i<K; i++){ | ||
| temp = (temp+w[i].n) % N; |
| for(int i=0; i<N; i++){ | ||
| if(ans[i] == ""){ | ||
| ans[i] = "?"; | ||
| } | ||
| } |
There was a problem hiding this comment.
P2. main 함수에서 ans 벡터를 정의할 때 vector<char> ans(N, '?');로 해준다면 따로 이 과정을 거치지 않아도 되겠죠?
| cin>>N>>K; | ||
|
|
||
| vector<wheel> w(K);//K만큼 할당 안해주고 값 입력받으면 for 문 안돌아간다. | ||
| vector<string> ans(N, ""); |
There was a problem hiding this comment.
여기서도 마찬가지로 알파벳 하나가 입력으로 주어지기 때문에 string 자료형이 아닌 char형으로 정의해줘도 돼요!
| cout<<str; | ||
| } | ||
| else{ | ||
| int n=sum%N;//출력 시작 인덱스 |
There was a problem hiding this comment.
P2. n을 여러번 사용해줬네요 코드의 가독성과 쉬운 유지보수를 위해 최대한 다른 변수로 정의해주는 게 좋아요!
| y%=x; | ||
| swap(x,y); | ||
| } | ||
| return y; |
There was a problem hiding this comment.
반복문을 사용해 유클리드 호제법을 구현해주셨네요! 잊지 않고 swap해주시는 것까지 완벽합니다😎
| gcd=gcdIter(x, y); | ||
|
|
||
| x/=gcd; | ||
| y/=gcd; |
| #include <cmath> | ||
| using namespace std; | ||
|
|
||
| const int RANGE=10000001; |
There was a problem hiding this comment.
입력 범위를 상수로 선언해주셨군요! 아주 좋습니다👍👍
| is_prime[j]=false; | ||
| } | ||
| } | ||
| } |
There was a problem hiding this comment.
에라토스테네스의 체의 구현도 완벽합니다😍😍
인적사항
학번 : 2271092
이름 : 이한선
과제제출
기존제출 : 1735, 2840, 6588
추가제출 :