コーディング規約括弧内の半角スペースは必要でしょうか

コーディング規約についての質問です。
仕事で他の方が書いたコードを見ていると、以下のような書き方によく遭遇します。

if ( $hoge == 1 )
{
return getResult( $template->str( $hoge ) );
}

個人的には「ifの後ろに半角スペースを空ける」「演算子の前後に半角スペースを空ける」というルールがあるのは納得できるのですが、括弧内の先頭と末尾に半角スペースが入るのがどうにも気持ち悪いです。
ならば直せばいいとも思うのですが、自分が浅学なだけで一般的な書き方であるのかもと悩んでいます。
担当しているエンジニアは既に退職している為意図を聞くことができず、他の方は半角スペースの書き方についてどのように考えていらっしゃるのか知りたいです。

otn

2021/02/19 14:51

質問の仕方が悪かったですね。括弧内の先頭と末尾に半角スペースを入れるコーディングルールを採用されている方は、一体どういう理由でそのルールを採用したのでしょう？他人の決めたルールに倣って採用したので自分での理由は無いという以外の方がいらっしゃれば教えて下さい。かな。

行動規範の内容に同意します

回答3件

表記方法はチームによってことなるのでチームが決めた方針に合わせるだけです。
決まってなければ極端に合理性のない書き方でなければ裁量によります。
チームで運用するときはたいてい統合ツール上で行うでしょうから
あまり個人の趣向は反映されないと思いますが・・・

投稿2021/02/19 05:39

yambejp

総合スコア116724

ベストアンサー

担当しているエンジニアは既に退職している為意図を聞くことができず、他の方は半角スペースの書き方についてどのように考えていらっしゃるのか知りたいです。

既に回答のある通り、チーム内で合意が取れていれば他の「一般的な認識」よりも優先されるかと思います。

個人的には、コーディング規約（今回の質問の様な形式的な部分）を人間の力で守るのはただの無駄な労力なので、IDEがサポートしているメジャーなコーディング規約に沿い、IDEの自動整形機能によって解決するべきと考えます。

その辺を考えると、(フレームワークで別のコーディング規約が採用されている場合を除いては)
PSR-2一択になり、PSR-2では制御構文については

5. Control Structures
The general style rules for control structures are as follows:

There MUST be one space after the control structure keyword
There MUST NOT be a space after the opening parenthesis
There MUST NOT be a space before the closing parenthesis
There MUST be one space between the closing parenthesis and the opening brace
The structure body MUST be indented once
The closing brace MUST be on the next line after the body
The body of each structure MUST be enclosed by braces. This standardizes how the structures look, and reduces the likelihood of introducing errors as new lines get added to the body.

5.1. if, elseif, else
An if structure looks like the following. Note the placement of parentheses, spaces, and braces; and that else and elseif are on the same line as the closing brace from the earlier body.

<?php
if ($expr1) {
    // if body
} elseif ($expr2) {
    // elseif body
} else {
    // else body;
}
The keyword elseif SHOULD be used instead of else if so that all control keywords look like single words.