facebook twitter hatena line email

「Php/コーディング規約」の版間の差分

提供: 初心者エンジニアの簡易メモ
移動: 案内検索
(規約指摘フォーマット)
行30: 行30:
 
  $c = new class1();
 
  $c = new class1();
  
==規約指摘フォーマット==
+
==規約指摘フォーマット(コードレビュー)==
 
*120文字(psr2 120文字ソフトリミット)超えてますが、そのままいきます? 参考:http://www.php-fig.org/psr/psr-2/ 2.3. Lines
 
*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
 
*カンマの後に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
 +
*ifの後と"){"の間はspace入れたほうが良いかもです。参考:http://www.php-fig.org/psr/psr-2/ 5.1. if, elseif, else
 +
*この変数は不要でそのまま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();

規約指摘フォーマット(コードレビュー)