[team-03][BackEnd] todo-list 1주차 #1 #40

nohriter · 2022-04-06T08:09:38Z

안녕하세요 3조의 백엔드 개발을 맡은 짱민과 노리입니다. 😁

테이블 설계와 api 설계에 대한 고민 EC2 작업이 주로 진행되어 진행사항이 많지 않은점 양해 부탁드립니다! 🙇

📝 진행

API명세서
데이터 모델링을 진행하였습니다. card, move_record 로 구분하였습니다.

EC2 서버에 MySQL 을 설치하여 커넥션 테스트까지 진행하였습니다.
Mock 컨트롤러를 이용하여 dummy data를 넣고, CardList를 JSON 형식으로 반환하도록 구현하였습니다.

리뷰 감사합니다. 😁

- Java 11 + Spring Boot(2.6.6) + Gradle + Spring Web + Lombok + Jdbc + mysqlDriver - 불필요한 파일 삭제(help.md) - gitignore 윈도우, 맥, 자바, 인텔리제이, 그레이들 추가

- 응답을 위한 CardsResponse DTO 생성

ksundong · 2022-04-06T12:03:30Z

android/README.md

@@ -0,0 +1 @@
+# Android


포함되면 안되는 파일이 포함된 것 같아요. 제거하고 다시 리뷰요청해주세요~~

안녕하세요 디온! PR전에 더 꼼꼼히 확인을 했어야 했는데 미흡했던점 죄송합니다!!
안드로이드 폴더 삭제 완료하여 재 요청 드렸습니다!! 감사합니다!!

ksundong

안녕하세요! 노리 & 짱민!! 리뷰어 dion입니다.
첫번째 협업 프로젝트네요. 즐겁게 진행해보시면 좋을 것 같습니다.
특별히 리뷰 드릴 내용은 없고, Lombok만 의존성에서 제거해주세요!
고생하셨습니다~

ksundong · 2022-04-06T14:40:25Z

backend/todo-app/src/main/java/team03/todoapp/controller/CardsResponse.java

+
+import lombok.Getter;
+
+@Getter


저는 가급적이면 롬복 사용을 추천드리지 않습니다.
롬복 의존성은 제거부탁드려요~

학습하는 단계에서 내부동작 방식을 경험하기위해서 롬복사용을 지양한다고 이해하면 될까요~?
아니면 롬복자체를 디온께서는 지양하시는 편이실까요?

저는 실무에서도 가급적이면 롬복을 지양합니다.
최근엔 코틀린을 사용해서 언어차원에서 지원해주니 복합적인 생각이 들긴합니다.

그와는 별개로, 제가 롬복 사용을 학습단계에서 추천드리지 않는 이유는 다음과 같습니다.

롬복을 적용할 만큼 코드가 복잡한지 자문해보시면 좋습니다. 보일러 플레이트 코드를 작성하는데 너무 많은 시간이 든다면 그 때 제한적으로 도입해도 늦지않습니다.

제가 생각하기엔 아직까지는 전혀 필요가 없어보이네요.

롬복은 설계를 숨깁니다. 보기엔 좋아보이지만, delombok을 해보면 엄청 많은양의 코드가 만들어져 있는 경우가 많습니다.

보통 대다수 코드는 이렇게 해서 코드스멜이 생깁니다.

롬복은 롬복 자체를 이해하지 않으면, 내가 코드를 변경했을 때 생기는 여파를 파악하기 어렵습니다.(변경에 취약합니다.)

그래서 정말 복잡한 프로젝트가 아닌이상 추천드리진 않아요.

ksundong · 2022-04-06T14:41:38Z

backend/todo-app/src/main/java/team03/todoapp/controller/CardsResponse.java

