Skip to content

Instantly share code, notes, and snippets.

@3846masa
Created October 20, 2014 05:54
Show Gist options
  • Save 3846masa/6218a252ea5b75385552 to your computer and use it in GitHub Desktop.
Save 3846masa/6218a252ea5b75385552 to your computer and use it in GitHub Desktop.
if文の中が1行のとき,どれがベストなのか
/*
if文の中が1行のとき,どれがベストなのか
コメント募集(下記以外の方法でも可)
*/
// 01
if (true) println("true");
// 02
if (true)
println("true");
// 03
if (true){println("true");}
// 04
if (true){
println("true");
}
// 05
if (true)
{
println("true");
}
@kvvzr
Copy link

kvvzr commented Oct 20, 2014

機能をついかしたときにdiffを取ると1, 2, 3の場合は余計な行を追加しなければいけないので、僕は4みたいに書く。細かいことを言うとif (true) {のようにスペースを入れる。if分の場合は5のようにはしないかなー。

@3846masa
Copy link
Author

なるほど...

@3846masa
Copy link
Author

ためになる

@satoruhiga
Copy link

ちょっとこのif外してみようかなって時に楽なので2,5をつかってます

// if (true)
  println("true");

// if (true)
{
 println("true");
}

@3846masa
Copy link
Author

なるほど! 確かに使いやすそうですね!!

@toolbits
Copy link

昔は 1 の書き方を多用していましたが、ここ 10 年くらいは以下で統一してます。
if (true) {
}

理由ですが、1行も複数行の範疇に含まれるからです。
1行の時だけ条件わけすると無駄に書く時に覚えないと行けない条件が増えるからです。
例えばプログラム的にソースコードを処理する時などにはなるべく判別するべき条件が少ない方がバグがでにくいです。

@toolbits
Copy link

ちなみに C++ の場合には変数スコープの話も絡んできて括弧ありとなしではまるで意味が違うので、できれば1行も複数行も統一して置いた方が良いと思います。

@3846masa
Copy link
Author

スコープ絡んでくると,囲むほうが無難な気もしてきますね....
なるほど,参考になります!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment