入职了一个多月了，天天改 BUG，有点待不下去了，想要大家批评

V2EX = way to explore

V2EX 是一个关于分享和探索的地方

现在注册

已注册用户请登录

• 请不要在回答技术问题时复制粘贴 AI 生成的内容

这是一个创建于 2082 天前的主题，其中的信息可能已经有所发展或是发生改变。

入职了一个多月了，天天改 BUG，有点待不下去了。

想听听大家的建议。

坐标杭州，年后换了一家工作工作了一个多月，在公司从事后台开发。

公司还是挺不错了，六七百号人，福利待遇各方面都可以，公司制度方面也不错。

面试这家公司的时候，感觉自己很满意。

入职后深刻的体会到了面试造飞机，工作拧螺丝。

然后代码的质量很差，感觉一个自营的项目弄成了外包，Bug 管理平台天天都有 Bug。

感觉待下去心态有点崩。

然后头几天，前端同事要我改一个 Bug，然后我就看了一半代码，我就把这个 Bug 改了。然后数据也是对了，也向他确认了过了几天又跟我说这个地方有 Bug，我说我不是改了吗？怎么又错了。

原因是我看代码的时候，可以说就看了一半就把这个 Bug 改了，不知道影响了后面的 if else。

感觉自己有点待不下去了。

我举一个项目里面的例子，大家伙看看

data = [];
for(user as users){
    flag = true
    if(user.type == user.type){
        flag = false
    }
    if(flag){
        data.push(user)
    }
}

可以看到这个 data 数组长度永远为 1，我不知道是怎么过滤这个数据的，反正就很恶心。

有点想撤了。

第 1 条附言 · 2019-04-15 17:41:19 +08:00

给为大哥，你们不要研究是什么语言哦，我是自己杂交了一下。

然后上面的代码我是回忆的，感觉我回忆的有点问题。

```
data = []
for(user as users){
getSomeThingByUserId()
flag = true
for(a as data){
if(a.type = a.type){
flag = false
}
}
if(flag){
data.push(user)
}
}

```

69 条回复 • 2019-04-16 13:06:27 +08:00

rcer

2019-04-14 20:43:05 +08:00

data = users.where(xxx);
心疼

nocrush

2019-04-14 20:44:46 +08:00 via Android

@rcer 我应该怎么办啊

FreeDog

2019-04-14 20:51:03 +08:00

@nocrush 偷偷告诉你... 其实大鹅厂也有这样的代码... 很可能是刚换语言的新手写的，不过被我发现就会立刻给重构掉（逃跑

jousca

2019-04-14 21:09:11 +08:00

@FreeDog 鹅厂大佬。

kyuuseiryuu

2019-04-14 21:13:05 +08:00

问你面试的人：“你面试我的时候问那么多牛皮的问题，为什么还招到写出这辣鸡代码的傻逼？”

fsafdasfsdafsd

2019-04-14 21:15:25 +08:00

改 bug 正常，只是刚入职熟悉业务和架构的手段。

rcer

2019-04-14 21:26:24 +08:00

@nocrush #2 哪儿都有操蛋的代码，比如我现在写的 rcer，充满了魔法。

eret9616

2019-04-14 21:33:19 +08:00

@kyuuseiryuu 哈哈哈哈

AngryMagikarp

2019-04-14 21:34:02 +08:00

没有 BUG 怎么让你安心 996。

nanaw

2019-04-14 21:38:38 +08:00 via Android

那就修炼吧 https://coderlmn.github.io/frontEndCourse/unmaintainable.html

dazhangpan

2019-04-14 21:38:44 +08:00

进公司之后一定不要只想着完成任务，要抓住机会搞自己的项目，吃领导画的大饼，也要学会给领导画大饼，要人要资源要团队，学会多做 PPT，增加自己的 visibility，就不用天天和这些 bug 缠斗了。

Linxing

2019-04-14 21:42:41 +08:00 via iPhone

没有 code review 吗

x7395759

2019-04-14 21:42:57 +08:00

多读书。

easylee

2019-04-14 21:45:46 +08:00

三字真言大军马上要来，真准备要撤了的话，不妨先找好下家。

YzSama

2019-04-14 21:49:46 +08:00 via iPhone

@Linxing code review 活在别人家公司里，反正都会以没时间、老板认为效率太低为由拒绝掉。

Fx8m

2019-04-14 21:55:34 +08:00

@nanaw 大佬，神了！

nocrush

2019-04-14 21:56:01 +08:00 via Android

@Linxing 没有 review

nocrush

