0903 코드리뷰 가이드 (튜티용)

• 함수가 너무 많은 일을 하고 있진 않은지 체크
  1. 함수 안에서 정렬함수 호출하는 경우
• 출력부 아니라면 void 함수 지양
• 벡터 사용 권장
• 1316
  1. 그룹단어 체크하는 부분 함수화 권장
  2. 시간복잡도 O(n^2)으로 작성된 코드는 O(n) 풀이법 생각하도록 유도
  3. 혹시 인덱스 참조가 런타임 에러 가능성이 있진 않은지 확인
• 13458
  1. 부감독관 배치시 샘플 코드 기준 line 36 부분이 중복되진 않은지 확인
• 1026
  1. 배열 하나는 오름차순 다른 하나는 내림차순 정렬했는지 확인 (했으면 👍 붙여주기)
• 1431

  1. cmp 함수 인자 형식 맞는지 체크

  2. if문 잘 끊어갔는지 확인

  3. 숫자 판별하는 조건문에서 이 부분 체크하기

     //문제 없는 코드
     if(isdigit(str[i])
     if(str[i]>='0'&&str[i]<='9') //isdigit 함수 안내
     
     //문제 있는 코드
     if(str[i]<='9') //'0'보다 큰거 체크안하면 버그 위험 큼 p1. 직접적으로 어떤 문제인지는 언급하지 않기
     

     

  4. cmp 마지막 사전순 조건에서 line 23 처럼 안하고 뭔가...복잡한 무언갈 쓰진 않았는지 확인

• 1946
  1. 함수화될 부분 있는지 확인
  2. pair 말고 구조체를 사용했다면 pair 사용 유도
  3. 가장 높은 면접심사 순위 갱신하는 과정에서 min 같은 함수(필요 X)를 쓰진 않았는지 확인
• 10994
  1. 그냥 재귀로 잘 구현했으면 OK
  2. 너무 비효율적이면 안내
• 11399
  1. O(n^2)으로 풀었다면 O(n)으로도 풀 수 있다는거 안내
• 11651
  1. pair 사용 유도