看板 Soft_Job 關於我們 聯絡資訊
我的建議是,如果你沒有辦法說服你的老闆,那表示你也還沒有通透為什麼 書上要這樣寫,也只是知其然不知其所以然,那就再努力點把更多細節搞懂, 對你也不是壞事。 比方說吧,建構子到底是要參數吃到飽還是分開寫get/set,跟他放在你整個 架構的哪一層有很大的關係。跟將來維護的頻率也有很大的關係,不是一定 哪個好哪個不好。 把更多細節搞清楚也是未來溝通很重要的工具,你也許今天說服不了你老闆, 輸了沒關係,卻可以讓你變得更強大。總有一天你會說服下一個老闆的。 但是結論是,既然他是老闆,照他說的改吧。之後如果要維護改code,因為 之前的彈性都沒了,現在時間要比較多,他也只能吞了。所以,記得發個 email給老闆確認code review的結果,免得到時後到打一靶說,工程師亂寫... ※ 引述《purin88 (原來我是憤怒的鄉民)》之銘言: : code review時,主管說暫存變數可省記憶體,不用一直建立變數佔記憶體,我就說"重 : 構"這本書作 : 者建議別這樣做,我就拿下面這個"重構"作者的網址 : https://sourcemaking.com/refactoring/split-temporary-variable : 他就說這個作者有問題,說我跟他寫一樣出去別人 : 會笑我 : 接著,我程式有用簡單工廠模式,就像head first design patten的內容一樣建立pizza : 店的工廠,他又 : 說為什麼要建立抽象的pizza店,建立A pizza加盟店,B pizza加盟店,我說每間pizza店 : 產生pizza囗味,方法不同,他又說建立A pizza店,B pizza店 : 產生物件浪費記憶體,為何不用switch case判定 : 是A或B,直接寫各店pizza的作法及口味,產生pizza的作法何必封 : 裝在A pizza物件,或B物件中,全寫在pizza這個程式中,寫一個類別靜態方法回傳pizza : 一樣的,他沒看過design patten,也覺得四人幫在亂寫一通,建立物件是浪費記憶體 : https://rongli.gitbooks.io/design-pattern/content/chapter1.html : https://dotblogs.com.tw/joysdw12/archive/2013/06/23/design-pattern-simple-fact : ory-pattern.aspx : 然後談到建立物件,我是用set get的方式設置參數,他就覺得為什麼不用建構子把好幾 : 個參數丟進去,我一樣拿出 : https://sourcemaking.com/refactoring/smells/long-parameter-list : http://teddy-chen-tw.blogspot.tw/2014/04/3long-parameter-list-divergent-change : .html?m=1 : 重構的作者是建議參數不用丟太多,建立一個物件, : 設定物件的值,把物件丟進建構子,或方法參數中,然後我這樣跟我主管說,他又說我沒 : 腦袋嗎 : 沒辦法判定這個作者有問題 : 參數本來就全丟給建構子,讓建構子去塞,即便 : 參數很多也沒關係,說我物件導向沒學好 : 反正一直在對我人身攻擊,即使我提到重構 : 設計模式,對他來說就是爛書,作者亂寫 : 請問我該如何是好? -- ※ 發信站: 批踢踢實業坊(ptt.cc), 來自: 75.17.244.220 ※ 文章網址: https://www.ptt.cc/bbs/Soft_Job/M.1462639192.A.F90.html
Deltaguita: 既然有code review 就有PR紀錄 05/08 01:34
Blueshiva: 出事的時候老闆要從大老闆那脫罪,你有PR/email是有啥 05/08 01:44
Blueshiva: 用啦 XDD 05/08 01:44
yourinfo: 是能出什麼事?最慘的不就是找人重寫而已,很正常啦 05/08 09:25
yourinfo: 而且重寫可能是一年後了,或許更久以後才發生 05/08 09:26
storyn26383: 一個會動的程式,就是好程式 (X 05/08 10:34
storyn26383: 說不定就永遠不會改了呢 xD 05/08 10:34
storyn26383: 我是認為建構子一堆參數我很難 new 05/08 10:35
storyn26383: 如果真要全在建構子做,用 option 模式可能會好些 05/08 10:36
littlethe: 我也傾向不在建構子加參數,建構子加一堆參數不好改 05/08 11:06