面向对象编程 - 有关设计和访问相关数据的问题

Question

假设您在 Timer< 中有以下代码/代码>： 它的主要目标是倒计时并在基于 Swing 的 GUI 上显示它。 计时器是游戏的一部分，用于通过首先找到解决方案的用户来决定谁是获胜者。

Action updateClockAction = new AbstractAction() {

    public void actionPerformed(ActionEvent e) {

        JLabel secLabel = m_GameApplet.GetJpanelStartNetGame().GetJlabelSeconds();
        secLabel.setFont(new java.awt.Font("Lucida Handwriting", 1, 36));
        secLabel.setForeground(Color.red);
        secLabel.setText(Integer.toString(m_TimerSeconds));
        if (m_TimerSeconds > 0) {
            m_TimerSeconds--;
        } else if (m_TimerSeconds == 0) {
            m_Timer.stop();
            m_GameApplet.GetJpanelStartNetGame().GetJlabelSeconds().setText("0");
            m_GameApplet.GetJpanelStartNetGame().GetJbuttonFinish().setVisible(false);
            //Checking whether time ended for both players and no solution was recieved
            if (!m_WasGameDecisived) {
                System.out.println("Tie - No one had a solution in the given time");
            }
        }
    }
};
m_Timer  = new Timer(1000, updateClockAction);

现在，我在客户端实现中有两个主要包 A.GUI - 持有所有摇摆 Jpanels 等。 B.LogicEngine

现在，在 LogicEngine 中，我有一个名为 GameManager 的类，它应该管理和存储有关游戏的信息。

我当前的实现一般是这样的： 我有一个位于 GUI 包上的 JpanelMainGame JpanelMainGame 包含 JPanelGameBoard，它引用（或换句话说包含）GameManger 的实例，该实例位于不同的包 - LogicEngine 上 包。

所以我的问题是：

1. 上面的所有计时器定义代码应该放在哪里？

   A.在 JpanelMainGame 中？ （JpanelMainGame 应该有对它的引用）。

   B.在GameManger中作为游戏数据的一部分

2. 在您给出的每个解决方案中，我应该如何从匿名内部类中访问所有标签信息？因为我只能使用命令访问外部类的成员： OuterClass.this.member。

3. 有关当前设计的任何评论。

非常感谢

Answer 1

我重复我在另一个问题中给出的答案：

我强烈反对从实现的角度组织包，例如控制器、数据等。我更喜欢按功能对它们进行分组，即功能1、功能2等。如果某个功能相当复杂并且需要大量功能类，然后（并且只有那时）我创建像上面一样的子包，即 feature1.controllers、feature1.data 等。

Answer 2

好吧，不是用问题来回答问题，但是……

• Timer 的使用频率是多少？
• 定时器编码是通用的还是特定于某个类的？

如果计时器是通用的，那么我会说它属于 LogicEngine 包树中的某个位置。但是，如果 Timer 只能在 GUI 元素中使用，那么它属于 GUI 包树。

作为一般经验法则（不要太用力地敲击敏捷鼓），您应该只编写现在有意义的代码，但不要害怕以后更改它。

示例：您正在代码中创建 AbstractAction() 类型的匿名内部类。如果简单地使用 updateClockAction 变量，这就很好（尽管我回避匿名类）。但是，一旦您的编码导致您需要跨越 updateClockAction 的边界（通过 OuterClass.this.member），那么我断言需要进行一些重构：提取这个内部类并使其成为一个完全限定的类。这样，您可以对 updateClockAction 对象的内部状态进行适当的控制（使用 cstr、getter 等）

编辑：添加了请求的示例

public class Testing {
    private Timer timer; /* INIT this from somewhere.... */

