Step4 carracing #5541

rkaehdaos · 2024-04-17T14:35:24Z

No description provided.

- 이전 study test depreacated 주석처리

- pmd로 ncss 15line 강제화 - jdk21 apply - 기존 에 * 에러 정의 차이 리플렉션을 사용하여 생성자를 호출하면, 생성자 내부에서 발생하는 예외는 InvocationTargetException에 감싸져서 발생합니다. 이 때문에 예외가 직접적으로 UnsupportedOperationException가 아니라 InvocationTargetException으로 포착되는 것입니다. * 최적화 * Car 상수 사용 및 보완 * formatting * 가독성 * workflow 수정 * workflow 수정 * pmdtest4에 step4 넣음 * workflow에 pmd summary 추가 * 15라인 rule은 main 코드에만 ㅋㅋ * Extract PMD info and update step summary * test dependon pmdmain * pmd 실패여도 진행 * xmllint 사용 제거 * 메시지 추출 부분 수정 * 일단 원복 * awk로 개선된 스크립트 설명 파일 처리: awk를 사용하여 <file> 태그로 시작하고 </file> 태그로 끝나는 블록을 찾습니다. 이는 awk '/<file name=/,/<\/file>/' 명령으로 처리합니다. 파일 내부 처리: awk 스크립트는 각 파일 블록을 처리하면서 파일 이름을 추출하고, 각 위반 사항의 정보를 파싱합니다. 파일 이름은 gensub() 함수를 사용하여 추출하고, 위반의 라인 번호, 메시지, URL을 추출하여 출력합니다. Markdown 형식: 추출된 정보는 콘솔에 출력되고 pmd_summary.md 파일에 Markdown 형식으로 기록됩니다. 최종적으로, 이 Markdown 파일은 GitHub Actions 단계 요약에 추가됩니다. 장점 성능: awk는 스트림을 처리하면서 발생하는 데이터를 효율적으로 처리할 수 있으므로, 큰 XML 파일에 대해서도 성능상의 이점이 있습니다. 간결성: awk를 사용하면 복잡한 grep 및 sed 조합을 하나의 명령으로 대체할 수 있어 스크립트가 더욱 간결해집니다. 유지 관리: 스크립트가 간결해지므로 유지 관리가 용이해지며, 변경 사항을 적용하기 쉬워집니다. * awk로 개선된 스크립트2 - 텍스트 한정 변경된 부분 설명 태그 내용 처리 로직: <violation> 태그를 만나면 내용 캡처를 시작하고, </violation> 태그를 만나면 내용 캡처를 종료하며, 그 사이에 있는 모든 텍스트를 content 변수에 누적합니다. 텍스트 출력: 최종적으로 각 violation의 설명이 content 변수에 저장되고, 이를 Markdown 링크의 텍스트로 사용하여 출력합니다. 이 수정을 통해 PMD 요약 정보에는 각 violation의 상세 설명이 <violation> 태그 사이의 텍스트로 정확히 제공됩니다. 이 방법은 XML 데이터에서 필요한 정보만을 추출하여 보다 명확하고 유용한 요약을 생성하는 데 도움이 됩니다. * message 수정 * workflow-PR 생성 * NCSS 5->15 * NCSS 5->15 * NCSS 15->5 * README.md 수정 * onPRTest * onPRTest - $GITHUB_STEP_SUMMARY * onPRTest * Comment SUMMARY Report on PR body check * body-path: 'PR_summary.md' * body-path: 'PR_summary.md' * PUSH_summary.md 에 모으기 * PR_summary.md 에 total count 추가 * push workflow는 pr이 있으면 작동하지 않음 * pr workflow type 지정 * PR시에는 build task * github.event.pull_request삭제 - pr workflow에서만 true 가능 * types 가독성 * graalvm 적용 * 정리 * 언어 정리 * pmd pmdtest는 전부 제외, pmdmin은 step4만 * 메서드라인은 15라인 * 문제가 없을시 pmd report 요약 액션 에러 수정 * push workflow - 문제가 없을시 pmd report 요약 액션 에러 수정 * PR 멈추기 * push test * push test2 * error 수정: branches-ignore: ['*'] * total_violations 출력 수정 * 극단적 test * 극단적 test2 * 극단적 test3 * PMD 리포트 요약 복원 * grep 명령 실패 허용 * jacoco summary 지워지는 거 방지 * PR workflow 활성화 --------- Co-authored-by: GeunChang Ahn <[email protected]>

* psvm 클래스 rename * 패키지 이동 * 메서드명 변경 * CarRacingRunner 생성 * 단일책임 원칙 --------- Co-authored-by: GeunChang Ahn <[email protected]>

#3) conflict 해결 ㅠ

…acing

* conflict ㅎㅐ결 * RandomMovingStrategy 작성 * stragegy 패턴 적용 * 포맷팅

neojjc2 · 2024-04-18T05:08:34Z

#tag @neojjc2

neojjc2

안녕하세요 근창님 🙇

4단계도 무난하게 잘 구현 해주셨습니다 💯
소소한 의견 몇 가지 드렸는데요,
마지막 단계 진행하시면서 수정 검토 해주시면 좋을 것 같네요 😄

이제 어느덧 마지막 단계만 남았습니다.
조금만 더 힘내 주세요 🙏

neojjc2 · 2024-04-18T05:09:11Z

README.md

+- [x] 단일 책임 원칙으로 메서드가 되어 있는지 확인
+- [ ] 모든 로직에 단위 테스트 구현 - UI 제외
+- [x] UI를 `InputView`, `ResultView`로 분리


요구사항 및 체크사항 좋습니다 💯

neojjc2 · 2024-04-18T05:10:40Z

src/main/java/step4_winner/Step4Application.java

