[4week] 원재 -> 나영 코드리뷰

[dnjswoc]

4주차 코드리뷰

1220_Magnetic

• 1과 2가 연달아 나오면 교착상태가 된다는 접근은 저랑 비슷한 것 같습니다.
• 하지만 boolean을 이용한 조건문으로 답을 구하는 것은 저에게 굉장히 참신한 방법이었습니다. 정말 신선하고 많이 놀랐습니다.

  for col in range(length):   # 열 순서대로 배열 순회
      find = False
      for row in range(length):
          if arr[row][col] == 1:   # '순서가 1,2 이렇게 되면 교착상태 1 1 2 도 가능'  처음 1 끝 2면 교착상태 1개로 판단
              find = True
          if arr[row][col] == 2:   # 1을 찾고 그다음 2가 나오면 find로 인해 그 교착상태임을 확인 가능
              if find:
                  max_deadlock+=1    # 교착 합 갱신
                  find = False  # 그러고나서 다시 처음부터 찾아야하니 find False로 돌려줌

• 1220_1 파일은 접근법은 같아 보이지만, 코드로 표현하는 방법을 다르게 한 것 같았습니다.
• 리스트가 아닌 string을 사용해서 답을 구하는 방법도 좋아보였습니다.

2805_농작물수확하기

• 행을 순회하면서 전체 행의 절반을 넘어갔을 때 열의 범위가 줄어드는 것을 표현하는 법이 비슷한 듯 달라서 신기했습니다.

  for col in range(mid - (N - row - 1), mid + (N - row)):  # 원재
  for col in range(row - center_col, N - (row - center_col)):  # 나영

• 그리고 저는 농작물 수확하기 문제를 풀면서 처음에 while문을 이용해서 풀었는데 한 번 참고해보시고, while문으로 문제를 푸는 두려움을 줄이셨으면 좋겠습니다.

  while row < num:                # 행을 0부터 N-1까지 반복
      farm_sum = arr[row][col]    # 반복하는 행의 농작물의 초깃값은 기준으로 삼은 N//2 열의 값으로 설정한다.
      for delta in dj:            # 위에서 설정한 기준값으로부터 델타 탐색하여 농작물을 더해준다
          if k == 0: continue     # k == 0의 경우는 continue
          for i in range(1, k+1):  # i를 1부터 k+1까지 반복
              farm_sum += arr[row][col + delta*i]  # 델타 탐색을 통해 행의 농작물 수확량을 더해준다.
      total_sum += farm_sum       # 총 농작물 수확량에 행의 농작물 수확량을 더해준다
      if row < col:               # 행이 열의 기준값 N//2까지는
          k += 1                  # k를 1씩 증가시키고
      else:                       # 행이 열의 기준값 N//2를 넘어서면
          k -= 1                  # k를 1씩 감소시킨다
      row += 1                    # 행을 1씩 증가

4466_최대 성적표 만들기

• 저는 선택 정렬 쓰는 법을 다 까먹었는데 굉장히 잘 쓰셨네요. 코드리뷰 덕분에 다시 배워갑니다.
• 4466_1 파일에는 sort함수로 정렬을 하셨던데, 역시 내장함수는 쓰면 쓸수록 좋네요ㅎㅎ
• 이 문제도 전제적인 접근법은 저와 유사했습니다.

5431_민석이의 과제 체크하기

• 5431 파일도 문제 접근법과 코드로 표현하는 방법이 제가 푼 방법이랑 비슷했습니다. 코드도 참 깔끔하구요.
• 5431_1 파일에는 아래와 같이 0으로 된 리스트를 만드셨던데, 우리가 알고리즘 문제 풀면서 배운 visited를 잘 활용한 것 같습니다.

  check_submit = [0] * (N + 1)  # 냈는지 확인하기 위한 빈배열 생성

총평

• 나영님 코드를 보면 항상 느끼는 거지만, 코드를 참 깔끔하게 잘 짜시는 것 같습니다.
• 접근법은 비슷한 것 같은데 코드로 표현하는 방식이 참 다양한 것 같습니다.
• 코드 리뷰하면서 코드를 표현하는 방법도 많이 알게 되었고, 덕분에 많이 배워갑니다.
• 문제 푸느라 수고하셨습니다!!

[skdud5126]

원재님 코드리뷰 해준신거 차근차근 읽어보면서 제가 푼 코드를 다시 보게되었습니다. 좋은 말씀과 조언해주셔서 감사합니다! 아직 while에 겁먹는다는게 눈에 보이셨군요 ㅎㅎ while문 쓸 때 무한루프에 걸려서 웬만하면 안쓰려고 했었는데 원재님 해주신 말씀처럼 사용에 대해 두려워하지 않고 시도해봐야겠습니다 ㅎㅎ 원재님도 더티원재에서 클린원재로 되신거 정말 큰 발전입니다 코드리뷰 감사합니다 ㅎㅎ IM 다 같이 뿌셔봅시다!

[hyerongii]

더티원재에서 클린원재ㅐㅋㅋㅋㅋㅋㅋㅋㅋㅋㅋㅋㅋㅋㅋㅋ