分离后（不考虑接口），这种分离是否正确？

Question

假设类的结构如下（MVC控制器）：class Controller {      public function action1() {            if(condition1)             {                // ……             }            // elseif(condition2){……}            // elseif(conditionN){……}            else             {                // ……             } }      public function action2() {            if(condition1)             {                // ……             }            // elseif(condition2){……}            // elseif(conditionN){……}            else             {                // ……             } } }分离后（不考虑接口）：class Controller_Action1 extends Controller {      public function condition1() {             …… }      public function condition2() {             …… }      public function conditionN() {             …… } }class Controller_Action2 extends Controller {      public function condition1() {             …… }      public function condition2() {             ……             }      public function conditionN() {             …… }

Answer 1

从分工合作的角度来说，这么分开挺好的。

在symfony框架中，多个actions可以写在一个actions.class.php文件里面，也可以每一个action只写在一个XxxAction.class.php里，根据实际情况发现还是分开写好，这样很容易把工作分给更多的人完成

当然你这里还没有分离到每个action一个文件的程度，但是相对比最开始所有actions一个文件来说，还是要合理不少。如果一个团队的分工就是按照module的概念（module是某几个actions的集合）去分的，那么你这么分其实也足够了

Answer 2

你这里用的是继承，继承的目的是提取共性， 所以你应该展示你的controller类，才好判断这个设计的合理性。