[🐣질문] 초보 지저분한 코드인지 평가좀

[익명(211.104)]

[익명(lobster7503)]

gm.currentHP2처럼 1하고 2하고 무슨 차이인지 명확하게 안보이는거? - dc App

[익명(lobster7503)]

그냥 단순하게 기능이 없이 123 이어지는거면 배열로 해두는게 명시적인거 같은데 - dc App

[익명(119.75)]

지저분함 일단 변수명부터 손좀 보세요

[tpf(arrow3322)]

주석을 좀 다는 게 좋아보이고, 각 조건문 안에 다 똑같은 코드 같은데 함수 선언해서 쓰면 더 깔끔해 보일 듯

[Indie 1(221.167)]

일단 매직넘버는 좋지못해 jpbyte가 뭔지 모르겠지만 enum을 활용하는게 좋아보임 그리고 나눗셈 할때 0 체크도 하는게 좋음 그리고 계산 공식도 중복 코드니까 함수로 빼던가 하고 그리고 B 자료형이 float인데 float.Parse를 왜 또 함?

[익명(211.104)]

ㄳㄳ B는 decimal 입니다

[익명(218.151)]

이언어는 switch문이 없나?

[Indie 2(218.37)]

GPT에서 물어보셈. 여기서 조언도 좋지만, 더 많은걸 배울 수 있음.

[익명(211.104)]

다들 ㄳ 많은걸 배웠습니다.

[Indie 3(180.66)]

크아악!! 이 코드는 대체 뭐냐!!!

[Ray.(choxu0826)]

B가 decimal 이면 (float)decimal 하면 되지 않음?

[Ray.(choxu0826)]

그리고 if문 내부에 패턴 보이는데, gm.jpbyte % 4 == 1, 2, 3 이런식으로 바꿀수도 있을듯?

[익명(211.104)]

ㄳㄳ 스트링으로 형변환 한 이유가 decimal은 바로 float으로 형변환 안 되길래 이중변환 했음.. 사람들이 decimal은 잘 안 쓰는지 딴 사람들도 잘 모르더라 ㅋㅋ

[Ray.(choxu0826)]

명시적으로 (float)B가 안된다는거임?

[Ray.(choxu0826)]

switch 패턴 매칭으로switch (gm.jpbyte){ case int n when n % 4 == 1: //동작부 break; case int n when n % 4 == 2: //동작부 break; case int n when n % 4 == 3: //동작부 break;}

[Ray.(choxu0826)]

이런식으로 바꾸는거 고려해봐 저런식으로 패턴 유지될거같으면

[익명(211.104)]

오 (float) 은 첨 보는데 해보니까 이건 되네 ㅇㅋ ㄳ

[211214(tomatoss)]

줄임말이랑 damage 오타 없애야할듯

[noble(altshake)]

변수명만........

[익명(222.238)]

currenthp1,2,3이 구별 가능해?

[익명(222.238)]

혹시 멀티게임임? jpbyte는 패킷이고?