个人比较反感的一些写法

lyxxxh 的个人博客 / 1 / 13 / 创建于 5年前 / 更新于 5年前

这只是我个人的感觉

() 里面的数字代变我个人讨厌程度。1-5吧

这些令我反感的，我都做过…

重要逻辑不拆分 (4)

这种才是最为讨厌的,一旦出bug(也很容易出),调试是非常麻烦的。

对于订单结算这些复杂操作,我认为必须要用事件之类的来解耦。

用DB查询 (3)

DB::table('user')->find(1);

模型关联: 减少非常多代码，两表或者三表关联过于噩梦 (主要原因)
统一入口: 用模型的关于这个表操作全部都是这个模型
模型自带的功能

一堆的if (2)

if(! $request->name) return $this->error("姓名不能为空");
if(! $request->phone)  return $this->error("电话不能为空");
if(! $request->phone)  ....
if...

个人觉得, 如果表单验证需要2个if以上就该用验证器或封装。

封装”完美”代码 (2)

获取轮播图

$this->bannerService->getBanner();
//这个bannerService -> bannerRepository -> Banner(Model)

实属蛋疼,对于简单的项目,尽量不封装才是最好的。

laravel

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

专心学习不瞎搞

讨论数量: 13

刘天承

课程读者 44 声望 / php开发工程师 @ 保密

哈哈哈哈哈

5年前评论

梧桐树下

70 声望

卧槽，你是远萌传媒的程序员呀。这家公司之前面过呢，老板有点逗 :joy:

L学习不停

讲讲怎么个逗法，吃瓜群众喜闻乐见

梧桐树下（作者）

@L学习不停我是最早面的，当时一个技术也没有。最后谈好配电脑，反悔不配。还跟我说，行政比你高点工资，毕竟要招个漂亮的。直接吐了

@dengminfeng 确实是有点逗，序员还要求长相了，哈哈哈

oneSmile

@dengminfeng 毕竟要招个漂亮的也太狠了

@L学习不停哈哈，不是程序员，是行政人员这种，说明程序员不值钱

调调

电脑都不配，太搞笑了吧。。

Wi1dcard

@dengminfeng 就冲漂亮行政，入职！😂

aen233

吃瓜群众吃个瓜，哈哈哈，博主写的几个也是我反感的，但是很多时候还是要遵循团队规定

lyxxxh

1.3k 声望

@dengminfeng :see_no_evil:

Epona

站长 1.8k 声望 / 石头剪刀部长 @ Poper

if 什么的还好，写那种一行的if 不加括号的看着才蛋疼😂

@Epona

??? 你是不是看过我以前的文章我为什么会从程序不喜欢加{}到加{}

我最近老毛病又犯了不想加{}

file
(这个return false没用了多余代码)

现在我觉得 {}不加也可以，需要缩进合理很容易区分代码块

你这个checkStock方法阅读性让人一言难尽

没，我只是单纯的喜欢全加上括号😂，哈哈。讲道理，if条件多了，如果只是在一个地方用，没什么太大的必要封装，多个地方公用的话，还是要封装一下的

lyxxxh （作者）（楼主）

哪里难了吗

有本事你function也别加括号 🤣

@调调 php语法不允许但是js我还真不加

this.http('course',{"type":this.type}).then( val => this.list = val )

getters: {
  getWebsocket: state => state.websocket
},

如果只有一行代码,js加{}是浪费空间。

犯二青年

这里的方法接收值我会用小驼峰，哈哈哈，我是格式强迫症患者😃

yonghuming945

课程读者 11 声望

我改过同事二十多个if条件套用的代码 :joy:

tianlan

那是真的难搞

file

公司以前换过电脑配置，那些配置… 你懂的所以我是自带电脑的这也是我比较喜欢这个公司的原因之一
其他方便一般般吧。。

你肯定是几年前的，因为两年前我都在这个公司

@dengminfeng

