吐槽下前端烂代码

2022 年 3 月 12 日
 ChefIsAwesome

获取了这样的一个数据:

	
    data: {
    	foo: {
        	bar,
        },
    }

接着他把这数据传给组件。传的是 foo 的数据,字段名却写成了 bar 。


	<Component :bar="foo" />  

接下里他写了好几十个组件,全都这么干。this.bar.bar 这样的写法频频出现,他也无所谓,破罐子破摔了。

我这是造了多大的孽,才得来重构这种鬼东西。

4585 次点击
所在节点    职场话题
25 条回复
neutrinos
2022 年 3 月 12 日
老板听到重构就觉得你在摸鱼
gouflv
2022 年 3 月 12 日
说实话,我自己写的 vue 项目,都不太敢重构
EPr2hh6LADQWqRVH
2022 年 3 月 12 日
这几个框架层层盘剥,前端的可维护性简直就是笑话
EPr2hh6LADQWqRVH
2022 年 3 月 12 日
前端最常见的维护手段就是整体重写,重写之后等着过两年再次重写
tonytonychopper
2022 年 3 月 12 日
说明了 lint 和 cr 的重要性= =
shakaraka
2022 年 3 月 12 日
没有类型就是这个样子
vivipure
2022 年 3 月 12 日
项目开发 code review 还是有必要的
kunkunzhang
2022 年 3 月 12 日
一定要 cr ,条件允许的话一个 pr/mr 最好两个人以上 cr 通过再合,大家互相纠正,为了保证代码质量也为了一起进步
hhjswf
2022 年 3 月 12 日
前端有什么重构的必要吗
wu67
2022 年 3 月 12 日
讲真, 如果不是公用组件, 其实根本不用管太多, 因为说不定下周就改了, 或者写完丢上去就再也不会动了, 思考简直浪费时间, 在老板面前也讨不到好...

个人认为只有反复 /多处使用的的公共组件才有规划和重构的需要, 至于其他的, 能把数据逻辑和命名捋清就行
liyang5945
2022 年 3 月 12 日
让你见识见识我司大神的代码
Terminator0826
2022 年 3 月 13 日
this.爸.爸 🌝
yanqing07
2022 年 3 月 13 日
@liyang5945 看起来是 jQ 时代的产物。如果是那个时代的代码,我觉得可以理解。那时候没有 lodash ,没有 filter/map/reduce 这些大家都是层层循环的多
liyang5945
2022 年 3 月 13 日
@yanqing07 #13 是 jq 时代的没错,但是到 2020 年他仍然在写这种代码,真的恶心人
darkengine
2022 年 3 月 13 日
如果是完全接手了,用 IDE 给重命名了吧,不然不知道啥时候一个没注意就出 bug 了
kingjpa
2022 年 3 月 13 日
@liyang5945 糟糕,这代码是我喜欢的风格
cszchen
2022 年 3 月 13 日
@liyang5945 糟糕,是心动的感觉
liyang5945
2022 年 3 月 13 日

@kingjpa #16
@cszchen #17 再来一点,注意行号
kingjpa
2022 年 3 月 13 日
@liyang5945 我收回我说的话
langzibuhuishou
2022 年 3 月 13 日
只会一点点前端,看了楼上截图的代码。带入场景,如果是自己接手的话,已经开始头疼恶心了。

这是一个专为移动设备优化的页面(即为了让你能够在 Google 搜索结果里秒开这个页面),如果你希望参与 V2EX 社区的讨论,你可以继续到 V2EX 上打开本讨论主题的完整版本。

https://www.v2ex.com/t/839892

V2EX 是创意工作者们的社区,是一个分享自己正在做的有趣事物、交流想法,可以遇见新朋友甚至新机会的地方。

V2EX is a community of developers, designers and creative people.

© 2021 V2EX