2

A previous version of my program was designed to display multiple series of magic trick images all with the same type of ActionListener. For example, when displaying all the tricks with a set time between each image, I did this by using a class called TimeListener that extends ActionListener. I make a new instance of that class and then use a Timer to tell it to fire an event every duration, where duration is a variable number of milliseconds.

TimeListener timeListener = new TimeListener();
Timer displayTimer = new javax.swing.Timer(duration, timeListener);
displayTimer.start();

The series were displayed in an order that was input by the user, and a control structure within the TimeListener determined which image to display next.

I now need to set it up so that different tricks have different ActionListeners. I'd like to set them up as follows:

ActionListener[] actionListeners = new ActionListener[trickSettings.length];
for(int i = 0; i < trickSettings.length; i++) {
    String pace = trickSettings[i].getPace();
    String trick = trickSettings[i].getTrick();
    if(pace.equals("Set Pace") || pace.equals("Set Pace Free Pause") || pace.equals("Set Pace Timed Pause") || pace.equals("Set Pace Subgoal Pause")) {
        actionListeners[i] = new TimeListener(trick, pace);
    }
    else if(pace.equals("Yoked Pace")) {
        actionListeners[i] = new YokedListener(trick, pace);
    }
    else if(pace.equals("Self Pace")) {
        actionListeners[i] = new SelfListener(trick, pace);
    }
}

However, the previous approach just ran everything from the class' constructor. If I take this new approach though I need to be able to switch which ActionListener is active. I'm pretty sure this can't be switched from the constructor. Do I need to make an ActionListener to tell when these ActionListeners are done so that it can switch to the next one? Is there a simpler way to do this?

Edit My Solution

private class MasterListener {
    public void newListener() {
        for(int i = 0; i < trickSettings.length; i++) {
            if(trickSettings[i].getIsDone() == false) {
                start(trickSettings[i], 0, true);
                inBegin = true;
                mouse = new Mouse(trickSettings[i]);
                addMouseListener(mouse);
                timeListener = new TimeListener(trickSettings[i]);
                displayTimer = new Timer(trickSettings[i].getDuration(), timeListener);
                displayTimer.start();
                break;
            } 
        }
    }
}

And I just kicked it off by creating an instance of MasterListener in the constructor and calling newListener() on it. Everytime I reach the end of a trick I just call newListener() on my MasterListener object.

Note: the start method executes a new SwingWorker for preloading images in the background.

4

1 に答える 1

2

データとコードを混在させることは避けます。つまり、データを「ハードコーディング」することは避けます。これは、あなたがやろうとしていることです。代わりに、私は...

  • くびきかどうかに関係なく、単一のトリック、そのペースを保持する新しいクラスを作成します。
  • プログラムのモデルには、このタイプのオブジェクトのコレクションが含まれている可能性があります。
  • このクラスの現在のオブジェクトの状態を使用して、1 つの ActionListener がどのように動作するかを決定します。ActionListener は、オブジェクトの状態に応じてさまざまなメソッドを呼び出すことができることに注意してください。

あなたのコメント/私の返信を編集する

実際には、TrickSettings と呼ばれるそのすべての情報を含むクラスが既にあります。このクラスに、表示された画像の数/表示されたかどうかに関する情報も含める必要がありますか? 上記のコードで、trictSettings は TrickSettings の配列です。

はい。必要なのはsize()、そのトリックの画像コレクションの を返すことだけです。

あなたの 3 番目の点に関する私の懸念は、すべてのトリックに対して同じタイミングで ActionListener を起動させたくないということです。

そのため、反復している現在の TrickSettings オブジェクトから取得したタイミングをタイマーに使用させます。大きな問題ではない。

たとえば、ファイルからタイミングを読み取る「Yoked Pace」などのペースのトリックや、ユーザーがクリックして進めるペースの「Self Pace」のトリックはさらに異なります。以前は、代わりに MouseAdapter クラスを使用していました...

これは、プログラムのロジックによって簡単に対応できます。

編集
作業がうまくいったことをうれしく思います。あなたには解決策があるので、私のアイデアをテストするために昨日作成したものを投稿します。多分それはあなたにいくつかのアイデアを与えることができます：

import java.awt.BorderLayout;
import java.awt.Color;
import java.awt.Dimension;
import java.awt.Graphics;
import java.awt.GridLayout;
import java.awt.event.ActionEvent;
import java.awt.event.ActionListener;
import java.awt.event.MouseAdapter;
import java.awt.event.MouseEvent;
import java.awt.image.BufferedImage;
import java.util.ArrayList;
import java.util.Iterator;
import java.util.List;
import java.util.Random;

import javax.swing.*;

public class TimerSpeedEg extends JPanel {
   private static final int BI_W = 150;
   private static final int BI_H = BI_W;
   private static final int SHOW_TRICKS_COUNT = 3;
   private List<Icon> icons;
   private Random random = new Random();
   private List<ShowTricks> showTricksList = new ArrayList<>();

   public TimerSpeedEg() {
      icons = createIcons();


      setLayout(new GridLayout(1, 0));
      add(createShowTricksPanel(BI_W, BI_H, "Fast", 100, false));
      add(createShowTricksPanel(BI_W, BI_H, "Slow", 1000, false));
      add(createShowTricksPanel(BI_W, BI_H, "Self-Paced", 0, true));

   }