@lyxxxh 有补贴就爽吧，没的话一点都不爽呢

哈哈，自带电脑不是喜欢的原因吧，他之前就是反悔不给配台式机，要我自己带电脑。而且还是单休的制度，现在不知道是不是

@dengminfeng 是单休的，不过自己带电脑更爽啊

小广告：来看看我司 - 所有工程师齐刷刷 iMac + 一块额外的 DELL 4K 屏 https://join.rightcapital.com/

xingkong12138

见习助教 190 声望

@yonghuming945 牛皮，这种代码，看着就头疼

largezhou

课程读者 682 声望 / Ctrl+CV工 @ LZzzz

我看过同事，前面的数据库更新/插入操作返回的结果存在 $bool, $bool1, $bool2... 中，其中还有其他方法调用，方法调用方法，都是这种返回值。

最后来一个：

if ($bool && $bool1 && $bool2 && ... && $bool16) {
    DB::commit();
    // 其他逻辑
} else {
    DB::rollback();
    return false;
}

完美收官，，，

largezhou （作者）

@调调那个可不止，，，他这十几个可能失败的步骤，均没有任何错误信息。你流程失败，只知道那十几个中，至少有一个失败，，，就说你懵不懵逼

这个可太秀了。。。

变量名取好一点也是不错

qufo

@调调关键是变量取好名了之后，你也不知道是哪个出问题了。你只知道这16个之中有至少一个假值而已。

@qufo 而且这是最外层的，，，有可能某个 $boolx，还是某个方法调用返回的结果，而方法里面也是好几个 $bool，，，哈哈哈哈哈

这个可太秀了+1

_Daniel

哈哈哈我也这样干过很长一段时间只要把这些结果
$bool && $bool1 && $bool2 && ... && $bool16
都返回了就知道是哪个错了

游离不2

181 声望 / 后端代码搬运工 @ 不二科技

最讨厌之一：无线嵌套 if-else

qingshui

俄罗斯套娃

0Robert0

课程读者 28 声望

敢问各位见过一个控制器类6、7千行代码，几乎把整个模块的所有功能代码全都耦合在这一控制器里，然后里面动不动来个方法就是大几百行，还有不明所以的大段空白、随意且不整齐的缩进，掺杂着令人费解的变量名称，一言难尽的逻辑，到处复制粘贴的冗余代码，最最重要的是整个控制器里没多少注释…..

萧潇

见过有这个味道哈哈

0Robert0 （作者）

@萧潇同样可怜的大佬抱抱

Jea

几万行都见过。。

@Jea 我去，几万行。。。这可有点厉害，不，能在这种文件上增删改代码厉害死了，不过按照这个代码量推算项目应该不小啊，团队里就没有代码审查流程或者就没人提出异议吗

人厶八夂

正在维护，上万行的

@人厶八夂老哥厉害

@0Robert0 老项目的业务代码，编辑器打开都卡，基本没什么问题就没人去管了

我现在的一个app项目就是这样，所有方法都在一个控制器qwq...

fatrbaby

280 声望 / 首席换水官 @ 名剑风流

不遵循psr规范的代码均不能忍受。

Dennis_Ritchie

898 声望 / Golang,Rust，K8s，Docker,Rust,C/C++，数据库爱好者，操作系统爱好者 @ Coder

天下武功，无坚不摧，唯快不破，在老板的眼里，功能才是最重要的

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

帮助

个人比较反感的一些写法

这只是我个人的感觉

重要逻辑不拆分 (4)

用DB查询 (3)

一堆的if (2)

封装”完美”代码 (2)

社区赞助商

关于 LearnKu

资源推荐

服务提供商

其他信息

个人比较反感的一些写法

这只是我个人的感觉

重要逻辑不拆分 (4)

用DB查询 (3)

一堆的if (2)

封装”完美”代码 (2)

社区赞助商

关于 LearnKu

资源推荐

服务提供商

其他信息

请登录