维护屎山，心态炸了

[rhacker1995]

传参用 map ，返回用 map ，啥都往里塞，面向对象封装是对程序员最起码的要求吧。
异常 catch 了既不打印，也不嵌套另一个异常再抛出，报啥错都不知道。
造各种各样的轮子，Apache 有这么多 util 类，还非得从 csdn 的拷贝代码，又不见得写的有多好。
http client 这种线程安全的东西放到方法局部变量，不能连接复用，并发一多连接吃满了。
数据库第一范式都不遵守，在各种字段里塞 json ，关键是这些 json 里的字段还要结构化查询，索引用不上，数据库查崩多少次了，还偷这种懒。
typo ，蹩脚的英文看着满屏 idea 的提示就难受

[rhacker1995]

负责 Code review 的人多少年没敲代码，只管需求上线，不管实现。

[DinnyXu]

这种项目维护的意义在哪？ 跳槽吧，别耽误自己

[fishCatcher]

换个角度想，你把这些重构或者优化，再包装一下跟老板吹一吹，那不就是升职的好机会？

如果整个项目都很完善，那自己离变成客服或者被强制 325 就不远了

[potatowish]

维护屎山是个技术活，资深的开发都是完美的绕过前人留下的坑，把屎山再堆高一层

[itechify]

review 等于帮忙合下代码🙃

[thedrwu]

如果领导开明，你的晋升机会来了。
如果领导不开明，在屎山上浇点肥，git blame 不到自己就不用背锅。

[Felldeadbird]

没有屎的存在 就不会聘请开发了。我已经再屎上多年了，习惯了铲屎和拉屎。

[mineralsalt]

和同事一起维护一个项目, 我也很难受, 一点编码规范都没有. 随便举个例子, a.equals("success") , 明明可以避免的空针在错误, 就是视而不见, 每次看到这种写法我都想说, 但是又怕说多了人家不高兴

[pmispig]

我们的异常 catch 直接吃掉了，啥都不打。没有打印错误日志就是没错误。
http client 我倒是持反对意见，我讨厌用 httpclient 连接池的人

[az467]

之前也干过.....
反射屎山用 javassist 一点点地换
自造垃圾轮子爱用用 反正不维护
就是传参用全局变量和 Map 我是实在没办法 直接绕开写了个 v2

[ferock]

你工资多少？心态好，能改一点是一点，今天心情不好，实现需求早点下班写自己的开源项目呗…

[Kontinue]

typo ，蹩脚的英文看着满屏 idea 的提示就难受
----
表示用 idea 看到这种那是真的难受，满屏波浪线。。。

[zw1one]

@fishCatcher 不可能的，能把代码维护成这样的领导，一定是不关注技术实现的，只考虑业务功能是否能用。对他来说，重构反而是没有产出的资源浪费，并且要承担重构后出问题的风险。

[buubiu]

全部满足数据库三大范式也不一定是好事吧

[seven123]

我现在每天都处于被喂屎中，想改变现状却发现我一个人的力量还是太小，花时间重构一部分代码，领导觉得风险大，给暂停了，领了年终奖也想跑路了，有时间先刷刷题吧。

[plcx]

要不是我们项目只有我一个后端，我差点以为你跟我在同一个公司

[cxe2v]

程序员总觉得代码写得好看是公司运转的重要基石

[q4336431]

巧了，前段时间刚在屎山留下新屎

[ScepterZ]

@fishCatcher 大部分公司，重构都不算绩效内容，虽然大家都懂这事又重要又有难度又收益大

[kensoz]

继续拉屎把

[bingoshe]

你觉得这就是屎山？能看得懂的就不算哈哈哈

[JamesR]

这个其实不算太严重，我见过难以看懂的某个大段代码，性质比楼主说得严重得多。

这方面主要看公司给得时间充足不，充足的话还是可以维护的，也算有意义的工作；如果公司一个劲催赶工，那还是早早准备换个工作吧。

[9dP06m83vIV00l72]

资深的开发都是把前人留下的屎山裹上一层英雄的外衣，然后把屎山再堆高一层；

[liyhu]

光抱怨没啥用

[Cbdy]

弄不下去可以跑路

[shyangs]

@buubiu

資料庫違反第一正規化, 說明可能連系統文件、開發文件都沒有. 否則文件裡應該會註明違反的原因 (比如為了效能, 減少 JOIN 等等).

沒有 code review 就算了, 沒有開發文件, 有點慘.

[leafre]

旧代码死都不要动，要我动我就离职

[xuanbg]

既然是屎山，那就让这屎山更高一点，更臭一点好了🐶

[qjbcnrs]

屎山最起码还有代码，我现在的项目，外包给别人，然后跑路了，手上只有一个 war 包，只能反编译出来。慢慢改，要吐了。

[18000rpm]

程序员的浪漫

[supuwoerc]

要么跑要么一起拉，重构？老板给这点钱配吗？

[yangzzzzzz]

一起拉 一起拉 到时候出问题，线上环境救急救火，年终奖少不了

[lyj5251]

用塑料袋包住然后继续拉。

[jackbrother]

@fishCatcher 想多了，出问题你第一个走人

[JDog]

"新人和老人的区别就是面对一坨屎山，新人会大吃一斤。老人会贤淑的避开最臭的那部分屎，然后灵巧的在保证屎山不垮的情况下把自己的屎再拉一层上去"