※ 引述《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/cn.aspx?n=bbs/Soft_Job/M.1462672912.A.E0B.html