PHPでの関数作成に関するヒント
- 関数名は動詞にすべきです。
- 各関数の名前は、その関数が行うことを正確に反映している必要があります。
- 関数は、その名前から明らかに推測されることのみを行い、他のことを行うべきではありません。
- 各関数は、単一のアクションのみを実行すべきです。
- 関数内でヘルパー関数を使用してください。
-
関数のコードの長さは、
10から15行を超えないようにするのが良いでしょう。 - 長い関数は、複数のヘルパー関数に分割することをお勧めします。
-
関数名には一般的に受け入れられている接頭辞を使用してください:
show、get、set、calc、create、change、check。 - 重複するコードは関数に抽出してください。
以下のコードにはどのような欠点があるか書き出し、修正してください:
<?php
function sum($arr) {
$res = 0;
foreach ($arr as $elem) {
$res += $elem;
}
return $res / count($arr);
}
?>
以下のコードにはどのような欠点があるか書き出し、修正してください:
<?php
function func($arr1, $arr2) {
$res1 = 0;
foreach ($arr1 as $elem) {
$res1 += $elem;
}
$res2 = 0;
foreach ($arr2 as $elem) {
$res2 += $elem;
}
return $res1 / $res2;
}
?>
以下のコードにはどのような欠点があるか書き出し、修正してください:
<?php
function getSum($arr) {
$res = 0;
foreach ($arr as $elem) {
$res *= $elem;
}
return $res;
}
?>