Conversation
| } | ||
| int main() { | ||
| int n; | ||
| int maxnum=-1000000000, minnum=1000000000; |
There was a problem hiding this comment.
P3. 0이 많아지는 경우에는 1e9과 같이 제곱으로 표현하면 한눈에 숫자의 크기를 알 수 있어서 좋아요!
| if(result>maxnum){ | ||
| maxnum=result; | ||
| } | ||
| if(result<minnum){ | ||
| minnum=result; | ||
| } |
There was a problem hiding this comment.
P3. 최솟값, 최댓값 갱신을 쉽게 해주는 min(), max()에 대해서도 알아보시면 좋을 것 같아요! 하지만 지금처럼 작성해주셔도 좋습니다~
참조: cplusplus.com/min | cplusplus.com/max
| if(cnt==n) | ||
| { | ||
| if(result>maxnum){ | ||
| maxnum=result; | ||
| } | ||
| if(result<minnum){ | ||
| minnum=result; | ||
| } | ||
| } | ||
| else{ |
There was a problem hiding this comment.
P2. if문 안에 return을 포함하면 else 키워드를 작성할 필요가 없어요! else를 쓰면 안의 내용을 중괄호로 한 번 더 감싸야 해서 번거롭기도 하고 인덴테이션도 깊어지기 때문에 if-return 구조에 익숙해지는 것을 추천드립니다🤗
| } | ||
| } | ||
|
|
||
| else{ |
There was a problem hiding this comment.
p2. 여기도 위에 코멘트 단 것과 같이 if-return를 사용해서 else를 생략하는 건 어떤가요??
| else{ | ||
| int temp=0; | ||
| for(int i=0; i<n; i++){ | ||
| if(temp!=nums[i]){ |
There was a problem hiding this comment.
이전에 사용한 숫자를 temp에 저장해서 중복 수열을 제외하셨네요! 다른 방법으로 nums에 set을 사용하는 방식도 한 번 고려해보시면 좋을 것 같아요. set을 사용하면 아예 입력 받을 때 중복되는 숫자를 제외할 수 있고, c++의 set은 자동으로 오름차순 정렬되기 때문에 따로 정렬 함수를 호출할 필요도 없어요👍
|
|
||
| for(int i=0; i<m; i++){ | ||
| cout << answer[i] << '\n'; | ||
| } |
There was a problem hiding this comment.
p3. 출력하는 코드는 별도의 메소드로 분리하면 조금 더 각 메소드의 기능이 명확하고 코드가 깔끔해질 것 같아요😊
인적사항
학번: 2071079
이름: 이하린
과제제출
기존 제출: 14888, 15665
추가 제출: 20055