Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[트리] 2116020 전유진 #328

Open
wants to merge 134 commits into
base: 2116020-전유진
Choose a base branch
from

Conversation

YujinJeon888888
Copy link

@YujinJeon888888 YujinJeon888888 commented May 23, 2023

인적사항

학번: 2116020
이름: 전유진


과제 제출

기존 제출 : 3190,5630
추가 제출 :


SeoYeomm and others added 30 commits February 28, 2023 19:56
jk0527 and others added 26 commits May 2, 2023 01:49
@kwakrhkr59 kwakrhkr59 changed the base branch from main to 2116020-전유진 May 24, 2023 03:50
어려워요 홍홍
Copy link
Contributor

@dbswn dbswn left a comment

Choose a reason for hiding this comment

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

안녕하세요 유진님 👍😄 코드도 너무 깔끔하고 주석도 작성해주셔서 리뷰 드리기 너무 좋았습니다!!!

p1은 반드시 수정해주셨으면 하는 부분이고, p2,p3는 참고만 해주셔도 괜찮습니다!수용하시고 싶은 부분있으시면 저 리뷰어로 호출해주세요~!수고하셨습니다🥰

#include <iostream>
using namespace std;

struct Node {
Copy link
Contributor

Choose a reason for hiding this comment

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

P3
구조체는 실제 알고리즘 테스트에서 사용하기 어려우니 map<int, <int, int>>을 사용해 보는 것도 좋을 것 같습니다!

Copy link

@kwakrhkr59 kwakrhkr59 left a comment

Choose a reason for hiding this comment

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

[트리 필수 구현 코드리뷰 완료]
안녕하세요 유진님~ 트리 필수 구현문제 코드리뷰 완료되셨습니다! queue를 사용해 변환 정보를 저장해주신 부분이 상당히 인상깊었습니다😊 주석을 자세하게 써주셔서 너무 좋았습니다👍👍 몇가지 코멘트 남겨드렸습니다! 확인해보시고 질문있으시면 언제든 리뷰어로 호출해주세요! 수고 많으셨습니다~


// 뱀의 이동 방향
const int dx[] = {0, 1, 0, -1}; // 동, 남, 서, 북
const int dy[] = {1, 0, -1, 0};

Choose a reason for hiding this comment

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

이동 방향을 잘 설정해주셨네요! 아주 좋습니다😍

int x;
char c;
cin >> x >> c;
directions.push({x, c});

Choose a reason for hiding this comment

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

queue를 사용해 변환 정보를 저장해주셨군요! 이 방법도 좋네요😍

// 이동한 칸에 사과가 없으면 꼬리 제거
if (board[nx][ny] == 0) {
int tailX = snake.front().first;
int tailY = snake.front().second;

Choose a reason for hiding this comment

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

변수, 상수, 함수명 별로 규칙을 통일해주세요! 저희 튜터링에서는 변수명은 스네이크 표기법을 컨벤션으로 하고 있습니다😊

if (nx < 0 || nx >= n || ny < 0 || ny >= n || board[nx][ny] == 2) {
cout << time + 1 << '\n'; // 1부터 시작하므로 +1
break;
}

Choose a reason for hiding this comment

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

P3. 메인 부분은 최대한 입출력만 담당하도록 작성해주세요! 지금과 같은 경우에는 게임이 종료되는 시간을 계산해서 반환하는 부분을 함수화해서 분리할 수 있을 것 같네요😁

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.

9 participants