中括弧は独自の行に表示する必要がありますか？[閉まっている]

中かっこは独自の行にあるべきですか？あなたはそれについてどう思いますか？

if (you.hasAnswer()) {
    you.postAnswer();
} else {
    you.doSomething();
}

またはそれがあるべき

if (you.hasAnswer())
{
    you.postAnswer();
}
else
{
    you.doSomething();
}

あるいは

if (you.hasAnswer())
    you.postAnswer();
else
    you.doSomething();

建設的にしてください！理由を説明し、経験を共有し、事実と参考文献でバックアップします。

私が学生だったとき、私は同じ行に中括弧を置いていたので、行が少なくなり、コードがより少ないページに印刷されます。行の中で唯一のものとして印刷された単一のブラケット文字を見るのは面倒です。（環境、紙の無駄）

ただし、大規模なアプリケーションをコーディングする場合、中括弧のみを使用した一部の行を許可すると、「グループ化」の感覚を考慮して手頃な価格になります。

どのスタイルを選択する場合でも、関連するコードの複数のスタイルを処理するためにあなた自身の脳にとってオーバーヘッドにならないように一貫性を保ってください。さまざまなシナリオ（上記など）では、さまざまなスタイルを使用してもかまいません。高いレベルで「コンテキストを切り替える」のは簡単です。

3番目の方法を実行しないでください。

中かっこをスキップすると、最初にいくつかのキーストロークを節約できますが、次に来るコーダーは、ブロックが中かっこが欠落していることに気付かずにelse句に何かを追加します。

他の人のためにコードを書いてください。

長い間、私は彼らが同等の価値がある、または非常に近いと主張していたので、正しい選択をすることによって得られる利益はそれについて議論するコストをはるかに下回っていました。

ただし、一貫性を保つことは重要です。それで、コインをひっくり返してコードを書くことにしましょう。

プログラマーがこのような変化に抵抗するのを見たことがあります。乗り越えろ！私は私のキャリアの中で何度も切り替えました。PowerShellではなくC＃で異なるスタイルを使用します。

数年前、私はチーム（約20人の開発者）で作業しており、入力を求めてから決定を下し、それをすべてのコードベースに適用することにしました。決定するのに1週間かかります。

たくさんのうめき声とアイローリング。たくさんの「私は自分のやり方が好きだ、なぜならそれはより良いからだ」が実質はない。

質問のより細かい点を検討しているときに、誰かがこの問題を同じ行に括弧で囲む方法で対処する方法を尋ねました。

void MyFunction(
    int parameterOne,
    int parameterTwo) {
    int localOne,
    int localTwo
}

パラメータリストがどこで終了し、ボディが開始するかはすぐにはわからないことに注意してください。と比較：

void MyFunction(
    int parameterOne,
    int parameterTwo) 
{
    int localOne,
    int localTwo
}

世界中の人々がこの問題にどのように対処しているかを読んで、開き中かっこの後に空白行を追加するパターンを見つけました。

void MyFunction(
    int parameterOne,
    int parameterTwo) {

    int localOne,
    int localTwo
}

視覚的な中断を行う場合は、ブレースを使用して行うこともできます。そうすると、視覚的な切れ目も一定になります。

編集：K＆Rを使用する場合の「余分な空白行」ソリューションの2つの選択肢：

1 /関数の引数を関数本体とは異なるインデント

2 /最初の引数を関数名と同じ行に配置し、新しい行の引数をその最初の引数に合わせます

例：

1 /

void MyFunction(
        int parameterOne,
        int parameterTwo) {
    int localOne,
    int localTwo
}

2 /

void MyFunction(int parameterOne,
                int parameterTwo) {
    int localOne,
    int localTwo
}

/編集

私はまだ一貫性が他の考慮事項よりも重要であると主張しますが、前例が確立されていない場合、ブレースオンネクストラインが道です。

基本的なルールは次のとおりです。

