Re: [討論] 主管不認同書本的知識,說我沒學好程設

作者: CLFJ   2016-05-08 00:39:49
我的建議是,如果你沒有辦法說服你的老闆,那表示你也還沒有通透為什麼
書上要這樣寫,也只是知其然不知其所以然,那就再努力點把更多細節搞懂,
對你也不是壞事。
比方說吧,建構子到底是要參數吃到飽還是分開寫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
: 重構的作者是建議參數不用丟太多,建立一個物件,
: 設定物件的值,把物件丟進建構子,或方法參數中,然後我這樣跟我主管說,他又說我沒
: 腦袋嗎
: 沒辦法判定這個作者有問題
: 參數本來就全丟給建構子,讓建構子去塞,即便
: 參數很多也沒關係,說我物件導向沒學好
: 反正一直在對我人身攻擊,即使我提到重構
: 設計模式,對他來說就是爛書,作者亂寫
: 請問我該如何是好?
作者: Deltaguita (貝里斯)   2016-05-08 01:34:00
既然有code review 就有PR紀錄
作者: Blueshiva (龍野南雲)   2016-05-08 01:44:00
出事的時候老闆要從大老闆那脫罪,你有PR/email是有啥用啦 XDD
作者: yourinfo (...)   2016-05-08 09:25:00
是能出什麼事?最慘的不就是找人重寫而已,很正常啦而且重寫可能是一年後了,或許更久以後才發生
作者: storyn26383 (販賣機)   2016-05-08 10:34:00
一個會動的程式,就是好程式 (X說不定就永遠不會改了呢 xD我是認為建構子一堆參數我很難 new如果真要全在建構子做,用 option 模式可能會好些
作者: littlethe (東周流浪漢)   2016-05-08 11:06:00
我也傾向不在建構子加參數,建構子加一堆參數不好改

Links booklink

Contact Us: admin [ a t ] ucptt.com