[请益] coding style差太多怎办?

楼主: prag222 (prag)   2021-01-19 21:02:29
大家好
小弟上上份工作快离职前
听到新进的同事说
他都习惯把程式写成一个一个小的function
后来离职我花了一点时间学习设计模式
和了解SOLID原则
我越觉得这种作法很OK
我大概也知道这应该是重复说高手说过的话
所以后来找到工作
专案自己一个人开发
也没主管强制要求程式该怎么写
变照着 之前同事说的话去开发
让程式码 程式码也是有结构性架构性的
而不是一个function写几百行几千行
mvc Model层也是切得很干净
Model层写的就像api
controller带参数给MODEL层捞资料出来
不过我最近的公司
完全不是这种做法
虽然是MVC不过还是下SQL查出资料
看到function写几百行我看了就昏(业务逻辑)
为了符合公司专案的coding style有点辛苦
基本上我速度也差不多折损一半也有了
不过尽可能把程式码写成一个一个小单元应该也没错吧
毕竟单元测试
跟我最近看重构的书也是建议这样
上份工作有改到open source的专案
好像也是这样
是很难看的懂 但扩充维护修改都很轻松
作者: aidansky0989 (alta)   2021-01-19 21:10:00
能动就好的公司建议逃
作者: knives   2021-01-19 21:11:00
下sql会很累,你还太菜,快逃吧
作者: lturtsamuel (港都都教授)   2021-01-19 21:15:00
mvc跟sql的关连是...?
作者: accessdenied (存取违规)   2021-01-19 21:19:00
程式码写成一个一个小单元,应该要增加可读性才对,怎么到你手上变成“是很难看懂,但维护轻松”?
作者: iamshiao (CircleHsiao)   2021-01-19 21:45:00
谁规定 mvc 不能下 sql?
作者: x246libra (楓)   2021-01-19 21:52:00
他的很难看懂 应该是指 程式码 会跳来跳去吧 有用接口Imp 通常不会放在同一个档案
作者: fiiox3 (飙速宅男)   2021-01-19 21:54:00
我大概懂你意思...我公司目前就是这样,看到头很晕同样逻辑东西不断复制贴上
作者: alihue (wanda wanda)   2021-01-19 21:56:00
mvc 还是要下 sql,虽然有些 orm 会额外包一层语法,但是专案一大,还是 sql 比较好维护。
作者: jj0321 (JJ与你倒数唷)   2021-01-19 21:56:00
哈哈 一个.cs档塞3~4万行程式码还是照样维护呀钱给超多还是吞下去继续做
作者: a740125 (哈哈)   2021-01-19 22:01:00
骑驴找马吧,这种环境待太久不太好
作者: longlyeagle (长鹰宝宝实验室)   2021-01-19 22:10:00
这个不叫 coding style 叫做有没有把程式写好
作者: devilkool (对猫毛过敏的猫控)   2021-01-19 22:13:00
看不懂什么mvc下sql
作者: wulouise (在线上!=在电脑前)   2021-01-19 22:13:00
原本还想要战tab跟space,进来竟然...!!
作者: j0958322080 (Tidus)   2021-01-19 22:14:00
还以为是我们公司XD
作者: airtsubasa (伪学姊)   2021-01-19 22:19:00
应该是指不是.where .select ,from a in b复杂sql或跨数据库用套件下也是痛苦啦
作者: devilkool (对猫毛过敏的猫控)   2021-01-19 22:28:00
那下SQL和MVC或是不是烂code无关啊XD不过进到架构烂的公司除非钱很多否则我也会想离职
作者: james732 (好人超)   2021-01-19 23:13:00
旧code能正常运作的就不要碰它,有bug要修再趁机重构有兴趣可以参考91的课程,很详细的说要怎么做从不可测试的烂code→可测试的烂code→可测试的好code不过个人觉得如果对薪水之类没帮助就不要乱碰它 XD
作者: mercurycgt68 (发芽的吉它手)   2021-01-19 23:16:00
这种好习惯还难看懂原因只有四种,我都亲身碰过:1. 命名差 2. 文件/注解没写好 3. 没有靠IDE帮忙跳转/peek 4. 对方是智障; 看您的行文风格,应该不是4
作者: alan3100 (BOSS)   2021-01-19 23:38:00
如果你只碰orm没碰过需要sql应该是你摸过的系统都太小
作者: luke72 (ccc)   2021-01-19 23:47:00
所谓的code style就是主管,前辈,掌权者说了算再多的书 大神文章 google设计模式 先问你薪水谁给的差太多怎么办?前辈的code你只能跟着阿 不然还能怎么办等你抓到机会抓到权力 才能慢慢导到你理想的方式去而且要想想既有的code为什么长这样 改成理想的样子能动吗很多菜鸟读了一些文章就以为自己超强 改下去才发现爆光光
作者: Kazimir (Kazimir)   2021-01-19 23:55:00
要是没看过某种架构或者pattern会觉得比较难看懂我觉得
作者: luke72 (ccc)   2021-01-19 23:55:00
就好像我有一次在牙医手术台 菜鸟医师刀开到一半跑去求救
作者: luke72 (ccc)   2021-01-19 23:56:00
"为什么跟教科书上的图不一样" 我:.......
作者: noahleft (NoahLeft)   2021-01-20 01:20:00
就一边工作一边注解一边refactor书上的范例都很理想 实务上不是人人都懂SOLID而且你会说看到头昏就表示你自己也还没很熟悉业务逻辑所以就一边工作一边注解确保你理解业务逻辑跟假设熟悉以后再根据SOLID补就好(不是推倒重来
作者: youtuuube000 (小孩)   2021-01-20 01:26:00
应该是命名太差造成看不懂吧..
作者: noahleft (NoahLeft)   2021-01-20 01:27:00
像前面版友建议的。可以先理解为啥要SOLID而不是书上说这样比较好
作者: vi000246 (Vi)   2021-01-20 01:33:00
orm跟sql都要学啊 orm有效能瓶颈的我是觉得要先学会看懂烂code 改得动烂code才能体会OOP的美好
作者: WaterLengend (Leeeeeeeeooooooo)   2021-01-20 06:57:00
我觉得这不是style 纯粹是之前写太烂
作者: del680202 (HANA)   2021-01-20 07:10:00
连调整都做不到 还是转行吧
作者: taikobo (勉强になるなぁ...)   2021-01-20 07:42:00
coding style 跟 code quality 是二回事...
作者: KanzakiHAria (神崎・H・アリア)   2021-01-20 08:36:00
测试能过随便你改
作者: shooter555 (shooter)   2021-01-20 08:45:00
测试能过 然后内容可维护性太低就是猪队友的做法
作者: testPtt (测试)   2021-01-20 09:27:00
没维护到一堆复制贴上的没资格抱怨啦
作者: xo1100 (虾蒸河粉)   2021-01-20 09:42:00
遇过同一个变量用到一千多行还在用的不然就是好几层ifelse 然后三四个变量在变的
作者: t19960804 (泥好吗)   2021-01-20 10:30:00
一堆智障senior也都是各种复制贴上 一堆function快百行
作者: shooter555 (shooter)   2021-01-20 10:50:00
好几层ifelse然后好几个变量这我也遇过 很想砍掉重练然后变量命名还是看不出含意的
作者: lazarus1121 (...)   2021-01-20 11:00:00
我连if的条件看到一堆括号or或and都受不了写出这种烂code的人可能还自以为是逻辑大师
作者: brianhsu (坟墓)   2021-01-20 11:38:00
ninja code,工程师保护自我价值 XD
作者: alihue (wanda wanda)   2021-01-20 12:01:00
作者: newhandfun (新手方)   2021-01-20 13:29:00
可读性跟效能有时候也是要做取舍,我觉得可能要搞清楚商业逻辑再看看
作者: wayne5668944 (wayne5668944)   2021-01-20 15:43:00
谁规定一定要用orm? 复杂的东西orm 根本超难处理好吗...
作者: ChungLi5566 (中坜56哥)   2021-01-20 16:09:00
那是以前VB时代留下来的包袱
作者: oachan (阿诚)   2021-01-20 22:43:00
如果还只是一般工程师,那只能从手边的做起,别人的code 尽量看,自己先维护好自己的程式码如果为了重构反而拖累开发时程,会被叮到飞起来的,等未来带团队或主管在尝试传播想法
作者: viper9709 (阿达)   2021-01-21 00:16:00
推楼上
作者: uioty (uioty)   2021-01-21 01:26:00
我待的第一间公司主管带得很好,刚进去的那阵子会很仔细的review我的coding style,第二间就真的是能动就好 code有时候看到会觉得公司招人标准到底在哪...
作者: brianhsu (坟墓)   2021-01-21 09:29:00
能动就好的 code 肯定一堆啦,我还去过那种为了抢快什么code smell 都有,标准教科书负面教材案例大全的新创。XD
作者: luke72 (ccc)   2021-01-21 11:19:00
以前修OS时老师是微软出身,他说windows也是这样..教科书都太理想仅供参考,winxp一堆没照课本做的
作者: wulouise (在线上!=在电脑前)   2021-01-22 21:16:00
BUG出来会害人停机的..不管怎样一定是先补起来啊,是取舍
作者: dogocreat   2021-01-24 00:38:00
有时候太复杂的业务需求orm反而效能不好
作者: superpandal   2021-01-24 13:51:00
所以说为何当一个好公司的元老很重要 一句话说出职场生态

Links booklink

Contact Us: admin [ a t ] ucptt.com