4-Backend-seungchan #24

s-chan-o · 2024-07-23T13:45:24Z

💡 개요

마인드웨이 4과제 수행

📃 작업내용

CRUD기능을 가진 Board구현

🔀 변경사항

🙋‍♂️ 질문사항

💀 어려웠던 점

ta2ye0n · 2024-07-24T01:17:06Z

src/main/java/com/example/board_crud/crud/Dto/BoardResponseDto.java

+public class BoardResponseDto {
+    private String title;
+    private String content;
+    private LocalDateTime createdAt;


작성된 시간과 수정된 시간을 반환하는 Board entity에는 이런 필드가 없어요 무슨 시간을 반환하시는건가요?

시간에 관한 필드를 생성하려고 했었는데 삭제를 안했던것 같습니다.
050af7d

ta2ye0n · 2024-07-24T01:22:10Z

src/main/java/com/example/board_crud/crud/controller/BoardController.java

+
+    private final BoardService boardService;
+
+    @Autowired


@Autowired를 사용한 생성자 주입보다 @RequiredArgsConstructor를 사용한 생성자 주입 방법를 사용하는게 좋을 것같아요

@Autowird을 활용한 생정자 주입은 필드 주입이라고 합니다
필드 주입 방법은 매우 간단해서 편리하지만 이러한 방법말고는 장점이 없어 스프링 4.3부터는 사용하지 않는 것을 권장하고 있습니다

ae66a75
피드백 반영 했습니다.

ta2ye0n · 2024-07-24T01:24:59Z

src/main/java/com/example/board_crud/crud/controller/BoardController.java

+        this.boardService = boardService;
+    }
+    //글 등록
+    @PostMapping("/boards")


공통적인 url은 @RequestMapping 통해 한번만 적고도 사용할 수 있어요

@RequestMapping은 클래스와 메서드에 사용할 수 있지만 위치에 따라 의미가 다르니 찾아보고 수정해주세요

007626e
피드백 반영 완료했습니다

src/main/java/com/example/board_crud/crud/service/BoardService.java

ta2ye0n · 2024-07-24T01:27:56Z

src/main/resources/application.properties

@@ -0,0 +1,6 @@
+
+spring.datasource.url=jdbc:mysql://localhost:3307/mindwaytest?serverTimezone=Asia/Seoul&useSSL=false&allowPublicKeyRetrieval=true


많은 프로젝트에서 application.properties 파일보다 application.yml파일이 가독성이 더 좋고 사용자 친화적이어서 yml 파일 더 많이 사용해요
그래서 yml 파일 사용해보는것도 좋을거 같아요

8abe05e
피드백 반영 했습니다.

ta2ye0n · 2024-07-24T01:28:14Z

src/main/resources/application.properties

@@ -0,0 +1,6 @@
+
+spring.datasource.url=jdbc:mysql://localhost:3307/mindwaytest?serverTimezone=Asia/Seoul&useSSL=false&allowPublicKeyRetrieval=true
+spring.datasource.username=root


username이나 password 같은 민감한 정보는 외부에 노출되면 안되기 때문에 커밋을 할때 환경변수 처리해서 올려주세요

ecca6f6
피드백 반영 했습니다.

ta2ye0n · 2024-07-24T01:28:51Z

src/main/java/com/example/board_crud/crud/service/BoardService.java

