这是一个创建于 933 天前的主题，其中的信息可能已经有所发展或是发生改变。
- 变量命名用拼音，不用英语
- 不分离代码，一个 JS 写 3000 多行，一个类 2000 多行
- 拒绝格式化，不改
- 到处复制粘贴，修改代码的方式就是把原来的代码复制一份和之前的逻辑用 if-else 分开，这也是导致他经受过的类代码量陡增的原因
最终导致，他的代码没法调试，一个小 bug ，让我调了两个小时，最终通过打日志才解决。
气得我边骂边捶桌子调式。
- 不分离代码，一个 JS 写 3000 多行，一个类 2000 多行
- 拒绝格式化，不改
- 到处复制粘贴，修改代码的方式就是把原来的代码复制一份和之前的逻辑用 if-else 分开，这也是导致他经受过的类代码量陡增的原因
最终导致，他的代码没法调试，一个小 bug ，让我调了两个小时，最终通过打日志才解决。
气得我边骂边捶桌子调式。
 |
1
wd 2022-04-26 19:49:48 +08:00 via iPhone
他的代码为什么不是他自己调试？
|
 |
2
fnd 2022-04-26 19:50:11 +08:00
润
|
 |
3
liangkang1436 OP @wd 因为他去别的项目组了，我留下来殿后
|
|
4
liangkang1436 OP 同一段代码到处复制，愣是不抽象出一个工具类，结果导致逻辑修改的时候到处找，我不止一次跟项目经理说，重构吧，项目经理说没时间，凑活用
|
|
5
liangkang1436 OP 项目最开始的原班人马大部队已经去进行别的项目的开发了，但是现有的项目需要有开发继续维护，我就是那个留下来的人，经过长时间的迭代，代码已经不满足现有的需求，甚至还成为了阻碍，急需一次重构升级，但是没有时间搞，就这么拧巴着，最终，今天晚上爆发了
|
|
6
liangkang1436 OP 算了，摆烂吧，下班回家自己玩自己的项目的时候上点心就行了
|
 |
7
sprite82 2022-04-26 20:17:55 +08:00  22
代码和你有一个能跑就行[doge]
|
 |
8
v2eb 2022-04-26 20:20:28 +08:00 via Android 1
我的同事临离职前写了好多 bug.
|
 |
9
imzcg2 2022-04-26 20:20:41 +08:00 via Android
您确定是重构？不是重头开始？技术债，哪那么容易还，尤其是加需求的时候，让原来开发的人过来都想跑路，更何况你
|
|
10
liangkang1436 OP @imzcg2 别扎了
|
 |
11
arthas2234 2022-04-26 20:30:10 +08:00
他还在项目组？那不是继续留后患？
贴脸输出：代码截图，发邮件，抄送领导 |
 |
12
echo1937 2022-04-26 20:32:16 +08:00
@liangkang1436 #5 我司的项目历来如此。
|
 |
13
potatowish 2022-04-26 20:35:19 +08:00 via iPhone
去别的项目组继续祸害
|
 |
14
ericls 2022-04-26 20:48:37 +08:00 via iPhone
没有人教吗？谁 merge 的？ CI 怎么过的 review 怎么过的？
有没有可能你们公司奖励这种你唾弃的代码呢？ 至少应该跟团队谈谈 看看大家是怎么想的 |
 |
15
InDom 2022-04-26 20:50:44 +08:00
发帖是靠通灵嘛 🐶
|
|
16
liangkang1436 OP @ericls 功能是没问题的，只是维护起来很麻烦，前期为了项目进度，领导也默许了
|
|
17
ericls 2022-04-26 20:58:21 +08:00 via iPhone
@liangkang1436 吸取经验教训吧 下次用这个为理由 让团队争取给自己更多时间去优化代码结构 理由就是为了以后更好维护
|
 |
18
littlewing 2022-04-26 21:24:19 +08:00
能跑吗？能跑就行
|
 |
19
github2020 2022-04-26 23:05:10 +08:00
团队负责人不管吗？
|
 |
20
FreshOldMan 2022-04-26 23:06:50 +08:00
格式化上 eslint ，代码问题上 code review 啊
|
 |
21
feather12315 2022-04-26 23:10:43 +08:00 via Android
说 review 的，需求来了催上线，匆匆忙忙写好，整理一堆文档，谁都想早上线早省事，不想持续加班几天整整这堆即将上线的代码，写代码的人烦，code review 的也烦。
|
 |
