首页   注册   登录
V2EX = way to explore
V2EX 是一个关于分享和探索的地方
现在注册
已注册用户请  登录
nocrush
V2EX  ›  程序员

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

  •  
  •   nocrush · 2019-04-14 20:36:22 +08:00 · 8304 次点击
    这是一个创建于 482 天前的主题,其中的信息可能已经有所发展或是发生改变。

    入职了一个多月了,天天改 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
        1
    rcer   2019-04-14 20:43:05 +08:00
    data = users.where(xxx);
    心疼
    nocrush
        2
    nocrush   2019-04-14 20:44:46 +08:00 via Android
    @rcer 我应该怎么办啊
    FreeDog
        3
    FreeDog   2019-04-14 20:51:03 +08:00
    @nocrush 偷偷告诉你... 其实大鹅厂也有这样的代码... 很可能是刚换语言的新手写的,不过被我发现就会立刻给重构掉(逃跑
    jousca
        4
    jousca   2019-04-14 21:09:11 +08:00
    @FreeDog 鹅厂大佬。
    kyuuseiryuu
        5
    kyuuseiryuu   2019-04-14 21:13:05 +08:00   ❤️ 4
    问你面试的人:“你面试我的时候问那么多牛皮的问题,为什么还招到写出这辣鸡代码的傻逼?”
    fsafdasfsdafsd
        6
    fsafdasfsdafsd   2019-04-14 21:15:25 +08:00
    改 bug 正常,只是刚入职熟悉业务和架构的手段。
    rcer
        7
    rcer   2019-04-14 21:26:24 +08:00
    @nocrush #2 哪儿都有操蛋的代码,比如我现在写的 rcer,充满了魔法。
    eret9616
        8
    eret9616   2019-04-14 21:33:19 +08:00
    @kyuuseiryuu 哈哈哈哈
    AngryMagikarp
        9
    AngryMagikarp   2019-04-14 21:34:02 +08:00
    没有 BUG 怎么让你安心 996。
    nanaw
        10
    nanaw   2019-04-14 21:38:38 +08:00 via Android   ❤️ 13
    dazhangpan
        11
    dazhangpan   2019-04-14 21:38:44 +08:00   ❤️ 11
    进公司之后一定不要只想着完成任务,要抓住机会搞自己的项目,吃领导画的大饼,也要学会给领导画大饼,要人要资源要团队,学会多做 PPT,增加自己的 visibility,就不用天天和这些 bug 缠斗了。
    Linxing
        12
    Linxing   2019-04-14 21:42:41 +08:00 via iPhone
    没有 code review 吗
    x7395759
        13
    x7395759   2019-04-14 21:42:57 +08:00
    多读书。
    easylee
        14
    easylee   2019-04-14 21:45:46 +08:00
    三字真言大军马上要来,真准备要撤了的话,不妨先找好下家。
    YzSama
        15
    YzSama   2019-04-14 21:49:46 +08:00 via iPhone
    @Linxing code review 活在别人家公司里,反正都会以没时间、老板认为效率太低为由拒绝掉。
    Fx8m
        16
    Fx8m   2019-04-14 21:55:34 +08:00
    @nanaw 大佬,神了!
    nocrush
        17
    nocrush   2019-04-14 21:56:01 +08:00 via Android
    @Linxing 没有 review
    nocrush
        18
    nocrush   2019-04-14 21:57:25 +08:00 via Android
    @kyuuseiryuu 不敢问
    yidinghe
        19
    yidinghe   2019-04-14 22:01:40 +08:00 via Android
    楼主自己稍微重构一下,不要看到哪有问题就只改哪,斟酌一下逻辑的组织是不是有问题。也不要觉得是给别人擦屁股,主要还是为了自己以后再改起来轻松点。
    sonicjam
        20
    sonicjam   2019-04-14 22:10:38 +08:00
    别问,问就是离婚,哦不,离职。
    0myun
        21
    0myun   2019-04-14 22:27:09 +08:00
    @nanaw #10 66666 分享给朋友了
    nocrush
        22
    nocrush   2019-04-14 22:32:59 +08:00 via Android
    @yidinghe 嗯嗯,谢谢你。感觉只能重构,bug 的位置,项目是很庞大的,php node java 什么都有。而且有点大材小用,设计得非常复杂
    nealwx
        23
    nealwx   2019-04-15 07:43:03 +08:00 via Android
    继续坚持下去吧,很多时候新人都是从解 BUG 开始的
    ccraohng
        24
    ccraohng   2019-04-15 08:18:47 +08:00 via Android
    额,这是什么语言,不是为 0 吗
    jxf2008
        25
    jxf2008   2019-04-15 09:46:58 +08:00
    @ccraohng
    长的挺像 python,我看着也是 0
    不过我很久没用 python 了。。。元祖,字典的详细很多都网了
    daodao116
        26
    daodao116   2019-04-15 09:50:06 +08:00
    重构啊,天下公司一般黑,哪个公司都有这个问题了。看到有问题的,就重构掉,然后跟主管谈谈,把这些给他看,然后讲讲自己的重构思路,让他觉得你是有能力喝想法的,有理有据的要求更有挑战性的任务,脱离这样的死循环。
    RobertWu
        27
    RobertWu   2019-04-15 10:01:48 +08:00
    楼主之前在 THS 工作过?
    nocrush
        28
    nocrush   2019-04-15 10:43:51 +08:00 via Android
    @RobertWu ths 是什么
    nocrush
        29
    nocrush   2019-04-15 10:47:26 +08:00 via Android
    @ccraohng 哈哈,比如说 这个 for as 我觉得 我是用 php 的 这个 data.push 我觉得我是用 js 的,因为不想百分百还原,所以就凑了一段代码。
    l00t
        30
    l00t   2019-04-15 11:01:47 +08:00   ❤️ 1
    不知道你在说什么东西…… 改 bug 你只看一半代码就动手了?这工作态度可不行啊。
    simen513
        31
    simen513   2019-04-15 11:06:55 +08:00
    1 )加上自动化测试,每个提交或者每天至少都跑一遍 CI
    2 ) Code review,相关的人都要 review,这样就能避免你没能考虑到你不知道的而别人熟悉的部分
    你们公司至少要完成以上这两点,才能正规化
    stebest
        32
    stebest   2019-04-15 11:11:54 +08:00
    if(a == a) then "逃跑"
    onfuns
        33
    onfuns   2019-04-15 11:14:46 +08:00
    同杭州,方便透露什么公司吗防止踩雷
    nocrush
        34
    nocrush   2019-04-15 11:50:15 +08:00 via Android
    @l00t 可能你看了那个地方的代码,你或许不会这样批评我,这里我确实也有问题。
    jowan
        35
    jowan   2019-04-15 12:00:54 +08:00
    长度应该是 0 啊
    nocrush
        36
    nocrush   2019-04-15 12:29:08 +08:00 via Android
    @jowan 你想想,data 是为空的 第一次执行到 for 是不执行的
    nocrush
        37
    nocrush   2019-04-15 12:31:05 +08:00 via Android
    @jowan 对对对,我说错了,可能我回忆代码的时候少了一点东西。
    mushishi
        38
    mushishi   2019-04-15 13:15:58 +08:00
    面试了 Java 高级开发工程师,入职后,已经写了一个星期的 PHP, 零基础的 PHP 开发。真是无语
    eqing
        39
    eqing   2019-04-15 13:16:53 +08:00
    离职快一个月了,投的简历还没有回复
    goodleixiao
        40
    goodleixiao   2019-04-15 13:21:37 +08:00
    其实觉得这样挺好的,没事做改 bug。 连 bug 都不用改了,那得多闲,用不着那么多人了。 我看你这代码有可能是应届生写,这样才需要你这样的高手来。
    Chenamy2017
        41
    Chenamy2017   2019-04-15 13:26:05 +08:00
    有问题去解决问题,这才是有价值的,而不是抱怨。如果一个单位什么问题都没有,还要你干什么。
    nocrush
        42
    nocrush   2019-04-15 13:32:22 +08:00 via Android
    @goodleixiao 不是应届生,工作比我久,好像是以前不用这个语言
    Foreverdxa
        43
    Foreverdxa   2019-04-15 13:36:05 +08:00
    @nanaw,你发的这个貌似是让代码更难读吧。。。
    Lin0936
        44
    Lin0936   2019-04-15 13:37:01 +08:00
    ```
    var isVisible = (int)value > 0 ? true : false;
    ```
    感受一下
    goodleixiao
        45
    goodleixiao   2019-04-15 13:38:15 +08:00
    @nocrush 见过这样的人,我司就有这样代码,一帮老人写的,以前搞 c 好多年,凑合着写出这样的代码。 多写单元测测,可以解决大部分问题,实在看不下删了重来。
    mars0prince
        46
    mars0prince   2019-04-15 13:45:32 +08:00
    给你说一个潜规则吧,很多人,会故意把代码写的复杂,或者写的除了自己谁也看不懂,这样除了他自己就没人愿意维护或者根本维护不了。然后他可以一遍又一遍的提优化方案,不可替代性越来越高,他就可以升职加薪了
    qq316107934
        47
    qq316107934   2019-04-15 13:50:47 +08:00 via Android
    @Lin0936 请教下,这个更好的写法是啥,没看出什么问题。
    Lin0936
        48
    Lin0936   2019-04-15 14:14:00 +08:00
    @qq316107934 本身没问题,重复了而已
    ljh0585
        49
    ljh0585   2019-04-15 14:50:36 +08:00
    我们还有
    r.count = 2300000 + Math.floor((n - 2300000) * 0.7) + 16165
    这种代码,我是真的没看懂他这是在干嘛。。
    macfred
        50
    macfred   2019-04-15 16:16:18 +08:00 via Android
    看到大家都在提 code review,大家都是怎么做的?如何看到每周集中一次下班后开会评个三小时以上这种搞法?
    zhea55
        51
    zhea55   2019-04-15 16:26:49 +08:00
    @qq316107934 脱了裤子放屁,见怪不怪了。

    ```javascript

    var isVisible = value > 0;
    ```
    msaionyc
        52
    msaionyc   2019-04-15 16:31:29 +08:00
    老哥,这个是正常的,改 bug 是熟悉项目的最好方式,可能之前公司部分人员能力不够写出来不好的代码,你多优化对你以后的生涯也是有帮助的,而且有些也不一定是烂代码,而是奇葩需求下的产物
    whoami9894
        53
    whoami9894   2019-04-15 16:35:51 +08:00 via Android
    @ccraohng
    JS,type==type 感觉是笔误而已

    @jxf2008
    ?从哪儿看出长得像的?你确定你用过 Python 吗
    TomVista
        54
    TomVista   2019-04-15 16:47:57 +08:00
    @zhea55 你要是不说,我真找不出来毛病 23333333333
    jxf2008
        55
    jxf2008   2019-04-15 17:23:09 +08:00
    @whoami9894
    确定
    nocrush
        56
    nocrush   2019-04-15 17:30:06 +08:00 via Android
    @whoami9894 你们不要在乎语言,我是为了防止被认出,杂交了一下。
    win7pro
        57
    win7pro   2019-04-15 17:54:04 +08:00
    公司:改 bug 都不淡定,怎么敢把创造性的需求派给你
    yawn852
        58
    yawn852   2019-04-15 18:05:36 +08:00 via iPhone
    我也是刚入职一个月,也是整理代码,改 bug,就做了几个小需求。
    ZSeptember
        59
    ZSeptember   2019-04-15 18:36:22 +08:00
    不知道大公司,我们公司也是 1000 人左右。
    反正,我觉得大部分的人都菜,包括所谓的高级,架构师。
    lzhnull
        60
    lzhnull   2019-04-15 18:44:28 +08:00 via Android
    哈哈哈,你重构完了一会,还是后继有人吐槽你重构的代码的
    nocrush
        61
    nocrush   2019-04-15 18:47:55 +08:00 via Android
    @lzhnull 嗯嗯,是的是这样,但是 这个 bug 是怎么上线的,去重是这样去重的
    unco020511
        62
    unco020511   2019-04-15 19:10:14 +08:00
    摸鱼一时爽,一直摸一只爽
    leexy
        63
    leexy   2019-04-15 19:15:13 +08:00
    能改 bug 不错了, 不是天天写 bug
    babedoll
        64
    babedoll   2019-04-16 08:09:06 +08:00
    @mars0prince
    我司就有这样的人,年方三十五,代码写的不如本科生,美其名曰优化余地多,其实就是技术垃圾。

    现在在我司混成茅坑里的臭石头,人人都不愿与他一起做项目。
    ganbuliao
        65
    ganbuliao   2019-04-16 09:05:28 +08:00
    flag = true
    if(user.type == user.type){
    flag = false
    }
    真是神了 这不就相当于
    if(1){
    flag = false
    }

    这么简单的东西都改不好,搞笑的吗
    mars0prince
        66
    mars0prince   2019-04-16 10:30:44 +08:00
    @babedoll 哈哈,最后你们都离职了,他升值了
    babedoll
        67
    babedoll   2019-04-16 10:55:17 +08:00
    @mars0prince 话倒不是这么说,如果说稳定,确实可以做到,因为我们公司从不裁人。
    但是升值就不一定了,毕竟年轻人跳槽工资就翻翻了,他沉浸在垃圾代码中,项目没人一起做,只能维持现状带带实习生了。
    techme
        68
    techme   2019-04-16 10:59:22 +08:00
    最近我们公司绩效改革,改 bug 不算工作量,一直改就一直没绩效工资
    是时候撤退了
    LimeVista
        69
    LimeVista   2019-04-16 13:06:27 +08:00 via Android
    如果是静态语言,编译器都会警告吧
    关于   ·   FAQ   ·   API   ·   我们的愿景   ·   广告投放   ·   感谢   ·   实用小工具   ·   1619 人在线   最高记录 5168   ·     Select Language
    创意工作者们的社区
    World is powered by solitude
    VERSION: 3.9.8.5 · 30ms · UTC 00:12 · PVG 08:12 · LAX 17:12 · JFK 20:12
    ♥ Do have faith in what you're doing.