刚入职,也进了一个大坑,如何优雅让领导知道这个坑很大 - V2EX
V2EX = way to explore
V2EX 是一个关于分享和探索的地方
现在注册
已注册用户请  登录
sea516

刚入职,也进了一个大坑,如何优雅让领导知道这个坑很大

  •  
  •   sea516 2017 年 9 月 8 日 5285 次点击
    这是一个创建于 3152 天前的主题,其中的信息可能已经有所发展或是发生改变。

    三年 oc 开发经验,目前公司用 swift,刚拿到之前项目,model,controller,view 没有 base 基类,毫无面向对象思想,完全面向过程编程,一个页面约束冲突几百个,项目还算正常运行,但是维护、开发新功能实在是太难受~ 计划重构,但又感叹工作量大,而且刚接触 swift,对了,就我一个人开发。大佬们给点建议

    27 条回复    2017-09-09 17:27:10 +08:00
    ivyliner
        1
    ivyliner  
       2017 年 9 月 8 日
    直接说啊
    0ZXYDDu796nVCFxq
        2
    0ZXYDDu796nVCFxq  
       2017 年 9 月 8 日 via iPhone
    你得先了解下之前的开发是谁,还在不在公司,免得得罪人。
    sea516
        3
    sea516  
    OP
       2017 年 9 月 8 日
    @gstqc 不在,我填坑
    learnshare
        4
    learnshare  
       2017 年 9 月 8 日
    仔细分析,列出报告,附上长期改进计划
    misaka19000
        5
    misaka19000  
       2017 年 9 月 8 日 via Android
    离职
    scys
        6
    scys  
       2017 年 9 月 8 日
    感觉你领导知道坑已经很大,所以找到"人"来填 --------> "人" == "你"
    所以你提出的不是坑大,而是怎么填才好。
    sea516
        7
    sea516  
    OP
       2017 年 9 月 8 日
    @scys 对,是啊,我得先说清楚,我填了哪些坑。不然领导不知道你做了啥
    sea516
        8
    sea516  
    OP
       2017 年 9 月 8 日
    @misaka19000 不会,有问题就解决
    acros
        9
    acros  
       2017 年 9 月 8 日 via iPhone
    然后楼主看到 main 文件有一行字:
    2017.9 - 我就是维护不下去了才离职的....

    然后看下 git 最近修改文件记录,有一行:
    2017.9 - 现在代码是第二次重构的结果....
    tgfbeta
        10
    tgfbeta  
       2017 年 9 月 8 日
    什么叫没有 Base 类? Composition Over Inheritance 啊,要面向 Protocol 啊骚年。
    xzg1993
        11
    xzg1993  
       2017 年 9 月 8 日
    刚完成项目重构,整个人都舒服了,。。。历时一个月
    JacksonBond
        12
    JacksonBond  
       2017 年 9 月 8 日
    给了多少钱
    sea516
        13
    sea516  
    OP
       2017 年 9 月 8 日
    @tgfbeta 哦 不懂,现在还停留在面向对象编程
    sea516
        14
    sea516  
    OP
       2017 年 9 月 8 日
    @JacksonBond - -很难回答这个问题
    mewpoi
        15
    mewpoi  
       2017 年 9 月 8 日
    以前也遇到过这样的问题,后悔没早提出问题严重性,导致后来为了维护一堆问题耽误很多没必要的时间,实际上是,在大的坑,只要你不整个重做,坑就会越填越深,以后开发也会受阻,当然第二条路就是辞职走人,把坑留给下一个接盘侠
    runningman
        16
    runningman  
       2017 年 9 月 8 日
    慢慢改造呗
    aitaii
        17
    aitaii  
       2017 年 9 月 8 日
    领导:招你来就是改我的代码的。
    wengjin456123
        18
    wengjin456123  
       2017 年 9 月 8 日 via Android
    重构
    owenliang
        19
    owenliang  
       2017 年 9 月 8 日
    你本来就是接盘侠。
    nanjishidu
        20
    nanjishidu  
       2017 年 9 月 8 日   1
    重构耗时多。边填坑 边改进。时间长了就发现已经改一遍了。。。亲历
    game3108
        21
    game3108  
       2017 年 9 月 8 日
    有信心就用 oc 全部重构算了。
    没信心就用 swift 全部重构。
    没时间的话,就慢慢修 bug 一样的写吧。。
    DingSoung
        22
    DingSoung  
       2017 年 9 月 8 日   1
    前年我来现在这家公司,去年 iOS leader 走了,我接收才开始重构
    有些老大确实不懂
    自己辛苦两三个月,没少挨说

    重构了,今后开发轻松些,可以继续个两年,
    不重构,天天恶心死痛苦死
    sulang
        23
    sulang  
       2017 年 9 月 8 日
    重构,否则无路可走~~~刚重构完后端代码
    CodeMozart
        24
    CodeMozart  
       2017 年 9 月 9 日
    让公司把我招进去吧,我陪你一块重构
    eyp82
        25
    eyp82  
       2017 年 9 月 9 日
    其实我怀疑很多所谓的重构只是因为别人写的代码你一时半会理解不了, 或者说记不得这么多细节. 你自己重构了一遍感觉很舒服是因为都是你自己写的, 各种细节你很清楚. 所以后期维护起来感觉很好.
    很大可能你走之后, 下一个维护的人接手你的代码还是要骂人, 还是要按照自己的意思重构一遍.
    LeoEatle
        26
    LeoEatle  
       2017 年 9 月 9 日 via iPhone
    接受别人的代码很常见,大公司也这样的,如果还在就问别人是最快的,不在如果有文档就看文档
    重构当然是最优方案,但是其实很难保证自己能顾及到所有需求点,可能不知道原先为什么这么设计,后来发现是需求所致
    sicongwang
        27
    sicongwang  
       2017 年 9 月 9 日
    @eyp82 我擦, 碰到知音了, 我也是这种感觉, 每次碰到重构的时候技术总监总是一顿吹, 说我们这次重构是要解决哪些技术瓶颈, 但是实际上是人手流动太大, 导致新来的人接不了手, 要重写一遍后面的人才能真正介入这些项目, 不然业务根本就不理解。还有的时候重构是技术部没有接到什么新的项目, 新的需求, 为了找一找存在感, 所以用重构来填充这些等项目的时间, 不显得无所事事的样子。
    关于     帮助文档     自助推广系统     博客     API     FAQ     Solana     1208 人在线   最高记录 6679       Select Language
    创意工作者们的社区
    World is powered by solitude
    VERSION: 3.9.8.5 76ms UTC 17:15 PVG 01:15 LAX 10:15 JFK 13:15
    Do have faith in what you're doing.
    ubao msn snddm index pchome yahoo rakuten mypaper meadowduck bidyahoo youbao zxmzxm asda bnvcg cvbfg dfscv mmhjk xxddc yybgb zznbn ccubao uaitu acv GXCV ET GDG YH FG BCVB FJFH CBRE CBC GDG ET54 WRWR RWER WREW WRWER RWER SDG EW SF DSFSF fbbs ubao fhd dfg ewr dg df ewwr ewwr et ruyut utut dfg fgd gdfgt etg dfgt dfgd ert4 gd fgg wr 235 wer3 we vsdf sdf gdf ert xcv sdf rwer hfd dfg cvb rwf afb dfh jgh bmn lgh rty gfds cxv xcv xcs vdas fdf fgd cv sdf tert sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf shasha9178 shasha9178 shasha9178 shasha9178 shasha9178 liflif2 liflif2 liflif2 liflif2 liflif2 liblib3 liblib3 liblib3 liblib3 liblib3 zhazha444 zhazha444 zhazha444 zhazha444 zhazha444 dende5 dende denden denden2 denden21 fenfen9 fenf619 fen619 fenfe9 fe619 sdf sdf sdf sdf sdf zhazh90 zhazh0 zhaa50 zha90 zh590 zho zhoz zhozh zhozho zhozho2 lislis lls95 lili95 lils5 liss9 sdf0ty987 sdft876 sdft9876 sdf09876 sd0t9876 sdf0ty98 sdf0976 sdf0ty986 sdf0ty96 sdf0t76 sdf0876 df0ty98 sf0t876 sd0ty76 sdy76 sdf76 sdf0t76 sdf0ty9 sdf0ty98 sdf0ty987 sdf0ty98 sdf6676 sdf876 sd876 sd876 sdf6 sdf6 sdf9876 sdf0t sdf06 sdf0ty9776 sdf0ty9776 sdf0ty76 sdf8876 sdf0t sd6 sdf06 s688876 sd688 sdf86