规范 - 不要使用缩写

循环变量的改进:

不好

foreach($people as $v)

foreach($people as $person)

函数名的改进:

不好

class UserRepository {
    public function fetch($billingId){}
}

class UserRepository {
    public function fetchByBillingId($id){}
}

方法名的单词组合不宜超过 3 个,超过三个可能意味着你需要重构你的方法

public function preAndShipAndNotifyUser()

不过也不是绝对,有时候只是为了单纯的提高可读性

public function newQueryWithoutScope($scope)

来源:laracasts

本作品采用《CC 协议》,转载必须注明作者和本文链接
讨论数量: 4

foreach($people as $v) // 我统一用 v ,免得各种各样的明明看得痛苦。只限定于最里层。多层就不适合了。

4年前 评论
心智极客 (楼主) 4年前

@dvaknheo 这编码的姿势未免会造成可读性低极限,我的团队编写习惯( 建议 )

foreach($articles as $article){
    // todo
}
4年前 评论
dvaknheo 4年前
dvaknheo 4年前

我都是用 $k => $v,正如 for 循环用 $i 一样,如果再嵌套一层的话就是 $k2 => $v2,因为这是在循环里的临时变量。
而且PHP没有块级作用域,如果用 $people as $person 的话,循环完之后 $person 依然存在,一般情况下都不会使用 unset 删除临时变量,所以可能会产生了变量名冲突的问题

4年前 评论
dvaknheo 4年前

很多时候仅仅是因为懒,毕竟命名并不是一件容易的事情。

4年前 评论

讨论应以学习和精进为目的。请勿发布不友善或者负能量的内容,与人为善,比聪明更重要!