1. プロジェクトの既存のコーディング標準に従ってください。
2. コーディング標準がなく、他の誰かが所有する既存のコードベースを編集している場合-どれだけ好きでも嫌いでも、既存のコードのスタイルと一貫性を保ってください。
3. グリーンフィールドプロジェクトに取り組んでいる場合は、他のチームメンバーと話し合い、公式または非公式のコーディング標準について合意に至ります。
4. 唯一の開発者としてグリーンフィールドプロジェクトに取り組んでいる場合は、自分で考えてから、冷酷に一貫性を保ってください。

外部からの制約がない場合でも、既存の（広く使用されている）コーディング標準またはスタイルガイドラインを探して、それに従うことを（IMO）最適です。独自のスタイルを展開する場合、数年後には後悔する可能性が高くなります。

最後に、既存のスタイルチェッカーとコードフォーマッタを使用して実装/実装可能なスタイルは、手動で「強制」する必要があるスタイルよりも優れています。

最初の方法の利点は、垂直方向にコンパクトであるため、より多くのコードを画面に収めることができることです。そのため、この方法をお勧めします。私が2番目の方法を支持して聞いた唯一の議論は、開きブラケットと閉じブラケットのペアリングが簡単になるということですが、ほとんどのIDEにはそのためのキーボードショートカットがあり、実際には間違ったステートメントです-開きブラケットと閉じブラケットのペアリングの代わりに同じインデントレベルで閉じ括弧を「ブロックの開始」式（if、else、for、while）にペアリングできるため、ブロックの開始位置を簡単に判断できます。

前のfor / while / ifコンストラクトがすでにブロックの開始を視覚的に示している場合、ブラケットだけのために行全体を無駄にする理由はありません。

そうは言っても、ブロックの終わりとそのインデント構造を目に見える形で示すために何かが必要なので、閉じ括弧はそれ自身の行にあるべきだと思います。

私は好む

if (you.hasAnswer())
{
    you.postAnswer();
}
else
{
    you.doSomething();
}

以上

if (you.hasAnswer()) {
    you.postAnswer();
} else {
    you.doSomething();
}

行you.postAnswer();が一目で読みやすく、見つけやすいからです。2番目の方法では、上にある線と混ざり合って（you.hasAnswer()）、読むために目を集中させる必要があります。

私は最初の方法を好みます。中かっこはまったく別の行に値しません。

問題は、中括弧は重要ではないということです。それらは単なる構文上のゴミです。これは、コードの目的、目的、実装方法を理解するために絶対に不要です。これらは、使用可能な画面スペースが少ないためにオペレータの視覚的なグループ化が不可能だった古いスタイルのCのような言語へのオマージュです。

括弧なしで問題ない言語（Python、Haskell、Ruby）があります。これは、ブレースがゴミであることを確認するだけであり、可能であればいつでもそれらのための行に値するべきではありません：

if (you.hasAnswer()){
    you.postAnswer();
}else{
    you.doSomething();
}

Pythonを使用して、引数を完全に回避します。

中括弧の位置は

メタデータ

プログラマーによってIDEで構成可能。そうすれば、作成者に関係なく、すべてのコードの厄介な中括弧は同じように見えます。

場合によります。

JavascriptまたはjQueryでコーディングしている場合、最初の形式を使用します。

jQuery(function($) { 
    if ($ instanceOf jQuery) { 
        alert("$ is the jQuery object!"); 
    } 
}); 

しかし、C＃でコーディングする場合は、2番目の形式を使用します。これは、C＃でそれを行う標準的な方法だからです。

public int CalculateAge(DateTime birthDate, DateTime now) 
{ 
    int age = now.Year - birthDate.Year; 
    if (now.Month < birthDate.Month 
        || (now.Month == birthDate.Month && now.Day < birthDate.Day)) 
        age--; 
    return age; 
} 

あなたの例を書くことができることに注意してください

if (you.hasAnswer())
    you.postAnswer();
else
    you.doSomething();

C＃で。

この例の間違いを見ることは私にとって難しいので、私は最初のものを好む。

if (value > maximum);
{
    dosomething();
}

この例よりも

if (value > maximum); {
    dosomething();
}

