这是一个创建于 1079 天前的主题，其中的信息可能已经有所发展或是发生改变。
传参用 map ，返回用 map ，啥都往里塞，面向对象封装是对程序员最起码的要求吧。
异常 catch 了既不打印，也不嵌套另一个异常再抛出，报啥错都不知道。
造各种各样的轮子，Apache 有这么多 util 类，还非得从 csdn 的拷贝代码，又不见得写的有多好。
http client 这种线程安全的东西放到方法局部变量，不能连接复用，并发一多连接吃满了。
数据库第一范式都不遵守，在各种字段里塞 json ，关键是这些 json 里的字段还要结构化查询，索引用不上，数据库查崩多少次了，还偷这种懒。
typo ，蹩脚的英文看着满屏 idea 的提示就难受
异常 catch 了既不打印，也不嵌套另一个异常再抛出，报啥错都不知道。
造各种各样的轮子，Apache 有这么多 util 类，还非得从 csdn 的拷贝代码，又不见得写的有多好。
http client 这种线程安全的东西放到方法局部变量，不能连接复用，并发一多连接吃满了。
数据库第一范式都不遵守，在各种字段里塞 json ，关键是这些 json 里的字段还要结构化查询，索引用不上，数据库查崩多少次了，还偷这种懒。
typo ，蹩脚的英文看着满屏 idea 的提示就难受
 |
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 包，只能反编译出来。慢慢改，要吐了。
 |
 |
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
"新人和老人的区别就是面对一坨屎山，新人会大吃一斤。老人会贤淑的避开最臭的那部分屎，然后灵巧的在保证屎山不垮的情况下把自己的屎再拉一层上去"
|
Select Language