等号加空格

$a=1; // 新手用
$a = 1; // 1年+工作经验都会变成这样

挺有趣的事实, 通过看 我以前代码 & 同事以前代码 & 一些新手的代码,
都会慢慢过渡加 空格 的。

变量重复

个人感觉一部分的代码的优化

一眼可以看出, code 变量出现两次。

 let code = this.getCodeByArea(value);
 if( isArea) 
   code = this.getAreaCode(value,check,type)

这样重写变量不香吗?

调用两次 this.getAreaCode 速度会慢些,
我只能说: 慢个 0.1ms 重要吗?

(当然最优的是: 是处理下参数最后再调用函数)

不用else

个人感觉一部分的代码的优化

写代码,我个人不用 else 的。

还有些人类迷惑行为:

if( $res == "hello")
    return true;
else 
    return false;

请问 else 有啥用
见过挺多这种 迷惑行为。

拒绝使用数字做状态

// 即使数据库有注释 但是看也麻烦 (我想只有新手才会用数字)
$order->status = 3;   

// 好处: 1. ide可以点击跳转过去  2. 一眼看懂
$order->status = Order::STATUS_SUCCESS;

使用本地作用域代替Where

这和 [拒绝使用数字做状态] 有类似,都是为了更直观理解代码。
Where:

Address::Where('user_id',Auth::id())->get()

使用作用域:

模型添加

public function scopeCurrUser($query)
{
    return $query->where('user_id', Auth::id());
}

调用:
Address::CurrUser()->get()

本地作用域优势:

复用 (建议创建个 模型基类)
直观

(作用域见: 局部作用)

模型可以考虑使用query()

直接用

一些代码写法推荐

query()

个人感觉一部分的代码的优化

区别

query() 有 ide 提示。
query() 更快,因为他不会经过这两个魔术方法。(这点速度可以忽略就一点点)

用异常替代错误的返回

return

一些代码写法推荐

异常

个人感觉一部分的代码的优化

异常优势

在 子方法 或者说在 任何方法,都可以 中止 后面的代码, 然后运行异常的处理。

这是 return 无法做到的。

推荐

可以看下这个: 重构改善既有代码的设计—笔记
(虽然我没怎么仔细看过)

本作品采用《CC 协议》，转载必须注明作者和本文链接

专心学习不瞎搞

本帖由系统于 4年前自动加精

lyxxxh

1.3k 声望

专心瞎搞不学习

33 人点赞

你将学到如 RESTFul 设计风格、PostMan 的使用、OAuth 流程，JWT 概念及使用和 API 开发相关的进阶知识。

以构建论坛项目 LaraBBS 为线索，展开对 Laravel 框架的全面学习。应用程序架构思路贴近 Laravel 框架的设计哲学。

讨论数量: 24

一个人的江湖

Laravel 8.x 译者 345 声望 / 开发 @ 一个人的江湖

if 不加花括号，我裂开 :joy: :joy:

4年前评论

lyxxxh （楼主）

哈哈

程序员祝融

110 声望

接受不了 if 不加花括号 {}，哈哈哈

4年前评论

风吹过有夏天的味道

课程读者 336 声望 / php小菜鸟 @ myself

复杂逻辑先写 汉字描述 再写具体代码

1.xxx
 // todo
  1.1xxx
   // todo
  1.2xxx
  // todo
2.xxx
// todo

4年前评论

一个人的江湖

Laravel 8.x 译者 345 声望 / 开发 @ 一个人的江湖

if 不加花括号，我裂开 :joy: :joy:

4年前评论

lyxxxh （楼主）

哈哈

Lime

课程读者 0 声望

:+1:学到了，感谢大佬分享！

4年前评论

小学毕业生

课程读者 176 声望

对模型查询加query那个不太建议，如果要ide提示加个minxin 写多了方法都记住了

4年前评论

sunny123456

啥意思？？？

小学毕业生（作者）

