4주차 코드리뷰

핵심: 저도 진문님도 주석을 생활화 해보아요!

1220_magnetic

for문으로 j(열) 먼저, 다음으로 i(행) 탐색하는 것은 저와 같은 생각입니다.

for j in range(matrix_width): # 행렬의 열을 탐색할 것이기 때문에 임의변수를 j로 설정. 이후의 for문에서는 상수로 사용.
    column_number_list = []
    for i in range(matrix_width):
        if matrix[i][j] != 0: # i가 변수이고 j가 상수, i는 0에서99까지의 값을 가짐

보통 행을 먼저 탐색, 다음으로 열을 탐색하며 i 다음 j를 적곤 합니다. i와 j 및 행과 열의 순서가 헷갈릴 수 있는데 잘 적어주신 점이 인상깊습니다.
```
column_number_list.append(matrix[i][j])
```

빈 배열에 1 혹은 2가 저장된 인덱스를 찾아 그 값 자체를 넣는 점 좋았습니다.

    for k in range(len(column_number_list)-1):
        if column_number_list[k+1] == column_number_list[k]+1: # 1 차이나는 연속된 수 이면 result +=1 동작
            result += 1

개인적으로 이 부분은 주석이 없어서 이해하기 어려웠습니다. len(column_number_list)-1 에서 왜 -1을 적었는지 궁금합니다...!
```
        if column_number_list[k+1] == column_number_list[k]+1: # 1 차이나는 연속된 수 이면 result +=1 동작
            result += 1
```
if문 굉장히 신기하고 [k+1]과 [k]+1을 활용하여 1과 2 모두 검사할 수 있는 점에서 감탄했습니다. 이 코드 생각해내기까지 시간이 얼만큼 걸렸는지 궁금합니다. 저는 아직 이런 코드를 잘 못 만들어서요. 정말 깔끔하고 규칙 있는 검사라고 생각합니다

2805_harvest

좋은 풀이 잘 봤습니다. 다만 언더바가 두 개 이상 포함되는 변수명은 오히려 가독성이 떨어질 수 있으니, middle_of_farm 대신 middle_value 혹은 middle_farm으로 해보는 건 어떨까요? (제 개인적인 생각입니다!)

for i in range(farm_size):
        if middle_of_farm >= 0:
            result_list.append(sum(matrix[middle_of_farm][i:farm_size-i])) # 한 row씩 올라가면서 수확에 해당하는 행의 일부분을 모두 더함
            middle_of_farm -= 1

if문의 조건이 >0 이어서 독특하다 생각했는데, middle을 하나씩 줄여가면서 0을 중심으로 만드는 점이 독창적인 발창입니다.

주석 #한 row씩 내려가면서 더함 부분도 리스트 슬라이싱을 활용한 점이 특이했고, 다양한 방식으로 문제를 풀어나가는 능력이 있으신 것 같습니다.

    else:
        reverse_farm = (farm_size // 2) +k
        result_list.append(sum(matrix[reverse_farm][i-(farm_size//2):farm_size-i+(farm_size//2)])) # 한 row씩 내려가면서 더함
        reverse_farm += 1
        k += 1

4466_max_table

수정 전 코드를 만들 때, 부분집합을 만든다는 발상이 독특했습니다.
수정 후 버블소팅 내림차순으로 정렬한 코드는 제 코드와 흡사합니다.
```
# 진문 코드
return sum(arr[:choice]) # 슬라이싱을 활용한 리턴
```
그러나 저는 위 코드를 구현하지 못하여 다음과 같이 for문을 활용하였습니다.
```
# 가현 코드
for k in range(select_subject) : # 내림차순 정렬한 리스트만큼 반복
    total_score += score[k] # 리스트 앞부터 합 구함
```
이 코드를 짜면서도 더 좋은 방법이 없을까, 하였는데 sum 함수와 슬라이싱의 조합이 참 센스 있어 보입니다!

5431_project_check

모든 학생을 대상으로 for문을 순회하는 건 저와 같지만, if문과 not in을 함께 활용한 점이 보다 간단히 문제를 풀어나가는 문법이라는 것을 깨닫게 되었습니다.

# 진문 코드
for number in students_list:
    if number not in submits_list:
        F_list.append(number)

저는 2중 for문을 돌며 모든 학생의 번호와 제출한 학생의 번호를 하나씩 대조했기 때문에 depth가 깊어졌습니다.

# 가현 코드
for i in all_num : # 모든 학생의 번호만큼 반복
    for j in submit_num : # 제출한 학생의 번호만큼 반복
        if i == j : # 만약 제출했다면
            break # 2중 for문 깨트리고 모든 학생의 번호 중 다음 번호 검사

그리고 다음 코드로 오름차순으로 하나씩 정리하는 것이 꽤 정교해 보입니다.

# 진문 코드
for i in range(len(F_list)-1):
    if F_list[i] > F_list[i+1]:
        F_list[i],F_list[i+1] = F_list[i+1], F_list[i]
for elem in F_list:
    print(elem,end=' ')
return ''

저는 for-else를 활용하여 간단히 해결해 보았습니다. 그러나 단점이 있다면 꽤나 파이써닉해서, 다른 언어로 알고리즘을 푼다면 쉽게 접근하지 못할 듯한... 오늘도 배워갑니다.

# 가현 코드
for j in submit_num : # 제출한 학생의 번호만큼 반복
        if i == j : # 만약 제출했다면
            break # 2중 for문 깨트리고 모든 학생의 번호 중 다음 번호 검사
        else : # 2중 for문에서 제출하지 않았다면
            continue # 2중 for문의 다음 숫자 순회
    else: # for문의 break가 걸리지 않으면 실행되며, i가 submit_number에 해당되지 않았을 때
        not_num.append(i)

kigahy / SIMPLER_AlgorithmStudy

[4week] 가현 -> 진문 코드리뷰 #21

4주차 코드리뷰

핵심: 저도 진문님도 주석을 생활화 해보아요!

1220_magnetic

2805_harvest

4466_max_table

5431_project_check

magnetic

harvest

max_table