+ // init
+ InputView inputView = new InputView(new Scanner(System.in));
+ ResultView resultView = new ResultView();
+ CarRacingRunner carRacingRunner = new CarRacingRunner(inputView, resultView);


이미 main이 있기 때문에 CarRacingRunner가 inputView, resultView에 대한 의존성을 가져갈 필요는 없을 것 같습니다 🙇

neojjc2 · 2024-04-18T05:10:57Z

src/main/java/step4_winner/domain/Car.java

+ this.name = inputName;
+ }
+
+ public void move(MovingStrategy movingStrategy) {


💯 💯 💯 💯 💯

neojjc2 · 2024-04-18T05:11:13Z

src/main/java/step4_winner/domain/Car.java

+ private int distance = MIN_DISTANCE;
+ private final String name;
+
+ public Car(String inputName) {


검증 잘 해주셨습니다 👍

neojjc2 · 2024-04-18T05:11:46Z

src/main/java/step4_winner/strategy/RandomMovingStrategy.java

+
+public class RandomMovingStrategy implements MovingStrategy {
+
+ private final Random random = new Random();


Suggested change

private final Random random = new Random();

private static final Random RANDOM = new Random();

neojjc2 · 2024-04-18T05:12:42Z

src/main/java/step4_winner/strategy/MovingStrategy.java

@@ -0,0 +1,5 @@
+package step4_winner.strategy;
+
+public interface MovingStrategy {


@FunctionalInterface가 붙여지면 더 좋을 것 같습니다 🙏
이 부분은 한번 참고만 해주시면 좋을 것 같아요.

https://tecoble.techcourse.co.kr/post/2020-07-17-Functional-Interface/

neojjc2 · 2024-04-18T05:14:17Z

src/main/java/step4_winner/view/ResultView.java

+ String winners = cars.stream()
+ .filter(car -> car.getDistance() == maxDistance)
+ .map(Car::getName)
+ .collect(Collectors.joining(", "));


요구사항 구현 잘 해주셨습니다 👍
소소한 의견이긴 한데 우승자를 구하는 핵심비즈니스 로직이 ResultView에 있는게
약간 아쉽긴 하네요 😄
뭔가 레이싱의 경기 결과를 담당하는 도메인이 생기고 그 쪽으로 위임해보시면 어떨까요??
의견 드려 봅니다 🙇

neojjc2 · 2024-04-18T05:27:40Z

src/test/java/step4_winner/domain/RaceTest.java

+ assertThat(anyCarMoved).isTrue();
+ }
+
+}


테스트 꼼꼼하게 작성 잘 해주셨습니다 💯

rkaehdaos added 20 commits April 17, 2024 21:43

step1 squash

42287c7

step2 squash

3351eca

step3 squash

d059c8e

docs(README): STEP4 README.md 작성

51b02c6

feat(car): Move, getDistance 구현

5ca13e3

feat(car): name 필드

37c02f1

feat(car): Car 구현 완료 및 정리

9dd8bb2

feat(Race): Race 생성자 및 getter 처리

64c8765

feat(Race): Race runRound

ed667b4

- 이전 study test depreacated 주석처리

feat(Input): Input 구현

cac66cb

feat(result): 자동차 이름과 함께 움직임 출력

cc64200

feat(result): 최종 우승자 출력: 1명 이상 일 수 있음

fb6ca7d

feat(result): result 구현 완료

01771dc

check: UI를 InputView, ResultView로 분리

3641b3f

Step4 - 단일 책임원칙 확인

96082e8

* psvm 클래스 rename * 패키지 이동 * 메서드명 변경 * CarRacingRunner 생성 * 단일책임 원칙 --------- Co-authored-by: GeunChang Ahn <[email protected]>

Merge remote-tracking branch 'origin/step4_carracing' into step4_carr… (

04ab98c

#3) conflict 해결 ㅠ

Merge branch 'rkaehdaos' into step4_carracing

d655ebe

Merge branch 'rkaehdaos' into step4_carracing

8f9e43a

Merge remote-tracking branch 'origin/step4_carracing' into step4_carr…

6ab6e1b

…acing

rkaehdaos closed this Apr 17, 2024

conflict 해결 및 전략 패턴 적용 (#4)

3a88c8d

* conflict ㅎㅐ결 * RandomMovingStrategy 작성 * stragegy 패턴 적용 * 포맷팅

rkaehdaos reopened this Apr 17, 2024

neojjc2 self-requested a review April 18, 2024 05:08

neojjc2 self-assigned this Apr 18, 2024

neojjc2 approved these changes Apr 18, 2024

View reviewed changes

neojjc2 merged commit d1ae780 into next-step:rkaehdaos Apr 18, 2024
2 checks passed

rkaehdaos deleted the step4_carracing branch April 19, 2024 04:26

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Step4 carracing #5541

Step4 carracing #5541

rkaehdaos commented Apr 17, 2024

neojjc2 commented Apr 18, 2024

neojjc2 left a comment

neojjc2 Apr 18, 2024

neojjc2 Apr 18, 2024

neojjc2 Apr 18, 2024

neojjc2 Apr 18, 2024

neojjc2 Apr 18, 2024

neojjc2 Apr 18, 2024

neojjc2 Apr 18, 2024

neojjc2 Apr 18, 2024


		public class RandomMovingStrategy implements MovingStrategy {

		private final Random random = new Random();

	private final Random random = new Random();
	private static final Random RANDOM = new Random();

		@@ -0,0 +1,5 @@
		package step4_winner.strategy;

		public interface MovingStrategy {

Step4 carracing #5541

Step4 carracing #5541

Conversation

rkaehdaos commented Apr 17, 2024

neojjc2 commented Apr 18, 2024

neojjc2 left a comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment