PHP(に限らずですが)可読性の良い書き方について

Question

プログラミングに少しずつ慣れてきたところで、
可読性というものをもう少し気にしなければと思うようになりました。

実際、まだまだ未熟な点もあり、書いている処理自体がそもそもスマートでないのも問題なのですが、
ちょっとしたスペースやインデントの使い方、改行だけでももう少し改善できたらと思い、質問させて頂きました。

下記はPHPになりますが、

//①変数などのイコール位置を合わせるか？
$hogehoge = 'apple';
$color         = 'red';

//②ifと()と{}の間に半角スペースを入れるか？ →if(){}なのかif () {}なのか
if ($hoge == 'apple') {

 //③変数と文字列のつなげ方はどれが良いか
 echo 'apple is '.$color.'.'; //基本シングル''+.で繋ぐ
 echo "apple is $color."; //基本ダブル""で囲む
 echo 'りんごは' . $color . 'です。'; //接続する.の前後にスペースを入れる
 $str     =  'りんごは'.htmlspecialchars($color).'です。'; //ここではhtml使ってませんがこういう変換をするときや
 $query = 'SELECT * FROM `hoge` WHERE `color` = ' . mb_convert_encoding($color) . ''; //SQL文にするときなど

} else {
 echo 'not apple.';
}

かなり雑で、場合によるよ！！というものもありますが、こうしたほうがメリットがある。可読性がある。
こんな書き方しません！ここはこだわったり統一しています！という書き方があれば、教えて頂けますと幸いです。

ちなみに私は
if(◯◯){
//処理
}
と、スペース入れない派だったのですが、他の方のソースをみているとir (◯◯) { と間にスペースを入れたり、
コメントアウトも「// 処理」のようにスペース入れていたりしてどっちがいいのかな〜と思っています。
( カッコ内 )にもスペースを入れている方も見ますが、何度も(カッコ)が出てきてスペースがあるものと無いものがあって、
どういう基準で入れてるのかな？と思いましたので、スペース入れる派の方がいればコメントいただけると嬉しいです。

また、インデントについてですが、処理と処理の間を改行しているとき、空行にもインデントを入れていますか？

//④インデントを「ー」で表しています。if終了後〜$fooまでの空行にも、インデントを入れるか否か
ーー$hoge = 'インデント';
ーー
ーーif ($hoge = 'インデント') {
ーーー//処理
ーー}
ーー
ーー
ーー$foo = 'スペース';

技術の関係ない質問で恐れ入りますが、何卒よろしくお願い致します。

Answer 1

一つだけ指摘します

$query = 'SELECT * FROM `hoge` WHERE `color` = ' . mb_convert_encoding($color) . '';

これは、SQLインジェクション攻撃の対象となりますので、やってはダメです
フレームワークとしてCakePHPを使うのであればフレームワークが用意するやり方に従うか、またはバインド変数を使うようにするほうがよいと思います

Answer 2

何が正解というものはないです。
オープンソースのフレームワークなど使っているのであれば、それに合わせるのが良いと思います。

すでにいろいろなところがコーディング規約を作っているので、自分の書き方に合ったものを探してそれに合わせていくほうが良いです。

ただ、あまり規約にこだわりすぎると、それがストレスになって作業がはかどりません。
たとえば、転職や派遣先の変更などでコーディング規約を変えざるを得ない時にクセとなって出てしまい大変です。
必要最低限かつ簡潔なものを選択することをお勧めします。

Answer 3

phpのコーディング規約、いくつかあります。
PSR-2準拠を頑張るというのはいかがでしょうか。

参考：
PHPコーディング規約まとめ - Qiita
http://qiita.com/hshimo/items/04be1f432240c58300f4

Answer 4

半角スペースをいれるのは、文字の詰まりを防ぎ
かつ、修正するときにプログラムのコード消しすぎを極力抑える為だと思います。
急に見たことのないのバグがでて、よくよく調べてみたら
いつのまにか、かっこの片割れ消しちゃってたってのはよくあるので。

また、質問から逸れますがif文で異常処理、正常処理を分ける場合、以下のように心がけています。

if (異常ケースの場合){
    異常処理

} else {
    正常処理

}

異常処理はだいたい短いので、一目で異常処理と正常処理を確認できます。
逆に書いてしまうと、長い正常処理を見てからでないと異常処理を確認できません。

最後に、空行へのインデントですが、入れてません。
必要性を感じない為です。