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

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

  •  
  •   nocrush · 6 天前 · 6609 次点击

    入职了一个多月了,天天改 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 条附言  ·  5 天前
    给为大哥,你们不要研究是什么语言哦,我是自己杂交了一下。

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

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

    ```javascript

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

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

    现在在我司混成茅坑里的臭石头,人人都不愿与他一起做项目。
        65
    ganbuliao   4 天前
    flag = true
    if(user.type == user.type){
    flag = false
    }
    真是神了 这不就相当于
    if(1){
    flag = false
    }

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