有次接到个PM要改版一个功能的UI设计，前端童鞋还没敲定页面的时候，我先看了看这个功能的后台，我擦...简直是惨不忍睹。。对PM来说是改版UI，对我这么有点代码洁癖的来说就是优化代码。
首先我能肯定的是，原来这个童鞋对面向对象理解的不太好。。其次对代码的整合不够...我就简单log下。。
先说面向对象理解的问题，在一个入口方法里，去调用该类自己的私有方法，这里传参的时候竟然把类属性的值都传过去了。。当时我就汗了，，显然在被调用的方法里，直接就可以获取这些类属性的值了。。为什么还要去传呢...
再有我说整合能力，这个入口方法是这样做的，取前端传过来的key，通过这个key值的不同，去调用类里的各个不同的方法，而且原来他自己都已经把各个方法名取的和key的值一样了。然而惨不忍睹的是，如果说几个switch的case，这么写不会让人觉得啥。。重点是这个key的值将近有20个...这再这么写下去代码就太糟糕了。。显而易见的是这几十行代码可以看出来是可以用相同代码的。正好php有个函数叫call_user_func()，可以派上用场。比如这样call_user_func(array($this, $_GET[‘key‘]));让实例去调用它的key名字的方法。这里有个小问题，比如原来代码里面有的几个不同key，但走了相同的方法。
这时，可以这样构造一个魔术方法__call()，然后在这个魔术方法里，把这几个特殊key构建成个数组，如果$_GET[‘key‘]在这里头的，就共同去调用这个方法。
仔细看，类里面还有很多值得优化的地方。其实一个类被设计的不该过于臃肿，一个类只需要负责好它自己的事就好了。这里的优化，我就抽取出来几个方法新建了一个类。
顺便我觉得改别人的代码很有意思，发现不合自己意就开始喷哈哈~