PTT職涯區 / Soft_Job (軟體人)

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

看板Soft_Job (軟體人)作者 (prag)時間3年前 (2021/01/19 21:02), 編輯推噓22(25推 3噓 65→)
留言93則, 41人參與, 3年前最新討論串1/3 (看更多)
大家好 小弟上上份工作快離職前 聽到新進的同事說 他都習慣把程式寫成一個一個小的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
推
01/19 21:10, 3年前 , 1F
能動就好的公司建議逃
01/19 21:10, 1F
噓
01/19 21:11, 3年前 , 2F
下sql會很累，你還太菜，快逃吧
01/19 21:11, 2F
噓
01/19 21:15, 3年前 , 3F
mvc跟sql的關連是...?
01/19 21:15, 3F
→
01/19 21:19, 3年前 , 4F
程式碼寫成一個一個小單元，應該要增加可讀性才對，
01/19 21:19, 4F
→
01/19 21:19, 3年前 , 5F
怎麼到你手上變成「是很難看懂，但維護輕鬆」？
01/19 21:19, 5F
→
01/19 21:45, 3年前 , 6F
誰規定 mvc 不能下 sql?
01/19 21:45, 6F
→
01/19 21:52, 3年前 , 7F
他的很難看懂 應該是指 程式碼 會跳來跳去吧 有用介面
01/19 21:52, 7F
→
01/19 21:52, 3年前 , 8F
Imp 通常不會放在同一個檔案
01/19 21:52, 8F
推
01/19 21:54, 3年前 , 9F
我大概懂你意思...我公司目前就是這樣，看到頭很暈
01/19 21:54, 9F
→
01/19 21:54, 3年前 , 10F
同樣邏輯東西不斷複製貼上
01/19 21:54, 10F
推
01/19 21:56, 3年前 , 11F
mvc 還是要下 sql，雖然有些 orm 會額外包一層語法，但是
01/19 21:56, 11F
→
01/19 21:56, 3年前 , 12F
專案一大，還是 sql 比較好維護。
01/19 21:56, 12F
推
01/19 21:56, 3年前 , 13F
哈哈 一個.cs檔塞3~4萬行程式碼還是照樣維護呀
01/19 21:56, 13F
→
01/19 21:57, 3年前 , 14F
錢給超多還是吞下去繼續做
01/19 21:57, 14F
→
01/19 22:01, 3年前 , 15F
騎驢找馬吧，這種環境待太久不太好
01/19 22:01, 15F
推
01/19 22:10, 3年前 , 16F
這個不叫 coding style 叫做有沒有把程式寫好
01/19 22:10, 16F
→
01/19 22:13, 3年前 , 17F
看不懂什麼mvc下sql
01/19 22:13, 17F
推
01/19 22:13, 3年前 , 18F
原本還想要戰tab跟space,進來竟然...!!
01/19 22:13, 18F
推
01/19 22:14, 3年前 , 19F
還以為是我們公司XD
01/19 22:14, 19F
→
01/19 22:19, 3年前 , 20F
應該是指不是.where .select ,from a in b
01/19 22:19, 20F
→
01/19 22:20, 3年前 , 21F
複雜sql或跨資料庫用套件下也是痛苦啦
01/19 22:20, 21F
→
01/19 22:28, 3年前 , 22F
那下SQL和MVC或是不是爛code無關啊XD
01/19 22:28, 22F
→
01/19 22:30, 3年前 , 23F
不過進到架構爛的公司除非錢很多否則我也會想離職
01/19 22:30, 23F
→
01/19 23:13, 3年前 , 24F
舊code能正常運作的就不要碰它，有bug要修再趁機重構
01/19 23:13, 24F
→
01/19 23:13, 3年前 , 25F
有興趣可以參考91的課程，很詳細的說要怎麼做
01/19 23:13, 25F
→
01/19 23:15, 3年前 , 26F
從不可測試的爛code→可測試的爛code→可測試的好code
01/19 23:15, 26F
→
01/19 23:16, 3年前 , 27F
不過個人覺得如果對薪水之類沒幫助就不要亂碰它 XD
01/19 23:16, 27F
推
01/19 23:16, 3年前 , 28F
這種好習慣還難看懂原因只有四種，我都親身碰過：1
01/19 23:16, 28F
→
01/19 23:16, 3年前 , 29F
. 命名差 2. 文件/註解沒寫好 3. 沒有靠IDE幫忙跳
01/19 23:16, 29F
→
01/19 23:16, 3年前 , 30F
轉/peek 4. 對方是智障; 看您的行文風格，應該不
01/19 23:16, 30F
→
01/19 23:16, 3年前 , 31F
是4
01/19 23:16, 31F
推
01/19 23:38, 3年前 , 32F
如果你只碰orm沒碰過需要sql應該是你摸過的系統都太小
01/19 23:38, 32F
推
01/19 23:47, 3年前 , 33F
所謂的code style就是主管,前輩,掌權者說了算
01/19 23:47, 33F
→
01/19 23:48, 3年前 , 34F
再多的書 大神文章 google設計模式 先問你薪水誰給的
01/19 23:48, 34F
→
01/19 23:50, 3年前 , 35F
差太多怎麼辦？前輩的code你只能跟著阿 不然還能怎麼辦
01/19 23:50, 35F
→
01/19 23:50, 3年前 , 36F
等你抓到機會抓到權力 才能慢慢導到你理想的方式去
01/19 23:50, 36F
→
01/19 23:52, 3年前 , 37F
而且要想想既有的code為什麼長這樣 改成理想的樣子能動嗎
01/19 23:52, 37F
→
01/19 23:53, 3年前 , 38F
很多菜鳥讀了一些文章就以為自己超強 改下去才發現爆光光
01/19 23:53, 38F
→
01/19 23:55, 3年前 , 39F
要是沒看過某種架構或者pattern會覺得比較難看懂我覺得
01/19 23:55, 39F
→
01/19 23:55, 3年前 , 40F
就好像我有一次在牙醫手術台 菜鳥醫師刀開到一半跑去求救
01/19 23:55, 40F
→
01/19 23:55, 3年前 , 41F
正常吧
01/19 23:55, 41F
→
01/19 23:56, 3年前 , 42F
"為什麼跟教科書上的圖不一樣" 我：.......
01/19 23:56, 42F
推
01/20 01:20, 3年前 , 43F
就一邊工作一邊註解一邊refactor
01/20 01:20, 43F
→
01/20 01:21, 3年前 , 44F
書上的範例都很理想 實務上不是人人都懂SOLID
01/20 01:21, 44F
→
01/20 01:22, 3年前 , 45F
而且你會說看到頭昏就表示你自己也還沒很熟悉業務邏輯
01/20 01:22, 45F
→
01/20 01:23, 3年前 , 46F
所以就一邊工作一邊註解確保你理解業務邏輯跟假設
01/20 01:23, 46F
→
01/20 01:24, 3年前 , 47F
熟悉以後再根據SOLID補就好（不是推倒重來
01/20 01:24, 47F
推
01/20 01:26, 3年前 , 48F
應該是命名太差造成看不懂吧..
01/20 01:26, 48F
推
01/20 01:27, 3年前 , 49F
像前面版友建議的。可以先理解為啥要SOLID
01/20 01:27, 49F
→
01/20 01:28, 3年前 , 50F
而不是書上說這樣比較好
01/20 01:28, 50F
推
01/20 01:33, 3年前 , 51F
orm跟sql都要學啊 orm有效能瓶頸的
01/20 01:33, 51F
→
01/20 01:33, 3年前 , 52F
我是覺得要先學會看懂爛code 改得動爛code
01/20 01:33, 52F
→
01/20 01:33, 3年前 , 53F
才能體會OOP的美好
01/20 01:33, 53F
推
01/20 06:57, 3年前 , 54F
我覺得這不是style 純粹是之前寫太爛
01/20 06:57, 54F
推
01/20 07:10, 3年前 , 55F
連調整都做不到 還是轉行吧
01/20 07:10, 55F
→
01/20 07:42, 3年前 , 56F
coding style 跟 code quality 是二回事...
01/20 07:42, 56F
噓
01/20 08:36, 3年前 , 57F
測試能過隨便你改
01/20 08:36, 57F
→
01/20 08:45, 3年前 , 58F
測試能過 然後內容可維護性太低就是豬隊友的做法
01/20 08:45, 58F
→
01/20 09:27, 3年前 , 59F
沒維護到一堆複製貼上的沒資格抱怨啦
01/20 09:27, 59F
→
01/20 09:42, 3年前 , 60F
遇過同一個變數用到一千多行還在用的
01/20 09:42, 60F
→
01/20 09:43, 3年前 , 61F
不然就是好幾層ifelse 然後三四個變數在變的
01/20 09:43, 61F
推
01/20 10:30, 3年前 , 62F
一堆智障senior也都是各種複製貼上 一堆function快百
01/20 10:30, 62F
→
01/20 10:30, 3年前 , 63F
行
01/20 10:30, 63F
→
01/20 10:50, 3年前 , 64F
好幾層ifelse然後好幾個變數這我也遇過 很想砍掉重練
01/20 10:50, 64F
→
01/20 10:52, 3年前 , 65F
然後變數命名還是看不出含意的
01/20 10:52, 65F
→
01/20 11:00, 3年前 , 66F
我連if的條件看到一堆括號or或and都受不了
01/20 11:00, 66F
→
01/20 11:02, 3年前 , 67F
寫出這種爛code的人可能還自以為是邏輯大師
01/20 11:02, 67F
推
01/20 11:38, 3年前 , 68F
ninja code，工程師保護自我價值 XD
01/20 11:38, 68F
推
01/20 12:01, 3年前 , 69F
https://bit.ly/2XYGmsy 我都這樣寫
01/20 12:01, 69F
→
01/20 12:02, 3年前 , 70F
01/20 12:02, 70F
→
01/20 13:29, 3年前 , 71F
可讀性跟效能有時候也是要做取捨，我覺得可能要搞清
01/20 13:29, 71F
→
01/20 13:29, 3年前 , 72F
楚商業邏輯再看看
01/20 13:29, 72F
推
01/20 15:43, 3年前 , 73F
誰規定一定要用orm? 複雜的東西orm 根本超難處理好
01/20 15:43, 73F
→
01/20 15:43, 3年前 , 74F
嗎...
01/20 15:43, 74F
→
01/20 16:09, 3年前 , 75F
那是以前VB時代留下來的包袱
01/20 16:09, 75F
推
01/20 22:43, 3年前 , 76F
如果還只是一般工程師，那只能從手邊的做起，別人的cod
01/20 22:43, 76F
→
01/20 22:43, 3年前 , 77F
e 儘量看，自己先維護好自己的程式碼
01/20 22:43, 77F
→
01/20 22:44, 3年前 , 78F
如果為了重構反而拖累開發時程，會被叮到飛起來的，等未
01/20 22:44, 78F
→
01/20 22:45, 3年前 , 79F
來帶團隊或主管在嘗試傳播想法
01/20 22:45, 79F
→
01/21 00:16, 3年前 , 80F
推樓上
01/21 00:16, 80F
推
01/21 01:26, 3年前 , 81F
我待的第一間公司主管帶得很好，剛進去的那陣子會很仔細的
01/21 01:26, 81F
→
01/21 01:26, 3年前 , 82F
review我的coding style，第二間就真的是能動就好 code有
01/21 01:26, 82F
→
01/21 01:26, 3年前 , 83F
時候看到會覺得公司招人標準到底在哪...
01/21 01:26, 83F
→
01/21 09:29, 3年前 , 84F
能動就好的 code 肯定一堆啦，我還去過那種為了搶快什麼
01/21 09:29, 84F
→
01/21 09:29, 3年前 , 85F
code smell 都有，標準教科書負面教材案例大全的新創。
01/21 09:29, 85F
→
01/21 09:29, 3年前 , 86F
XD
01/21 09:29, 86F
推
01/21 11:19, 3年前 , 87F
以前修OS時老師是微軟出身，他說windows也是這樣..
01/21 11:19, 87F
推
01/21 11:20, 3年前 , 88F
教科書都太理想僅供參考，winxp一堆沒照課本做的
01/21 11:20, 88F
推
01/22 21:16, 3年前 , 89F
BUG出來會害人停機的..不管怎樣一定是先補起來啊，是取
01/22 21:16, 89F
→
01/22 21:16, 3年前 , 90F
捨
01/22 21:16, 90F
→
01/24 00:38, 3年前 , 91F
有時候太複雜的業務需求orm反而效能不好
01/24 00:38, 91F
→
01/24 13:51, 3年前 , 92F
所以說為何當一個好公司的元老很重要 一句話說出職
01/24 13:51, 92F
→
01/24 13:51, 3年前 , 93F
場生態
01/24 13:51, 93F
‣ 更多 prag222 的文章
文章代碼(AID): #1W1jXgBp (Soft_Job)
Soft_Job 近期熱門文章
更多 近期熱門文章 >>
PTT職涯區 即時熱門文章
更多 即時熱門文章 >>
‣ 更多 prag222 的文章
文章代碼(AID): #1W1jXgBp (Soft_Job)