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

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

  •  
  •   nocrush · 245 天前 · 7814 次点击
    这是一个创建于 245 天前的主题,其中的信息可能已经有所发展或是发生改变。

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

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

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

    ```javascript

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

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

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

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