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

V2EX = way to explore

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

现在注册

已注册用户请登录

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

入职了一个多月了，天天改 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

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