4 Backend chanul by chanul0817 · Pull Request #25 · Team-MindWay/MindWay-8th-Assignment

chanul0817 · 2024-07-23T14:21:35Z

💡 개요

마인드웨이 과제 수행

📃 작업내용

CRUD기능을 가진 Board구현

🔀 변경사항

🙋‍♂️ 질문사항

💀 어려웠던 점

ta2ye0n · 2024-07-24T00:31:59Z

+import java.util.List;
+
+@Service
+@AllArgsConstructor


@AllArgsConstructor는 모든 필드 값을 파라미터로 받는 생성자를 생성 어노테이션이라 서비스 로직에서 사용하기에 적합하지 않아보여요

ta2ye0n · 2024-07-24T00:38:50Z

+    }
+
+    @PutMapping("/{id}")
+    public Long updateBoard(@PathVariable Long id, @RequestBody BoardRequestDto requestDto) {


board를 수정하는 메서드 인데 반환하는 값이 Long인 이유가 있나요?

ta2ye0n · 2024-07-24T00:43:44Z

+public class BoardController {
+    private final BoardService boardService;
+
+    public BoardController(BoardService boardService) {


생성자를 만들어서 BoardService를 사용하는 것이 아닌 @RequiredArgsConstructor를 통한 의존성 주입을 통해 boardService를 사용하는게 더 좋을거 같아요

ta2ye0n · 2024-07-24T00:51:42Z

+@Entity(name = "boards") // DB 테이블 이름을 "boards"로 설정
+public class Board {
+    // 글 고유 아이디
+    @GeneratedValue(strategy = GenerationType.AUTO)


현재 기본키 생성 전략을 GenerationType.AUTO로 해두어 하이버네이트가 자동으로 전략을 선택하게끔 됬는데 DB 버전에 따라 선택되는 전략이 달라질 수 있으므로 사용하는 DBMS에 맞는 전략을 사용하시면 좋을거 같아요

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

+    @Transactional
+    public Long deleteBoard(Long id) {
+        boardRepository.deleteById(id);
+        return id;


삭제하는 메서드에서 삭제한 board의 id를 다시 반환하는 이유가 있나요?

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

+
+    @Transactional
+    public Long deleteBoard(Long id) {
+        boardRepository.deleteById(id);


받아온 id를 바로 삭제하는게 아니라 이 board의 id가 있는지 확인하는 로직이 있으면 좋을 것 같아요

ta2ye0n · 2024-07-24T01:08:49Z

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


많은 프로젝트에서 application.properties 파일보다 application.yml파일을 더 많이 사용해요
그래서 yml 파일 사용해보는것도 좋을거 같아요

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

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


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

ta2ye0n · 2024-07-24T01:13:55Z

+    }
+
+    @PostMapping("/")
+    public BoardResponseDto createBoard(@RequestBody BoardRequestDto requestDto) {


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

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

yusung103 · 2024-07-24T05:38:10Z

+        return boardService.findOneBoard(id);
+    }
+
+    @PutMapping("/{id}")


@PutMapping보다 @PatchMapping이 나아보입니다.
기본적인 http 메서드에 대해 알아보시는걸 추천드려요

yusung103 · 2024-07-24T05:42:43Z

+
+@Repository
+public interface BoardRepository extends JpaRepository<Board, Long> {
+    List<BoardListResponseDto> findAllByOrderByModifiedAtDesc();


이 메서드 사용하지 않고 있는데 넣으신 이유가 있나요?

로컬에서 실행했을 때 ModifiedAt이라는 필드가 없지만 여기서 ModifiedAt이라는 필드를 기준으로 정렬하려해서 오류가 터져 실행이 되지 않습니다

ta2ye0n

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

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

init

66baa4c

ta2ye0n reviewed Jul 24, 2024

View reviewed changes

yusung103 reviewed Jul 24, 2024

View reviewed changes

ta2ye0n reviewed Jul 24, 2024

View reviewed changes

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

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

Uh oh!

Conversation

chanul0817 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!

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 left a comment • 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!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

ta2ye0n left a comment •

edited

Loading