PTT評價

[請益] coding style差太多怎辦?

看板Soft_Job標題[請益] coding style差太多怎辦?作者
prag222
(prag)
時間推噓23 推:26 噓:3 →:65

大家好
小弟上上份工作快離職前
聽到新進的同事說
他都習慣把程式寫成一個一個小的function
後來離職我花了一點時間學習設計模式
和了解SOLID原則
我越覺得這種作法很OK
我大概也知道這應該是重複說高手說過的話
所以後來找到工作
專案自己一個人開發
也沒主管強制要求程式該怎麼寫
變照著 之前同事說的話去開發
讓程式碼 程式碼也是有結構性架構性的
而不是一個function寫幾百行幾千行
mvc Model層也是切得很乾淨
Model層寫的就像api
controller帶參數給MODEL層撈資料出來

不過我最近的公司
完全不是這種做法
雖然是MVC不過還是下SQL查出資料
看到function寫幾百行我看了就昏(業務邏輯)

為了符合公司專案的coding style有點辛苦
基本上我速度也差不多折損一半也有了

不過盡可能把程式碼寫成一個一個小單元應該也沒錯吧
畢竟單元測試
跟我最近看重構的書也是建議這樣
上份工作有改到open source的專案
好像也是這樣
是很難看的懂 但擴充維護修改都很輕鬆






--

※ PTT 留言評論
※ 發信站: 批踢踢實業坊(ptt.cc), 來自: 150.117.70.191 (臺灣)
PTT 網址

aidansky098901/19 21:10能動就好的公司建議逃

knives01/19 21:11下sql會很累,你還太菜,快逃吧

lturtsamuel01/19 21:15mvc跟sql的關連是...?

accessdenied01/19 21:19程式碼寫成一個一個小單元,應該要增加可讀性才對,

accessdenied01/19 21:19怎麼到你手上變成「是很難看懂,但維護輕鬆」?

iamshiao01/19 21:45誰規定 mvc 不能下 sql?

x246libra01/19 21:52他的很難看懂 應該是指 程式碼 會跳來跳去吧 有用介面

x246libra01/19 21:52Imp 通常不會放在同一個檔案

fiiox301/19 21:54我大概懂你意思...我公司目前就是這樣,看到頭很暈

fiiox301/19 21:54同樣邏輯東西不斷複製貼上

alihue01/19 21:56mvc 還是要下 sql,雖然有些 orm 會額外包一層語法,但是

alihue01/19 21:56專案一大,還是 sql 比較好維護。

jj032101/19 21:56哈哈 一個.cs檔塞3~4萬行程式碼還是照樣維護呀

jj032101/19 21:57錢給超多還是吞下去繼續做

a74012501/19 22:01騎驢找馬吧,這種環境待太久不太好

longlyeagle01/19 22:10這個不叫 coding style 叫做有沒有把程式寫好

devilkool01/19 22:13看不懂什麼mvc下sql

wulouise01/19 22:13原本還想要戰tab跟space,進來竟然...!!

j095832208001/19 22:14還以為是我們公司XD

airtsubasa01/19 22:19應該是指不是.where .select ,from a in b

airtsubasa01/19 22:20複雜sql或跨資料庫用套件下也是痛苦啦

devilkool01/19 22:28那下SQL和MVC或是不是爛code無關啊XD

devilkool01/19 22:30不過進到架構爛的公司除非錢很多否則我也會想離職

james73201/19 23:13舊code能正常運作的就不要碰它,有bug要修再趁機重構

james73201/19 23:13有興趣可以參考91的課程,很詳細的說要怎麼做

james73201/19 23:15從不可測試的爛code→可測試的爛code→可測試的好code

james73201/19 23:16不過個人覺得如果對薪水之類沒幫助就不要亂碰它 XD

mercurycgt6801/19 23:16這種好習慣還難看懂原因只有四種,我都親身碰過:1

mercurycgt6801/19 23:16. 命名差 2. 文件/註解沒寫好 3. 沒有靠IDE幫忙跳

mercurycgt6801/19 23:16轉/peek 4. 對方是智障; 看您的行文風格,應該不

mercurycgt6801/19 23:16是4

alan310001/19 23:38如果你只碰orm沒碰過需要sql應該是你摸過的系統都太小

luke7201/19 23:47所謂的code style就是主管,前輩,掌權者說了算

luke7201/19 23:48再多的書 大神文章 google設計模式 先問你薪水誰給的

luke7201/19 23:50差太多怎麼辦?前輩的code你只能跟著阿 不然還能怎麼辦

luke7201/19 23:50等你抓到機會抓到權力 才能慢慢導到你理想的方式去

