实践中的重构06_方法调用顺序和性能

有空都去读一下
《重构》，
第9章 简化条件表达式 （Simplifying Conditional Expressions）

首先我想说的是你这个不是重构， 起码不是代码重构， 而是逻辑的重构。


重构的前提是在不改变代码逻辑的前提下，改给代码换一种结构，一增加可读性、可扩展性。。。。。。


你当前的重构明显改变了逻辑，就是第一个方法调用结果如果为true， 就不调用第二个了，或许你后面的isGardUser方法的调用本身也是必须的， 比如会在远程的服务器上生成验证记录什么的。那你这样去掉反而不对。

你现在的重构实际上就是逻辑上重新整理。

感觉问题除性能外，还在于：
* WS 或 Heavy方法，在API或项目内没有重视（管理问题）
* 原代码质量不好，接手较痛苦（如，result 魔术） （培训问题）

认可

什么是result魔术？

建议弄个简单的权限框架来搞定这个事情，这样维护也很easy。

代码的自我注释的确是很好，这也是大家推荐的做法。
   但是它也有不足，它不可能100%的说明自身，最多60，70%，在很多的模糊场景下，有的隐含条件我觉得必须写明注释，要不一两个月后，也许你自己忘了，更别说别人维护。

有个东西叫做自动测试

自动测试又不测性能的
这种地方既然为了优化，还是写个注释吧。如果不写，这段代码我是看不出楼主刻意强调的用意的

写注释不是问题，
问题是注释要写成什么样。
怎么保证注释和代码能同步更新。