[💬] 얀데레 코드 리뷰어는 왜 이걸 /25로 설정한다고 하는거임?

[Indie(180.224)]

이걸 25로 나눠지니까 / 25로 하자는건 진짜 이 코드보다 병신같이 짠건데 이거보면서 낄낄대는게 내 입장에선 그렇게 웃길 수가 없노.

25로 나눠지는건 그냥 값 경계인거고 게임해보고 수치가 별로다 하면 23, 50, 72 이렇게 바뀔 수가 있는데 그럼 그때가서 다시 /25인거 풀고 또 나눈다고? 데이터 자료형에 대한 이해가 그냥 없는거임. 케이스적으로 변경가능성이 존나 높은 상수값을 간접적으로 하드코딩하는거랑 다른게 없음.

json으로 가져올거까지 지원할거 아니다 하면 젤 위에다가 [0, 25, 50, 75, 100] 이렇게 선언하고 affection 인풋받아서 이터레이션돌아서 인덱스 반환해주는 함수 만들어야 값이 변경되도 동적으로 판단해서 지원이 되는거고, 저 코드 제일 문제점은 Affection이란 단일요소로 결정되는게 두 가지 멤버변수 상태로 존재하면 나중에 한쪽만 업데이트될 가능성이 있고 그런게 진짜 문제인거지 저딴게 아님. 내용보면 그렇게 어펙션레벨이 자주 참조될 것도 아닌거 같고 그냥 get AffectionLevel하고 따로 변수로 두지않는게 나음. 진짜 복잡한 연산이다 해서 연산결과를 상태로 캐싱할 수준은 아닌거같고. 캐싱한다고 해도 Affection, AffectionLevel간의 관계는 그냥 순수 static 함수로 만들고 하거나 정 연산을 줄이겠다하면 UpdateAffection함수에서 동시에 업데이트를 해주는게 가장 최선임.

내 기준에선 저걸 /25로 처박는게 진짜 코드리뷰 꽉꽉 적어주고싶은 코드임

[익명(211.208)]

뭔데 갑자기 화났어 그영상도 한 5년전 영상아님? 그리고 짤만보면 상수값 %수치를 0~5단계로 나누는코드라 /25 하란거 아녔을까 굳이 분기문 쓰기보다는

[글쓴 Indie(180.224)]

그게 가장 변경이 쉬운 부분임. 25단위로 된다는 보장이 없기 때문에 /25라는건 그냥 하드코딩을 하는거랑 다름이 없다는거지. 롤 챔피언 공격력이 36이라고 그냥 공격했을때 아 어차피 36이니까 champ.damage가 아니라 36적는거랑 간접적으로 똑같음.

[글쓴 Indie(180.224)]

근데 저걸 보면서 필요이상으로 깔깔대니까 내 입장에선 병신같다고 느낀거임 ㅋㅋㅋㅋ 지들이 수정한 코드가 더 병신인걸 모르고

[익명(211.208)]

걍 상숫값 매직넘버 넣어서 나눠버리는게 이상한거라는거로 이해했는데 맞나? 뭐 나중에 바뀌는거면 위 코드도 조건 바꾸는건 동일한데다 만약에 20으로 바뀌는게 걱정이면 내경우엔 따로 define클래스에다 상숫값 적어두고 써가지고

[익명(211.208)]

아 코드 비웃는거때문에 그런거구나 영미권에서 얀심이 하도 그 뭐라하냐 호감? 필수요소? 비슷해가지고 그걸로 컨텐츠뽑느라 그런거 아닐까 그 왜 개발자가 우유마시는짤 유명하잖아

[글쓴 Indie(180.224)]

@ㅇㅇ(211.208) 마즘 ㅇㅇ 어차피 저런 상수값은 변하기 마련인데 그럼 변경을 최소화하는 방법은 동적으로 돌아갈 수 있게 배열로 경계들을 명시하고 그걸 코드변경없이 변경하게 하는게 제일 깔끔한거고, 그정도까지 변경할 필요성이 존재하지않는 경우엔 그냥 저렇게 글루코드로 박아도 아무 문제 없음. 뭐 개발자 본인이 /25로 한 수치로 해야지~ 생각하면서 저렇게 짜면 문젠데 그냥 글루코드해놓고 게임테스트하면서 게임경험에 맞게 수정한 최종결과물이 저거였다하면 욕하기 어렵다는거임. 왜냐면 저거 /25로 바꿔봤자 아무 쓸모없거든

[글쓴 Indie(180.224)]

@ㅇㅇ(211.208) 그럴수도있고.. 나한테 공감까지 해주노. 그냥 나도 누가 얘기하길래 써봤음. 별 의미없음

[익명(211.208)]

그치 뭐 저거하나 바꾼다고 연산이 좋아질거 같지도 않고, /25로 바꾼다고 갑자기 업데이트콜 줄어드는것도 아닌데 ㅋㅋ

[Indie 1(211.200)]

이게 그 유명한 조건문이군

[dd(directly1991)]

그럼 죽어 같은거지 지루하고 현학적임 < 그럼 죽어

[익명(118.235)]

영상 링크 좀