Soft_Job 板


LINE

※ 引述《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 : 重構的作者是建議參數不用丟太多,建立一個物件, : 設定物件的值,把物件丟進建構子,或方法參數中,然後我這樣跟我主管說,他又說我沒 : 腦袋嗎 : 沒辦法判定這個作者有問題 : 參數本來就全丟給建構子,讓建構子去塞,即便 : 參數很多也沒關係,說我物件導向沒學好 : 反正一直在對我人身攻擊,即使我提到重構 : 設計模式,對他來說就是爛書,作者亂寫 : 請問我該如何是好? 我重構的方法是用樣板簡單工廠合併 1.模組由樣板實作 2.產品由工廠實作 在工廠階段通過參數switch決定要什麼樣板 並實作樣板內容 我認爲你老大要的是動態時期注入什麼方法 這的確在初期會減少記憶體,在測試上也比較彈性 再來我會開始定義領域物件,透過IResult泛型做統一回傳class,並定義該工廠的Reapon se class裡面定義不同樣板Domain class, 重構以後程式結構大概如下 http://i.imgur.com/XSxp7Pv.jpg
我重構得前提是維持住現有的商務邏輯,透過封裝和釐清權責後再重構,先決定領域,將 相關的領域從原本大雜燴的程式抽離,將共同部份放到樣本主類別,子類別決定細節,工 廠階段則決定要何種子類別。 這樣閱讀性和效能說真的比原先好多 我重構也會找組長和長官討論多請益,基本上codereview前先討論,並能說明該方法好處 在哪,我想不會被刁難 --



※ 發信站: 批踢踢實業坊(ptt.cc), 來自: 123.192.166.3
※ 文章網址: https://webptt.com/m.aspx?n=bbs/Soft_Job/M.1462672912.A.E0B.html







like.gif 您可能會有興趣的文章
icon.png[問題/行為] 貓晚上進房間會不會有憋尿問題
icon.pngRe: [閒聊] 選了錯誤的女孩成為魔法少女 XDDDDDDDDDD
icon.png[正妹] 瑞典 一張
icon.png[心得] EMS高領長版毛衣.墨小樓MC1002
icon.png[分享] 丹龍隔熱紙GE55+33+22
icon.png[問題] 清洗洗衣機
icon.png[尋物] 窗台下的空間
icon.png[閒聊] 双極の女神1 木魔爵
icon.png[售車] 新竹 1997 march 1297cc 白色 四門
icon.png[討論] 能從照片感受到攝影者心情嗎
icon.png[狂賀] 賀賀賀賀 賀!島村卯月!總選舉NO.1
icon.png[難過] 羨慕白皮膚的女生
icon.png閱讀文章
icon.png[黑特]
icon.png[問題] SBK S1安裝於安全帽位置
icon.png[分享] 舊woo100絕版開箱!!
icon.pngRe: [無言] 關於小包衛生紙
icon.png[開箱] E5-2683V3 RX480Strix 快睿C1 簡單測試
icon.png[心得] 蒼の海賊龍 地獄 執行者16PT
icon.png[售車] 1999年Virage iO 1.8EXi
icon.png[心得] 挑戰33 LV10 獅子座pt solo
icon.png[閒聊] 手把手教你不被桶之新手主購教學
icon.png[分享] Civic Type R 量產版官方照無預警流出
icon.png[售車] Golf 4 2.0 銀色 自排
icon.png[出售] Graco提籃汽座(有底座)2000元誠可議
icon.png[問題] 請問補牙材質掉了還能再補嗎?(台中半年內
icon.png[問題] 44th 單曲 生寫竟然都給重複的啊啊!
icon.png[心得] 華南紅卡/icash 核卡
icon.png[問題] 拔牙矯正這樣正常嗎
icon.png[贈送] 老莫高業 初業 102年版
icon.png[情報] 三大行動支付 本季掀戰火
icon.png[寶寶] 博客來Amos水蠟筆5/1特價五折
icon.pngRe: [心得] 新鮮人一些面試分享
icon.png[心得] 蒼の海賊龍 地獄 麒麟25PT
icon.pngRe: [閒聊] (君の名は。雷慎入) 君名二創漫畫翻譯
icon.pngRe: [閒聊] OGN中場影片:失蹤人口局 (英文字幕)
icon.png[問題] 台灣大哥大4G訊號差
icon.png[出售] [全國]全新千尋侘草LED燈, 水草

請輸入看板名稱,例如:Gossiping站內搜尋

TOP