PHP配列のカンマは？前に置く？後ろに置く？

Question

配列を書くときに、カンマの位置をどうしようか悩んでいます。
以下のA、B、Cについて、みなさまのご意見をお聞かせください

PHP配列のカンマの位置は？

A：前に置く派
B：後ろに置く派
C：状況による派（例えば？）

（理由もあればうれしいです）

// A：前に置く派
$array = [
    ['id'=>1, 'name'=>'いち', 'site'=>'yahoo', 'drink'=>'beer']
    ,['id'=>2, 'name'=>'にい', 'site'=>'apple', 'drink'=>'wine'] 
    ,['id'=>3, 'name'=>'さん', 'site'=>'yahoo', 'drink'=>'beer'] 
];

// B：後ろに置く派
$array = [
    ['id'=>1, 'name'=>'いち', 'site'=>'yahoo', 'drink'=>'beer'],
    ['id'=>2, 'name'=>'にい', 'site'=>'apple', 'drink'=>'wine'], 
    ['id'=>3, 'name'=>'さん', 'site'=>'yahoo', 'drink'=>'beer'] 
];

私はAの方がカンマ忘れのミスがなくていいと思いますが、Aだと一番上の行をコメントアウトするときに、二行目のカンマを削除しないといけないという点で面倒だと感じています

あとはJavaScriptで、jsfiddleでAにするとエラーマークが出るのでその点でもAは不満です

などなど、みなさまのご意見を参考にさせていただきたく思います。
宜しくお願いいたします

Comments

[hentaiman]

javascriptの話を出しているけど、phpの場合とjavascriptの場合で回答変わると思うのですがPhpの場合の回答でいいんですか？

[picnicpi]

なぜどう変わるのかぜひ知りたいです！

[hentaiman]

javascriptだとふるーいブラウザ向けの末尾カンマつけない書き方で書く事に慣れているみたいな回答もあるかなと

[picnicpi]

そういったこともあるんですね！ふるーいブラウザ（？）とやらは、末尾にカンマをつけるBだとエラーなのでしょうか？

[hentaiman]

そうです　今ではお目にかかれません
回答しようと思ったけどAでもBでもどうでも良い派なので選択肢に無いな

[picnicpi]

あれれそうでしたか（´◦ω◦｀）

Answer 1

まず、今行っている開発のプロジェクトのフレームワークのコーディング規約を確認し、
取り決めがあればそれに従います

参考: PHPコーディング規約まとめ - Qiita

この確認は PHP_CodeSniffer というツールで自動的に確認することが可能です

squizlabs/php_codesniffer - Packagist

また、エディターでも誤りがないか常に確認を行うことができます

• phpcs - Visual Studio Marketplace
• PHP_CodeSniffer - Help | PhpStorm

このような、コーディングルールの細かい部分に関しては、
近年では、意見が分かれる割に生産性に差が生まれない、ということで、
ツールで強制的に統一してカスタマイズもほぼ行わない、というのがトレンドのようです

Comments

[picnicpi]

そうなんですね。ありがとうございます

Answer 2

私も基本的にB派です。（見やすさ的に！）

Comments

[picnicpi]

見やすさは頭が揃ってるということですが？ではこうだといかがでしょうか。

// A：前に置く派
$array = [
_____['id'=>1, 'name'=>'いち', 'site'=>'yahoo', 'drink'=>'beer']
____,['id'=>2, 'name'=>'にい', 'site'=>'apple', 'drink'=>'wine']
____,['id'=>3, 'name'=>'さん', 'site'=>'yahoo', 'drink'=>'beer']
];

※スペースが入らなかったので「_」を使いました

[tr_09]

行のカウント、という意味で末にあったほうが見やすいという意味でした！！すみませんわかりづらくて…

[picnicpi]

いえ、はやとちりすみません
カウントですか…カンマの位置で行のカウントの利便性が上がるというのは…？少しイメージできませんでした

Answer 3

PHPって配列末尾のカンマを許容するから、行で区切るなら常時カンマをつけるのが一番楽だと思います。
結構な言語がこのような挙動をするので、自分は基本的にBのスタイルを採用しています。

<?php
$data = [
    ["hello", "world"],
    ["hello", "PHP"],
];
print_r($data);

実行例

Comments

[picnicpi]

ありがとうございます
常時カンマをつけるというのは統一してていいですね。

Answer 4

私は圧倒的にB派です。
理由は「頭が揃って見た目が綺麗だから」です。

SQLは下記のように前カンマ派ですが。

SELECT
    a
   ,b
   ,c
FROM
    table

これは「1列目以外を変更もしくはコメントアウトする時に便利だから」です。

でも、これってPHPの配列でも一緒ですね。
結局は好みですが、プロジェクトのコーディング規約があれば、それに従う必要がありますね。

Comments

[picnicpi]

そういえばSQLは最後のカンマが許容されないですし、先頭カンマをPHPよりもよく見かけますね。ありがとうございます。