[Rule] if, else, for, while, do statement에서 single-line에도 curly brace를 필수로 한다.

[luffy-song]

AS-IS: Rule of Standard

if (foo) foo++;

TO-BE

if (foo) {
    foo++;
}

제안하고 싶은 내용 및 의견

코드 리뷰를 진행하다가 @inbeom 이 curly brace 없는 single-line의 if statement는 읽다 가끔 놓치는 경우가 있다는 코멘트를 주셔서 짚고 넘어가는 것도 좋을 것 같아 이슈라이즈 하게 되었습니다. https://github.com/titicacadev/triple-content-web/pull/418#discussion_r350486379

참고: https://eslint.org/docs/rules/curly

---

댓글로 이모지 또는 새로운 의견을 남겨주세요~
ex) - 3️⃣ : 나는 요래조래

• 1️⃣ : 현행
• 2️⃣ : 제안대로 수정
• 3️⃣ : 기타 의견

[luffy-song]

1️⃣ 속내를 드러내자면 저는 심플한 게 좋기 때문에 single-line은 brace 없이 쓰는 게 좋을 것 같아서 위의 제안은 저의 제안이 아니라, 그냥 한다면의 to-be 정도로 여겨주시면 좋을 것 같아요 ㅎㅎㅎ

[appear]

저도 1️⃣을 선호하는 편이었으나 뭔가 트리플에서는 2️⃣처럼 더 많이 사용하고있네요 🛩
1️⃣도 큰무리는 없지 않을까싶어서 저도 1️⃣한표

[boxersb]

ASI 를 활용한 ; 생략 컨벤션을 쓰고 있기 때문에 single-line statement 가 좀 읽기 어려운것에 동의합니다.
그럼에도 불구하고 ㅋㅋ 저도 한줄로 끝나는게 좋기 때문에 1️⃣ 입니다..

이거 룰을 좀 더 strict 하게 할 수 있나 봐야할것 같아요..

1️⃣ - 한줄로 끝나는 statement 는 아래에 반드시 빈라인을 넣는다, 한 줄로 끝나는 statement 는 { } 를 쓰면 안된다.

정도로 갈 수 있으면 좋겠습니다.

[inbeom]

저는 2️⃣와 3️⃣의 중간인 것 같아요. 생략은 허용하더라도 개행은 있었으면 좋겠어요 ㅋㅋㅋㅋ

if (foo)
    foo++;

[polysiya]

제 취향은 2️⃣ 입니다 하하

[giwan-dev]

저도 2️⃣가 취향입니다ㅎㅎㅎ

[yceffort]

저는 2️⃣입니다. ~나떼는말이야~~ 개행 안하면 혼나고 그랬어~

[appear]

이거 토레스한테도 물어볼게요

[steve-triple]

저도 2️⃣번 ㅎ

[boxersb]

아... 이거 깜빡하고 있었네요.. 2️⃣ 로 수렴된것 같군요..

작업 들어가겠습니다!

[luffy-song]

앗 우디가 merge하면 담에 제가 PR올리려고 했는데 ㅎㅎㅎ 삽 안 뜨셨다면 제가 할게유

[boxersb]

아아.. 그러시죠 그럼!