22
micean 2022-04-26 23:31:34 +08:00 via Android 8
为了项目进度 = 没有能力重构
都是借口而已，写个漂亮的业务代码能花多少时间？一直写垃圾代码就是没有那个能力而已，见多了 |
 |
23
buffzty 2022-04-27 00:21:47 +08:00
这根本不是问题啊,他代码垃圾你就多排时间呗.时间够就重写.跟领导要足够多的时间就行.
2 个小时的需求就要 2 天. 1 天的要 5 天 这还是事吗 |
 |
25
maypu 2022-04-27 08:49:20 +08:00
楼主说的不是我的同事吗，哈哈哈
|
 |
26
darkengine 2022-04-27 08:49:25 +08:00
还 review 个毛线啊人都走了，有时间有能力的话，你就大刀阔斧的改吧
|
 |
27
beisilu 2022-04-27 08:58:52 +08:00
我现在状况跟你差不多。但是比你还要惨一点：
1.很多没有被调用的重复垃圾代码。（怀疑是凑行数） 2.明显暴露在线上的 undefined 错误。 关键是，因为原来的人全去做其他东西了，我是做移动端被拉进来顶前端的，现在除了多要时间也不知道该怎么办了。 |
 |
28
IvanLi127 2022-04-27 08:59:28 +08:00 via Android
变量用拼音，还不如用中文呢！哈哈哈哈
|
 |
29
wobuhuicode 2022-04-27 09:27:12 +08:00
这有什么办法呢。等你跳多几家公司你就发现每家公司都会有这样的代码
|
 |
30
yuthelloworld 2022-04-27 09:55:49 +08:00
摆烂吧
|
 |
31
txy3000 2022-04-27 10:04:22 +08:00
屎山才是常态 符合热力学第三定律🐶
|
 |
32
daimubai 2022-04-27 10:06:48 +08:00
大家重构完测试人员不是还需要重新测试一遍吗，如果业务逻辑复杂的话跑不了自动化，这属不属于给人家增加工作量啊
|
 |
33
ufan0 2022-04-27 10:08:01 +08:00
@micean #22
“ 为了项目进度 = 没有能力重构 都是借口而已，写个漂亮的业务代码能花多少时间？一直写垃圾代码就是没有那个能力而已，见多了 ” 把我心里话用语言描述出来了，一直不知道怎么用语言表述~ |
 |
34
stroh 2022-04-27 10:08:19 +08:00 1
3000 行算什么，我们公司好多项目一页 js 1 万行到 3 万行，全部不复用，相似模块从来不提出去，一个函数重复写 10 遍，就因为传值不同，命名大多是词不对意的英文或者不知名的简写，map 从来不 key index ，代码中充斥着 console.log()，控制台 100 多个警告和报错，只是代码能跑，索尼、荣耀、海尔、等大厂都在用我们这套系统，对了，代码还没做压缩...当然没人想搞那屎山一样的代码，变相的保护的源码...
|
 |
35
Vitta 2022-04-27 10:12:13 +08:00
不懂就问，为啥不能用拼音啊，虽然我没用，我甚至都想用汉字
|
 |
36
l00t 2022-04-27 10:34:10 +08:00
又不是不能用…… 重构不如重写。
|
 |
37
root01 2022-04-27 11:24:03 +08:00
拼音又不是看不懂
|
 |
38
zw1one 2022-04-27 11:33:09 +08:00
摆烂，找领导摊牌：这是坨屎，宁看怎么办？你不管我也不管了。
|
 |
39
Dkngit 2022-04-27 11:33:58 +08:00
政企有些事务意译英文也是无法理解的，还不如英译用拼音
|
|
40
buffzty 2022-04-27 11:37:52 +08:00
@beisilu 如果你每天有效工作时间 2 小时,那就至少要 5 天排期.或者十天也行.让领导知难而退 还让之前的人改,或者让你重写. 我遇到屎山都是要么不改 要么重构
|
|
41
liangkang1436 OP @stroh 想想我都头疼
|
|
42
liangkang1436 OP @root01 拼音首字母缩写
|
 |
43
zxxufo008 2022-04-27 12:29:54 +08:00
不能润，还是建议自己精简下，因为估计以后还会有坑
|
 |
44
mosfet 2022-04-27 13:11:10 +08:00
你们组老大咋想的呢？
之前留下来的能跑的代码，我看了一会儿就觉得要重构。给老大看了，也觉的是要重构 然后我理清业务逻辑后，直接重写了，花了一个多星期吧 |
 |
46
ychost 2022-04-27 13:42:19 +08:00
这种代码 CR 肯定过不了
|
 |
47
abenx 2022-04-27 13:47:13 +08:00
我怀疑你再说我
|
 |
49
slmaaw 2022-04-27 13:51:30 +08:00 via Android
接屎山都是这样的感觉 最有效的做法是找个人把屎山丢出去 实在丢不出去只能重构 降低心智负担
|
 |
50
seres 2022-04-27 13:54:37 +08:00
变量用拼音可以用英语不好解释，用首字母缩写就过分了。。。
|
 |
51
aimechan 2022-04-27 13:58:09 +08:00
一般这种情况，PM 也是屎，只顾当前自身利益不管产品长期发展，纵容时间紧凑就压缩质量，是在造成后患，反正出了问题也不是他 PM 的问题，把矛盾转嫁到执行层。
|
 |
52
dongdongdong 2022-04-27 14:22:57 +08:00
不会是我吧
|
 |
53
fyxtc 2022-04-27 14:34:19 +08:00
非杠，其实我觉得这种情况，自己可能也需要反思一下，为什么我会碰到这样的同事
|
 |
54
bzsh 2022-04-27 14:38:04 +08:00
@feather12315 大实话
|
 |
55
ChefIsAwesome 2022-04-27 14:42:35 +08:00
前两天刚碰上个字段名叫 childs 的。见怪不怪。
|
 |
56
Bongzyu 2022-04-27 14:43:22 +08:00 3
这有啥，你那是没见过在注释里撒谎的😭
|
 |
57
dongtingyue 2022-04-27 15:34:05 +08:00
- 变量命名用拼音，不用英语
因为不知道用啥英文。。。例如代币新代币，维护列表运维列表 - 不分离代码，一个 JS 写 3000 多行，一个类 2000 多行 - 到处复制粘贴 半天时间完成一个功能没时间优化，怎么简单怎么来 - 拒绝格式化，不改 下单子花时间优化代码 |
 |
58
edward1987 2022-04-27 15:42:11 +08:00
@Bongzyu #56 哈哈哈哈看来有故事
|
 |
59
sun1993 2022-04-27 15:55:40 +08:00
我的建议是赶紧润
|
 |
60
onionKnight888 2022-04-27 16:14:16 +08:00
润啊
|
 |
61
sogwsc 2022-04-27 17:00:59 +08:00
@arthas2234 一般领导只关心怎么让项目继续平稳的运行下去
即使上一个人天坑 也只会安排你赶紧填上 |
 |
62
cat9life 2022-04-27 17:21:51 +08:00
快跑.jpg
|
 |
63
snowlyg 2022-04-27 17:59:47 +08:00
哪里的公司，怎么招进去的
|
 |
64
Shiu 2022-04-27 18:12:46 +08:00
跟我司代码如出一辙
|
 |
65
kyleduo 2022-04-27 18:50:30 +08:00
有些木匠懂得榫卯，有些熟悉钉子。家具都能立起来，只有木头，知道不同滋味。
|
 |
66
cjban 2022-04-27 19:33:04 +08:00 via iPhone
这个主题配合头像简直完美哈哈哈哈哈哈哈
|
 |
67
ghost024 2022-04-27 19:37:39 +08:00
那你是没见过我之前的一个跑路的同事是用随机数来表示报表的其中一个百分比的数据，我看到代码的时候人都傻了
|
|
68
liangkang1436 OP @ghost024 你永远不知道你的同事写代码的时候脑子里在想什么
|
|
70
ghost024 2022-04-27 19:52:38 +08:00
@liangkang1436 他写了注释，说因为这个统计的数不准确,所以直接 0.8+0.2*随机数，保证数据好看，但是就是对不上:)
|
|
71
liangkang1436 OP @ghost024 66666966
|
 |
72
ochatokori 2022-04-27 20:01:58 +08:00 via Android
我同事也是
加功能就上千行的代码复制粘贴出一个新文件改一小点。 vue 组件里的样式不写 scope 也不写组件内的父级类名直接改 ui 库组件的样式。 处理冲突看都不看直接应用自己的。 我想打屎她 |
 |
73
lsdvincent 2022-04-29 17:32:06 +08:00 via iPhone
又不是不能用
|
 |
74
haah 2022-05-01 18:03:57 +08:00
这不是你同事的错！只能说贵司没有建立质量保证体系。
|
Select Language