PHPの書き方について！！

Question

友人とPHPのコードの書き方についてどっちが見やすく綺麗か話し合ってます。
まあ、簡単に言うとただのリファクタリングですね！

// pattern - 1
class FB
{
  function __construct()
  {}
  function fizzBuzz()
  {
    $fizz = 3;
    $buzz = 5;
    if( (0 < $fizz && $fizz < 100) && (0 < $buzz && $buzz < 100) )
    {
      for( $i = 1 ; $i <= 100 ; $i++ )
      {
        $f = $i % $fizz;
        $b = $i % $buzz;
        if( $f === 0 && $b === 0 )
        {
          echo "FIZZ BUZZ({$i})<br />";
        }
        elseif( $f === 0 )
        {
          echo "FIZZ({$i})<br />";
        }
        elseif( $b === 0 )
        {
          echo "BUZZ({$i})<br />";
        }
        else
        {
          echo "{$i}<br />";
        }
      }
    }
    else
    {
      echo '入力は 1 以上 100 未満';
    }
    return 0;
  }
}

// pattern - 2
class FB {
  function __construct() {}
  function fizzBuzz() {
    $fizz = 3;
    $buzz = 5;
    if ((0 < $fizz && $fizz < 100) && (0 < $buzz && $buzz < 100)) {
      for ($i = 1 ; $i <= 100 ; $i++) {
        $f = $i % $fizz;
        $b = $i % $buzz;
        if($f === 0 && $b === 0) {
          echo "FIZZ BUZZ({$i})<br />";
        } elseif($f === 0) {
          echo "FIZZ({$i})<br />";
        } elseif($b === 0) {
          echo "BUZZ({$i})<br />";
        } else {
          echo "{$i}<br />";
        }
      }
    } else {
      echo '入力は 1 以上 100 未満';
    }
    return 0;
  }
}

もっと綺麗で見やすい書き方等あればアドバイス下さい！

※例としてのコードなのでFizzBuzz問題のシンプル化を突っ込まれても困るのでご了承下さいWW

Answer 1

リファクタリングというよりコーディング規約の問題ではないでしょうか？

特別な理由がない限り（フレームワーク独自のルールがある等）、
PHPのコーディング規約はPSRに従えばいいと思います。
PSR

PHP界隈のお偉いさん？があつまって決めたルールで日本語の翻訳してくれている人もいます。
（ぐぐれば出てくると思います。）

掲題のコードでいう制御構造のブロック開始位置はPSR-2に記載があります。

Answer 2

おそらくあなたの言っている
「FizzBuzz問題のシンプル化」を(含めてコードを整理したり)することが
「リファクタリング」です。

さて、{}の位置ですが、
複数人での開発：規約に従う。
一人で組むけど誰かが見るかもしれない：言語で一般的な方に合わせる。
誰も見ない：自分の好み。
でいいんじゃないでしょうか。

Answer 3

IDE を使っている人ならツールの自動整形を使うんじゃないでしょうか。
これは綺麗で見やすいというだけでなく、表記揺れが少なくなるので統一性の問題も解決してくれます（ツールがバラけた場合はまた別の問題が発生する気もするけど…）。
ちなみに Eclipse（Pleiades の PHP 版インストール直後）で整形した結果はほぼ２番目のようです。
事例だと半角スペースをつかっているように見えるけど Eclipse だとタブが使われるようです。
後はだいたい一緒っぽい。