Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

[예지니어스] 지하철 정보 관리 - 인수테스트 미션 제출합니다. #48

Merged
merged 45 commits into from
May 23, 2020
Merged

[예지니어스] 지하철 정보 관리 - 인수테스트 미션 제출합니다. #48

merged 45 commits into from
May 23, 2020

Conversation

YejiAhn
Copy link

@YejiAhn YejiAhn commented May 9, 2020

안녕하세요 :)
지하철 정보 관리 - 인수테스트 미션 제출합니다.

YejiAhn added 21 commits May 6, 2020 17:43
@YejiAhn
feature: 지하철 노선 API CRUD 기능 구현 완료
4825ecb
@YejiAhn
feature: 프론트엔드와 연결과 기능 목록 작성
8757414
@YejiAhn
feature: 경로 설정
f7b0dc1
@YejiAhn
feature: 지하철 노선 추가 및 조회 API 사용
0cd7d7e
@YejiAhn
feature: 지하철 노선 삭제 API 사용
1289814
@YejiAhn
feature: 지하철 노선 조회 API 생성 및 사용
b5b17a1
@YejiAhn
feature: Update 불완전 구현
6809f05
@YejiAhn
feature: Update 구현
24e6e03
@YejiAhn
feature: 지하철 노선에 역 추가와 역 제거 및 역 조회 기능 구현
90df513
@YejiAhn
feature: 인수테스트(LineStationAcceptanceTest) 진행 중
ca0535d
@YejiAhn
feature: 인수테스트(LineStationAcceptanceTest) 완성
9e6b243
@YejiAhn
feature: 지하철 추가 버그 수정
c97a19a
@YejiAhn
docs: 5단계 요구사항 README.md에 추가
c707d50
@YejiAhn
feature: 지하철 구간관리 페이지
810336b
@YejiAhn
feature: 추가된 노선 띄우는 기능 미완성 구현
fdd952b
@YejiAhn
refactor: async와 await 추가로 구간관리 페이지에서 노선 확인 가능하게 수정
dcb9bc3
@YejiAhn
refactor: station api 연결
236411b
@YejiAhn
feature: Line Station DB 연결
9936c87
@YejiAhn
feature: 구간 추가 및 삭제 구현
dd6cc73
@YejiAhn
docs: 요구 기능 구현 상황 업데이트
1cbd138
@YejiAhn
refactor: 주석 및 콘솔메세지 코드 삭제
716790a
kang-hyungu
kang-hyungu requested changes May 12, 2020
View reviewed changes
Copy link

@kang-hyungu kang-hyungu left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

안녕하세요 예지니어스, 리뷰어 구구입니다
미션 구현 잘 하셨어요 👏
아직 프론트, 테스트 코드는 피드백 남기지 않았어요
피드백 확인하시고 수정하셔서 리뷰 재요청 주시면 관련 부분도 피드백 남길게요
궁금한 점 생기면 코멘트 남기거나 dm 주세요

build.gradle Show resolved Hide resolved
src/main/java/wooteco/subway/admin/controller/LineController.java Outdated Show resolved Hide resolved
src/main/java/wooteco/subway/admin/controller/LineController.java Outdated Show resolved Hide resolved
src/main/java/wooteco/subway/admin/controller/LineController.java Outdated Show resolved Hide resolved
src/main/java/wooteco/subway/admin/controller/LineController.java Outdated Show resolved Hide resolved
src/main/java/wooteco/subway/admin/service/LineService.java Show resolved Hide resolved
src/main/java/wooteco/subway/admin/service/LineService.java Outdated Show resolved Hide resolved
src/main/java/wooteco/subway/admin/service/LineService.java Outdated Show resolved Hide resolved
src/main/java/wooteco/subway/admin/service/LineService.java Outdated Show resolved Hide resolved
src/main/java/wooteco/subway/admin/service/LineService.java Outdated Show resolved Hide resolved
YejiAhn added 13 commits May 12, 2020 22:37
@YejiAhn
refactor: LineStation이 출발역인지 아닌지 확인하는 메서드 추가
06cefaf
@YejiAhn
refactor: service 구성하는 필수 멤버에 final 변수 추가
c5d27dc
@YejiAhn
refactor: CUD 메서드에 @transactional 어노테이션 추가
b95d7eb
@YejiAhn
refactor: toLineStation static 메서드에서 일반 메서드로 변경
a7657cd
@YejiAhn
refactor: Line 안 쓰이는 생성자 삭제
f82054f
@YejiAhn
refactor: 스트림에 get() 대신 orElseThrow() 사용
a84dffd
@YejiAhn
refactor: StationResponse DTO에 안 쓰이는 메서드 삭제
21258b6
@YejiAhn
test: 테스트 관련 로직들 보완
6b529c0
@YejiAhn
refactor: 중복되는 부분 메서드로 추출
033b9c8
@YejiAhn
refactor: api에 도메인 대신 Dto 노출
5f5a919
@YejiAhn
refactor: bgColor 필드명 backgroundColor로 변경
e9747a8
@YejiAhn
refactor: Line 클래스 메서드 분리 및 depth 줄이는 리팩터링
1843f79
@YejiAhn
test: 테스트 코드 보완
3b20f2c
kang-hyungu
kang-hyungu requested changes May 15, 2020
View reviewed changes
Copy link

@kang-hyungu kang-hyungu left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

예지니어스, 피드백 반영 잘 하셨어요 👏
간단하게 몇 가지 피드백 남겼어요
수정해서 요청 주시면 반영할게요!
궁금한 점 생기면 코멘트 남기거나 dm 주세요

src/main/java/wooteco/subway/admin/controller/mock/MockController.java Outdated Show resolved Hide resolved
src/main/java/wooteco/subway/admin/dto/StationResponse.java Outdated Show resolved Hide resolved
src/main/java/wooteco/subway/admin/service/LineService.java Outdated Show resolved Hide resolved
src/main/resources/schema.sql Outdated Show resolved Hide resolved
src/main/resources/static/js/views/AdminLine.js Outdated Show resolved Hide resolved
src/main/resources/static/js/views/AdminLine.js Outdated Show resolved Hide resolved
src/main/resources/static/js/views/AdminLine.js Outdated Show resolved Hide resolved
src/test/java/wooteco/subway/admin/domain/LineTest.java Show resolved Hide resolved
kang-hyungu
kang-hyungu requested changes May 19, 2020
View reviewed changes
Copy link

@kang-hyungu kang-hyungu left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

예지니어스, 피드백 반영한 부분 확인했어요
테스트 코드도 깔끔하게 잘 추가하셨어요 👍
pr은 머지 처리하겠습니다. 구현하느라 수고하셨어요

src/main/java/wooteco/subway/admin/domain/Line.java Outdated Show resolved Hide resolved
src/main/java/wooteco/subway/admin/domain/Line.java
private LocalTime startTime;
private LocalTime endTime;
private int intervalTime;
private Set<LineStation> stations;
private Set<LineStation> stations = new HashSet<>();
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

stations 관련 로직들이 많네요
일급 컬렉션으로 추출하면 중복 코드를 제거하고 좀더 코드 복잡도를 줄일 수 있을거에요

@woowahan-pjs woowahan-pjs merged commit 12ae111 into woowacourse:yejiahn May 23, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@kang-hyungu kang-hyungu kang-hyungu requested changes

@woowahan-pjs woowahan-pjs woowahan-pjs approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@YejiAhn @kang-hyungu @woowahan-pjs