楼主:
saladim (杀拉顶)
2024-07-22 23:55:30小鲁目前在一家还算大的公司工作 现在有两三位顶大的junior的同事
写程式的习惯让我觉得是不是要赶快跑了 举两个例子好了
他们都喜欢if-else combo, 没巢状到波动拳那么深 但就是动不动就if-else
三层 然后三层里面还会再if-else
第二个例子就是如果有function 90%(50~100行)适合他们想要的用途,
他们就会copy整个function 然后修改一下后"整段"code插到他们需要的地方
光是上面两个例子已经让我的头超大的 原本公司的codebase已经长得像科学怪人
了 然后又有他们持续"加持" 根本就没办法理解
更惨的是 跟他们讲他们也不当一回事 又有顶大当红硕论护体 一副你也不敢砍我的底气
公司也没人想要当坏人 code review也没人出声 而且大家都知道 上市公司每个都喜欢
有学历的人当门面 反正真正主力有人会扛 XDDDD
每次改到他们经手过的code都很痛苦 若是要帮忙擦屁股根本擦不完阿 因为一直拉....
自己宝贵的时间也都被吃掉了
这样是不是该走了比较好?
我知道烂code到处都是啦 但是至少不能一直拉吧 又是都讲不听的 更资深的也都能闪就闪
但大家轮流中枪.............
作者: lee457088 2024-07-23 00:10:00
你不如放段 code 举个例我们才知道严重程度有多大呀~
作者:
soga5731 (Letiz)
2024-07-23 00:16:00打不赢就加入,你code写得再好薪水会拿比较多?
作者:
DrTech (竹科管理处网军研发人员)
2024-07-23 00:26:00不知道为什么要强调学历。你换了更好的工作,只会有更多学历好的顶大硕。
作者:
wulouise (在线上!=在电脑前)
2024-07-23 00:27:00code review建立在平等尊重,没有的话的确没什么好说的不是你的理由不够好,就是太minor有更重要的事
作者:
DrTech (竹科管理处网军研发人员)
2024-07-23 00:29:00贵公司主管有在意这些事吗? 不要当个小兵,操皇帝的心。
楼主:
saladim (杀拉顶)
2024-07-23 00:32:00不是操皇帝的心 是影响到我了
作者:
ko27tye (好滋好滋)
2024-07-23 00:33:00薪水还行的话就练练粪坑里挑黄金的能力 烂code单位到处都是 你迟早要面对的
楼主:
saladim (杀拉顶)
2024-07-23 00:33:00提到学历+当红硕论的原因 文中有写了不是要讨论程式or review 哲学, copy 100行的程式插到另外一个地方 很难相信有单位接受每次都这样做 还是再说一次 这边不是讨论哲学...是在提一件很基本的事情....
作者:
acgotaku (otaku)
2024-07-23 00:47:00如果程式已经肥大老旧到等待重构,旧 method 被大量呼叫我是觉得这种方式不能说不好,你也没更安全的做法实作要重构不知历史没单元测试的旧方法,是我也想动拉
作者:
neo5277 (I am an agent of chaos)
2024-07-23 00:52:00你们单位有code rule嘛?
作者:
acgotaku (otaku)
2024-07-23 00:53:00这种只能等待某个时机点 有人开始重视并重构专案
会提到学历是不是抠抠不平衡要不找主管谈到愿意擦屎要不就离职
作者:
neo5277 (I am an agent of chaos)
2024-07-23 00:54:00100行感觉还好阿
主管在会议还默许这样,是我打不赢就加入,干嘛因为别的智障换工作
作者:
acgotaku (otaku)
2024-07-23 00:57:00大点软件公司一年总会有三到五成精力重构不合需求旧专案
你就拉的比他们更凶更超过 改天就换他们求你改写法了
作者:
neo5277 (I am an agent of chaos)
2024-07-23 01:23:00感觉是银行
没有会当人的主管吗?这种写法review都会被喷吧?
作者: c8dog (草莓猪乌梅猪百香狗) 2024-07-23 01:43:00
因为这种原因想走,在别的公司还是会遇到的,如果没有改变环境的能力,还是别想东想西
作者: superpandal 2024-07-23 01:53:00
我遇到的都是老屁股或有权的在乱搞 有时候会怀疑人生到底这样有什么意义不过这篇毕业没多久的人就像老油条一样想着保饭碗拖时间 真的有前途 比我傻傻受伤害好多了现在职场就是谁跟你好来好去当好人
作者:
MoonCode (MoonCode)
2024-07-23 02:06:00红明显 建议不要内耗 好好刷题跳槽
作者:
Phater (肥特)
2024-07-23 02:20:00junior都吃不下那要你review是橡皮图章喔
作者:
alihue (wanda wanda)
2024-07-23 06:15:00离职
作者:
knives 2024-07-23 06:36:00新人就是这样,功能能跑就好了,除非有人叫你重构,不然别动别人的程式
作者:
pot1234 (锅子)
2024-07-23 07:11:00为什么要改他们经手过的code,他们不能自己改吗?话说你们公司不code review吗
作者:
brucetu (sec)
2024-07-23 07:16:00快跑垃圾公司
作者: ericthree (如果 她这样动人) 2024-07-23 08:06:00
钱多吗? 主管不在意的话就大家一起写烂code啊
烂Code哪里都有,很在意就面试先直球问这点二的情况我也遇过,真要说优点就是低耦合新人进来第一天就能直接改code,改坏了就那个func坏掉而已,免洗好用而且你帮他们改code干嘛?最烦新人进公司一知半解在那自以为的改code,有的是历史问题
作者:
jobintan (Robin Artemstein)
2024-07-23 08:38:00那是人家顶大资深工程师厉害的地方,他们先到处屙屎,等接手的菜鸡处理不来,自己再带菜鸡清理屎山,到时候OKR就能说是自己的credit了。
作者: vinex518 2024-07-23 08:51:00
离职
作者:
NDark (溺于黑暗)
2024-07-23 09:06:00if elss 本身并没有错 都要看情况
作者:
Lhmstu (lhmstu)
2024-07-23 09:38:00没差吧,你知道这样是烂就足够了,他们早晚有一天会知道
没设定程式风格的公司 你是主管吗 不是的话 管很宽喔
有讲过就好,不然就看你有没有心跟主管提或是下去带人啦啦啦
作者:
ji3g4zo6 (黑人闭嘴欧巴马)
2024-07-23 10:07:00同58楼 不懂就问 不然就离职就好
我现在写也都会直觉式写if else,之后再重构现在有AI就方便多了,只要再code review 就好
作者:
zxzx8059 (zxzx8059)
2024-07-23 10:43:00有些chip原厂出来的sdk塞了满满巢状switch-case +if-else + do-while combo
作者: alex70266 (小眼) 2024-07-23 11:13:00
看主管态度啊,主管觉得没差的话就…适当的切好自己的interface就好。除非你是mentor,不然真的bug等他们自己处理coding sense这种东西强求不来的
作者: alex70266 (小眼) 2024-07-23 11:19:00
我自己的经验就是让他们自己痛几下才会学乖,那就不要太早出手做些什么
作者: worf 2024-07-23 11:24:00
你主管都不care了
作者: yamagishi (山岸刑务官) 2024-07-23 11:35:00
纯看你文章内容,这么 junior 的问题用 gpt 尬一下就好ㄌ
作者:
fatb (胖逼=口=)
2024-07-23 11:40:00哈欠 不是人的问题一率都不是问题
作者:
labbat (labbat)
2024-07-23 11:58:00用一堆SETcc MOVcc CMPccXADD会比较高级吗
code review你还不是连个屁都不敢放 问就是离职
作者:
arhtur945 (AnthonyBennet)
2024-07-23 12:58:00Request changes?
作者:
angusyu (〒△〒)
2024-07-23 13:32:00问就是离职,怕你不敢
作者:
pilor (Formosa)
2024-07-23 13:48:00没图没依据
作者: psychic (过路人) 2024-07-23 13:51:00
自己写自己的,又没碍到你 interface切干净就好
动不动就if-else三层 然后三层里面还会再if-else为什么不直接说四层?
不知能不能找一个写得好的专门 review 跟整理/清理程式PR 都要过他,他负责整理好再并
当到 senior 还没办法在 code review 上管动 junior是怎么把自己搞得这么卑屈?2008就在发文 根本超级老 怎么会连几个刚毕业的都管不动 到底人缘多差会靠北 if-else 的人 往往是想要搞一些 strategy patern 或依赖反转之类的东西 但又常不喜欢写文件 最后就是一大坨没有文件又过度设计的屎 还不如几个 if-else
有本事就到有厉害同事的公司啊,没本事的话你也跟你同事半斤八两,不要在那边自我优越了
作者:
knives 2024-07-23 16:39:00已经16年了阿,怎么还在这当这种小事
作者:
ma721 (UndeadJ)
2024-07-23 16:50:00是不是傻,他们比你赚的清松
作者: hobnob (hobnob) 2024-07-23 18:03:00
第一次看到工作十几年还压不过Junior的
作者:
gpctv (gpctv)
2024-07-23 18:26:00本来想分享我的类似经验,但我看被嘴成这样,想想还是算了。
作者: cylee (Artisan) 2024-07-23 18:49:00
你觉得写的不好,那有提出你的解法吗? Code review 要出声review 时每次开炮,我就不信不会改
作者: HelloPTT 2024-07-23 19:14:00
问题可能不在if else,有做好分类、单一职责吗
作者:
Csongs (西歌)
2024-07-23 19:42:00其实真的可以叫他请ai code review
作者:
ma721 (UndeadJ)
2024-07-23 20:16:00半桶水响叮当,问ai都比你强
作者:
uncle925 (-新潮x乐活x便捷)
2024-07-23 22:14:00问就是:你听过Copilot吗
作者:
ECMA 2024-07-23 23:55:00还在手写程式喔 呵呵
楼主:
saladim (杀拉顶)
2024-07-24 00:37:00ㄟ 不是啦 很多人没看文中有写 都有跟对方讲过了 也有其他相对资深的也有讲过 而且大家都是尊重平等理解的状况下去看程式码跟讨论的 这是很自然的事 是不是大家把自己或是自己的体会带入才会觉得别人都在乱当? 现在这时代早就只能用尊重平等理解的方式来讲话了 当然也有提出写法 这都是标准作业流程阿 也不只是我也有其他几位有同样感想不是格式问题 遇过得比较能体会 很难用例子去描述 例子很难显现那种杂乱跟让人觉得思路很乱的方式传炸弹阿~~~块陶阿~~~
作者: boomberm (boom) 2024-07-24 02:32:00
code review不要让他过就好,有什么难的
每间公司都有这种问题,跳到其它公司只是又是另一个坑而己。反正要烂大家一起烂,只要没显示error就好
作者:
mathrew (Joey)
2024-07-24 05:53:00同意楼上,除非你能力真的够,去那种管很好的公司
你可以Design Pattern重构啊,但如果你同事们很菜的话高机率他们会看不懂,反而被告状说改成这样只有你看得懂影响到整个部门,怕事的主管就会宁可大家都看得懂方便出事情后人洒下去一起帮忙issue tracking,也不要只有你一个人处理
相同阶级的人讲了没用,上级主管又不想管所以你操这个心要干嘛?既然有真正主力在做事那又跟你有什么关系?你就是想嘴人家写一手烂Code而已
作者:
pig2014 (Rocking Man)
2024-07-24 10:01:00coding 如果不在同一个等级,就不该在同间公司。如果你跳不走充其量代表你algo或是design 也不够强不能被更强的公司认可,那在一起人各有长也没啥好抱怨的
作者: cylee (Artisan) 2024-07-24 12:31:00
如果对方真的是讲不听,跟主管说也不想管的话,那就算了公事公办,code review 就是严格把关,卡到他们改为止
有没有一种可能你就算去NV Google还是都是这种code居多 不是不想refacotr 而且成本太高
作者:
Abbee (阿比)
2024-07-24 14:44:00code review是资深带资浅在作的 你带不动他们那就是你资历不够 无法说服别人 都说他们是ju了 你不教怎么变se
作者:
jily (吉利)
2024-07-24 16:11:00每个人看别人的code都很痛苦XD 无法习惯表示不适合这一行
作者:
acgotaku (otaku)
2024-07-25 01:57:00重构都是成本拉,写完压测 e2e 什么测试都要重新跑一遍
作者:
invidia (莎莉竟然会飞)
2024-07-25 09:16:00copy 的那个 function 你故意请他们改个小东西他们就会懂有多难 maintain 了吧
作者: tsaigi (菜鸡) 2024-07-25 16:43:00
没人在意你自己在那操心干嘛
作者:
hooll111 (Katsudon)
2024-07-26 12:44:00知道是烂code 又不在review上讲 那codereview 干嘛
作者:
ck237 (白色小鸡)
2024-07-26 13:24:00觉得你这样有点问题,写if else很正常啊有四层判定式不就是需要这个判定,你要资浅的写递回本来就强人所难
作者: windlll (我要工作阿) 2024-07-26 15:53:00
能不能动?能。有解决问题吗?有。有人喊效率吗?没有。结案….鬼事太多,到后来没心力优化这些问题,理想很棒,现实赚钱要紧
老实讲你遇到的真的还好 还有更多可怕的 例如不懂因地制宜强硬要你按照最潮的做法做那才叫痛苦我宁愿遇到解决问题的同事也不要遇到增加问题的同事 给你参考有时候过于匠气真的会气死
作者:
acenova (归零)
2024-07-29 14:44:00程式能动真的没必要花时间在这上面,除非你没事做了
不要浪费时间在这些没kpi的事情大家都是混口饭吃而已 东西挂掉也别太在意
作者:
Ghamu (猫丸)
2024-09-08 23:49:00讲不通或许可以考虑分享文章的方式指桑骂槐 也可以说是单纯技术分享这样?会不会比较对事不对人就等于透过另一个权威reference 做担保 而不是你个人的意见要别人接受