で; {終わる行よりも間違っているように見える;ので、私はそれに気付く可能性が高くなります。

1）のわずかなバリエーションを好む

if (you.hasAnswer()) {
    you.postAnswer();
} // note the break here
else {
    you.doSomething();
}

どうして？

• ブレースを常に自分の行に置くと読みやすさが低下すると思います。私の画面には、一定量のソースコードのみを収めることができます。ブラケットスタイル2）は、多くのネストされたループと条件付きのヒーブアルゴリズムを非常に長くします。

• しかし、私は欲しい elseので、新しい行に開始するifとelse、視覚的に、一緒に属しています。の前にブラケットがある場合else、何が何に属しているかを見つけるのははるかに困難です。

• 3）自分自身を失格にする。かっこを外して忘れてしまうと、どんな悪いことが起こり得るかを知っています。

私はどこかの本の著者がコードを次のようにフォーマットすることを望んでいることをどこかで読みました。

if (you.hasAnswer())
{
    you.postAnswer();
}
else
{
    you.doSomething();
}

しかし、出版社からのスペースの制約は、これを使用しなければならなかったことを意味しました。

if (you.hasAnswer()) {
    you.postAnswer();
} else {
    you.doSomething();
}

今はそれが本当かどうかわかりません（もう見つけられないので）が、本では後者のスタイルが非常に普及しています。

個人的なレベルでは、次のように別の行の括弧を好む：

a）新しいスコープを示します
b）ミスマッチがある場合に見つけやすくなります（ただし、これはエラーを強調するIDEの問題ではありません）。

ああ、 1つの真のブレーススタイル。

聖なる道のために、リチャードの「私の道か高速道路」のストールマンでさえ、すべてが聖道のために作られています。

その男は非常に多くのことについてとても間違っていましたが、GNUはブレースに関してはスポットオンです。

[更新]私は光を見て、今オールマンを崇拝しています

2番目の例では、読みやすさが非常に重要です。他の方法をブロックするかどうかを見て我慢できない=（

簡単な答え：デバッグが簡単なものは何ですか？

// Case 1:
void dummyFunction() {
  for (i = 0; i != 10; ++i) {
    if (i <= 10)
      std::cout << "i is: " << i << "\n";
      std::cout << 10 - i << " steps remaining\n";

      // Some hard work here
      // which is really hard
      // and does take some screen estate
    }
    else
      std::cout << "We'll never get there";
  }
} // COMPILER ERROR HERE


// Case 2:
void dummyFunction()
{
  for (i = 0; i != 10; ++i)

    if (i <= 10)
    {
      std::cout << "i is: " << i << "\n";
      std::cout << 10 - i << " steps remaining\n";

      // Some hard work here
      // which is really hard
      // and does take some screen estate
    }
    else
      std::cout << "We'll never get there\n";
  }
} // COMPILER ERROR HERE

どの場合に最初に問題を診断しましたか？

私は個人的な好みのためにあまり気にしない（WhitesmithのとAlを含む他の多くのスタイルは、。があります）、私はそれがコードと読んで自分の能力阻害しない限り...あまり気にしないデバッグを。

「無駄スペース」の議論に関しては、私はそれを購入しません。論理グループの間に空白行を追加して、プログラムをより明確にする傾向があります...

誰も気付かないことではありませんが、これが中括弧が条件と同じ行に属する理由です（非常に長い条件を除き、それはエッジケースです）：

Cでは、これは有効な構成体です。

while（true）;
{
    char c;
    getchar（）; //入力待ち
}

早く！このコードは何をしますか？「入力を求める無限ループ」と答えた場合、あなたは間違っています！入力すらしません。でキャッチされwhile(true)ます。最後にセミコロンがあることに注意してください。このパターンは実際にはより一般的であるように思われます。Cでは、ブロックの先頭で変数を宣言する必要があるため、新しい変数が開始されました。

コード行は思考です。中括弧は、条件またはループを含む思考の一部です。したがって、それらは同じ行に属します。

私は最初の方法が好きです。IMOがすっきりしていて、よりコンパクトで、気に入っています。

編集：ああ、3番目。それはさらに小さく/きれいなので、私は可能な限りそれが一番好きです。

あなたはそれを書くことができます：

you.hasAnswer() ? you.postAnswer() : you.doSomething();