2019-04-14 21:57:25 +08:00 via Android

@kyuuseiryuu 不敢问

yidinghe

2019-04-14 22:01:40 +08:00 via Android

楼主自己稍微重构一下，不要看到哪有问题就只改哪，斟酌一下逻辑的组织是不是有问题。也不要觉得是给别人擦屁股，主要还是为了自己以后再改起来轻松点。

sonicjam

2019-04-14 22:10:38 +08:00

别问，问就是离婚，哦不，离职。

0myun

2019-04-14 22:27:09 +08:00

@nanaw #10 66666 分享给朋友了

nocrush

2019-04-14 22:32:59 +08:00 via Android

@yidinghe 嗯嗯，谢谢你。感觉只能重构，bug 的位置，项目是很庞大的，php node java 什么都有。而且有点大材小用，设计得非常复杂

nealwx

2019-04-15 07:43:03 +08:00 via Android

继续坚持下去吧，很多时候新人都是从解 BUG 开始的

ccraohng

2019-04-15 08:18:47 +08:00 via Android

额，这是什么语言，不是为 0 吗

jxf2008

2019-04-15 09:46:58 +08:00

@ccraohng
长的挺像 python,我看着也是 0
不过我很久没用 python 了。。。元祖，字典的详细很多都网了

daodao116

2019-04-15 09:50:06 +08:00

重构啊，天下公司一般黑，哪个公司都有这个问题了。看到有问题的，就重构掉，然后跟主管谈谈，把这些给他看，然后讲讲自己的重构思路，让他觉得你是有能力喝想法的，有理有据的要求更有挑战性的任务，脱离这样的死循环。

RobertWu

2019-04-15 10:01:48 +08:00

楼主之前在 THS 工作过？

nocrush

2019-04-15 10:43:51 +08:00 via Android

@RobertWu ths 是什么

nocrush

2019-04-15 10:47:26 +08:00 via Android

@ccraohng 哈哈，比如说这个 for as 我觉得我是用 php 的这个 data.push 我觉得我是用 js 的，因为不想百分百还原，所以就凑了一段代码。

l00t

2019-04-15 11:01:47 +08:00

不知道你在说什么东西…… 改 bug 你只看一半代码就动手了？这工作态度可不行啊。

simen513

2019-04-15 11:06:55 +08:00

1 ）加上自动化测试，每个提交或者每天至少都跑一遍 CI
2 ） Code review，相关的人都要 review，这样就能避免你没能考虑到你不知道的而别人熟悉的部分
你们公司至少要完成以上这两点，才能正规化

stebest

2019-04-15 11:11:54 +08:00

if(a == a) then "逃跑"

onfuns

2019-04-15 11:14:46 +08:00

同杭州，方便透露什么公司吗防止踩雷

nocrush

2019-04-15 11:50:15 +08:00 via Android

@l00t 可能你看了那个地方的代码，你或许不会这样批评我，这里我确实也有问题。

jowan

2019-04-15 12:00:54 +08:00

长度应该是 0 啊

nocrush

2019-04-15 12:29:08 +08:00 via Android

@jowan 你想想，data 是为空的第一次执行到 for 是不执行的

nocrush

2019-04-15 12:31:05 +08:00 via Android

@jowan 对对对，我说错了，可能我回忆代码的时候少了一点东西。

mushishi

2019-04-15 13:15:58 +08:00

面试了 Java 高级开发工程师，入职后，已经写了一个星期的 PHP, 零基础的 PHP 开发。真是无语

eqing

2019-04-15 13:16:53 +08:00

离职快一个月了，投的简历还没有回复

goodleixiao

2019-04-15 13:21:37 +08:00

其实觉得这样挺好的，没事做改 bug。连 bug 都不用改了，那得多闲，用不着那么多人了。我看你这代码有可能是应届生写，这样才需要你这样的高手来。

Chenamy2017

2019-04-15 13:26:05 +08:00

有问题去解决问题，这才是有价值的，而不是抱怨。如果一个单位什么问题都没有，还要你干什么。

nocrush

2019-04-15 13:32:22 +08:00 via Android

@goodleixiao 不是应届生，工作比我久，好像是以前不用这个语言

Foreverdxa

2019-04-15 13:36:05 +08:00

@nanaw，你发的这个貌似是让代码更难读吧。。。

Lin0936

2019-04-15 13:37:01 +08:00

```
var isVisible = (int)value > 0 ? true : false;
```
感受一下

goodleixiao

2019-04-15 13:38:15 +08:00

