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

Week1: 기본과제(2)+심화과제+키워드과제 #2

Open
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

hyeyeonnnnn
Copy link
Member

@hyeyeonnnnn hyeyeonnnnn commented Oct 12, 2023

📌 기본 과제 (seminar)
📌 심화 과제
📌 키워드 과제

https://www.notion.so/1-ab44337c027543e5b9b31764eab5b8f7?pvs=4

@hyeyeonnnnn hyeyeonnnnn changed the title Week1: 기본과제(1)+심화과제+키워드과제 Week1: 기본과제(2)+심화과제+키워드과제 Oct 12, 2023
Copy link
Member

@unanchoi unanchoi left a comment

Choose a reason for hiding this comment

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

깔끔하게 구현했네요 ~ 수고하셨습니다!

public class Response<T> {
private String code;
private String status;
private boolean suceess;
Copy link
Member

Choose a reason for hiding this comment

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

오타가 있는 것 같습니다!!

@Hong0329
Copy link

P5. 심화 과제까지 구현하시느라 고생 많으셨습니다~! 특히 생각 과제에서 Generic에 대한 설명이 너무 잘되어있어서 정독 했습니다 :) 고생하셨어요~

Copy link

@jumining jumining left a comment

Choose a reason for hiding this comment

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

P5. 심화구제 구현한 내용 확인했습니다. 수고하셨습니다 :)

Comment on lines +3 to +14
public class HealthCheckResponse {
//private String status;
private static final String STATUS = "OK";
public String status;

public HealthCheckResponse() {
//this.status = status;
this.status = STATUS;
}

}

Choose a reason for hiding this comment

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

P4. 이 부분에 주석 처리한 코드는 삭제하지 않고 남겨둔 이유가 혹시 있을까요? 주석의 의도가 혹시 딱히 있지 않다면 지우는 것도 좋을 것 같습니다 :)

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