[%] 얼리 리턴 썼다가 코드 리뷰 존나 까이고 왔다

[익명(am1196)]

좆같은 misra ㅋㅋ

- dc official App

[익명(121.140)]

얼리 리턴 왜 안된대?

[익명(am1196)]

misra C 15.5 - dc App

[익명(am1196)]

그거에 위배돼서 정적분석 검사기에 걸린대 - dc App

[익명(183.102)]

권장사항이지 강제 아닌데

[익명(183.102)]

머,, 거기 컨벤션이 그렇다면 따르는게 맞긴 함

[익명(121.129)]

저렇게 하려면 if else로 스코프 분리해서 짜야 겠네 모든 코드를 끝까지 다 봐야 하니 좀 피곤하겠다

[익명(am1196)]

7중 if문 뒤질거 같음 ㅋㅋ - dc App

[익명(59.11)]

븅신같은 룰이네

[익명(am1196)]

ㄹㅇㅋㅋ - dc App

[라면의근본삼양라면(211.234)]

왜 쓰면 안 됨? 얼리 리턴의 본질적인 문제가 뭔데

[익명(am1196)]

케이스 잘못 나누면 잘못된 리턴을 하거나 아래 부분 로직이 작동 안 할 수 있단게 요지인데 if 중첩도 그건 똑같지 않나싶다 - dc App

[라면의근본삼양라면(211.234)]

??? ㅋㅋㅋ 너말대로 if 중첩도 똑같기도 하고 그런 로직적인 부분은 테스트코드로 잡을 문제지, 정적 분석이나 컨벤션의 영역이 아닌데 ㅋㅋㅋ

[익명(am1196)]

이직했는데 왜 SI를 거르라는지 알게 되는 하루였음 ㅋㅋ - dc App

[익명(chartman)]

아래 부분 로직이 할당받은 리소스를 해제하는 코드들이라면 어떨까?

[익명(am1196)]

ㄴRAII는 어따팔아먹었노 게이야 - dc App

[익명(am1196)]

ㄴ애초에 그랬으면 내가 그렇게 하지도 않았을거고 - dc App

[코더입니다(nowadays3724)]

대충 for문안의 break쓰면 위험하다는 사람을 본 느낌

[익명(220.94)]

그거 룰끄면 되는데 참나