上一主题 下一主题
ScriptCat,新一代的脚本管理器脚本站,与全世界分享你的用户脚本油猴脚本开发指南教程目录
返回列表 发新帖

已老实!公司的代码再也不敢乱改了!

[复制链接]
  • TA的每日心情
    开心
    昨天 10:48
  • 签到天数: 33 天

    [LV.5]常住居民I

    46

    主题

    32

    回帖

    156

    积分

    中级工程师

    积分
    156
    发表于 2024-12-31 11:32:36 | 显示全部楼层 | 阅读模式

    本帖最后由 jinchamchanwaji 于 2024-12-31 14:05 编辑

    很小的事但很有意思,分享给初入职场的兄弟们!遇到“丑代码”改不是不改~
    顺便提一下,如果你计算机、软件工程、电子等相关专业本科及以上学历,欢迎来共事。前端/后端/测试均可投,技术大厂。😆

    ● 感谢大家支持🤩

    亲身经历

    我第一次接手老代码的时候,映入我眼帘的就是侧边栏满页的提示以及代码下面的众多波浪线,以及提交代码时的提示,如下图:

    我内心 OS:
    1)大干一场,把**波浪线全干掉!
    2)同事这写的也太不优雅了吧,改成我这样!
    3)这代码怎么也没格式化,我来 Ctrl + Alt + L 格式化一波!

    已老实,求放过

    干掉**波浪线,将代码改 ”优雅“ 结局如下:
    1)不声不吭动了同事代码,换来同事怒骂,毕竟人家逻辑写好,然后你按你想法来搞,也没有跟人家商量。
    2)后续领导找你加需求,你发现原来之前的代码有妙用,你悔不当初,被扣绩效。
    3)格式化后,在项目修改记录上面是你的修改,这代码出问题,负责人先来找你。

    说说我的

    代码能跑不要动

    前几日我要在老项目中,新增一点小功能,在新增完功能后,我扫了一眼代码,发现有几处逻辑根本不会执行,比如:抛异常后,执行删除操作类似,我也不会去义愤填膺的去干掉这块代码,毕竟我想到一点!项目都跑七八年没出问题了,能跑就别动它。

    代码强迫症不要强加于别人

    前几天看见了这样一个讨论:

    这样的事情其实在小公司经常发生,你觉得它写的不优雅,封装少,可能是别人也有别人的难处,至少不能将自己想法强加于别人,比如领导突然来一个需求,跟你说今天你得完成,然后第二天这个需求,你要这样改、再给我加点新需求上去,你能想到的封装其实只是你冷静下来,而且没有近乎疯狂的迭代需求得到的想法,当你每天都要在原代码上面疯狂按照领导要求修改,可能你会有自己的看法。

    新增代码,尽量不影响以前逻辑

    新增代码的时候,尽量按照以前的规则逻辑来进行,比如我改的一个老项目,使用的公司自己写的一套 SQL 处理逻辑,我总不能说不行!我用不惯这个!我要用 MyBatis!!!!那真的直接被 T 出门口了。

    尊重他人代码风格

    每个人的代码风格都有所不同,这个很正常,不同厨师的老师教法不一样,做出的味道还不一样呢,没有最好的代码,只有更适合的代码,刚好我就有这样的例子:

    我注入 Spring 依赖喜欢用构造注入、用 Lombook 的注解 @RequiredArgsConstructor 注入,我同事喜欢 @Autowired ,我能说他不准用这个吗,这个是人家的习惯,虽然 Spring 也不推荐使用这个,但改不改这个都不会影响公司收益,反而能少一件事情,促进同事友好关系,哈哈哈哈,我是这样认为的。

    处理好同事之间的关系

    哈哈哈哈这个真的就是人情事故了,你换位想象一下,如果你写的幸幸苦苦的代码,新来的同事或者实习生,来批评你的代码不规范,要 Diss 你,偷偷改你代码,就算他说的超级对,你心里都十分不好受,会想一万个理由去反驳。

    我一般如果需求需要改动同事的代码,我会先虚心的向同事请求,xx哥,我这个需求要改动你这边的代码来配合一下,你来帮我一起看看,你这部分的代码这样改合理吗,或者你自己改下你自己的部分,然后我合并一下~ 谢谢 xx哥。

    image.png

  • TA的每日心情
    慵懒
    3 小时前
  • 签到天数: 859 天

    [LV.10]以坛为家III

    31

    主题

    559

    回帖

    1596

    积分

    荣誉开发者

    积分
    1596

    荣誉开发者新人进步奖油中2周年生态建设者新人报道挑战者 lv2油中3周年喜迎中秋

    发表于 2024-12-31 16:11:14 | 显示全部楼层

    螺蛳粉山就是这么形成的🙄
    另外,同一个项目的代码应该统一风格,比如设置eslint,统一规则自动格式化

    回复

    使用道具 举报

  • TA的每日心情
    慵懒
    3 小时前
  • 签到天数: 859 天

    [LV.10]以坛为家III

    31

    主题

    559

    回帖

    1596

    积分

    荣誉开发者

    积分
    1596

    荣誉开发者新人进步奖油中2周年生态建设者新人报道挑战者 lv2油中3周年喜迎中秋

    发表于 2024-12-31 16:12:16 | 显示全部楼层

    steven026 发表于 2024-12-31 16:11

    [md]螺蛳粉山就是这么形成的🙄
    另外,同一个项目的代码应该统一风格,比如设置eslint,统一规则自动格式化[ ...

    @李恒道 触发关键字了

    回复

    使用道具 举报

  • TA的每日心情
    擦汗
    2024-12-18 11:32
  • 签到天数: 194 天

    [LV.7]常住居民III

    730

    主题

    6233

    回帖

    6977

    积分

    管理员

    非物质文化遗产社会摇传承人

    积分
    6977

    荣誉开发者喜迎中秋油中2周年生态建设者

    发表于 2024-12-31 16:36:25 | 显示全部楼层
    steven026 发表于 2024-12-31 16:12
    [md]@李恒道 触发关键字了
    ![](https://bbs.tampermonkey.net.cn/data/attachment/forum/202308/29/09080 ...

    是特性!
    混的人。
    ------------------------------------------
    進撃!永遠の帝国の破壊虎---李恒道

    入驻了爱发电https://afdian.com/a/lihengdao666
    回复

    使用道具 举报

  • TA的每日心情
    开心
    昨天 10:48
  • 签到天数: 33 天

    [LV.5]常住居民I

    46

    主题

    32

    回帖

    156

    积分

    中级工程师

    积分
    156
    发表于 2025-1-2 11:06:56 | 显示全部楼层

    是滴😆😆
    回复

    使用道具 举报

    发表回复

    本版积分规则

    快速回复 返回顶部 返回列表