7

我当前的应用程序有一个 JFrame,其中大约 15 个操作存储为 JFrame 中的字段。每个动作都是一个匿名类,其中一些很长。

将动作分解成自己的类是否常见,可能在称为动作的子包中?

如果不是,通常如何驯服这种复杂性?

谢谢

4

3 回答 3

8

如果您的操作可能是可重用的(例如,通过键盘快捷键、其他菜单、其他对话框等),特别是如果它们可以直接在底层模型(而不是 UI)上工作,那么通常会更好不要将它们作为匿名类。

相反,创建一个单独的包,并为每个包创建类。

通常,不直接实例化这些也是有意义的,而是使用某种管理器来定义常量并初始化和返回动作集,以便您可以例如在不同版本中提供不同的动作集或仅设置某些动作用于内部发布。

最后,检查您的操作是否可以重构为类层次结构。它们通常可以,这样可以节省代码复制,还可以帮助您增加健壮性(例如,在让操作执行之前检查某些条件)。

于 2009-01-15T19:59:51.067 回答
4

这就是我通常的做法。每个动作都有自己的类,该类具有对“app”对象的引用,因此它可以获取所需的资源。我通常有一个操作管理器来保存所有操作,因此有一个地方可以访问它们,也有一个地方可以更新它们的启用和内容。

最终这也变得难以管理,此时您应该开始考虑使用应用程序框架,如 Eclipse RCP、NetBeans 框架、JIDE 等。如果您想支持用户定义的键盘映射和类似的东西,尤其如此。

于 2009-01-15T19:49:43.990 回答
2

我所做的是为动作类创建一个包(实际上是包树),然后根据上下文实例化每个类。几乎我所有的动作类都是抽象的,带有抽象方法来获取上下文(ala Spring)。

public abstract class CalcAndShowAction extends AbstractAction {
    //initialization code - setup icons, label, key shortcuts but not context.

    public void actionPerformed(ActionEvent e) {
        //abstract method since it needs ui context
        String data = getDataToCalc();

        //the actual action - implemented in this class, 
        //  along with any user interaction inherent to this action
        String result = calc(data);  

        //abstract method since it needs ui context
        putResultInUI(result);
    }
    //abstract methods, static helpers, etc...
}

//actual usage
//...
button.setAction(new CalcAndShowAction() {
    String getDataToCalc() {
        return textField.getText();
    }

    void putResultInUI(String result) {
        textField.setText(result);
    }
});
//...

(抱歉有任何错误,我是在此文本框中手动编写的,而不是在 IDE 中)。

于 2009-01-15T20:34:27.923 回答