+import lombok.Getter;
+
+@Getter
+public class CardsResponse {


이름을 봤을 땐 Card 목록일 것 같은데 아니네요. 네이밍이 조금 아쉽습니다.

CardResponse 가 좋을것 같습니다 ! 😅

ksundong · 2022-04-06T14:43:01Z

backend/todo-app/src/main/java/team03/todoapp/controller/MockController.java

+public class MockController {
+
+    @GetMapping("/cards")
+    public Map<String, List<CardsResponse>> getCards() {


Map으로 응답객체를 사용하는 것은 지양해주세요.
DTO를 만들어 반환시켜주세요.
이런건 알려드린 적이 없는 것 같은데...!!

DTO를 통해서 응답할 수 있도록 하겠습니다!

ksundong · 2022-04-06T14:44:13Z

backend/todo-app/src/main/java/team03/todoapp/TodoAppApplication.java

+	public static void main(String[] args) {
+		SpringApplication.run(TodoAppApplication.class, args);
+	}


여기는 들여쓰기가 Tab으로 되어있는데요, 가급적이면 프로젝트 전반적으로 일치시켜주시면 좋을 것 같습니다.

들여쓰기를 Tab으로 일치시켜서 진행하겠습니다!

ksundong · 2022-04-06T14:44:52Z

backend/todo-app/build.gradle

+dependencies {
+//	implementation 'org.springframework.boot:spring-boot-starter-jdbc'
+	implementation 'org.springframework.boot:spring-boot-starter-web'
+	compileOnly 'org.projectlombok:lombok'


롬복 의존성은 빼주세요!

롬복의존성 제거해서 진행하도록 하겠습니다!

[BE] API Server - 전체 History를 조회하도록 기능 구현

…반환하도록 변경 (#40)

[iOS-bibi] NewCardView constraint 지정 및 NewCardViewController 생성

* feat: [#33] Header Template 추가 * feat: [#33] HeaderModel menu 상태 관련 메서드 추가 - menuStatus update 메서드 - menuStatus get 메서드 * feat: [#33] HeaderView render 기능 추가 * feat: [#33] HeaderView 이벤트 등록 메서드 추가 * feat: [#33] HeaderModel, HeaderView 생성하는 class Header 추가 * feat: [#33] menuBtn 클릭핸들러_menuStatus update하기 * feat: [#40] History Template 추가 * feat: [#40] HistoryView render 기능 추가 * feat: [#43] History closeBtn 이벤트 등록 * feat: [#43] History animation 기능 추가 * feat: [#33][#43] menuBtn, closeBtn 클릭핸들러, 히스토리 애니메이션 호출 * feat: [#43] HistoryView 생성하는 class History 추가 * feat: Controller, Header, History 인스턴스 생성 * fix: 기존 body 내부 요소 주석처리 * feat: [#34] alert창 기본 동작 구현 * refactor: utils폴더구조 수정 * fix: 인스턴스생성할 필요 없는 클래스 객체로 수정 - 수정하면서 생기는 render 문제 해결 * refactor: [#34] 템플릿리터럴 코드 분리 * fix: git 충돌 해결 * fix: [#36] 카드 write모드 스타일 수정 * feat: [#35] 컬럼 및 카드 템플릿 추가 * feat: [#36] 카드추가 버튼 클릭 후 카드박스 노출 기능 추가 * feat: [#36] 카드추가 기능 * feat: [#36] 카드추가 취소 기능 추가 * feat: [#39] 드래그앤 드롭 도착 잔상 및 맨끝에 추가 제외한 기본 기능 구현 * chore: 드래그기능과 카드추가기능 이벤트 충돌로 인한 주석 * feat: [#37] 카드수정 기능 추가 - 카드 write 모드 스타일 추가 - 수정기능 구현하면서 생기는 오류들 해결 * chore: 지우지 못한 주석 삭제 * feat: [#38] 카드 삭제 기능 구현 * fix: [#37] cardView의 remove 메서드 사용 * feat: [#38] 카드삭제 버튼 mouseover, mouseout이벤트 추가 Co-authored-by: khj <[email protected]> Co-authored-by: HongJungKim <[email protected]>

* Feat: 카드 등록 #21 (#35) * Design: 버튼 컴포넌트에서 focus도 hover와 같은 효과 주도록 변경 * Design: todo-item 컴포넌트의 border 기본값을 지정. border 기본값 : 1px solid transparent 마우스 이벤트에 따라 border가 추가되는 과정에서 레이아웃이 조금 변하는 현상을 없애기 위해 이벤트에 따라 border-color만 바꿔줌 * Refactor: db생성하는 함수 getDb로 분리 * Feat: 카드 삭제 버튼 하이라이트 추가 mouseover시 하이라이트 * Refactor: 카드 등록 로직 개선 * Feat: modal에 target element 설정 modal을 여는 경우 원인이 된 element 설정. modal을 닫는 경우 element값 null로 초기화 * Chore: 배포용 패키지 및 스크립트 추가 #36 (#37) * Feat: index.html의 css link를 main.scss의 url로 추가 * Chore: webpack 설정 변경, 패키지 추가 html-webpack-plugin, gh-pages패키지 추가 package.json에 배포용 스크립트 추가 * Fix: TodoColumns, TodoColumn init 메서드 변경 (#38) * Chore: dependency 이동 * Fix: TodoColumns, TodoColumn init 메서드 변경 forEach -> map으로 변경 * Design: drag and drop 위한 스타일 적용 (#40)

* feat: [#33] Header Template 추가 * feat: [#33] HeaderModel menu 상태 관련 메서드 추가 - menuStatus update 메서드 - menuStatus get 메서드 * feat: [#33] HeaderView render 기능 추가 * feat: [#33] HeaderView 이벤트 등록 메서드 추가 * feat: [#33] HeaderModel, HeaderView 생성하는 class Header 추가 * feat: [#33] menuBtn 클릭핸들러_menuStatus update하기 * feat: [#40] History Template 추가 * feat: [#40] HistoryView render 기능 추가 * feat: [#43] History closeBtn 이벤트 등록 * feat: [#43] History animation 기능 추가 * feat: [#33][#43] menuBtn, closeBtn 클릭핸들러, 히스토리 애니메이션 호출 * feat: [#43] HistoryView 생성하는 class History 추가 * feat: Controller, Header, History 인스턴스 생성 * fix: 기존 body 내부 요소 주석처리 * feat: [#34] alert창 기본 동작 구현 * refactor: utils폴더구조 수정 * fix: 인스턴스생성할 필요 없는 클래스 객체로 수정 - 수정하면서 생기는 render 문제 해결 * refactor: [#34] 템플릿리터럴 코드 분리 * fix: git 충돌 해결 * fix: [#36] 카드 write모드 스타일 수정 * feat: [#35] 컬럼 및 카드 템플릿 추가 * feat: [#36] 카드추가 버튼 클릭 후 카드박스 노출 기능 추가 * feat: [#36] 카드추가 기능 * feat: [#36] 카드추가 취소 기능 추가 * feat: [#39] 드래그앤 드롭 도착 잔상 및 맨끝에 추가 제외한 기본 기능 구현 * chore: 드래그기능과 카드추가기능 이벤트 충돌로 인한 주석 * feat: [#37] 카드수정 기능 추가 - 카드 write 모드 스타일 추가 - 수정기능 구현하면서 생기는 오류들 해결 * chore: 지우지 못한 주석 삭제 * feat: [#38] 카드 삭제 기능 구현 * fix: [#37] cardView의 remove 메서드 사용 * feat: [#38] 카드삭제 버튼 mouseover, mouseout이벤트 추가 * feat: [#78] 초기 컬럼, 카드 서버요청 및 렌더 * refactor: 컬럼, 카드 id data속성으로 변경 * fix: Header updateStatus 수정 (리뷰 반영) * fix: column querySelector 수정 * fix: [#78] cardlist 렌더 li.start 추가 * feat: [#74] 카드 추가 요청 구현 * feat: [#74] 카드 삭제 요청 구현 * feat: [#74] 카드 수정 요청 구현 * chore: 불필요한 코드 삭제 * chore: 불필요한 주석 제거 * feat: [#78] 초기 히스토리렌더 기능 추가 * fix: [#78] 히스토리 fetch요청 reqest 함수 import하는 방식으로 변경 * feat: [#78] 히스토리 볼때 시간 업데이트 기능 추가 * fix: [#78] 컬럼 fetch요청 request import하여 사용으로 수정 * chore: [#78] 히스토리 내용 을(를) 수정 * feat: [#86] 카드 추가, 수정, 삭제시 히스토리에 반영 * chore: 주석 삭제 Co-authored-by: khj <[email protected]> Co-authored-by: HongJungKim <[email protected]>

* feat : 테이블 인덱스 컬럼 추가 closes #15 * refactor: CardController 응답시 List 보내줄 수 있도록 수정 - sample data에 cardIndex가 status별로 0번만 있게 수정 closes #36 * refactor: 구글 코딩 컨벤션을 사용하여 코드 포매팅 설정하기 - 불필요한 임포트, 컨벤션 삭제 * refactor: controller 메소드이름 변경 * refactor: queryParameter를 사용 * refactor: addValues 메소드 사용 * refactor: cardRepository변경 - rowMapper를 재사용 - 요청에 사용되는 dto와 응답에 사용되는 dto를 분리 - repository 레이어에서는 도메인 객체사용하도록 변경 * refactor: jdbcTemplate에서 object를 반환하는 메소드 사용하도록 변경 * refactor: creatTime메소드 삭제 closes #38 * refactor: MYSQL 사용을 위한 schema.sql 수정 closes #40 * Bug: 배포 뒤 서버 오류 해결 * feat: 서버 cors에러 해결 Co-authored-by: leekm0310 <[email protected]>

드래그 기능 구현

leezzangmin and others added 5 commits April 4, 2022 17:51

Update README.md

e0f0b93

Build : Android , Backend 폴더 생성

15a07e4

Build : 백엔드 브랜치 생성

6d1153e

Build: 백엔드 프로젝트 초기 설정

966c627

- Java 11 + Spring Boot(2.6.6) + Gradle + Spring Web + Lombok + Jdbc + mysqlDriver - 불필요한 파일 삭제(help.md) - gitignore 윈도우, 맥, 자바, 인텔리제이, 그레이들 추가

feat: Mock 카드조회 기능 추가

984cbef

- 응답을 위한 CardsResponse DTO 생성

nohriter added the review-BE Improvements or additions to documentation label Apr 6, 2022

nohriter requested a review from ksundong April 6, 2022 08:09

nohriter changed the title ~~[team-03][BE] todo-list 1주차 #1~~ [team-03][BackEnd] todo-list 1주차 #1 Apr 6, 2022

ksundong reviewed Apr 6, 2022

View reviewed changes

Remove: 안드로이드 관련 파일 제거

9b7d99b

leezzangmin requested a review from ksundong April 6, 2022 13:31

ksundong approved these changes Apr 6, 2022

View reviewed changes

ksundong merged commit d418f91 into codesquad-members-2022:team-03 Apr 6, 2022

ku-kim pushed a commit that referenced this pull request Apr 12, 2022

Merge pull request #40 from ku-kim/feature/BE/GET_api_histories

b720734

[BE] API Server - 전체 History를 조회하도록 기능 구현

Dae-Hwa pushed a commit that referenced this pull request Apr 13, 2022

feat: default userId를 Cookie로 전달해주는 컨트롤러 생성 (#40)

02494c1

Dae-Hwa pushed a commit that referenced this pull request Apr 13, 2022

feat: HttpServletRequest에서 쿠키를 통해 userId를 가져오도록 적용 & Map을 Dto로 캡슐화하여 …

b1cd5d3

…반환하도록 변경 (#40)

Dae-Hwa pushed a commit that referenced this pull request Apr 13, 2022

refactor: update CASCADE 제거 (#40)

99b3ad4

GangWoon pushed a commit that referenced this pull request Apr 14, 2022

Merge pull request #40 from zbqmgldjfh/feature-addCardPopup

e30d3ac

[iOS-bibi] NewCardView constraint 지정 및 NewCardViewController 생성

hwicode pushed a commit that referenced this pull request Apr 15, 2022

feat: drag-drop-swipe-#40

50a3d75

snowjang24 pushed a commit that referenced this pull request Apr 18, 2022

Merge pull request #40 from ldldz/main

4d91442

드래그 기능 구현

		@@ -0,0 +1 @@
		# Android No newline at end of file


		import lombok.Getter;

		@Getter

[team-03][BackEnd] todo-list 1주차 #1 #40

[team-03][BackEnd] todo-list 1주차 #1 #40

Uh oh!

Conversation

nohriter commented Apr 6, 2022 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

📝 진행

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

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

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!

4 participants

nohriter commented Apr 6, 2022 •

edited

Loading