be/main/yeoshin_Java_Lv0 by duckgii · Pull Request #19 · 42cabi/on-boarding

duckgii · 2025-01-07T06:57:44Z

자바 Lv0 구현 완료했습니다.

로직

대욱이의 현재 분노 수치가 분노 임계값을 넘어가면 프로퍼티로 가지고 있는 flag를 변경하여 main에서 펀치를 호출하고 탈출할 수 있도록 작성했습니다.

saewoo1

자바를 처음 하신다고 들었는데, 정말 잘 작성하셨습니다~!!!
출력부 분리, 클래스의 책임과 기능분리에 대해 쪼금만 더 익숙해지시면 곧 자바 마스터이신데여??
고생 많으셨습니다!!!!!

saewoo1 · 2025-01-08T06:45:31Z

접근제어자가 멀까요? 생성자는 어떤 역할을 하는걸까여??
하나의 클래스에 함께 존재하는 변수와 메서드는 어떤 의미를 가져야 하는가.. 객체지향은 어렵다

아무 생각 없이 cpp처럼 만들었는데 자바에는. default라는게 있었군요. 새로운 접근제어자를 배울 수 있었습니다 감사합니다!!
다른 패키지에서도 생성할 수 있도록 public으로 변경했습니다.

saewoo1 · 2025-01-08T06:46:51Z

매직넘버를 사용해봅시다!!
41, 80과 같이 직접 변수를 넣어주기 보다는, 의미를 주면 가독성과 유지보수가 좋은 코드가 될거에여!!

넵 유지보수를 위한 코드를 고민해볼 수 있어서 좋았습니다. 수정완료했습니다 :)

saewoo1 · 2025-01-08T06:52:49Z

보통 Collection 객체의 변수명은 복수형으로 작성! ment -> ments 권장드립니다

ArrayList로 작성한건 구현체를 의도하신건가여?! 생성자 내에서 초기화 시에도 동일하게 작성을 하셨던데,
멤버변수 선언 시 타입을 String으로 정의하셨다면, 제네릭 타입 추론이 가능하기 때문에 new ArrayList<>(); 이렇게 생략도 가능합니다!

넵 조언 덕분에 제네릭에 대해 자세히 공부해볼 수 있었습니다 감사합니다 :)
수정 완료했습니다!!

saewoo1 · 2025-01-08T07:01:14Z

구현체로 선언 하고, 생성자에서 초기화 후 값을 채워넣는 방식이군용
그렇다면 List<String> ments = new ArrayList<>(List.of("멘트1", "멘트2", "멘트3"));
이렇게 하면 동일한 의미가 될 것 같습니다! List.of()는 불변 리스트를 반환하고, newArrayList<> 안에 넣어서 가변 리스트로 변환하면 좋을 것 같읍니다

ArrayList의 생성자 괄호 속에는 초기 용량 또는 Collection만 가능하다는 것을 알 수 있었습니다. 감사합니다 :)
수정 완료했습니다!!

saewoo1 · 2025-01-08T07:02:52Z

어디까지나 취향 차이지만, 무한루프보다는 탈줄조건을 while 안에 넣어주는건 어떠신가여?

네 가독성 측면에서 그게 더 좋을 것 같습니다!! 수정 완료했습니다.

saewoo1 · 2025-01-08T07:06:37Z

의도는 베리 굿인데여??
방식을 조금만 바꿔봅시다! 쩨이팍의 멘트와 정확히 중복되고 있으니 쩨이팍으로부터 멘트공격을 받으면, 분노 수치가 쌓이도록 바꿔보는건 어떠실까여?

각각의 랜덤 값은 클래스 생성 시에, 생성자에서 정해집니다.
리드미에서 대욱 클래스에 대해서 이렇게 설명하고있어서 어떤 방향으로 수정해야할지 감이 잘 잡히지 않습니다.

각 도발 문자열마다 가지는 도발 수치의 랜덤값이 다르기 때문에 제이팍으로부터 문자열을 받아서 그때 랜덤값을 생성하는 방식은 적용하지 않았습니다! 모두 동일한 범위의 랜덤값이라면 같은 명령어를 통해 설정할 수 있지만, 이 문제에서는 결국 하드코딩이 필요한 조건이라고 생각합니다. 멘트 부분은 static으로 선언해서 리터럴 문자열은 하나만 있도록 수정해보겠습니다!!

Ssuamje · 2025-01-08T11:47:36Z

백엔드 코드에서는 특히 하드코딩된 정적인 값(""이나 이미 매직넘버)을 조심하는 습관을 들이시면 좋을 것 같아요!

duckgii · 2025-01-08T13:03:36Z

자바를 처음 하신다고 들었는데, 정말 잘 작성하셨습니다~!!! 출력부 분리, 클래스의 책임과 기능분리에 대해 쪼금만 더 익숙해지시면 곧 자바 마스터이신데여?? 고생 많으셨습니다!!!!!