質問に答えるために; 以前は自分の行では中括弧を好んでいましたが、ブラウザでのセミコロンの自動挿入によるバグについて考える必要を避けるために、Javascriptでエジプト風のスタイルを使用し始めました。また、EclipseでJavaをコーディングするとき、デフォルトのブレーススタイルとの戦い（または構成）に興味がなかったので、その場合もエジプト人と一緒に行きました。今、私は両方で大丈夫です。

ここでのほぼすべての回答は、「あなたがすることは何でも、1つまたは2つに固執する」というバリエーションを言っています。

だから私はしばらくそれについて考え、私はそれがそんなに重要だとは思わないことを認めなければなりませんでした。誰もが、次のことは難しいと正直に言うことができますか？

int foo(int a, Bar b) {
    int c = 0;
    while(a != c)
    {
        if(b.value[a] == c) {
            c = CONST_A;
        }
        c++;
    }
    return c;
}

私は他の誰かについてはわかりません...しかし、私は精神的にスタイルを行き来するのに全く問題がありません。コードが何をするのかを理解するのに少し時間がかかりましたが、それはCのような構文をランダムに入力した結果です。:)

私のそれほど謙虚な意見では、中括弧を開くことはコードの可読性とほとんど完全に無関係です。上記のいくつかのコーナーケースでは、いずれかのスタイルが違いを生むことがありますが、ほとんどの場合、空白行を賢明に使用することでクリーンアップできます。

FWIW、私たちの職場のコーディングスタイルは、少し構造化されたフォーム1と修正されたフォーム3を使用します。（C ++）

            // blank line is required here
if (x) {
            //This blank line is required
   y = z;
}
            // blank line is required here too, unless this line is only another '}'

if (x) y = z; //allowed

if (x)
    y = z;  // forbidden

フォーム2を強く好む人がフォーム1のこのバージョンをより良く見つけられるかどうか、私は興味があります。なぜなら、空白行がより強い視覚的分離を与えるからです。

これはまだ提起されていないことに驚いています。ブロックをより簡単に選択できるので、2番目のアプローチの方が好きです。

ブレースが同じ列と独自の行で開始および終了する場合、マージンから選択するか、列0のカーソルで選択できます。これは通常、マウスを選択するとより寛大な領域になり、キーボードを選択するとより少ないキーストロークになります。

もともとは条件付きと同じ行でブレースを使用していましたが、切り替えたときに作業速度が加速することがわかりました。もちろん、昼夜ではありませんが、条件文の横にある中括弧を使用すると作業が少し遅くなります。

個人的には2番目の方法が好きです。

しかし、私がこれから実証する方法は、最高の仕事の安全をもたらすため、私の意見では最高です！私の大学の仲間の学生が宿題を手伝ってくれと頼みましたが、これが彼のコードの様子です。プログラム全体が1つのブロックのように見えました。興味深いのは、彼が作成したプログラムのバグの95％が、括弧の不一致によるものだということです。他の5％は、ブレースが一致すると明らかになりました。

