関数の波括弧の開始位置について

Question

私は全ての関数の波括弧の開始位置を丸括弧の次の行にしていますが、丸括弧のすぐよこにするメリットは何でしょうか？
そちらの方が見やすいという人のためでしょうか。

Answer 1

ifやforやwhileなどでは同一行に書くというスタイルでやっている場合は、揃えるのが一貫性のある姿勢だと思います。これらでも行を分けるのであればどうぞ分けてください。

ちなみに「関数定義のときだけ行を分ける」書き方にも理由があって、いにしえのC（ANSI Cの前）では関数の引数リストが

void foo(hoge, fuga, ...)  //引数は名前だけの宣言
int hoge, fuga;   //型の宣言です
...
{
   ...
}

という構文だったので、関数定義では自然と波括弧を行を分けて書くようになり、その名残が今も続いている・・・とされていると思います。関数定義以外では同じ行に書くのは、紙媒体で出版するときに行数を圧縮するためみたいなことが言われていますね。

私は学び始めの頃この意味がわからなかったので、すべて行を分けないで丸括弧の隣に書く書き方で覚えてしまいましたが。

K&Rスタイルで納得できないこと - 涼の成長記録
字下げスタイル - Wikipedia
プログラミング言語C - Wikipedia

Comments

[otn]

昔は gcc -traditional で、上記引数形式をコンパイルできていたのですが、いつの間にかできなくなっています。

Answer 2

いろいろな人がいるので、いろいろな流儀があります。
Man page of INDENT

{を同じ行に書く理由は行数を減らすためです。画面は有限なので、情報の少ない行（括弧だけの行）を減らしたほうが、1画面で見えるコード量が増えますので、読んだりデバッグしたりするとき効率的です。。
また、ifやwhile等と同じ行に{を書くスタイルの場合は、それと統一するという意味があります。

Answer 3

見やすいと感じるかどうかは人それぞれですね。今どきの若い人には想像もつかないことでしょうが、昔は「80桁×25行」が画面に表示できる最大文字数だったことが長い期間ありました。その中に納まるようになるべく行数を減らそうとしたものです。その習慣が今も続いているのでしょう。どこぞのコーディング規約では「1行は80文字以内」などと決めているところもあります。

まぁ、おそらく多くの人は特にメリットとか気にせずに、単に勉強の過程で参考にしたソースコードが制御文と同じ行に{を書くスタイルだったので、そういうものだと思ってそう書いているだけだとは思いますけどね。あとは、今どきのテキストエディターには大抵コード整形機能が付いていますが、デフォルトの設定では制御文と同じ行に書くスタイルのものも結構あります。

ちなみに、私も{は改行してから書く派です。ブロックの範囲が一瞬で判るのでプログラムの制御構造が把握しやすいということと、行単位での編集がしやすいというメリットがあります。デバッグの際も、「一時的にif文をコメントアウトしてブロックの中身を実行させてみたい」などということがやりやすいですし。

Answer 4

こんにちは。

既に複数の回答があるように、 { を関数定義と同じ行に書くのは書式ルールを単純化しつつ行数を削減できるというメリットがあります。これはこれで有用な視点と思います。
ただ、私は個人的な見解ですが、それはメリットよりデメリットの方が痛いと思うので、逆の視点を補足させていただきます。

C++で複雑な引数を持つ関数や複雑な条件式を書いていると、なが～～～～い文が超痛いです。
下記は今サンプルとして書いただけですが、実際のプログラミングの場面ではこれを遥かに越えるものが結構発生します。

#include <iostream>
#include <functional>
#include <vector>
#include <list>

std::list<int> extract(std::vector<int> const& iVector, std::function<bool(int)> iCondition) {
    std::list<int> ret;
    for (int elem : iVector) {
        if (!iCondition(elem)) {
            ret.push_front(elem);
        }
        if (iCondition(elem)) {
            ret.push_back(elem);
        }
    }
    return ret;
}

int main() {
    auto extracted = extract({-152, 44, 9, -29, 33}, [](int x){return x >=0;});
    for (int item : extracted) {
        std::cout << item << "\n";
    }
}

そのような時は、パラメータ毎に改行する人が多いと思いますが、その際に) {という行は泣きたくなる程 意味不明なので、パラメータ毎の改行をするのかどうかその度に悩みます。

更に、上記はまだforブロックやifブロックの中が僅かしか無いのでましですが、数行～十数行を越えてくるとブロックの区切りを見落とし易くなりますが、{と}のインデントを揃えることで対策になります。

#include <iostream>
#include <functional>
#include <vector>
#include <list>

std::list<int> extract
(
    std::vector<int> const& iVector,
    std::function<bool(int)> iCondition
)
{
    std::list<int> ret;
    for (int elem : iVector)
    {
        if (!iCondition(elem))
        {
            ret.push_front(elem);
        }
        if (iCondition(elem))
        {
            ret.push_back(elem);
        }
    }
    return ret;
}

int main()
{
    auto extracted = extract({-152, 44, 9, -29, 33}, [](int x){return x >=0;});
    for (int item : extracted)
    {
        std::cout << item << "\n";
    }
}

ただ、流石に下記は間延びしすぎて「見苦しい」レベルになので悩ましいのですが、私は長い文の分割に悩まないことやブロックの見落としを回避する方を優先することにしています。

if (foo)
{
    bar();
}
else
{
    baz();
}

Answer 5

改行は空白と同じ扱いと思いますので, 言語(コンパイラ?)的には改行の位置は問題では無いでしょう.
ですので, 「人のため」であっていると思います.

Answer 6

要するに関数の「宣言」を関数ブロックに含むかどうか、ということなんですけど、全体で統一が取れているならどっちだっていいと思います。

個人的には改行しない派です。if や for, while でも改行しないので、それと合わせている感じです。
if で改行しないのは、

if (...) {
  ...
} else {
  ...
}

と、ブロックの区切りが一行にまとまるから……ですかね。これが

if (...)
{
  ...
}
else
{
 ...
}

と、とくに else の部分が間延びしてしまう感じがします。（else if なら見やすいかも知れませんが）

Answer 7

そこらへんは人の好み、ということで。
同一行にしたほうが行数増えなくて見やすくなるって程度ですね