감사합니다!! 신경써서 알려주신 부분 모두 수정해서 내일 다시 올려보겠습니다 :)

duckgii · 2025-01-08T13:04:30Z

백엔드 코드에서는 특히 하드코딩된 정적인 값(""이나 이미 매직넘버)을 조심하는 습관을 들이시면 좋을 것 같아요!

넵!! enum 활용법에 대해서 조금 더 알아보고 적용해보겠습니다 감사합니다 ㅎ

chyo1

수고하셨슴다

chyo1 · 2025-01-12T05:19:06Z

이 부분 수정하고 안 뺀 거겠죠..?

chyo1 · 2025-01-12T05:20:17Z

변수로 출력형식 지정해둔 것 구웃

chyo1 · 2025-01-12T11:03:06Z

Suggested change

public String prevoke() {

public String provoke() {

오타 이슈

chyo1 · 2025-01-12T11:11:33Z

Suggested change

String ment;

int prevokePoint;

ment = jpark2.prevoke();

prevokePoint = daewoole.bePrevoked(ment);

String ment = jpark2.prevoke();

int prevokePoint = daewoole.bePrevoked(ment);

선언과 동시에 할당을 하는 것이 권장돼요

chyo1 · 2025-01-12T11:14:30Z

Suggested change

public boolean getPunchFlag() {

return (punchFlag);

}

public int bePrevoked(String ment) {

angryPoint += prevokePoints.get(ment);

prevokeCount++;

if (angryPoint > angryLimit)

punchFlag = false;

return (prevokePoints.get(ment));

public boolean isDaewooleAngered() {

return angryPoint > angryLimit;

}

public int bePrevoked(String ment) {

angryPoint += prevokePoints.get(ment);

prevokeCount++;

return prevokePoints.get(ment);

punchFlag 대신 (angryPoint > angryLimit)로 임계점을 판단하는 게 더 좋을 것 같습니다

duckgii · 2025-01-13T11:28:43Z

수고하셨슴다

감사합니다 수정완료했습니다!

duckgii · 2025-01-13T11:30:35Z

잘못 눌러서 닫아버렸네요;;

FEAT : finish LV_0

220a827

duckgii self-assigned this Jan 7, 2025

saewoo1 reviewed Jan 8, 2025

View reviewed changes

duckgii added 8 commits January 10, 2025 13:45

FIX : Jpark2 프로퍼티 이름 ments로 변경, 생성 방식 변경(List.of 사용)

de9a57d

FIX : while 탈출 조건 변경

c5cd3c9

FIX : 리터럴 하드코딩값 모두 변수로 변경

f84fc49

BUG : 오타 수정

741cf04

FIX : PUNCH_MENT 한 줄로 변경

9323238

FIX : checkPoint 출력 멘트 상수로 수정

87b6160

FIX : 메인 클래스 출력문 상수로 수정

6764cd1

FIX : 각 클래스의 생성자 public으로 변경

cf75d9d

chyo1 reviewed Jan 12, 2025

View reviewed changes

REFACTOR: 효원님 리뷰 내용 수정

8053391

duckgii closed this Jan 13, 2025

duckgii deleted the be/main/yeoshin_Java_Lv0 branch January 13, 2025 11:27

duckgii restored the be/main/yeoshin_Java_Lv0 branch January 13, 2025 11:27

duckgii reopened this Jan 13, 2025

-			String ment;
-			int prevokePoint;
-			ment = jpark2.prevoke();
-			prevokePoint = daewoole.bePrevoked(ment);
+			String ment = jpark2.prevoke();
+			int prevokePoint = daewoole.bePrevoked(ment);

-	public boolean getPunchFlag() {
-		return (punchFlag);
-	}
-	public int bePrevoked(String ment) {
-		angryPoint += prevokePoints.get(ment);
-		prevokeCount++;
-		if (angryPoint > angryLimit)
-			punchFlag = false;
-		return (prevokePoints.get(ment));
+	public boolean isDaewooleAngered() {
+		return angryPoint > angryLimit;
+	}
+	public int bePrevoked(String ment) {
+		angryPoint += prevokePoints.get(ment);
+		prevokeCount++;
+		return prevokePoints.get(ment);

Conversation

duckgii commented Jan 7, 2025

로직

Uh oh!

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

Choose a reason for hiding this comment

Uh oh!

duckgii Jan 14, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Ssuamje commented Jan 8, 2025

Uh oh!

duckgii commented Jan 8, 2025

Uh oh!

duckgii commented Jan 8, 2025

Uh oh!

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

duckgii commented Jan 13, 2025

Uh oh!

duckgii commented Jan 13, 2025

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

4 participants

duckgii Jan 14, 2025 •

edited

Loading