while(1){
i=0;
printf("Enter coded text:\n");
while((s=getchar())!='\n'){
         if(i%1==0){
            start=(char*)realloc(input,(i+1)*sizeof(char));
if(start==NULL){
printf("Memory allocation failed!");
exit(1);}
input=start;}
      input[i++]=s;}
start=(char*)realloc(input,(i+1)*sizeof(char));
if(start==NULL){
printf("Memory allocation failed!!!");
exit(1);}
input=start;
input[i]='\0';
                puts(input);

私の個人的な好みは最初の方法です。おそらくそれが最初にPHPを学んだ方法だからです。

単一行ifステートメントの場合、次を使用します

if (you.hasAnswer()) you.postAnswer();

そうではないyou.postAnswer();が、もっと長いもの、たとえばyou.postAnswer(this.AnswerId, this.AnswerText, this.AnswerType);最初のタイプに戻るとしたら：

if (you.hasAnswer) {
    you.postAnswer(this.AnswerId, this.AnswerText, this.AnswerType);
}

改行は使用しませんelse。また、ステートメントがある場合は、このメソッドを使用しません。

if (you.hasAnswer()) you.postAnswer();
else you.doSomething()

理論的な可能性ですが、私が今まで使用したことはありません。これはに変換する必要があります

if (you.hasAnswer()) {
    you.postAnswer();
} else {
    you.doSomething();
}

彼らはすべきではありません。私の最初の方法。

2番目の行を見ると、使用されていない行（最後の閉じ中かっこ以外に中かっこのみが含まれている行）があるため、コードの連続性が壊れているように感じます。私は通常、コードの目的またはこのような何かの分離を意味する空の行に特別な注意を払う必要があるため、それを速く読むことはできませんが、「この行は中括弧に属している」ことはありませんインデントの）。

とにかく、テキストを書くときと同じように、段落の先頭にインデントを追加することは、その前に空白行がある場合は不要です（段落変更の二重記号）、中括弧の行を無駄にする必要はありません適切にインデントします。

さらに、既に述べたように、画面に多くのコードを収めることができますが、そうでなければ少し逆効果になります。

プラットフォーム/言語/慣習に依存します

Javaの場合：

void someMethod() { 
     if (you.hasAnswer()) {
         you.postAnswer();
     } else {
       you.doSomething();
     }
}

C＃で

void someMethod() 
{ 
     if (you.hasAnswer()) 
     {
         you.postAnswer();
     } 
     else 
     {
       you.doSomething();
     }
}

Cで：

void someMethod() 
{ 
     if (you_hasAnswer()) {
         you.postAnswer();
     } else {
       you_doSomething();
     }
}

Javaの人がC＃コードで自分のスタイルを使用し、その逆の場合は嫌です。

私が言えることは、もしあなたが方法＃3のファンなら、あなたは地球上のすべてのIDEコードフォーマッタによって迫害されるだろうということです。

最初の方法を使用する理由は、それがよりコンパクトで、画面上でより多くのコードを使用できるためです。私自身、ブレースのペアリングに問題はありませんでした（if条件を追加する前にステートメントと一緒に常に記述し、ほとんどの環境では対応するブレースにジャンプできます）。

ブレースを視覚的にペアにする必要がある場合は、2番目の方法をお勧めします。ただし、それにより一度にコードが少なくなり、より多くのスクロールが必要になります。そして、少なくとも私にとっては、適切に配置されたブレースを持っているよりも、コードの読み取りに大きな影響を与えます。スクロールが嫌いです。繰り返しになりますが、1つのifステートメントをスクロールする必要がある場合は、大きすぎる可能性が高く、リファクタリングが必要です。

しかし; 最も重要なことは一貫性です。どちらか一方を使用してください-両方を使用しないでください！

12でプログラミングを初めて学んだとき、Microsoftのコーディングチュートリアルはそのようなものなので、中かっこを次の行に追加しました。そのとき、4スペースTABSでインデントしました。

数年後、私はJavaとJavaScriptを学び、同じ行の中括弧のコードを見たので、変更しました。また、2スペースのスペースでインデントを開始しました。

中括弧を揃えるが、スペースを無駄にしない4番目のオプションがあります。

if (you.hasAnswer())
{    you.postAnswer();
     i.readAnswer();
}
else
{   you.doSomething();
}

唯一の問題は、ほとんどのIDEのオートフォーマッターがこれにこだわっていることです。

コーディングの制約や標準がプロジェクトマネージャーによって設定されているプロジェクトに取り組んでいない限り、そのプロジェクトに取り組んでいるすべてのプログラマーがコーディング中に従う必要があるということは、すべてあなた次第です。

私は個人的には第一の方法を好むでしょう。

また、私はあなたが3番目の方法で見せたいものを手に入れませんでしたか？

それは間違った方法ではないでしょうか？たとえば、次のような状況を考えます。

if (you.hasAnswer())
  you.postAnswer();
else
  you.doSomething();

誰かがifブロックにさらにステートメントを追加したい場合はどうでしょうか？

その場合、3番目のメソッドを使用すると、コンパイラは構文エラーをスローします。

if (you.hasAnswer())
   you.postAnswer1();
   you.postAnswer2();
else
   you.doSomething();