这是一个创建于 2378 天前的主题，其中的信息可能已经有所发展或是发生改变。
入职了一个多月了，天天改 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)
}
}
```
然后上面的代码我是回忆的，感觉我回忆的有点问题。
```
data = []
for(user as users){
getSomeThingByUserId()
flag = true
for(a as data){
if(a.type = a.type){
flag = false
}
}
if(flag){
data.push(user)
}
}
```
 | 1 rcer 2019-04-14 20:43:05 +08:00 data = users.where(xxx); 心疼 |
 | 3 FreeDog 2019-04-14 20:51:03 +08:00 @nocrush 偷偷告诉你... 其实大鹅厂也有这样的代码... 很可能是刚换语言的新手写的，不过被我发现就会立刻给重构掉（逃跑 |
 | 5 Chemist 2019-04-14 21:13:05 +08:00  4 问你面试的人：“你面试我的时候问那么多牛皮的问题，为什么还招到写出这辣鸡代码的傻逼？” |
 | 6 fsafdasfsdafsd 2019-04-14 21:15:25 +08:00 改 bug 正常，只是刚入职熟悉业务和架构的手段。 |
 | 8 eret9616 2019-04-14 21:33:19 +08:00 @kyuuseiryuu 哈哈哈哈 |
 | 9 AngryMagikarp 2019-04-14 21:34:02 +08:00 没有 BUG 怎么让你安心 996。 |
 | 10 nanaw 2019-04-14 21:38:38 +08:00 via Android 13 |
 | 11 dazhangpan 2019-04-14 21:38:44 +08:00 11 进公司之后一定不要只想着完成任务，要抓住机会搞自己的项目，吃领导画的大饼，也要学会给领导画大饼，要人要资源要团队，学会多做 PPT，增加自己的 visibility，就不用天天和这些 bug 缠斗了。 |
 | 12 Linxing 2019-04-14 21:42:41 +08:00 via iPhone 没有 code review 吗 |
 | 13 quickma &nbs; 2019-04-14 21:42:57 +08:00 多读书。 |
 | 14 easylee 2019-04-14 21:45:46 +08:00 三字真言大军马上要来，真准备要撤了的话，不妨先找好下家。 |
 | 15 YzSama 2019-04-14 21:49:46 +08:00 via iPhone @Linxing code review 活在别人家公司里，反正都会以没时间、老板认为效率太低为由拒绝掉。 |
 | 18 nocrush OP @kyuuseiryuu 不敢问 |
 | 19 yidinghe 2019-04-14 22:01:40 +08:00 via Android 楼主自己稍微重构一下，不要看到哪有问题就只改哪，斟酌一下逻辑的组织是不是有问题。也不要觉得是给别人擦屁股，主要还是为了自己以后再改起来轻松点。 |
 | 20 sonicjam 2019-04-14 22:10:38 +08:00 别问，问就是离婚，哦不，离职。 |
| 22 nocrush OP @yidinghe 嗯嗯，谢谢你。感觉只能重构，bug 的位置，项目是很庞大的，php node java 什么都有。而且有点大材小用，设计得非常复杂 |
 | 23 nealwx 2019-04-15 07:43:03 +08:00 via Android 继续坚持下去吧，很多时候新人都是从解 BUG 开始的 |
 | 24 ccraohng 2019-04-15 08:18:47 +08:00 via Android 额，这是什么语言，不是为 0 吗 |
 | 26 daodao116 2019-04-15 09:50:06 +08:00 重构啊，天下公司一般黑，哪个公司都有这个问题了。看到有问题的，就重构掉，然后跟主管谈谈，把这些给他看，然后讲讲自己的重构思路，让他觉得你是有能力喝想法的，有理有据的要求更有挑战性的任务，脱离这样的死循环。 |
 | 27 RobertWu 2019-04-15 10:01:48 +08:00 楼主之前在 THS 工作过？ |
| 29 nocrush OP @ccraohng 哈哈，比如说 这个 for as 我觉得 我是用 php 的 这个 data.push 我觉得我是用 js 的，因为不想百分百还原，所以就凑了一段代码。 |
 | 30 l00t 2019-04-15 11:01:47 +08:00 1 不知道你在说什么东西…… 改 bug 你只看一半代码就动手了？这工作态度可不行啊。 |
 | 31 simen513 2019-04-15 11:06:55 +08:00 1 ）加上自动化测试，每个提交或者每天至少都跑一遍 CI 2 ） Code review，相关的人都要 review，这样就能避免你没能考虑到你不知道的而别人熟悉的部分 你们公司至少要完成以上这两点，才能正规化 |
 | 32 stebest 2019-04-15 11:11:54 +08:00 if(a == a) then "逃跑" |
 | 33 onfuns 2019-04-15 11:14:46 +08:00 同杭州，方便透露什么公司吗防止踩雷 |
 | 35 jowan 2019-04-15 12:00:54 +08:00 长度应该是 0 啊 |
 | 38 mushishi 2019-04-15 13:15:58 +08:00 面试了 Java 高级开发工程师，入职后，已经写了一个星期的 PHP, 零基础的 PHP 开发。真是无语 |
 | 39 eqing 2019-04-15 13:16:53 +08:00 离职快一个月了，投的简历还没有回复 |
 | 40 goodleixiao 2019-04-15 13:21:37 +08:00 其实觉得这样挺好的，没事做改 bug。 连 bug 都不用改了，那得多闲，用不着那么多人了。 我看你这代码有可能是应届生写，这样才需要你这样的高手来。 |
 | 41 Chenamy2017 2019-04-15 13:26:05 +08:00 有问题去解决问题，这才是有价值的，而不是抱怨。如果一个单位什么问题都没有，还要你干什么。 |
| 42 nocrush OP @goodleixiao 不是应届生，工作比我久，好像是以前不用这个语言 |
 | 43 Foreverdxa 2019-04-15 13:36:05 +08:00 @nanaw，你发的这个貌似是让代码更难读吧。。。 |
 | 44 Lin0936 2019-04-15 13:37:01 +08:00 ``` var isVisible = (int)value > 0 ? true : false; ``` 感受一下 |
| 45 goodleixiao 2019-04-15 13:38:15 +08:00 @nocrush 见过这样的人，我司就有这样代码，一帮老人写的，以前搞 c 好多年，凑合着写出这样的代码。 多写单元测测，可以解决大部分问题，实在看不下删了重来。 |
 | 46 mars0prince 2019-04-15 13:45:32 +08:00 给你说一个潜规则吧，很多人，会故意把代码写的复杂，或者写的除了自己谁也看不懂，这样除了他自己就没人愿意维护或者根本维护不了。然后他可以一遍又一遍的提优化方案，不可替代性越来越高，他就可以升职加薪了 |
 | 47 qq316107934 2019-04-15 13:50:47 +08:00 via Android @Lin0936 请教下，这个更好的写法是啥，没看出什么问题。 |
| 48 Lin0936 2019-04-15 14:14:00 +08:00 @qq316107934 本身没问题，重复了而已 |
 | 49 ljh0585 2019-04-15 14:50:36 +08:00 我们还有 r.count = 2300000 + Math.floor((n - 2300000) * 0.7) + 16165 这种代码，我是真的没看懂他这是在干嘛。。 |
 | 50 macfred 2019-04-15 16:16:18 +08:00 via Android 看到大家都在提 code review，大家都是怎么做的？如何看到每周集中一次下班后开会评个三小时以上这种搞法？ |
 | 51 zhea55 2019-04-15 16:26:49 +08:00 |
 | 52 msaionyc 2019-04-15 16:31:29 +08:00 老哥，这个是正常的，改 bug 是熟悉项目的最好方式，可能之前公司部分人员能力不够写出来不好的代码，你多优化对你以后的生涯也是有帮助的，而且有些也不一定是烂代码，而是奇葩需求下的产物 |
 | 53 whoami9894 2019-04-15 16:35:51 +08:00 via Android |
 | 55 jxf2008 2019-04-15 17:23:09 +08:00 @whoami9894 确定 |
| 56 nocrush OP @whoami9894 你们不要在乎语言，我是为了防止被认出，杂交了一下。 |
 | 57 win7pro 2019-04-15 17:54:04 +08:00 公司：改 bug 都不淡定，怎么敢把创造性的需求派给你 |
 | 58 yawn852 2019-04-15 18:05:36 +08:00 via iPhone 我也是刚入职一个月，也是整理代码，改 bug,就做了几个小需求。 |
 | 59 looplj 2019-04-15 18:36:22 +08:00 不知道大公司，我们公司也是 1000 人左右。 反正，我觉得大部分的人都菜，包括所谓的高级，架构师。 |
 | 60 lzhnull 2019-04-15 18:44:28 +08:00 via Android 哈哈哈，你重构完了一会，还是后继有人吐槽你重构的代码的 |
 | 62 unco020511 2019-04-15 19:10:14 +08:00 摸鱼一时爽,一直摸一只爽 |
 | 63 leexy 2019-04-15 19:15:13 +08:00 能改 bug 不错了, 不是天天写 bug |
 | 64 babedoll 2019-04-16 08:09:06 +08:00 |
 | 65 ganbuliao 2019-04-16 09:05:28 +08:00 flag = true if(user.type == user.type){ flag = false } 真是神了 这不就相当于 if(1){ flag = false } 这么简单的东西都改不好，搞笑的吗 |
| 66 mars0prince 2019-04-16 10:30:44 +08:00 @babedoll 哈哈，最后你们都离职了，他升值了 |
| 67 babedoll 2019-04-16 10:55:17 +08:00 @mars0prince 话倒不是这么说，如果说稳定，确实可以做到，因为我们公司从不裁人。 但是升值就不一定了，毕竟年轻人跳槽工资就翻翻了，他沉浸在垃圾代码中，项目没人一起做，只能维持现状带带实习生了。 |
 | 68 techme 2019-04-16 10:59:22 +08:00 最近我们公司绩效改革，改 bug 不算工作量，一直改就一直没绩效工资 是时候撤退了 |
 | 69 LimeVista 2019-04-16 13:06:27 +08:00 via Android 如果是静态语言，编译器都会警告吧 |
Select Language