[5week] 가현 -> 나영 코드리뷰

[kigahy]

5주차 코드리뷰

• 클린코드의 정석을 보여주며, 지금까지 겁나서 외면했던 함축적인 코드를 들여다보는 시간이 되었다
• 선언한 변수나 문장이 문제에서 어떤 것을 뜻하는지 쓰면 좋을 것 같다

IM_test

• 간단하고 함축적이라 이해하기 어려웠습니다 (...) 동경합니다
• swap = {0:1, 1:0} : 딕셔너리를 활용하여 키와 값으로 교환하는 방식이 좋았습니다

  if (j+1) % idx == 0:   # 같지 않은 인덱스의 배수인 인덱스 찾기
  start_light[j] = swap[start_light[j]]   # swap

• 나머지 연산 센스있습니다. 나머지 연산을 잘 활용하지 못하는 입장에서 배워갈 만하다고 생각하였습니다.

SWEA_10580_전봇대

• 저는 문제를 못풀었고, 제가 생각한 로직은 다음과 같습니다.
  1. N = int(input()) # 전깃줄 개수
  2. electrics = [list(map(int, input().split())) for _ in range] # 전선 위치들
  3. 현재 순회값보다 이전 electrics의 [0]과 [1] 인덱스가 현재 값보다 작-크거나, 크-작으면 count += 1
  4. 나 전의 모든 선을 고려해야 하므로 현재 전선 이전의 모든 값을 순회하는 2중 for문 형성

     for i in range(N-1):
     for j in range(i+1,N):

• 저는 현재 값 이전의 값을 고려하고자 하였는데, 나영님은 2중 for문의 range 범위에서 현재 값보다 다음 값을 고려한다는 점이 제 생각과 달랐습니다.

SWEA_12712_파리퇴치3

• + 모양과 x 모양으로 나눈다는 점은 동일하지만, 각각 함수로 정의한 점이 보기 깔끔하네요. 좋습니다!

SWEA_1860_진기의 최고급 붕어빵

• 문제도 코드도 어렵습니다 (미안) 잘 모르겠지만, 클린코드 같아요.

SWEA_3499_퍼펙트 셔플

• for i in range(N//2+(N%2)): # 홀수 일때도 가능 : 저는 if문을 통하여 홀수일 경우와 짝수일 경우를 각각 생각했는데 N%2를 활용한 게 좋아보입니다.
• 그런데 밑줄과 밑밑줄에도 N//2+(N%2)가 반복적으로 등장하는데, 이것을 따로 변수로 선언하면 더 깔끔한 코드가 나올 수도 있을 것 같아요.

SWEA_4408_자기 방으로 돌아가기

• 로직을 짜는 게 굉장히 어려워 보이는데, 홀수와 짝수를 같은 번호로 설정하여 2중 for문 한 개로 짜는 거 좋습니다.
• for i in range(start, end+1) 는 한 번에 한 학생이 이동하는 거리를 뜻하는 게 맞나요?

SWEA_9489_고대유적

• dxy = [[1,0],[0,1]] 우측 이동과 하단 이동은 저와 같습니다.
• if not arr[row][col]: if not으로 0 판별한 뒤 다음 for문으로 넘어가는 것 깔끔해서 좋습니다.
• 48행 for k in range(1, max(N,M)): 에서 max(N, M)을 하면 1이 얼마나 많은지와 상관 없이 배열의 행이나 열이 긴 값으로 골라지지 않나요? 어떻게 이 범위를 설정하게 된 건지 궁금해요!

[skdud5126]

[4408] 자기 방으로 돌아가기

• for문을 돌면서 start 부터 end 범위까지 학생이 이정도 범위를 지나갔다를 표시하기 위함입니다! 가현님이 이해하신게 맞습니다 ㅎㅎ

[9589] 고대유적

• 저는 델타탐색으로 각 원소들을 돌면서 유물을 확인하는데 문제에서 N과 M 어떤 값이 큰지를 안나타냈기 때문에 둘 중 큰값에 대해만큼 돌아도 밑에 조건문에서 범위가 벗어나면 반복문을 탈출한다 이뜻입니다! 만약 M을 범위로 설정하였을 때 N이 M보다 크면 N만큼 끝까지 탐색을 못하기때문입니다!!

---

• 가현님의 코드리뷰 감사합니다 ㅎㅎ 퍼펙트셔플 문제에서 저도 저 부분 변수로 따로 뺄까하다가 문제풀기 바빠서 과정을 생략해버렸네요 ㅎㅎ 조언 감사합니다! 다음엔 리뷰하시는 분들을 위해 변수 설정해서 보기 편하게 하는 습관 좀 들여야겠습니다..ㅎ
• 아무래도 저는 문제를 이해하면서 코드를 작성하다보니 변수설정한 것이나 그에 대한 주석이 생략이 되었습니다..ㅎ 주석다는 습관을 더 늘려보겠습니다 ㅎㅎ
• 그리고 팀장님과 팀원들 덕분에 늦게나마 신입으로 들어와 simpler 스터디 진행하면서 IM 취득할 수 있게되었습니다. 다들 으쌰으쌰하는 모습에 저도 열심히하게되어 좋은 결과가 나올 수 있었습니다. 저희 팀 모두다 A,B형까지 취득하는 그날까지!! 화이팅합시다!!! simpler 뽜이아~~~~