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

[prag222 (prag)]

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

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

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

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






--
※ 發信站: 批踢踢實業坊(ptt.cc), 來自: 150.117.70.191 (臺灣)
※ 文章網址: https://www.ptt.cc/bbs/Soft_Job/M.1611061354.A.2F3.html
推 aidansky0989: 能動就好的公司建議逃
噓 knives: 下sql會很累，你還太菜，快逃吧
噓 lturtsamuel: mvc跟sql的關連是...?
→ accessdenied: 程式碼寫成一個一個小單元，應該要增加可讀性才對，
→ accessdenied: 怎麼到你手上變成「是很難看懂，但維護輕鬆」？
→ iamshiao: 誰規定 mvc 不能下 sql?
→ x246libra: 他的很難看懂 應該是指 程式碼 會跳來跳去吧 有用介面
→ x246libra: Imp 通常不會放在同一個檔案
推 fiiox3: 我大概懂你意思...我公司目前就是這樣，看到頭很暈
→ fiiox3: 同樣邏輯東西不斷複製貼上
推 alihue: mvc 還是要下 sql，雖然有些 orm 會額外包一層語法，但是
→ alihue: 專案一大，還是 sql 比較好維護。
推 jj0321: 哈哈 一個.cs檔塞3~4萬行程式碼還是照樣維護呀
→ jj0321: 錢給超多還是吞下去繼續做
→ a740125: 騎驢找馬吧，這種環境待太久不太好
推 longlyeagle: 這個不叫 coding style 叫做有沒有把程式寫好
→ devilkool: 看不懂什麼mvc下sql
推 wulouise: 原本還想要戰tab跟space,進來竟然...!!
推 j0958322080: 還以為是我們公司XD
→ airtsubasa: 應該是指不是.where .select ,from a in b
→ airtsubasa: 複雜sql或跨資料庫用套件下也是痛苦啦
→ devilkool: 那下SQL和MVC或是不是爛code無關啊XD
→ devilkool: 不過進到架構爛的公司除非錢很多否則我也會想離職
→ james732: 舊code能正常運作的就不要碰它，有bug要修再趁機重構
→ james732: 有興趣可以參考91的課程，很詳細的說要怎麼做
→ james732: 從不可測試的爛code→可測試的爛code→可測試的好code
→ james732: 不過個人覺得如果對薪水之類沒幫助就不要亂碰它 XD
推 mercurycgt68: 這種好習慣還難看懂原因只有四種，我都親身碰過：1
→ mercurycgt68: . 命名差 2. 文件/註解沒寫好 3. 沒有靠IDE幫忙跳
→ mercurycgt68: 轉/peek 4.  對方是智障; 看您的行文風格，應該不
→ mercurycgt68: 是4
推 alan3100: 如果你只碰orm沒碰過需要sql應該是你摸過的系統都太小
推 luke72: 所謂的code style就是主管,前輩,掌權者說了算
→ luke72: 再多的書  大神文章  google設計模式  先問你薪水誰給的
→ luke72: 差太多怎麼辦？前輩的code你只能跟著阿  不然還能怎麼辦
→ luke72: 等你抓到機會抓到權力  才能慢慢導到你理想的方式去
→ luke72: 而且要想想既有的code為什麼長這樣  改成理想的樣子能動嗎
→ luke72: 很多菜鳥讀了一些文章就以為自己超強  改下去才發現爆光光
→ Kazimir: 要是沒看過某種架構或者pattern會覺得比較難看懂我覺得
→ luke72: 就好像我有一次在牙醫手術台  菜鳥醫師刀開到一半跑去求救
→ Kazimir: 正常吧
→ luke72: "為什麼跟教科書上的圖不一樣"  我：.......
推 noahleft: 就一邊工作一邊註解一邊refactor
→ noahleft: 書上的範例都很理想 實務上不是人人都懂SOLID
→ noahleft: 而且你會說看到頭昏就表示你自己也還沒很熟悉業務邏輯
→ noahleft: 所以就一邊工作一邊註解確保你理解業務邏輯跟假設
→ noahleft: 熟悉以後再根據SOLID補就好（不是推倒重來
推 youtuuube000: 應該是命名太差造成看不懂吧..
推 noahleft: 像前面版友建議的。可以先理解為啥要SOLID
→ noahleft: 而不是書上說這樣比較好
推 vi000246: orm跟sql都要學啊 orm有效能瓶頸的
→ vi000246: 我是覺得要先學會看懂爛code 改得動爛code
→ vi000246: 才能體會OOP的美好
推 WaterLengend: 我覺得這不是style 純粹是之前寫太爛
推 del680202: 連調整都做不到 還是轉行吧
→ taikobo: coding style 跟 code quality 是二回事...
噓 KanzakiHAria: 測試能過隨便你改
→ shooter555: 測試能過 然後內容可維護性太低就是豬隊友的做法
→ testPtt: 沒維護到一堆複製貼上的沒資格抱怨啦
→ xo1100: 遇過同一個變數用到一千多行還在用的
→ xo1100: 不然就是好幾層ifelse 然後三四個變數在變的
推 t19960804: 一堆智障senior也都是各種複製貼上  一堆function快百
→ t19960804: 行
→ shooter555: 好幾層ifelse然後好幾個變數這我也遇過 很想砍掉重練
→ shooter555: 然後變數命名還是看不出含意的
→ lazarus1121: 我連if的條件看到一堆括號or或and都受不了
→ lazarus1121: 寫出這種爛code的人可能還自以為是邏輯大師
推 brianhsu: ninja code，工程師保護自我價值 XD
推 alihue: https://bit.ly/2XYGmsy 我都這樣寫
→ alihue: https://bit.ly/3bPlRqy 更新連結
→ newhandfun: 可讀性跟效能有時候也是要做取捨，我覺得可能要搞清
→ newhandfun: 楚商業邏輯再看看
推 wayne5668944: 誰規定一定要用orm? 複雜的東西orm 根本超難處理好
→ wayne5668944: 嗎...
→ ChungLi5566: 那是以前VB時代留下來的包袱
推 oachan: 如果還只是一般工程師，那只能從手邊的做起，別人的cod
→ oachan: e 儘量看，自己先維護好自己的程式碼
→ oachan: 如果為了重構反而拖累開發時程，會被叮到飛起來的，等未
→ oachan: 來帶團隊或主管在嘗試傳播想法
→ viper9709: 推樓上
推 uioty: 我待的第一間公司主管帶得很好，剛進去的那陣子會很仔細的
→ uioty: review我的coding style，第二間就真的是能動就好 code有
→ uioty: 時候看到會覺得公司招人標準到底在哪...
→ brianhsu: 能動就好的 code 肯定一堆啦，我還去過那種為了搶快什麼
→ brianhsu:  code smell 都有，標準教科書負面教材案例大全的新創。
→ brianhsu: XD
推 luke72: 以前修OS時老師是微軟出身，他說windows也是這樣..
推 luke72: 教科書都太理想僅供參考，winxp一堆沒照課本做的
推 wulouise: BUG出來會害人停機的..不管怎樣一定是先補起來啊，是取
→ wulouise: 捨
→ dogocreat: 有時候太複雜的業務需求orm反而效能不好
→ superpandal: 所以說為何當一個好公司的元老很重要  一句話說出職
→ superpandal: 場生態
推 nayeonmywife: 別用ORM了吧… 調效能很慘