첫 파견지에서 처음으로 코드리뷰 받았는데 이상함

[익명(118.222)]

코멘트 더 세세하게 달아라, for문 중감연산자 전위로 하지 마라 이런거까진 좀 기분 나빠도 이해는 감

근데 if문 중첩하기 싫어서 일부로 3항연산자 처리한걸 중첩 if문으로 다시 바꾸라거나, boolean값을 == true로 꼭 쓰라는건 진짜 이해 안가네

하다못해 !대신 == false까진 어찌어찌 하겠는데 true 변수명이 isXXX인데 이것도 헷갈리는 사람이 있다면 걍 코딩하지 말아야 하는거 아닌가

[익명(125.178)]

니가노답임

[17번(39.7)]

중첩할일 아예 없애는게 아니면 수정지시할 가치가 없는데 ㄹㅇㅋㅋ

[17번(39.7)]

근데 뭐 결국 그사람들 뽑아놨으니 일은 시켜야하고 전체적인 생산성 생각해보면 좀 바보같이 짜는게 결론적으로 나을수도 있음ㅋㅋㅋㅋㅋㅋㅋㅋ

[익명(118.222)]

전위증감도 처음 지적 들어올땤 저렇게 하면 1부터 시작이라고 지적들어왔다…

[17번(39.7)]

ㅋㅋㅋㅋㅋㅋㅋㅋㅋㅋㅋㅋㅋㅋㅋㅋㅋㅋ

[17번(39.7)]

오타인진 모르겠는데 증감연산임

[익명(222.98)]

3항연산자가 오히려 가독성 해쳐서 그럴수도 있음. ==true는 가끔 미친 사람들이 자바에서 int형 변수를 if안에다 넣는 경우 있어서 그런듯?

[익명(222.98)]

뭐 코딩컨벤션 따로 정해진 게 있으면 따르는 게 맞는거고

[익명(118.222)]

자바는 그러면 어차피 컴파일 에러라서 그냥 true인지 false인지 독해력 안좋은 사람들땜에 저래 하라는거 같음

[익명(118.222)]

중첩 if는 진짜 화난다 근데. 내가 머리 깨지게 고민하면서 메소드 새로 만들며 2중 if문 다 없애놨는데 그걸 다시 돌리라니

[17번(39.7)]

근데 그거나 그거나 분기문 두개라 또이또이임

[익명(118.222)]

난 2중 3중 if문이 가독성이 안좋다고 생각해서 바꾼거거든 ㅠㅠ 성능상으론 차이 없는거야 알지

[17번(39.7)]

삼항연산자도 엔간해서 코드 직접 안보곤 나는 더 할말이 없네

[익명(175.223)]

자바에서 int형 넣으면 빨간줄 뜨는데?

[익명(222.98)]

아 그루비랑 헷갈렷네 ㅈㅅㅈㅅ

[야딩조무사(cir6174)]

불리언을 불리언이랑 비교하는건 오히려 안좋은 코딩 습관인데 - dc App