@sunny123456 /* @mixin Builder /

guoliang1994

见习助教 105 声望 / 技术小组长 @ bit-labs

官方代码规范PSR-2 照着这个来

4年前评论

lddtime

PSR 的规范好象是非官方的吧，而且编码规范建议看 12 的，2 已经废弃了！翻译：PSR-12 编码规范扩充

程序员祝融

110 声望

接受不了 if 不加花括号 {}，哈哈哈

4年前评论

李小明

课程读者 237 声望

我准备从新按照你的方式优化下我的程序代码

4年前评论

我的世界

你的名字和我的小学同学一样，不会是老同学吧

馨竹之乡

23 声望

我只能说：慢个 0.1ms 重要吗？一个函数0.1，10个函数呢？

4年前评论

zxk

10 个函数 1 ms 确实没影响。

nff93

@zxk 他的意思是10函数就是10ms

假如_丶

146 声望

我隐约的记得if不用花括号是代码不规范的一种。 :see_no_evil:

4年前评论

Adachi

252 声望

我也知道if不加花括号不规范，但是一行的if就是控制不住我自己，最主要我觉得一行的代码不加花括号，再换行好看，而且像if,foreach花括号我都换行 :joy:

4年前评论

lun1bz

见习助教 121 声望

除了作用域，全中 :joy:

4年前评论

啵啵

课程读者 12 声望

忙的时候不会思考怎么优雅，不忙的时候慢慢细磨。

4年前评论

臭鼬

见习助教 178 声望

这样写不行吗？

 let code  = isArea ? this.getAreaCode(value,check,type) : this.getCodeByArea(value);

4年前评论

lyxxxh （楼主）

可以的你这是更好的写法

李小明

不错

zxk

$code = match ((bool)$isArea) {
    true => this.getAreaCode(value,check,type),
    false =>  this.getCodeByArea(value)
};

Mr-houzi

@zxk PHP8语法？

____

课程读者 83 声望

没太看懂第二条 if else 哪里不好。。
不太赞同写一个本地作用域代替一个 where 条件，多个条件的情况下需要很了解每个作用域，但是直接 where 能解决的事情，个人认为这种有点过度封装，而且后期维护很难，还要再去看一下作用域的具体含义

4年前评论

lyxxxh （楼主）

变量重复是我见代码出现过最多

Laravel
$condition['city'] $condition['yr_name'] $condition['tel'] 出现那么多次！

至于作用域 出现两次以上才会考虑使用

上面那个 if else 还好,变量只出现两次。 (不算声明)
但我不喜欢,个人爱好。
我喜欢 合并。

风吹过有夏天的味道

@lyxxxh 这种代码看到就炸

飞飞鱼

@lyxxxh 看到这个我已经炸毛了 :flushed:

ZLSN

@lyxxxh 怎么让它们只出现一次？。。。

xxx

13 声望

不错,大部分都是跟你推荐的一样,尤其是query():joy:

4年前评论

giao哥

121 声望 / 最强王者 @ 阿里妹妹

我就是这样来写代码的哈哈

4年前评论

zhumengyang

课程读者 88 声望

代码写得好，换人换的早 :relieved:

4年前评论

yuancoder

45 声望

代码少了 else 看起来简洁了，但是很容易出现疏忽，如果条件不成立的话，还要往回看代码，很难看出覆盖了所有分支，增加了心智负担。

4年前评论

hawind

39 声望

感觉该啰嗦的不能省，因为写的时候开心了，回头改的时候可能因为一些小技巧，改出bug，JAVA那么繁琐还是那么流行，没有规矩不成方圆

4年前评论

风吹过有夏天的味道

课程读者 336 声望 / php小菜鸟 @ myself

复杂逻辑先写 汉字描述 再写具体代码

1.xxx
 // todo
  1.1xxx
   // todo
  1.2xxx
  // todo
2.xxx
// todo

4年前评论

JustSo

课程读者 16 声望

github上有一个项目叫 clean code，上面有你所有需要知道的写代码技巧。

4年前评论

cxylove

见习助教 0 声望

之前遇到个公司项目一个if 1000行代码还不算else里面，而且7-8层if层数，里面还有大量的foreach循环查数据库想想就刺激一个接口查询数据库上千次的都有，但是那个技术总监还JB说，我们不考虑重构，已经稳定运行了2年，第二天我就闪人了。

4年前评论

李小明

你这个太年轻了，一份工作而已，何必这么认真！能运行就好了，又不是不给你钱！站在技术总监的角度估计也很看不起你！先站稳脚步，做出几个项目来，技术总监看重你后，你想怎么玩，只要你自己不累就行！

颠倒的玉石

课程读者 166 声望 / 技术小垃圾 @ 山西证券

if不加花括号或者if写在一行换行能扣钱么.......

4年前评论

李小明

课程读者 237 声望

其实楼主只是建议而已不必太认真，想要优雅写法 ‘Python ’ 就好了，

3年前评论

jdzor

11 声望

代码加空格只是习惯性用 Ctrl+L 命令，另外使用模型类的人也是不嫌麻烦 :joy:

3年前评论

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

帮助

一些代码写法推荐

等号加空格

变量重复

不用else

拒绝使用数字做状态

使用本地作用域代替Where

模型可以考虑使用query()

直接用

query()

区别

用异常替代错误的返回

return

异常

异常优势

推荐

高认可度评论：

代码写得好，换人换的早 :relieved:

社区赞助商

关于 LearnKu

资源推荐

服务提供商

其他信息

一些代码写法推荐

等号加空格

变量重复

不用else

拒绝使用数字做状态

使用本地作用域代替Where

模型可以考虑使用query()

直接用

query()

区别

用异常替代错误的返回

return

异常

异常优势

推荐

高认可度评论：

代码写得好，换人换的早 :relieved:

社区赞助商

关于 LearnKu

资源推荐

服务提供商

其他信息

请登录