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

[ 정렬, 맵, 셋 ] 8월 23일 #1

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

Conversation

sari-harin
Copy link
Collaborator

인적사항

학번: 2071079
이름: 이하린

과제제출

기존 제출: 19636, 1431, 14425
추가 제출:

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.

[정렬, 맵, 셋 구현 코드 리뷰 완료]
안녕하세요 하린님~ 제출해주신 필수 구현문제 코드리뷰 완료되셨습니다! 이번 문제가 고려해야 하는 변수들이 많아서 까다로울 수 있는 문제였는데 아주 깔끔하고 간단하게 해결해주셨네요👍👍 워낙 로직을 깔끔하게 잘 짜주셔서 특별히 드릴 말씀은 없었네요😊 질문 있으시면 언제든 리뷰어로 호출해주세요! 수고 많으셨습니다~

if (abs(tr_w)>tr){
diet_life+=floor(tr_w/2.0);
}
w0+=diet_eat - day_life - diet_ex;

Choose a reason for hiding this comment

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

체중과 기초대사량의 변화를 아주 깔끔하게 계산해주셨네요😍

int w0=w1;
int tr_w;

for(int i=0;i<n;i++){

Choose a reason for hiding this comment

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

n은 이 for문 외에서는 사용되지 않는 변수네요! 이런 경우에는 while(n--)로도 간단하게 표현할 수 있을 것 같네요😊

Copy link

@sawoll sawoll left a comment

Choose a reason for hiding this comment

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

P3: 안녕하세요 하린님! 필수문제 1431번, 14425번 코드 리뷰 완료했습니다. 두 문제 모두 정렬을 사용하여 잘 풀어 주셨습니다.😊 단, 14425번의 경우에는 컨테이너 사용과 관련하여 코멘트 남겨드렸고, 1431번 또한 코드 구조 관련하여 코멘트 남겨드렸으니 확인해보시면 좋을 것 같습니다. 수고하셨습니다!😍

Comment on lines +11 to +19
int s1_sum=0, s2_sum=0;
for(int i=0;i<s1.size();i++){
if('0'<=s1[i] && s1[i]<='9'){
s1_sum+=s1[i]-'0';
}
if('0'<=s2[i] && s2[i]<='9'){
s2_sum+=s2[i]-'0';
}
}
Copy link

Choose a reason for hiding this comment

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

아주 깔끔한 로직으로 비교함수cmp를 잘 구현해 주셨네요!😊 핵심인 자릿수 합 구하기도 잘 작성해 주셨습니다. cmp에는 문제의 세 가지 조건을 비교하는 부분만 남기고, 이 부분은 따로 함수로 빼내어 cmp에서 호출하여 사용한다면 더 깔끔한 코드가 될 수 있을 것 같습니다!👍👍

Comment on lines +13 to +15
if('0'<=s1[i] && s1[i]<='9'){
s1_sum+=s1[i]-'0';
}
Copy link

Choose a reason for hiding this comment

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

그리고, string헤더의 isdigit 함수를 이용하여 문자의 숫자 여부를 판별할 수 있다는 점도 고려해 주시면 좋을 것 같습니다.😊

Comment on lines +21 to +28
sort(arr.begin(), arr.end());

for(int i=0;i<m;i++){
cin >> stm;
if(binary_search(arr.begin(), arr.end(), stm)){
num++;
}
}
Copy link

Choose a reason for hiding this comment

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

sort와 이진탐색을 이용하여 문제를 풀어 주셨네요!👍그런데, 직접 값을 정렬해 주지 않아도 값을 삽입하면 정렬된 상태로 저장해 주는 컨테이너가 있었죠! 그걸 사용해 보면 어떨까요?😉

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