代码被重构了，实话说，写的确实很烂。
问题1：service层调用了部分control层的东西。
原因：违反了依赖倒置的原则。
依赖倒置的原则：
1)上层依赖于下层。事实上，高层模块不应该依赖低层模块，两者都应该依赖其抽象；
2)抽象不应该依赖细节;
3)细节应该依赖抽象。
问题2：某一个类的职责分工不单一，导致结构混乱,逻辑不清楚。
原因：违反了单一职责原则。
单一职责原则：
不要存在多于一个导致类变更的原因。通俗的说，即一个类只负责一项职责。
这就是类级别的单一原则。方法级别的单一职责与此类似。
关于设计模式的六大原则，可以参考GOF（四人组）。
据说GOF是Java基础知识和J2EE框架知识之间一座隐性的"桥"。看来我也得读一读了。