   private JPanel createShowTricksPanel(int w, int h, String title,
         int speed, boolean selfPaced) {
      final ShowTricks showTricks = new ShowTricks(w, h);
      TrickSettings trickSettings = new TrickSettings(speed, selfPaced);
      for (Icon icon : icons) {
         trickSettings.addIcon(icon);
      }
      showTricks.setTrickSettings(trickSettings );
      JButton showButton = new JButton(new AbstractAction("Show") {

         @Override
         public void actionPerformed(ActionEvent arg0) {
            showTricks.show();
         }
      });
      JPanel btnPanel = new JPanel();
      btnPanel.add(showButton);
      JPanel panel = new JPanel(new BorderLayout());
      panel.setBorder(BorderFactory.createTitledBorder(title));
      panel.add(showTricks.getTrickLabel(), BorderLayout.CENTER);
      panel.add(btnPanel, BorderLayout.SOUTH);
      return panel;
   }

   private List<Icon> createIcons() {
      List<Icon> iconList = new ArrayList<Icon>();
      String[] strings = { "One", "Two", "Three", "Four", "Five", "Six",
            "Seven", "Eight", "Nine", "Ten" };
      for (int i = 0; i < strings.length; i++) {
         JLabel label = new JLabel(strings[i], SwingConstants.CENTER);
         label.setSize(BI_W, BI_H);
         label.setOpaque(true);
         Color color = new Color(random.nextInt(127) + 127,
               random.nextInt(127) + 127, random.nextInt(127) + 127);
         Color borderColor = new Color(color.getRed() - 100,
               color.getGreen() - 100, color.getBlue() - 100);
         label.setBackground(color);
         label.setBorder(BorderFactory.createLineBorder(borderColor, 20));
         BufferedImage bImg = new BufferedImage(BI_W, BI_H,
               BufferedImage.TYPE_INT_ARGB);
         Graphics g = bImg.getGraphics();
         label.paint(g);
         g.dispose();
         iconList.add(new ImageIcon(bImg));
      }
      return iconList;
   }

   private static void createAndShowGui() {
      TimerSpeedEg mainPanel = new TimerSpeedEg();

      JFrame frame = new JFrame("TimerSpeedEg");
      frame.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
      frame.getContentPane().add(mainPanel);
      frame.pack();
      frame.setLocationByPlatform(true);
      frame.setVisible(true);
   }

   public static void main(String[] args) {
      SwingUtilities.invokeLater(new Runnable() {
         public void run() {
            createAndShowGui();
         }
      });
   }
}

class ShowTricks {
   int prefW;
   int prefH;
   private JLabel trickLabel = new JLabel() {
      @Override
      public Dimension getPreferredSize() {
         return new Dimension(prefW, prefH);
      }
   };

   public ShowTricks(int prefW, int prefH) {
      this.prefW = prefW;
      this.prefH = prefH;
   }

   private TrickSettings trickSettings;

   public JLabel getTrickLabel() {
      return trickLabel;
   }

   public void setTrickSettings(TrickSettings trickSettings) {
      this.trickSettings = trickSettings;
   }

   public void show() {
      if (trickSettings == null) {
         return;
      }
      if (trickSettings.isSelfPaced()) {
         showSelfPaced();
      } else {
         showWithTimer();
      }
   }

   private void showWithTimer() {
      Timer timer = new Timer(trickSettings.getMiliseconds(), new TimerListener());
      timer.setInitialDelay(0);
      timer.start();
   }

   private void showSelfPaced() {
      trickLabel.setIcon(trickSettings.getIcon(0));
      trickLabel.addMouseListener(new MouseAdapter() {
         private int count = 1;

         @Override
         public void mousePressed(MouseEvent e) {
            if (count < trickSettings.getSize()) {
               trickLabel.setIcon(trickSettings.getIcon(count));
               count++;
            } else {
               trickLabel.removeMouseListener(this);
            }
         }
      });
   }

   private class TimerListener implements ActionListener {
      private int count = 0;

      @Override
      public void actionPerformed(ActionEvent evt) {
         if (count < trickSettings.getSize()) {
            trickLabel.setIcon(trickSettings.getIcon(count));
            count++;
         } else {
            ((Timer) evt.getSource()).stop();
         }
      }
   }
}

class TrickSettings implements Iterable<Icon> {
   private List<Icon> icons = new ArrayList<Icon>();
   private int miliseconds;
   private boolean selfPaced;

   public TrickSettings(int miliseconds, boolean selfPaced) {
      this.miliseconds = miliseconds;
      this.selfPaced = selfPaced;
   }

   public int getSize() {
      return icons.size();
   }

   public int getMiliseconds() {
      return miliseconds;
   }

   public boolean isSelfPaced() {
      return selfPaced;
   }

   @Override
   public Iterator<Icon> iterator() {
      return icons.iterator();
   }

   public Icon getIcon(int index) {
      return icons.get(index);
   }

   public void addIcon(Icon icon) {
      icons.add(icon);
   }
}
于 2013-09-26T21:19:12.570 に答える