Comment, Member MVC 기능 추가(Member login logout제외)#4
Open
junhyung8795 wants to merge 67 commits intodevfrom
Open
Comment, Member MVC 기능 추가(Member login logout제외)#4junhyung8795 wants to merge 67 commits intodevfrom
junhyung8795 wants to merge 67 commits intodevfrom
Conversation
1. 개발 도중 Comment는 UserProblemSet과 연관이 깊어서 차후 개발 영역을 서로 재분배해야할 필요성이 있음. 2. 테스트를 위해서 MyBatisConfig를 추가하고 MapperScan 적 3. ProblemMapper, QuizRoomMapper, ScoreMapper수정 4. UserMapper를 삭제하고 MemeberMapper를 추
1.로그인과 로그아웃기능은 아직 구현 X + ID와 더불어 이메일도 중복불허용 제약을 걸어야할 것 같음. 2. DB에 memberId가 아닌 그냥 사람이 로그인할 때 사용하는 id가 "ID"로대문자로 저장돼있음. 이거 id로 고치는건 어떨지 의논해야함. 3. LoginRequest와 MemberUpdateRequest라는 DTO를 추가로 만들어 로그인할 때 id, password 수정할 때는 닉네임과 이메일은 바꿀 수 있다고 가정하고 구현. 4.deleteQuizRoom은 오류때문에 잠시 주석처리 5. Mapper.xml에서 공통적으로 mapper태그나 DML관련 태그들에서 오류가 뜸. 수정 필요.
Contributor
|
…blem-mvc-development' into comment
…r의 수정로직 PutMapping -> PathchMapping으로 수정
…을 PuMaping -> PatchMapping으로 수정
1. CommentServiceImpl에 userProlemSetId의 타입 ->long 2. Comment Controller 예외처리 추가
1. 전체적으로 url이 난잡해서 컨트롤러의 url주소가 프론트단에서 괜찮을지 협의해봐야함
2. 특히 유저가 문제를 제작하고 등록할 때, 세트도 등록하고 문제들도 문제만 따로 모아놓는 테이블이 있어서 만약 유저가 문제를 등록한다면
세트 등록-> 등록한 결과물로 set을 리스폰스바디(ApiResponse)에 넣고 이 set안에 문제세트 Id가 담겨있으니 이걸로 다시 /api/user-problems/sets/{userProblemSetId}로 Post요청을 보내야 문제들도 안전하게 user_problem테이블에 담기게 된다.
1. 이제 회원가입 후에 user_score테이블에도 회원의 member_id가 올라감 2. 방장이 퀴즈방을 삭제하면 같은 퀴즈방에 있던 사람들도 quiz_room_member에서 삭제돼서 나감 처리가 됨. 3. problem 조회할때 전체 문제들 개수보다 쿼리문에 있는 limit가 크면 오류가 나서 일시적으로 limit을 받는 코드들을 주석처리하거나 삭제함.
1. memberId로 유저제작문제가 여러개 조회 가능하기에 UserProblemSetDao의 selectUserProblemSetByMemberId메서드의 반환값을 List<>로 변경하여 다중행을 받을 수 있게 변경 2. user_score테이블에 같은 memberId를 가지는 여러 행이 존재할 수 있어 user_score.memberId의 제약조건으로 UNIQUE를 추가 3. 유저제작 문제세트 추가-> 유저제작문제 추가 로 이어지는 로직 사이에 userProblemSetId가 브라우저로 전달되지 않는 문제가 있었음. -> UserProblemSetMapper.xml의 insertUserProblemSet 부분에 useGenerateKey와 keyProperty 속성을 추가하여 해결.
…into comment # Conflicts: # codeBoy_backend/src/main/resources/db/migration/DDL_schema.sql
1. UserProblemMapper.xml의 인서트문에 칼럼명에 category, comment_count도 들어가게끔 돼있어서 이를 제거함. 2. DDL이 원래 설계대로 돼있지 않아서 다시 수정
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
✨Comment Dao, Controller, Service, ServiceImpl, CommentMapper.xml 개발
✨Member DAO, DTO, Service, ServiceImpl, Controller, MemberMapper개발
1.로그인과 로그아웃기능은 아직 구현 X + ID와 더불어 이메일도 중복불허용 제약을 걸어야할 것 같음.
2. DB에 memberId가 아닌 그냥 사람이 로그인할 때 사용하는
id가 "ID"로대문자로 저장돼있음. 이거 id로 고치는건 어떨지 의논해야함.
3. LoginRequest와 MemberUpdateRequest라는 DTO를 추가로 만들어 로그인할 때 id, password
수정할 때는 닉네임과 이메일은 바꿀 수 있다고 가정하고 구현.
4.deleteQuizRoom은 오류때문에 잠시 주석처리
5. Mapper.xml에서 공통적으로 mapper태그나 DML관련 태그들에서 오류가 뜸. 수정 필요.
closes #2