「Php/コーディング規約」の版間の差分
提供: 初心者エンジニアの簡易メモ
(→規約指摘フォーマット) |
(→規約指摘フォーマット(コードレビュー)) |
||
行35: | 行35: | ||
*ifの後と"){"の間はspace入れたほうが良いかもです。参考:http://www.php-fig.org/psr/psr-2/ 5.1. if, elseif, else | *ifの後と"){"の間はspace入れたほうが良いかもです。参考:http://www.php-fig.org/psr/psr-2/ 5.1. if, elseif, else | ||
*ifの後と"){"の間はspace入れたほうが良いかもです。参考:http://www.php-fig.org/psr/psr-2/ 5.1. if, elseif, else | *ifの後と"){"の間はspace入れたほうが良いかもです。参考:http://www.php-fig.org/psr/psr-2/ 5.1. if, elseif, else | ||
− | * | + | *return前の変数は不要でそのままreturnが良いかもです。参考:https://dev.eclipse.org/sonar/rules/show/squid:S1488 |
2017年3月27日 (月) 11:56時点における版
psrコーディング規約
いろんなプロジェクトの共通部分をまとめたコーディング規約
http://9ensan.com/blog/programming/php/php-psr-coding-standards/
PHP_CodeSnifferをインストールして以下コマンドで確認可能
$ phpcs --standard=PSR2 Sample.php
pearコーディング規約
http://phpspot.net/php/man/pear/standards.html
zendコーディング規約
http://framework.zend.com/manual/ja/coding-standard.html
コーディング規約忘れがちまとめ抜粋
それぞれ以下の通りコーディングすることが好ましい。
foreach ($arr as $key => $value) { } class Class1 { public function Class1() { return 1; } } define('BASE', (dirname(__FILE__)); require_once 'ActionClass.php';
最後に括弧をつける
$c = new class1();
規約指摘フォーマット(コードレビュー)
- 120文字(psr2 120文字ソフトリミット)超えてますが、そのままいきます? 参考:http://www.php-fig.org/psr/psr-2/ 2.3. Lines
- カンマの後にspaceがありませんでした。 参考:http://www.php-fig.org/psr/psr-2/ 4.4. Method Arguments
- ifの後と"){"の間はspace入れたほうが良いかもです。参考:http://www.php-fig.org/psr/psr-2/ 5.1. if, elseif, else
- ifの後と"){"の間はspace入れたほうが良いかもです。参考:http://www.php-fig.org/psr/psr-2/ 5.1. if, elseif, else
- return前の変数は不要でそのままreturnが良いかもです。参考:https://dev.eclipse.org/sonar/rules/show/squid:S1488