@nocrush 见过这样的人，我司就有这样代码，一帮老人写的，以前搞 c 好多年，凑合着写出这样的代码。多写单元测测，可以解决大部分问题，实在看不下删了重来。

mars0prince

2019-04-15 13:45:32 +08:00

给你说一个潜规则吧，很多人，会故意把代码写的复杂，或者写的除了自己谁也看不懂，这样除了他自己就没人愿意维护或者根本维护不了。然后他可以一遍又一遍的提优化方案，不可替代性越来越高，他就可以升职加薪了

qq316107934

2019-04-15 13:50:47 +08:00 via Android

@Lin0936 请教下，这个更好的写法是啥，没看出什么问题。

Lin0936

2019-04-15 14:14:00 +08:00

@qq316107934 本身没问题，重复了而已

ljh0585

2019-04-15 14:50:36 +08:00

我们还有
r.count = 2300000 + Math.floor((n - 2300000) * 0.7) + 16165
这种代码，我是真的没看懂他这是在干嘛。。

macfred

2019-04-15 16:16:18 +08:00 via Android

看到大家都在提 code review，大家都是怎么做的？如何看到每周集中一次下班后开会评个三小时以上这种搞法？

zhea55

2019-04-15 16:26:49 +08:00

@qq316107934 脱了裤子放屁，见怪不怪了。

```javascript

var isVisible = value > 0;
```

msaionyc

2019-04-15 16:31:29 +08:00

老哥，这个是正常的，改 bug 是熟悉项目的最好方式，可能之前公司部分人员能力不够写出来不好的代码，你多优化对你以后的生涯也是有帮助的，而且有些也不一定是烂代码，而是奇葩需求下的产物

whoami9894

2019-04-15 16:35:51 +08:00 via Android

@ccraohng
JS，type==type 感觉是笔误而已

@jxf2008
？从哪儿看出长得像的？你确定你用过 Python 吗

TomVista

2019-04-15 16:47:57 +08:00

@zhea55 你要是不说,我真找不出来毛病 23333333333

jxf2008

2019-04-15 17:23:09 +08:00

@whoami9894
确定

nocrush

2019-04-15 17:30:06 +08:00 via Android

@whoami9894 你们不要在乎语言，我是为了防止被认出，杂交了一下。

win7pro

2019-04-15 17:54:04 +08:00

公司：改 bug 都不淡定，怎么敢把创造性的需求派给你

yawn852

2019-04-15 18:05:36 +08:00 via iPhone

我也是刚入职一个月，也是整理代码，改 bug,就做了几个小需求。

ZSeptember

2019-04-15 18:36:22 +08:00

不知道大公司，我们公司也是 1000 人左右。
反正，我觉得大部分的人都菜，包括所谓的高级，架构师。

lzhnull

2019-04-15 18:44:28 +08:00 via Android

哈哈哈，你重构完了一会，还是后继有人吐槽你重构的代码的

nocrush

2019-04-15 18:47:55 +08:00 via Android

@lzhnull 嗯嗯，是的是这样，但是这个 bug 是怎么上线的，去重是这样去重的

unco020511

2019-04-15 19:10:14 +08:00

摸鱼一时爽,一直摸一只爽

leexy

2019-04-15 19:15:13 +08:00

能改 bug 不错了, 不是天天写 bug

babedoll

2019-04-16 08:09:06 +08:00

@mars0prince
我司就有这样的人，年方三十五，代码写的不如本科生，美其名曰优化余地多，其实就是技术垃圾。

现在在我司混成茅坑里的臭石头，人人都不愿与他一起做项目。

ganbuliao

2019-04-16 09:05:28 +08:00

flag = true
if(user.type == user.type){
flag = false
}
真是神了这不就相当于
if(1){
flag = false
}

这么简单的东西都改不好，搞笑的吗

mars0prince

2019-04-16 10:30:44 +08:00

@babedoll 哈哈，最后你们都离职了，他升值了

babedoll

2019-04-16 10:55:17 +08:00

@mars0prince 话倒不是这么说，如果说稳定，确实可以做到，因为我们公司从不裁人。
但是升值就不一定了，毕竟年轻人跳槽工资就翻翻了，他沉浸在垃圾代码中，项目没人一起做，只能维持现状带带实习生了。

techme

2019-04-16 10:59:22 +08:00

最近我们公司绩效改革，改 bug 不算工作量，一直改就一直没绩效工资
是时候撤退了

LimeVista

2019-04-16 13:06:27 +08:00 via Android

如果是静态语言，编译器都会警告吧