Feature/#26 게시판기능구현 by imchanyang · Pull Request #49 · Battle-s/battle-school-backend

imchanyang · 2022-08-23T07:40:56Z

개요 ✋

전체/학교 게시판 작성 기능 구현

작업사항 🗒️

AccessToken으로 유저 정보를 어떻게 가져올까요?
Post테이블에 UserId를 넣어야합니다.

변경로직 🛠️

내용을 적어주세요.

yejipractice

진짜 고생 많으셨습니다 늦은 리뷰 죄송해요 ㅠㅠ 어려우셨을텐데 잘 작성하셨네요 특히 서비스는 아주 굿굿 : )

src/main/java/org/battles/battles/post/Dto/PostsSaveRequestDto.java

yejipractice · 2022-08-24T12:58:26Z

src/main/java/org/battles/battles/post/Post.java

-    @Column(nullable = true)
-    private Long schoolId;
+    @ManyToOne
+    @JoinColumn(name = "schoolId", nullable = true)


매핑 잘 하셨는데 지금 언뜻 서치해보니까 fk null이면 에러가 난다는.... 설계를 다시 해봐야겠네요 ㅠㅠㅠ

fk null 허용되지 않나요? JPA라서 그런가요??

jpa라서 그런건 아니고.. 아 fk가 pk 동시일때 null허용은 안되는구나 원래 db상으로는... 음 고민이되네요 null이 좋은 방식은 아닌 것 같긴 해요 ㅠㅠ 고민고민

음 더 알아보겠습니다. @yejipractice

Post의 schooId 컬럼에 값을 전부(전체/학교) 다 넣어준다.

Post의 schoolId 컬럼을 빼고 필요할 때 마다 User를 통해 조회한다.

정도 고민이 있는 것 같습니다.

개발하고 있는데 schoolID가 NULL이 되면 코드가 많아지기도 했습니다..

둘의 테이블을 나눠버린다?

src/main/java/org/battles/battles/post/PostController.java

src/main/java/org/battles/battles/post/PostService.java

yejipractice

고생하셨어요 코코 자꾸 수정 사항만 말씀 드리는 것 같아 죄송해요 ㅠㅠ 그래도 코코 코드가 점점 좋아보여요~! 고생하셨습니다 👍 💯

yejipractice · 2022-08-29T11:51:04Z

src/main/java/org/battles/battles/post/PostService.java

@@ -46,6 +51,7 @@ public PostsResponseDto saveEntire(String email, PostsSaveRequestDto requestDto)
    @Transactional
    public PostsResponseDto saveSchool(String email, PostsSaveRequestDto requestDto, Long schoolId) {


제가 말씀 드린 건 all entire 보다는 위에 saveSchool 보시면 학교를 저장한다는 의미로 보이거든요? 로직은 학교 게시글 저장인데? 그런 점에서 메소드 명이 조금 수정이 필요해보인다라고 말씀드린거였습니다! saveAll도 보시면 모두 저장한다는 걸로 해석되잖아요? 사실 전체 게시판에 글을 저장하는 건데.. 이 부분에 대해 어떻게 생각하실까요?? 고민한 흔적이 보여서 고생하셨어요ㅠㅠ!

yejipractice · 2022-08-29T11:51:45Z

src/main/java/org/battles/battles/post/PostService.java

+            throw new CPostNotFoundException();
+        }
+
+        System.out.println(new PostsResponseDto(post.get()));


코드 올릴 때에는 println은 빼주셔야 합니다~!

yejipractice · 2022-08-29T11:53:18Z

src/main/java/org/battles/battles/post/PostService.java

+
+        List<PostsResponseDto> postsResponseDtos = new ArrayList<>();
+
+        for (int i = 0; i < posts.size(); i++) {


이 부분은 .. 음 사실 문법적으로 맞지만 다르게 거의 쓰시거든요? 자바 공부해보시고 싶으시면 stream.map 한번 적용해보는 건 어떨까요? 해보시고 궁금한 점 있으시면 질문 주세용!!

yejipractice · 2022-08-29T11:55:52Z

src/main/java/org/battles/battles/post/Dto/PostsResponseDto.java

+
+    private String schoolName;
+
+    private String title;


코드 좋습니다!! 다만 api문서를 그렇게 작성했던 것 같긴 한데 순간 response로 모두 보내줘야 할까 싶은데 .. 어떻게 생각하세요? 모두 보내줘야겠다 싶으시면 이부분은 수정안하셔도 됩니다!

yejipractice · 2022-08-29T11:56:52Z

src/main/java/org/battles/battles/post/Post.java

@@ -45,9 +48,9 @@ public class Post extends TimeStamped {
    private Status status;

    @Builder


빌더는 빼주세요~!

imchanyang added 2 commits August 22, 2022 21:19

[feature] #26 게시판기능구현

e91803c

[feature] #26 전체/학교 게시글 작성 기능 구현

458542e

imchanyang added review 🙏 리뷰 요청 feature 💻 기능 추가 backend 🖥️ labels Aug 23, 2022

imchanyang requested a review from yejipractice August 23, 2022 07:40

imchanyang self-assigned this Aug 23, 2022

imchanyang added 2 commits August 23, 2022 16:44

[feature] #26 전체/학교 게시글 작성 기능 구현

95cf417

[feature] #26 전체/학교 게시글 작성 기능 구현 완료

bb014ab

yejipractice requested changes Aug 24, 2022

View reviewed changes

yejipractice linked an issue Aug 25, 2022 that may be closed by this pull request

[FEAT] 게시판 기능 구현 #26

Open

5 tasks

imchanyang added 4 commits August 28, 2022 17:15

[feature] #26 전체/학교 게시글 작성 기능 구현 완료

ae5fc0d

[feature] #26 전체/학교 게시글 작성 기능 구현 완료

22806c6

[feature] #26 전체/학교 게시글 작성 기능 구현 완료

41160b4

[feature] #26 전체/학교 게시글 조회

e65aa8d

yejipractice requested changes Aug 29, 2022

View reviewed changes

yejipractice reviewed Aug 29, 2022

View reviewed changes

		@@ -46,6 +51,7 @@ public PostsResponseDto saveEntire(String email, PostsSaveRequestDto requestDto)
		@Transactional
		public PostsResponseDto saveSchool(String email, PostsSaveRequestDto requestDto, Long schoolId) {


		List<PostsResponseDto> postsResponseDtos = new ArrayList<>();

		for (int i = 0; i < posts.size(); i++) {

		@@ -45,9 +48,9 @@ public class Post extends TimeStamped {
		private Status status;

		@Builder

Conversation

imchanyang commented Aug 23, 2022

개요 ✋

작업사항 🗒️

변경로직 🛠️

Uh oh!

yejipractice left a comment

Choose a reason for hiding this comment

Uh oh!

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!

imchanyang Aug 28, 2022 • 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!

Uh oh!

Uh oh!

Uh oh!

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

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

imchanyang Aug 28, 2022 •

edited

Loading