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

[week 2] 기본과제/생각과제/심화과제 #2

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

Conversation

Hong0329
Copy link
Collaborator

@Hong0329 Hong0329 commented Apr 21, 2023

기본과제

  1. 게시물 형식 정의와 각 필요한 DTO 정의
  • SecondSeminar/src/main/java/sopt/org/SecondSeminar/domain/board/Board.java
  • SecondSeminar/src/main/java/sopt/org/SecondSeminar/controller/board/dto/request/BoardRegisterRequestDto.java
    2.3,4 게시물 생성,조회(게시물 내용 조회), 제목으로 게시물 검색
  • SecondSeminar/src/main/java/sopt/org/SecondSeminar/controller/board/BoardController.java
  • SecondSeminar/src/main/java/sopt/org/SecondSeminar/service/BoardService.java

생각과제
https://canyon-rhythm-70f.notion.site/2-5ee558b81dc14674835370e98e1c3278

@Hong0329 Hong0329 changed the title Week#2 [week 1] 기본과제/생각과제 Apr 21, 2023
@Hong0329 Hong0329 changed the title [week 1] 기본과제/생각과제 [week 2] 기본과제/생각과제 Apr 21, 2023
Copy link

@SeorinY SeorinY left a comment

Choose a reason for hiding this comment

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

수고하셨습니다!

System.out.println(boardList.get(boardId.intValue() - 1).toString());

return boardId + "번 게시물이 등록되었습니다!";
}
Copy link

Choose a reason for hiding this comment

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

메서드 끝나도 들여쓰기 해주면 좋다고 들었어요!!
자바 코드 컨벤션 구글링하면 참고하실 수 있습니다!

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

제가 자바가 익숙하지 않아서 잘 몰랐는데 말씀해주셔서 감사합니다!! :)

@Getter
@AllArgsConstructor(access = AccessLevel.PRIVATE)
public class BoardRegisterRequestDto {
private String title;
Copy link

Choose a reason for hiding this comment

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

Dto는 값이 바뀌지 않는 객체기 때문에 final 로 변수를 선언해주면 조금 더 데이터가 보호될 것 같아요!

return "유저 조회 성공";
}

@GetMapping("user/search")
Copy link

Choose a reason for hiding this comment

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

"/user/search" 가 되면 더 좋은 Rest ful URI 로 만들 수 있을 것 같아요!

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Restful URI에 대해서 더 공부해보겠습니당!감사합니다 :)

@Hong0329 Hong0329 changed the title [week 2] 기본과제/생각과제 [week 2] 기본과제/생각과제/심화과제 May 5, 2023
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.

2 participants