Skip to content

[브루트포스] 3월 14일#6

Open
chami3i wants to merge 1 commit intomainfrom
04-assignment
Open

[브루트포스] 3월 14일#6
chami3i wants to merge 1 commit intomainfrom
04-assignment

Conversation

@chami3i
Copy link
Collaborator

@chami3i chami3i commented Mar 14, 2025

인적사항

학번: 2376320
이름: 홍채민

과제제출

기존 제출: 1063, 1476, 11723
추가 제출:

### 인적사항
학번: 2376320
이름: 홍채민

### 과제제출
기존 제출: 1063, 1476, 11723
추가 제출:
</br>
Copy link

@SYEON10 SYEON10 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

[1063 코드 리뷰 완료]

1063(P2)

안녕하세요, 홍채민 님!
자료구조를 개선하면 불필요한 코드가 없어지는 부분이 있어 코멘트 남겨두었습니다! 확인하시고 개선해보면 좋을 것 같아요.
과제하느라 수고 많으셨습니다.

Comment on lines +6 to +8
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

P2. 사전에 명령과 이동 방향에 대해 정의하신 점 아주 좋습니다! 그런데, 이 아이들을 묶어서 정의하면 더 좋지 않을까요? key-value 자료구조를 이용하시면 이동 방향을 8번 돌면서 검사하는 로직 없이도 깔끔하게 이동 기능을 구현할 수 있습니다!

Comment on lines +40 to +44
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

바로 킹의 위치를 옮기지 않고 새로운 위치를 먼저 확인하는 점 좋습니다!

Copy link

@zzangss zzangss left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

[1476번 코드리뷰 완료]

P3: 문제 너무 잘 풀어주셨습니다! 짧은 코멘트 남겨드렸으니 참고하시면 좋겠습니다. 이번 주도 수고 많으셨어요:)

Comment on lines +9 to +16
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

이 부분을 함수로 빼면 어떨까요? main 함수가 간결해져 코드 가독성과 재사용에 도움이 될 거 같습니다!

Copy link

@s0ye0ve s0ye0ve left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

[11723번 코드리뷰 완료]

11723(P3)

안녕하세요! 각 기능 모두 비트마스킹을 이용하여 잘 구현해주셨습니다. 짧은 코멘트 남겨드렸으니 확인 부탁드립니다. 수고 많으셨습니다! 🥰

Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

main 함수에서는 입력, 출력 정도만 수행하고 각 기능을 함수로 분리해주면 어떨까요?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants