1
rhacker1995 OP 负责 Code review 的人多少年没敲代码,只管需求上线,不管实现。
|
2
DinnyXu 2022-01-10 20:54:52 +08:00
这种项目维护的意义在哪? 跳槽吧,别耽误自己
|
3
fishCatcher 2022-01-10 21:12:10 +08:00 via iPhone 3
换个角度想,你把这些重构或者优化,再包装一下跟老板吹一吹,那不就是升职的好机会?
如果整个项目都很完善,那自己离变成客服或者被强制 325 就不远了 |
4
potatowish 2022-01-10 21:14:35 +08:00 via iPhone 7
维护屎山是个技术活,资深的开发都是完美的绕过前人留下的坑,把屎山再堆高一层
|
5
oneisall8955 2022-01-10 21:47:52 +08:00 via Android
review 等于帮忙合下代码🙃
|
6
thedrwu 2022-01-10 21:55:41 +08:00 via Android 4
如果领导开明,你的晋升机会来了。
如果领导不开明,在屎山上浇点肥,git blame 不到自己就不用背锅。 |
7
Felldeadbird 2022-01-10 23:14:08 +08:00
没有屎的存在 就不会聘请开发了。我已经再屎上多年了,习惯了铲屎和拉屎。
|
8
mineralsalt 2022-01-10 23:38:03 +08:00
和同事一起维护一个项目, 我也很难受, 一点编码规范都没有. 随便举个例子, a.equals("success") , 明明可以避免的空针在错误, 就是视而不见, 每次看到这种写法我都想说, 但是又怕说多了人家不高兴
|
9
pmispig 2022-01-10 23:41:35 +08:00
我们的异常 catch 直接吃掉了,啥都不打。没有打印错误日志就是没错误。
http client 我倒是持反对意见,我讨厌用 httpclient 连接池的人 |
10
az467 2022-01-11 00:12:03 +08:00
之前也干过.....
反射屎山用 javassist 一点点地换 自造垃圾轮子爱用用 反正不维护 就是传参用全局变量和 Map 我是实在没办法 直接绕开写了个 v2 |
11
ferock 2022-01-11 06:04:03 +08:00 via iPhone
你工资多少?心态好,能改一点是一点,今天心情不好,实现需求早点下班写自己的开源项目呗…
|
12
Kontinue 2022-01-11 08:52:28 +08:00
typo ,蹩脚的英文看着满屏 idea 的提示就难受
---- 表示用 idea 看到这种那是真的难受,满屏波浪线。。。 |
13
zw1one 2022-01-11 09:24:29 +08:00
@fishCatcher 不可能的,能把代码维护成这样的领导,一定是不关注技术实现的,只考虑业务功能是否能用。对他来说,重构反而是没有产出的资源浪费,并且要承担重构后出问题的风险。
|
14
buubiu 2022-01-11 09:35:01 +08:00
全部满足数据库三大范式也不一定是好事吧
|
15
seven123 2022-01-11 09:58:39 +08:00
我现在每天都处于被喂屎中,想改变现状却发现我一个人的力量还是太小,花时间重构一部分代码,领导觉得风险大,给暂停了,领了年终奖也想跑路了,有时间先刷刷题吧。
|
16
FANGYK 2022-01-11 10:09:51 +08:00
要不是我们项目只有我一个后端,我差点以为你跟我在同一个公司
|
17
cxe2v 2022-01-11 10:16:06 +08:00 2
程序员总觉得代码写得好看是公司运转的重要基石
|
18
q4336431 2022-01-11 10:26:51 +08:00
巧了,前段时间刚在屎山留下新屎
|
19
ScepterZ 2022-01-11 10:33:24 +08:00
@fishCatcher 大部分公司,重构都不算绩效内容,虽然大家都懂这事又重要又有难度又收益大
|
20
kensoz 2022-01-11 10:53:58 +08:00
继续拉屎把
|
21
bingoshe 2022-01-11 11:17:31 +08:00 1
你觉得这就是屎山?能看得懂的就不算哈哈哈
|
22
JamesR 2022-01-11 11:24:37 +08:00
这个其实不算太严重,我见过难以看懂的某个大段代码,性质比楼主说得严重得多。
这方面主要看公司给得时间充足不,充足的话还是可以维护的,也算有意义的工作;如果公司一个劲催赶工,那还是早早准备换个工作吧。 |
23
andytao 2022-01-11 11:37:16 +08:00
资深的开发都是把前人留下的屎山裹上一层英雄的外衣,然后把屎山再堆高一层;
|
24
liyhu 2022-01-11 12:35:27 +08:00
光抱怨没啥用
|
25
Cbdy 2022-01-11 13:10:37 +08:00 via Android
弄不下去可以跑路
|
26
shyangs 2022-01-11 13:13:08 +08:00
@buubiu
資料庫違反第一正規化, 說明可能連系統文件、開發文件都沒有. 否則文件裡應該會註明違反的原因 (比如為了效能, 減少 JOIN 等等). 沒有 code review 就算了, 沒有開發文件, 有點慘. |
27
leafre 2022-01-11 13:14:52 +08:00 via Android
旧代码死都不要动,要我动我就离职
|
28
xuanbg 2022-01-11 15:46:09 +08:00
既然是屎山,那就让这屎山更高一点,更臭一点好了🐶
|
29
zhongjun96 2022-01-11 16:18:56 +08:00
屎山最起码还有代码,我现在的项目,外包给别人,然后跑路了,手上只有一个 war 包,只能反编译出来。慢慢改,要吐了。
![image_1641889057870.png]( https://s2.loli.net/2022/01/11/6GSMlWF21kwavNb.png) |
30
18000rpm 2022-01-11 21:10:05 +08:00
程序员的浪漫
|
31
supuwoerc 2022-01-12 17:44:50 +08:00
要么跑要么一起拉,重构?老板给这点钱配吗?
|
32
yangzzzzzz 2022-01-13 15:27:16 +08:00
一起拉 一起拉 到时候出问题,线上环境救急救火,年终奖少不了
|
33
lyj5251 2022-01-13 17:53:30 +08:00
用塑料袋包住然后继续拉。
|
34
jackbrother 2022-01-17 10:23:56 +08:00
@fishCatcher 想多了,出问题你第一个走人
|
35
JDog 2022-01-19 11:37:07 +08:00
"新人和老人的区别就是面对一坨屎山,新人会大吃一斤。老人会贤淑的避开最臭的那部分屎,然后灵巧的在保证屎山不垮的情况下把自己的屎再拉一层上去"
|