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

看公司/别人代码需要每一句都读懂吗

  •  
  •   shuangdeyu · 2018-05-21 15:23:38 +08:00 · 5034 次点击
    这是一个创建于 2377 天前的主题,其中的信息可能已经有所发展或是发生改变。

    新人入职一个新公司,在看公司代码,有个疑问就是代码需要每个函数,每一句都弄懂吗,还是只要弄懂结构,模式,基本逻辑,要改的话能快速定位就好了?

    30 条回复    2018-05-22 11:01:55 +08:00
    jasonyang9
        1
    jasonyang9  
       2018-05-21 15:30:17 +08:00   ❤️ 1
    我怎么觉得你心里已经有答案了
    murmur
        2
    murmur  
       2018-05-21 15:36:30 +08:00   ❤️ 1
    先抄一个和你要做功能最接近的代码就行
    照着别人的风格写是有好处的
    首先要过第一个模块的代码走查
    其次大家风格一样就算烂他也能维护
    leaves7i
        3
    leaves7i  
       2018-05-21 15:37:44 +08:00 via Android
    跟随自己的内心
    msg7086
        4
    msg7086  
       2018-05-21 15:39:55 +08:00
    每个函数都弄懂?
    到你辞职的那天都不见得能做到每个函数都弄懂。
    jmc891205
        5
    jmc891205  
       2018-05-21 15:47:45 +08:00   ❤️ 3
    先把每个函数当作黑箱 然后把比较 high level 的 code 看懂
    然后有需要的话 再去研究每个函数里面是怎么做的
    pusidun
        6
    pusidun  
       2018-05-21 15:48:09 +08:00   ❤️ 31
    每个都弄懂不存在的
    我以前碰到一个项目求功率的函数
    ```
    double getPwrValue(HardwareInit& hw);
    double getPwrValue2(HardwareInit& hw);
    ```
    我看了半天,没弄懂这 2 个函数有什么区别,关键是这 2 个函数都用到了还没问题。
    我也要读取这里的功率,犹豫了半天


    写了个 getPwrValue3
    leaves7i
        7
    leaves7i  
       2018-05-21 15:57:32 +08:00 via Android
    @pusidun hhh
    ycz0926
        8
    ycz0926  
       2018-05-21 16:00:49 +08:00
    看了干嘛?
    找 bug or 离职对接?
    qf19910623
        9
    qf19910623  
       2018-05-21 16:15:10 +08:00
    @pusidun 目测还会有 getPwrValue4
    shuangdeyu
        10
    shuangdeyu  
    OP
       2018-05-21 16:32:49 +08:00
    @jasonyang9 个人倾向于后者,但不全部读懂,又怕到时候问你某个点说不上来[笑哭]
    xiaochocking
        11
    xiaochocking  
       2018-05-21 16:33:20 +08:00
    @pusidun
    `
    getPwrValue3( ){

    }


    `
    shuangdeyu
        12
    shuangdeyu  
    OP
       2018-05-21 16:34:20 +08:00
    @ycz0926 熟悉公司代码啊,总不能以后随心所欲自己写,而且公司代码的确有很多借鉴部分
    xiaochocking
        13
    xiaochocking  
       2018-05-21 16:34:37 +08:00
    @pusidun

    getPwrValue3( ){
    //随机返回 getPwrValue or getPwrValue2 的结果
    }
    jasonyang9
        14
    jasonyang9  
       2018-05-21 16:38:01 +08:00
    @xiaochocking #13

    ```
    getPwrValue4( ){
    // 加权平均返回 getPwrValue, getPwrValue2 and getPwrValue3 的结果
    }
    ```
    xrlin
        15
    xrlin  
       2018-05-21 16:38:11 +08:00
    自己写的代码过了一段时间都不一定能每条语句都弄懂。
    shuangdeyu
        16
    shuangdeyu  
    OP
       2018-05-21 16:39:59 +08:00
    @xrlin 哈哈哈哈,这样一想,好像没什么纠结的了
    palxie
        17
    palxie  
       2018-05-21 16:41:17 +08:00
    先整体再局部
    ZhLTE
        18
    ZhLTE  
       2018-05-21 16:42:31 +08:00
    有注释的 看个注释就完事了 没有的 看看返回参数和引用的地方也差不多能懂是干嘛的了
    scofieldpeng
        19
    scofieldpeng  
       2018-05-21 16:51:39 +08:00   ❤️ 4
    正好刚刚换公司,说下我个人如何快速上手的吧,仅供参考。

    1. 快速体验业务的产品,知道到底是做什么的,有哪些功能(鼠标点点点或者手机戳戳戳然后一边抓包分析)
    2. 找你的搭档或者最近的那个上司,明白你接下来要负责的事情,同时让他给你讲一下大概的架构哪些东西,宏观上大概先知道一下
    3. 知道你接下来要负责的事情后,然后看主要的流程和逻辑,如果是后端,瞅数据表,我相信,很多公司文档都不咋地,这个时候你可以和我一样顺手整理一波文档出来贴到公司的 wiki 中(如果有的话),有不懂得,及时找你的搭档或者最近的那个上司,然后问,没什么比直接问别人最快的了。但是,提问之前看下《提问的智慧》,我个人是神烦那种 2b 问题都没有尝试解决或者看过就问的伸手党
    4. 如果给你任务了,赶紧上手写!没有任务?要任务呗!
    5. 一定要记住,多沟通多交流,别一个人瞎折腾瞎看,否则吃力不讨好,别人还会怀疑你的能力。

    以上,你的印象分会高很多,且能够快速融入一个新环境。
    lsyAndroid
        20
    lsyAndroid  
       2018-05-21 18:33:46 +08:00 via Android
    不用,看懂结构,具体的修改再说
    UN2758
        21
    UN2758  
       2018-05-21 18:38:56 +08:00
    @pusidun #6 发现虫群,灭虫剂准备
    crazypudding
        22
    crazypudding  
       2018-05-21 18:59:46 +08:00 via Android
    @pusidun 2333 我们现在的代码就是这样,前几天改一个 bug,一看,几种 if 调来调去都是同一个方法
    ycz0926
        23
    ycz0926  
       2018-05-21 19:01:26 +08:00
    @lsyAndroid 这位仁兄说的在理
    cncqw
        24
    cncqw  
       2018-05-21 19:23:05 +08:00
    @pusidun 函数重写,比如加一个参数修改下逻辑啥的,但是又不能直接在原来的函数里面修改,会影响到其他使用这个函数的业务,一时想不到别的名字,再后面加个数字以示区分,这种在多人开发的项目里面很常见,我们之前基本每个函数都有函数 2 号,后来实在受不了,改成在原来的函数后面加 ByXxx(xxx=程序员名字)
    Chyroc
        25
    Chyroc  
       2018-05-21 20:32:03 +08:00
    chiyeung524
        26
    chiyeung524  
       2018-05-21 23:02:25 +08:00
    不需要,本来就是当成抽象工具用的,就好比如我们不需要弄懂 API 内部的代码,除非出问题了或者有其他需要
    guanghao11
        27
    guanghao11  
       2018-05-21 23:26:13 +08:00
    @pusidun 笑死了🤣
    mingyun
        28
    mingyun  
       2018-05-21 23:54:44 +08:00
    Mutoo
        29
    Mutoo  
       2018-05-22 07:37:00 +08:00
    面向黑盒编程。
    CasualYours
        30
    CasualYours  
       2018-05-22 11:01:55 +08:00
    更多的是看码识业务吧。
    关于   ·   帮助文档   ·   博客   ·   API   ·   FAQ   ·   实用小工具   ·   1602 人在线   最高记录 6679   ·     Select Language
    创意工作者们的社区
    World is powered by solitude
    VERSION: 3.9.8.5 · 25ms · UTC 16:59 · PVG 00:59 · LAX 08:59 · JFK 11:59
    Developed with CodeLauncher
    ♥ Do have faith in what you're doing.