Step4 - 🚀 4단계 - 자동차 경주(우승자) #5540

rkaehdaos · 2024-04-17T10:30:03Z

No description provided.

- Car는 전진한 거리 distance를 가진다. - Car는 random값이 4 이상이면 거리증가

- 초기화 streawm 이용 - 0 이하시 에러 - runRound 추가

- 아예 차가 나타나지 않는 현상 방지

build.gradle - deprecated 확인 옵션 추가

- 필드 추가 - 생성자 추가 - 5자 제한 추가 - getter 추가 - 테스트도 따라서 추가

- 상수 사용 - `null` check

- 테스트 수정

- 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: Kai Ahn <[email protected]>

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

Merge branch 'rkaehdaos' into step4_racing_winner

…racing_winner

rkaehdaos added 30 commits April 6, 2024 10:38

[step3] ready

0dd8dfa

[step3] 자동차 대수와 시도할 회수가 정확하게 입력된다 - 테스트 성공

7215746

[step3] 변수명 수정

ff61813

[step3] String.format대신 printf 사용

10a0630

[step3] OOP설계(Car)

028e02d

- Car는 전진한 거리 distance를 가진다. - Car는 random값이 4 이상이면 거리증가

[step3] OOP설계(Race): Race에는 n대의 차가 참여

8acc7e0

[step3] OOP설계(Race): 개선

8fc50d1

- 초기화 streawm 이용 - 0 이하시 에러 - runRound 추가

[step3] 객체를 이용한 프로그램 실행

5f11a0d

[step3] 리팩토링 - car 처음 distance 초기값 0->1

ea419ed

- 아예 차가 나타나지 않는 현상 방지

[step3] CarMoveTest 추가

1bd2064

[step3] CarMoveTest 개선

f41d845

[step3] RaceRoundTest: race가 1번이라도 열리면 100대중 1개는 전진 한다

d32c7a1

[step3] MainOutputTest (미완성)

32c1b3e

[step3] 테스트 성공

0831273

[test] 테스트 성공

178c321

[test] 필요 없는 부분 정리

2c4f988

[step3] 사용하지 않는provideOutput 메서드 삭제 -

8b013b5

[step3] resultView 생성

512e323

[step3] 워닝 수정

0a37b27

[step3] runraces로 운영 분리

3794990

[step3] InputView 추가

c85110c

[step3] InputView 반영

e5b9485

[step3] 라이브러리 버전업+롬복 추가

5d85c9a

[step3] 정리및 빈 LINE CHECK 추가

6d0e80b

[step3] action 정리

8f06528

docs(README): STEP4 README.md 작성

529b499

feat(car): Move 메서드 테스트 및 구현

c833ac9

build.gradle - deprecated 확인 옵션 추가

feat(car): getDistance 이미 구현

232c86a

feat(car): name 필드

0ff931b

- 필드 추가 - 생성자 추가 - 5자 제한 추가 - getter 추가 - 테스트도 따라서 추가

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

cc56bb6

- 상수 사용 - `null` check

rkaehdaos and others added 12 commits April 15, 2024 16:43

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

ed2bb8b

feat(Race): Race runRound

b5fff1c

fix(other): 예전 study 코드 deprecated부분 주석

aab2a46

feat(Input): Input 구현

158d494

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

d906437

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

5990847

feat(result): result 구현 완료

22219b1

- 테스트 수정

check: UI를 InputView, ResultView로 분리

4b2d71b

Step4 - 단일 책임원칙 확인

185b3ad

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

UI(views) 제외 모든 로직에 단위 테스트

518f233

들여쓰기 1인지 확인

173a82e

rkaehdaos closed this Apr 17, 2024

C

c3ea197

Merge branch 'rkaehdaos' into step4_racing_winner

rkaehdaos reopened this Apr 17, 2024

rkaehdaos added 5 commits April 17, 2024 20:35

C

39b45e4

Merge branch 'rkaehdaos' into step4_racing_winner

Merge remote-tracking branch 'origin/step4_racing_winner' into step4_…

2089a3a

…racing_winner

Merge remote-tracking branch 'origin/step4_racing_winner' into step4_…

b55c146

…racing_winner

Merge remote-tracking branch 'origin/step4_racing_winner' into step4_…

65e8f7b

…racing_winner

Merge remote-tracking branch 'origin/step4_racing_winner' into step4_…

238282b

…racing_winner

rkaehdaos closed this Apr 17, 2024

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Step4 - 🚀 4단계 - 자동차 경주(우승자) #5540

Step4 - 🚀 4단계 - 자동차 경주(우승자) #5540

rkaehdaos commented Apr 17, 2024

Step4 - 🚀 4단계 - 자동차 경주(우승자) #5540

Step4 - 🚀 4단계 - 자동차 경주(우승자) #5540

Conversation

rkaehdaos commented Apr 17, 2024