    public void myFunction() {
    /* 60 Seconds */
    long countDownTimeSEC = 60; 
    /* convert to Miliseconds */
    long countDownTimeMS = 1000 * countDownTimeSEC; 

    /* Get ref to label */
    JLabel label = m_GameApplet.GetJpanelStartNetGame().GetJlabelSeconds(); 

    /* Set once */
    label.setFont(new java.awt.Font("Lucida Handwriting", 1, 36)); 
    label.setForeground(Color.red);

    /* Set initial time */
    label.setText(Long.toString(countDownTimeSEC)); 

    /* Get ref to button */
    JButton button = m_GameApplet.GetJpanelStartNetGame().GetJbuttonFinish(); 

    /* Set up post Count Down list */
    ArrayList<AbstractAction> actsWhenComplete = new ArrayList<AbstractAction>();

    /* instantiate countdown object */
    CountDownClockAction cdca = new CountDownClockAction(label, countDownTimeMS, actsWhenComplete);
    this.timer  = new Timer(1000, cdca);

    /* Now that we have a timer, add the post Count Down action(s) to the  post Count Down list */
    actsWhenComplete.add(new CountDownFinishAction(label, button, this.timer));


    /* Finally, kick off the timer */
    this.timer.start();
}

public static class CountDownClockAction extends AbstractAction {
    private static final long serialVersionUID = 1L;
    private final JLabel labelToUpdate;
    private final long startMS;
    private long currentMS;
    private long timeMark;
    private final ArrayList<AbstractAction> actionsToExeWhenComplete;
    private boolean actionsExedFlag;

    public CountDownClockAction(
            final JLabel labelToUpdate, 
            final long startMS, 
            ArrayList<AbstractAction> actionsToExeWhenComplete
    ) {
        super();
        this.labelToUpdate = labelToUpdate;
        this.startMS = startMS;
        this.currentMS = startMS;
        this.timeMark = 0;
        this.actionsExedFlag = false;
        this.actionsToExeWhenComplete = actionsToExeWhenComplete;
    }

    @Override
    public void actionPerformed(final ActionEvent e) {
        /* First time firing */
        if (this.timeMark == 0) 
            this.timeMark = System.currentTimeMillis();

        /* Although the Timer object was set to 1000ms intervals, 
         * the UpdateClockAction doesn't know this, nor should it
         * since > or < 1000ms intervals could happen to do the fact that
         * the JVM nor the OS have perfectly accurate timing, nor do they
         * have instantaneous code execution properties.
         * So, we should see what the *real* time diff is...
         */
        long timeDelta = System.currentTimeMillis() - this.timeMark;

        /* Allow for the label to be null */
        if (this.labelToUpdate != null)
            labelToUpdate.setText(Long.toString((long)(currentMS / 1000)));

        if (currentMS > 0) {
            currentMS -= timeDelta;

        } else if (currentMS <= 0 && this.actionsExedFlag == false) {
            /* Ensure actions only fired once */
            this.actionsExedFlag = true;
            /* Allow for the label to be null */            
            if (this.actionsToExeWhenComplete != null)
                for (AbstractAction aa: this.actionsToExeWhenComplete)
                aa.actionPerformed(e);
        }

        /* Finally, update timeMark for next calls */
        this.timeMark = System.currentTimeMillis();
    }
}

public static class CountDownFinishAction extends AbstractAction {
    private final JLabel labelToUpdate;
    private final JButton buttonToUpdate;
    private final Timer timerToStop;

    public CountDownFinishAction(
            JLabel labelToUpdate,
            JButton buttonToUpdate, 
            Timer timerToStop
    ) {
        super();
        this.labelToUpdate = labelToUpdate;
        this.buttonToUpdate = buttonToUpdate;
        this.timerToStop = timerToStop;
    }

    @Override
    public void actionPerformed(final ActionEvent e) {
        /* Perform actions, allowing for items to be null */
        if (this.labelToUpdate != null)
            this.labelToUpdate.setText("0");

        if (this.buttonToUpdate != null)
                this.buttonToUpdate.setVisible(false);

            if (this.timerToStop != null)
                this.timerToStop.stop();
        }
    }
}