[5week] 원재 -> 가현 코드리뷰

[dnjswoc]

5주차 코드리뷰(원재 -> 가현)

IM test

• 접근 방식은 모두들 비슷한 것 같습니다. 첫 번째 전구부터 시작하여 목표 스위치와 다른 인덱스를 찾아 그 인덱스의 배수에 해당하는 전수를 바꿔 나가는 방식으로 접근한 것 같습니다. IM 축하드립니다~!~!
• 저는 문제를 풀 때 서로 다른 인덱스를 찾아 그에 해당하는 배수의 개수를 따로 구하고 풀었습니다. 하지만 가현님 코드를 보니 배수를 따로 구하지 않고, range에서 움직일 때 조절을 해주셨군요! 이거 참 신박했습니다. 혹시 이 코드에서 i+1을 해주다가 마지막 인덱스를 넘어가도 에러가 나지 않고 잘 마무리가 되나요...?

  
  times = N // (idx + 1)  # 배수의 개수(바꿔야 하는 스위치 개수)  -> 원재 코드
  for time in range(1, times + 1):  # 배수의 개수만큼 반복

for j in range(i, len(change), i+1) # 가현님 코드

- 원하는 리스트와 다른 상태일 때의 인덱스를 구할 때 원하는 <리스트의 전구가 1일 때, 원하는 리스트의 전구가 0일 때> 두 가지 경우를 나누어서 반복문을 쓰셨더라구요. 조건 밑에 있는 코드는 두 조건 다 같던데 서로 같지 않다는 표현을 쓰면 두 개의 for문을 하나로 줄일 수 있을 것 같습니다!
```python
 while start_arr != light_arr:  # 원재 코드

if data[i] == 1 and change[i] == 0 :  # 가현님 코드
if data[i] == 0 and change[i] == 1 :

• 그리고 출력문이 신기하더라구요.. 이거 무슨 뜻인가요??

  print("#%d %d" % (test_case, AnswerN))

  12712. 파리퇴치 3

• 이번 문제도 저랑 접근 방식은 똑같습니다. 코드 진행도 굉장히 유사한데요. 제가 코드 짤 때는 분사하는 M을 먼저 반복하고, 그 안에 델타 탐색을 해서 + 모양과 x 모양을 한 번에 처리했습니다. 코드 길이는 거의 유사한 것 같아 한 번 참고만 하시면 될 것 같습니다.

  
  for r in range(1, M):
  for di, dj in dij1:  # + 방향 합계
  for di, dj in dij2:  # x 방향 합계   -> 원재 코드

for cx, cy in cross_dxy : # + 모양의 4방향 각각 순회 for c in range(1, M) : for dx, dy in dia_dxy : # x 모양의 4방향 순회 for d in range(1, M) : # 가현님 코드

## 9489. 고대 유적
- 이 문제 푸느라 정말 고생하셨습니다. 사실 이 문제 절대 D2 아닙니다... 어려운 거 맞아요...
- 그리고 정답까지 거의 다 오신 것 같습니다. 아쉬워요ㅠㅠ
- dxy가 오른쪽 방향, 아래쪽 방향 모두 있으므로 반복을 돌면 가로로 혹은 세로로만 긴 값을 찾을 수 없을 것 같습니다. 그리고 가로, 세로 방향을 모두 찾을 수 있으므로 반복문이 두 개일 필요가 없을 것 같습니다.
```python
dxy = [[0,1], [1,0]]
for rx, ry in dxy : # 가로로 긴 값 시작

마무리

• 문제 푸시느라 수고하셨습니다... 우리 IM 땄으니까 자신감 가지시구요! 그리고 문제 정말 잘 푸셨습니다. 저는 일단 답만 나오면 된다는 주의이기 때문에 잘 하고 계십니다!!
• 다른 문제도 다 푸시면 마저 리뷰하겠습니다~~

[kigahy]

IM

원재님, 정성스러운 코드리뷰 진심으로 감사합니다. IM문제에서 인덱스를 벗어나는 건, 바로 밑에서 바로 어떤 행동을 하는 게 아니라 if문을 넣어서 에러가 안 났나 하고 짐작하고 있습니다.

                if change[j] == 0 :
                    change[j] = 1
                elif change[j] == 1 :
                    change[j] = 0

에서 if의 0과 1일 때만 가정한 것입니다. 그리고 for j in range(i, len(change), i+1) 에서 len(change)를 만족해야 i+1를 하므로, 인덱스 범위를 벗어나는 i+1은 실현되지 않은 것으로 짐작합니다. 챗GPT한테 물어보니 range는 유효한 범위 내에서만 형성한다고 합니다! AnswerN은 시험 스켈레톤 코드에 있던 변수로, 전구를 켜고 끈 횟수를 기록하는 변수입니다.

[dnjswoc]

사실 제 마음 같아서는 가현님 힘으로 처음부터 끝까지 스스로 문제를 푸는 양을 늘려 가셨으면 좋겠습니다! GPT도 좋은 참고자료가 될 수 있고, 지금 가현님께서 하시는 일이 많아서 처음부터 문제를 다 푸는 것이 많이 힘드시겠지만, 저는 제 스스로 문제를 끝까지 풀어본 것이 제 실력 향상에 많은 도움이 되었던 것 같습니다. 물론 강사님께서도 너무 오랜 시간 문제를 붙잡는 것은 추천하지 않는다고 하셨고, 공부 방법은 가현님이 선택하시는 것이니까 참고 바라는 마음에 글을 적습니다ㅎㅎ