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

[nocrush]

入职了一个多月了，天天改 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，我不知道是怎么过滤这个数据的，反正就很恶心。

有点想撤了。

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

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

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

```

[rcer]

data = users.where(xxx);
心疼

[nocrush]

@rcer 我应该怎么办啊

[FreeDog]

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

[jousca]

@FreeDog 鹅厂大佬。

[kyuuseiryuu]

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

[fsafdasfsdafsd]

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

[rcer]

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

[eret9616]

@kyuuseiryuu 哈哈哈哈

[AngryMagikarp]

没有 BUG 怎么让你安心 996。

[nanaw]

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

[dazhangpan]

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

[Linxing]

没有 code review 吗

[x7395759]

多读书。

[easylee]

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

[YzSama]

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

[Fx8m]

@nanaw 大佬，神了！

[nocrush]

@Linxing 没有 review

[nocrush]

@kyuuseiryuu 不敢问

[yidinghe]

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

[sonicjam]

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

[0myun]

@nanaw #10 66666 分享给朋友了

[nocrush]

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

[nealwx]

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

[ccraohng]

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

[jxf2008]

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

[daodao116]

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

[RobertWu]

楼主之前在 THS 工作过？

[nocrush]

@RobertWu ths 是什么

[nocrush]

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

[l00t]

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

[simen513]

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

[stebest]

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

[onfuns]

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

[nocrush]

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

[jowan]

长度应该是 0 啊

[nocrush]

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

[nocrush]

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

[mushishi]

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

[eqing]

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

[goodleixiao]

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

[Chenamy2017]

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

[nocrush]

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

[Foreverdxa]

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

[Lin0936]

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

[goodleixiao]

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

[mars0prince]

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

[qq316107934]

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

[Lin0936]

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

[ljh0585]

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

[macfred]

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

[zhea55]

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

```javascript

var isVisible = value > 0;
```

[msaionyc]

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

[whoami9894]

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

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

[TomVista]

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

[jxf2008]

@whoami9894
确定

[nocrush]

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

[win7pro]

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

[yawn852]

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

[ZSeptember]

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

[lzhnull]

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

[nocrush]

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

[unco020511]

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

[leexy]

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

[babedoll]

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

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

[ganbuliao]

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

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

[mars0prince]

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

[babedoll]

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

[techme]

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

[LimeVista]

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