luke7201/19 23:52而且要想想既有的code為什麼長這樣 改成理想的樣子能動嗎

luke7201/19 23:53很多菜鳥讀了一些文章就以為自己超強 改下去才發現爆光光

Kazimir01/19 23:55要是沒看過某種架構或者pattern會覺得比較難看懂我覺得

luke7201/19 23:55就好像我有一次在牙醫手術台 菜鳥醫師刀開到一半跑去求救

Kazimir01/19 23:55正常吧

luke7201/19 23:56"為什麼跟教科書上的圖不一樣" 我:.......

noahleft01/20 01:20就一邊工作一邊註解一邊refactor

noahleft01/20 01:21書上的範例都很理想 實務上不是人人都懂SOLID

noahleft01/20 01:22而且你會說看到頭昏就表示你自己也還沒很熟悉業務邏輯

noahleft01/20 01:23所以就一邊工作一邊註解確保你理解業務邏輯跟假設

noahleft01/20 01:24熟悉以後再根據SOLID補就好(不是推倒重來

youtuuube00001/20 01:26應該是命名太差造成看不懂吧..

noahleft01/20 01:27像前面版友建議的。可以先理解為啥要SOLID

noahleft01/20 01:28而不是書上說這樣比較好

vi00024601/20 01:33orm跟sql都要學啊 orm有效能瓶頸的

vi00024601/20 01:33我是覺得要先學會看懂爛code 改得動爛code

vi00024601/20 01:33才能體會OOP的美好

WaterLengend01/20 06:57我覺得這不是style 純粹是之前寫太爛

del68020201/20 07:10連調整都做不到 還是轉行吧

taikobo01/20 07:42coding style 跟 code quality 是二回事...

KanzakiHAria01/20 08:36測試能過隨便你改

shooter55501/20 08:45測試能過 然後內容可維護性太低就是豬隊友的做法

testPtt01/20 09:27沒維護到一堆複製貼上的沒資格抱怨啦

xo110001/20 09:42遇過同一個變數用到一千多行還在用的

xo110001/20 09:43不然就是好幾層ifelse 然後三四個變數在變的

t1996080401/20 10:30一堆智障senior也都是各種複製貼上 一堆function快百

t1996080401/20 10:30

shooter55501/20 10:50好幾層ifelse然後好幾個變數這我也遇過 很想砍掉重練

shooter55501/20 10:52然後變數命名還是看不出含意的

lazarus112101/20 11:00我連if的條件看到一堆括號or或and都受不了

lazarus112101/20 11:02寫出這種爛code的人可能還自以為是邏輯大師

brianhsu01/20 11:38ninja code,工程師保護自我價值 XD

alihue01/20 12:01https://bit.ly/2XYGmsy 我都這樣寫

alihue01/20 12:02https://bit.ly/3bPlRqy 更新連結

newhandfun01/20 13:29可讀性跟效能有時候也是要做取捨,我覺得可能要搞清

newhandfun01/20 13:29楚商業邏輯再看看

wayne566894401/20 15:43誰規定一定要用orm? 複雜的東西orm 根本超難處理好

wayne566894401/20 15:43嗎...

ChungLi556601/20 16:09那是以前VB時代留下來的包袱

oachan01/20 22:43如果還只是一般工程師,那只能從手邊的做起,別人的cod

oachan01/20 22:43e 儘量看,自己先維護好自己的程式碼

oachan01/20 22:44如果為了重構反而拖累開發時程,會被叮到飛起來的,等未

oachan01/20 22:45來帶團隊或主管在嘗試傳播想法

viper970901/21 00:16推樓上

uioty01/21 01:26我待的第一間公司主管帶得很好,剛進去的那陣子會很仔細的

uioty01/21 01:26review我的coding style,第二間就真的是能動就好 code有

uioty01/21 01:26時候看到會覺得公司招人標準到底在哪...

brianhsu01/21 09:29能動就好的 code 肯定一堆啦,我還去過那種為了搶快什麼

brianhsu01/21 09:29 code smell 都有,標準教科書負面教材案例大全的新創。

brianhsu01/21 09:29XD

luke7201/21 11:19以前修OS時老師是微軟出身,他說windows也是這樣..

luke7201/21 11:20教科書都太理想僅供參考,winxp一堆沒照課本做的

wulouise01/22 21:16BUG出來會害人停機的..不管怎樣一定是先補起來啊,是取

wulouise01/22 21:16

dogocreat01/24 00:38有時候太複雜的業務需求orm反而效能不好

superpandal01/24 13:51所以說為何當一個好公司的元老很重要 一句話說出職

superpandal01/24 13:51場生態

nayeonmywife03/21 19:06別用ORM了吧… 調效能很慘