今天看了别人的一段代码,看完之后"被虐"了千百遍,我真不知道他为何要这样写.
我建议新手们这样写!!!!!
首先变量以及方法的命名:大家基本都遵循驼峰,所以你就不要奇葩的写命名了.
//基本变量命名
//多个用户你的加一个s吧,所以要用$users不要再用$user了,这样你遍历的时候$value可以写成$user,是不是感觉很优雅
$users = User::where('active',true)->get();
//方法命名 你这样命名别人一看就知道你是用地址id去删除一个地址,因为删除地址有可能是用user_id去删除的,所以方法命名一定要让别人看出来你在干什么,用的是什么参数
public function deleteAddressById($id)
{
}
再者就是代码利用率!再者就是代码利用率!再者就是代码利用率!
重要的事情说三遍!!!!!
比如一个商城保存收货地址(这里举一个小小的例子)
你应该把保存收货地址写在AddressManager类里面,写成一个saveAddress方法
这样你在pc端和移动端个人中心、提交订单页面保存收货地址都调用这个方法.
这样你是不是就写了一个代码就把所以的事情都干了,当需求变更的时候你是不是只需要改AddressManager类里面saveAddress方法就可以了,其他的代码都不用动.这样保证你代码结构的完整性.
所以大家一定要保证代码的利用率!!!最后祝大家工作愉快.
网友评论