如果类名或对象名告诉你某些东西后,请不要在变量名中重复。
小坏坏:
class Car { public $carMake; public $carModel; public $carColor; //... }
好的方式:
class Car { public $make; public $model; public $color; //... }
使用默认参数而不是使用短路运算或者是条件判断
不好的做法:
这是不太好的因为 $breweryName 可以是 NULL.
function createMicrobrewery($breweryName = 'Hipster Brew Co.'): void { // ... }
还算可以的做法:
这个做法比上面的更加容易理解,但是它需要很好的去控制变量的值.
function createMicrobrewery($name = null): void { $breweryName = $name ?: 'Hipster Brew Co.'; // ... }
好的做法:
你可以使用 类型提示 而且可以保证 $breweryName 不会为空 NULL.
function createMicrobrewery(string $breweryName = 'Hipster Brew Co.'): void { // ... }
对比
使用 相等运算符
不好的做法:
$a = '42'; $b = 42;
使用简单的相等运算符会把字符串类型转换成数字类型
if( $a != $b ) { //这个条件表达式总是会通过 }
表达式 $a != $b 会返回 false 但实际上它应该是 true !
字符串类型 '42' 是不同于数字类型的 42
好的做法:
使用全等运算符会对比类型和值
if( $a !== $b ) { //这个条件是通过的 }
表达式 $a !== $b 会返回 true。
函数
函数参数(2 个或更少)
限制函数参数个数极其重要
这样测试你的函数容易点。有超过 3 个可选参数会导致一个爆炸式组合增长,你会有成吨独立参数情形要测试。
无参数是理想情况。1 个或 2 个都可以,最好避免 3 个。
再多就需要加固了。通常如果你的函数有超过两个参数,说明他要处理的事太多了。 如果必须要传入很多数据,建议封装一个高级别对象作为参数。
不友好的:
function createMenu(string $title, string $body, string $buttonText, bool $cancellable): void { // ... }
友好的:
class MenuConfig { public $title; public $body; public $buttonText; public $cancellable = false; } $config = new MenuConfig(); $config->title = 'Foo'; $config->body = 'Bar'; $config->buttonText = 'Baz'; $config->cancellable = true; function createMenu(MenuConfig $config): void { // ... }
内容版权声明:除非注明,否则皆为本站原创文章。