[투 포인터] 2076278 윤지윤 #316
Open
[투 포인터] 2076278 윤지윤 #316
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
kwakrhkr59
approved these changes
May 19, 2023
| cout <<-1; | ||
| }else{ | ||
| cout <<ans.first<<" "<<ans.second; | ||
| } |
| cin.tie(NULL); | ||
| cout.tie(NULL); | ||
| int T,k; | ||
| string w; |
There was a problem hiding this comment.
변수명의 표기법은 통일해서 사용해주세요! 참고로 저희 튜터링에서는 변수명은 스네이크 표기법을, 상수명은 대문자로 표기할 것을 권장해드리고 있습니다😊
|
|
||
| for (int i =0;i<w.length();i++){ | ||
| if(al[w[i]-'a']<k){ | ||
| continue; |
There was a problem hiding this comment.
여기서 if문을 통과하는 경우 continue를 만나게 되므로 아래 코드를 실행하지 않게 되죠! 아래 else문은 인덴테이션이 많아질 경우 작성하지 않으셔도 괜찮을 것 같네요😀
| max_len =max(len,max_len); | ||
| min_len = min(len,min_len); | ||
| } | ||
| } |
There was a problem hiding this comment.
인덴테이션이 많아지고 있네요! 3중 이상의 인덴테이션이 생기면 이 중괄호가 어디에 속한 중괄호인지 알아보기 어려워질 때가 있죠😥 if(w[i] != w[j]) { continue; }를 사용해 인덴테이션을 하나 제거해보면 어떨까요?
| cnt--; | ||
| } | ||
|
|
||
| } |
| for( int i=0;i<n;i++){ | ||
| cin >>seq[i]; | ||
| } | ||
| cout << cntSubseq(k,n,seq); |
kwakrhkr59
approved these changes
May 19, 2023
| cout <<-1; | ||
| }else{ | ||
| cout <<ans.first<<" "<<ans.second; | ||
| } |
There was a problem hiding this comment.
출력 형식만 한번 확인해주세요...! 이 부분이 놓치기 쉬운 부분이죠😂 딱 하나만 수정해주시면 코드는 잘 돌아갈 것 같네요! 다만 위에서 이중 for문을 돌면서 시간 초과가 날 것 같네요😥 cnt가 k보다 커지면 탐색할 의미가 없어지니 cnt==k가 되면 탐색을 중지시켜보면 어떨까요? 한번 시도해보시고 그래도 어려우시다면 다시 한 번 질문해주세요😁
|
[투포인터 추가제출 확인 완료] |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
인적사항
학번: 2076278
이름: 윤지윤
과제 제출
기존 제출 : 20922, 20437
추가 제출 : 14503
20437번 투포인터 사용으로 푸는 방법은 잘 모르겠어서 원래 제가 작성했던 방법대로라도 풀어보려고 했는데 계속 틀렸습니다가 뜹니다 ㅠㅠ
어떻게 고쳐야 할까요?