+    }
+    //모든 글 가져오기
+    public List<BoardListResponseDto> findAllBoard(){
+        try{


try-catch 문을 사용하신 이유가 있나요??

코드 실행 중 발생할 예외 처리를 위해 사용하고 있습니다

try-catch 구문을 사용해서 예외처리를 하는 것보다 Optional를 사용해서 예외처리하는게 더 좋을 것 같아요

cd2e132
피드백 반영 했습니다.

예외처리 Optionl 사용해서 잘 해준거 같은데 전체 글을 불러오는 상황에서 예외처리가 필요해보이지 않아요 굳이 수정할 필요는 없지만 다음에 개발할 때 참고해주세요!

ta2ye0n · 2024-07-24T01:29:56Z

src/main/java/com/example/board_crud/crud/controller/BoardController.java

+    //전체 목록 조회
+    @GetMapping("/boards")
+    public List<BoardListResponseDto> getAllBoards(){
+        return boardService.findAllBoard();


컨트롤러에서 반환할때 ResponseEntity 사용해서 반환값과 상태 코드를 함께 반환하면 좋을거 같아요

ResponseEntity는 적절한 상태 코드와 응답 헤더 및 응답 본문을 생성하여 클라이언트에 전달하기 위해 사용합니다

c3af8d1
피드백 반영 했습니다

ResponseEntity를 사용해서 return 할때는 생성자보다 Builer 패턴을 사용해요

생성자보다 Builder 패턴을 사용하는것이 유지보수에 좋고 의미가 직관적임

ta2ye0n · 2024-07-24T01:34:09Z

src/main/java/com/example/board_crud/crud/Dto/BoardListResponseDto.java

+    private LocalDateTime createdAt;
+    private LocalDateTime modifiedAt;
+    //Entity -> dto
+    public BoardListResponseDto(Board board){


entity에서 dto 변환하는 메서드가 두개 있는데 두 메서드가 title를 가져오는 내용으로 같은 기능을 하는 메서드같은데 두개를 만드신 이유가 있나요??

사용되지 않는 메서드는 지우는게 좋을것같아요

177f1fb
피드백 반영 했습니다.

ta2ye0n

글 작성과 글 수정할 때 같은 dto를 사용하고 있는데 분리해주세요

ex) 글작성 -> BoardWriteRequestDto 글수정 -> BoardUpdateRequestDto (이건 예시입니다 참고해서 수정해주세요)

ta2ye0n · 2024-07-24T06:10:07Z

src/main/java/com/example/board_crud/crud/entity/Board.java

+import lombok.*;
+
+@NoArgsConstructor
+@Entity(name = "users")


board에 관련된 entity인데 users로 저장 하는건 맞지 않는 것 같아 보여요

6536bf5
피드백 반영 했습니다.

yusung103 · 2024-07-24T06:32:52Z

src/main/java/com/example/board_crud/crud/service/BoardService.java

+@AllArgsConstructor
+public class BoardService {
+
+    private final BoardRepository boardRepository;


@AllArgsConstructor를 사용해도 문제는 없으나 필드에 final이 붙어서 @AllArgsConstructor대신에 @RequiredArgsConstructor를 사용하는것이 더 적절합니다
생성자 어노테이션에 대해 알아보시면 도움이 될거 같습니다

피드백 반영 했습니다.
c8be7a6

ta2ye0n · 2024-07-24T10:39:42Z

src/main/resources/application.yml

@@ -1,8 +1,8 @@
 spring:
    datasource:
        url: jdbc:mysql://localhost:3307/mindwaytest?serverTimezone=Asia/Seoul&useSSL=false&allowPublicKeyRetrieval=true


url 부분도 환경 변수 처리 해주세요

환경변수 처리는 {$USERNAME} 이런식으로 하면 됩니다

오류 수정 했습니다.
247e5c8

url 부분이 환경변수 처리가 되지 않았어요 다시 확인하고 환경변수 처리해주세요

다시 확인했습니다. 죄송합니다.
ffba8af

피드백 반영했습니다 감사합니다!
0dacb5e

s-chan-o · 2024-07-24T13:45:46Z

글 작성과 글 수정할 때 같은 dto를 사용하고 있는데 분리해주세요

ex) 글작성 -> BoardWriteRequestDto 글수정 -> BoardUpdateRequestDto (이건 예시입니다 참고해서 수정해주세요)

820fb3a
피드백 반영 했습니다.

s-chan-o added 2 commits July 23, 2024 22:42

init

c2d0a41

init

f6d89c6

ta2ye0n reviewed Jul 24, 2024

View reviewed changes

yusung103 reviewed Jul 24, 2024

View reviewed changes

s-chan-o added 6 commits July 24, 2024 19:15

fix : "피드백 반영"

050af7d

fix : "AllArgsConstructor->RequiredArgsConstructor"

c8be7a6

fix : "생성자 주입을 자동화"

ae66a75

fix : "use RequestMapping"

007626e

fix : "use properties->yaml"

8abe05e

fix : "use Environment variables"

ecca6f6

ta2ye0n reviewed Jul 24, 2024

View reviewed changes

s-chan-o added 9 commits July 24, 2024 19:41

fix : "Error"

247e5c8

fix : "Table name"

6536bf5

fix : "use ResponseEntity for state"

c3af8d1

fix : "use url Environment variables"

ffba8af

fix : "Environment variables"

0dacb5e

feat : "use Optional for Exception"

cd2e132

feat : "delete never use method"

177f1fb

feat : "delete never use method"

2700866

feat : "BoardWriteRequestDto"

820fb3a

s-chan-o added 2 commits July 24, 2024 22:46

no changes

8b7b45d

fix : type error

6ca5484

		@@ -0,0 +1,6 @@

		spring.datasource.url=jdbc:mysql://localhost:3307/mindwaytest?serverTimezone=Asia/Seoul&useSSL=false&allowPublicKeyRetrieval=true

		@@ -0,0 +1,6 @@

		spring.datasource.url=jdbc:mysql://localhost:3307/mindwaytest?serverTimezone=Asia/Seoul&useSSL=false&allowPublicKeyRetrieval=true
		spring.datasource.username=root

4-Backend-seungchan #24

Are you sure you want to change the base?

4-Backend-seungchan #24

Uh oh!

Conversation

s-chan-o commented Jul 23, 2024

💡 개요

📃 작업내용

🔀 변경사항

🙋‍♂️ 질문사항

💀 어려웠던 점

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

ta2ye0n Jul 24, 2024 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

ta2ye0n left a comment

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

ta2ye0n Jul 24, 2024 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

s-chan-o commented Jul 24, 2024

Uh oh!

ta2ye0n Jul 24, 2024 •

edited

Loading

ta2ye0n Jul 24, 2024 •

edited

Loading