交接同事的程式,物件錯綜複雜彼此糾纏,看code非常痛苦。
其中最讓我頭痛的是 他先把一個base 屬性給初始值寫在 當第一次讀取get 屬性時,判斷對應private 變數是不是null,
當判斷是null當下 呼叫function撈資料,
很多屬性都是depend on 那組base屬性,先判斷 自己的私變數是不是空 再去讀取前面base 那組property,,再去做篩選 塞回的私變數,
程式一堆這種一連串相生的屬性,看code快瘋掉了,
不知道大家覺得 這樣的寫法正常嗎?
要怎麼翻寫比較好解讀跟維護
請諸位大神賜教
長線成長還是可以多多想一下 design pattern :
https://skyyen999.gitbooks.io/-study-design-pattern-in-java/content/oodPrinciple.html
如果每次null後都是call同一個funtion,感覺還算單純就還好..久了也會成記憶
但如果裡面更錯綜複雜,那放在base就不優了,每次都找會到瘋掉XD
我自己也遇到很鳥的,一串SQL語法要兜15個funtion才完成...,看完都吐了
20年年資的上司:我不想看,我直接重寫
這種 if
通常是商業邏輯改變了,維護者
無法依照原開發者
的思路維護下去。
又或者是,維護者
不願承擔改變商業邏輯的風險或side effect
。
如果你要重構,還是讓你主管知道。畢竟,風險是有人要承擔的?