Problem with ActionListener firing multiple times

Question

I have a GUI class, GUIView, with a no-arg constructor and a static getInstance() method.
The GUI class has a JButton called btnB1 with a method:
     void addB1Listener(ActionListener al){
          btnB1.addActionListener(al);
     }The Controller class has an instance of the GUI and the Model.
The Controller has a constructor which assigns a new ActionListener to the JButton from the GUI.
private GUI m_gui;
     FTController(GUIView view){
          m_gui = view;
          m_gui.addButtonListener(new AddButtonListener());
     }The Controller has an inner class:
class AddButtonListener implements ActionListener{
          @Override
          public void actionPerformed(ActionEvent e) {
                      // do stuff
}The model has a constructor which accepts a GUIView.
My main method setups instances of the objects:
     private GUIView view;
     private Model model;
     private Controller controller;
     public static void main(String [] args){
          view = GUIView.getInstance();
          model = new Model(view);
          controller = new Controller(view);
     }This action listener for btnB1 is firing multiple times, but I don't understand why.
Edited by: user10199598 on Jan 9, 2012 2:56 PM

Answer 1

Here is the actual Controller class
import java.awt.event.ActionEvent;
import java.awt.event.ActionListener;
import java.io.IOException;
import javax.swing.JOptionPane;
import com.esri.arcgis.carto.FeatureLayer;
import com.esri.arcgis.carto.IActiveView;
import com.esri.arcgis.carto.Map;
import com.esri.arcgis.interop.AutomationException;
public class FTController{
     private FTView m_arraView;
     private FTModel m_arraModel;
     private Map m_map;
     FTController(FTView view, FTModel model, Map map){
          m_arraView = view;
          m_arraModel = model;
          m_map = map;
          m_arraView.addPointListener(new AddPointListener());
          m_arraView.addExitListener(new AddExitListener());
          m_arraView.addResetListener(new AddResetListener());
     public FTView getM_arraView() {
          return m_arraView;
     // Inner class used as the ActionListener for the btnAddPoint in FTView
     class AddPointListener implements ActionListener {
          @Override
          public void actionPerformed(ActionEvent e) {
               FeatureLayer fl = (FeatureLayer) m_arraModel.getLayerByName("arra_field_teams", m_map);
               try {
                    // Start the process to add a new feature point
                    m_arraModel.addPoint(fl, (IActiveView) m_map.getActiveView());
                    // Reset and dispose of the ARRA Field Teams form.
                    m_arraView.getCboTeam().setSelectedIndex(0);
                    m_arraView.getCboAgency().setSelectedIndex(0);
                    m_arraView.getTxtLatitude().setText("");
                    m_arraView.getTxtLongitude().setText("");
                    m_arraView.getTxtGridL().setText("");
                    m_arraView.getTxtGridN().setText("");
                    m_arraView.getTxtQuadrant().setText("");
                    m_arraView.getFtf3IC().setText("0.002");
                    m_arraView.getFtf3FC().setText("0.002");
                    m_arraView.getFtf3IO().setText("0.002");
                    m_arraView.getFtf3FO().setText("0.002");
                    m_arraView.getFtfAgxCartridge().setText("0.000");
                    m_arraView.getFtfBGCM().setText("50.000");
                    m_arraView.getFtfBGURHR().setText("0.002");
                    m_arraView.getTxtTimeOfReading().setText("");
                    m_arraView.dispose();
                    // Refresh the map window
                    m_map.getActiveView().refresh();
               } catch (AutomationException e1) {
                    e1.printStackTrace();
               } catch (IOException e1) {
                    e1.printStackTrace();
     } // end AddPointListener
     // Inner class used as the ActionListener for the btnExit in FTView
     class AddExitListener implements ActionListener{
          @Override
          public void actionPerformed(ActionEvent e) {
               m_arraView.getCboTeam().setSelectedIndex(0);
               m_arraView.getCboAgency().setSelectedIndex(0);
               m_arraView.getTxtLatitude().setText("");
               m_arraView.getTxtLongitude().setText("");
               m_arraView.getTxtGridL().setText("");
               m_arraView.getTxtGridN().setText("");
               m_arraView.getTxtQuadrant().setText("");
               m_arraView.getFtf3IC().setText("0.002");
               m_arraView.getFtf3FC().setText("0.002");
               m_arraView.getFtf3IO().setText("0.002");
               m_arraView.getFtf3FO().setText("0.002");
               m_arraView.getFtfAgxCartridge().setText("0.000");
               m_arraView.getFtfBGCM().setText("50.000");
               m_arraView.getFtfBGURHR().setText("0.002");
               m_arraView.getTxtTimeOfReading().setText("");
               m_arraView.dispose();
     } // end AddExitListener
     // Inner class used as the ActionListner for the btnReset in FTView
     class AddResetListener implements ActionListener{
          @Override
          public void actionPerformed(ActionEvent e) {
               FeatureLayer fl = (FeatureLayer) m_arraModel.getLayerByName("field_teams", m_map);
               try {
                    // Actually, "Reset" is deleting all features from the shapefile, poor choice of labels.
                    m_arraModel.resetFeatures(fl, (IActiveView) m_map.getActiveView(), (Map) m_map);
                    // Refresh the map window
                    m_map.getActiveView().refresh();
               } catch (AutomationException e1) {
                    e1.printStackTrace();
               } catch (IOException e1) {
                    e1.printStackTrace();
}Here is where the application starts:
import java.awt.event.MouseEvent;
import java.io.IOException;
import java.text.DecimalFormat;
import java.text.SimpleDateFormat;
import java.util.Date;
import javax.swing.UIManager;
import javax.swing.UnsupportedLookAndFeelException;
import com.esri.arcgis.addins.desktop.Tool;
import com.esri.arcgis.arcmap.Application;
import com.esri.arcgis.arcmapui.IMxDocument;
import com.esri.arcgis.arcmapui.MxDocument;
import com.esri.arcgis.carto.IActiveView;
import com.esri.arcgis.carto.IFeatureLayer;
import com.esri.arcgis.carto.Map;
import com.esri.arcgis.framework.IApplication;
import com.esri.arcgis.geodatabase.Feature;
import com.esri.arcgis.geodatabase.IFeatureClass;
import com.esri.arcgis.geometry.Point;
import com.esri.arcgis.interop.AutomationException;
public class FTTool extends Tool {
     private IApplication app;
     private IActiveView av;
     private IMxDocument mxDocument;
     private Map map;
     private FTView arraView;
     private FTModel model;
     private FTController controller;
     private int left;
     private int top;
      * Called when the tool is activated by clicking it.
      * @exception java.io.IOException if there are interop problems.
      * @exception com.esri.arcgis.interop.AutomationException if the component throws an ArcObjects exception.
     @Override
     public void activate() throws IOException, AutomationException {
     @Override
     public void init(IApplication app) throws IOException, AutomationException {
          this.app = app;
          try {
             try {
                    UIManager.setLookAndFeel(UIManager.getSystemLookAndFeelClassName());
               } catch (ClassNotFoundException e) {
                    // TODO Auto-generated catch block
                    e.printStackTrace();
               } catch (InstantiationException e) {
                    // TODO Auto-generated catch block
                    e.printStackTrace();
               } catch (IllegalAccessException e) {
                    // TODO Auto-generated catch block
                    e.printStackTrace();
               } catch (UnsupportedLookAndFeelException e) {
                    // TODO Auto-generated catch block
                    e.printStackTrace();
               // Initialize our map document, map and active view
               mxDocument = (IMxDocument)app.getDocument();
               av = mxDocument.getActiveView();
               map = (Map)mxDocument.getMaps().getItem(0);
               // Get an instance of the ARRAView JFrame, and setup the model and controller for this tool add-in
               arraView = FTView.getInstance();
//               arraView.addWindowListener(arraView);
               // Set up the model and controller objects
               model = new FTModel(arraView);
               controller = new FTController(arraView, model, map);
          } catch (AutomationException e) {
               e.printStackTrace();
          } catch (IOException e) {
               e.printStackTrace();
     @Override
     public void mousePressed(MouseEvent mouseEvent) {
          super.mousePressed(mouseEvent);
          // Cast IMxDocument into MxDocument so we can get the Parent which returns an Application
          MxDocument mxd = (MxDocument)mxDocument;
          Application application;
          // Create an Application object so we can get the left and top properties of the window
          //  so we can position the Field Teams GUI.
          try {
               application = new Application(mxd.getParent());
               left = application.getLeft() + 75;
               top = application.getTop() + 105;
          } catch (IOException e2) {
               e2.printStackTrace();
          try {
               // Call the model to convert the screen coordinates to map coordinates and project the point to WGS_1984
               Point p1 = new Point();
               p1.putCoords((double)mouseEvent.getX(), (double)mouseEvent.getY());
               Point p2 = (Point) model.getMapCoordinatesFromScreenCoordinates(p1, av);
               Point point = (Point)model.handleToolMouseEvent(mouseEvent, av);
             // Format the decimal degrees to six decimal places
               DecimalFormat df = new DecimalFormat("#.######");
               // Assign the point2 values to double
               double x = point.getX();
               double y = point.getY();
               // Set the text of the lat/long fields.
               arraView.getTxtLatitude().setText(df.format(y));
               arraView.getTxtLongitude().setText(df.format(x));
               // Set the Time of Reading text field
               Date now = new Date();
               SimpleDateFormat sdf = new SimpleDateFormat("HH:mm:ss");
               arraView.getTxtTimeOfReading().setText(sdf.format(now).toString());
               // Determine whether the mouse click point intersects the arra_grid_quad.
               //  If so, iterate over that feature cursor to update the gridl, gridn and quadrant fields...
               IFeatureLayer featLayer = (IFeatureLayer) model.getLayerByName("arra_grid_quads", map);     
               IFeatureClass featClass = featLayer.getFeatureClass();
               Feature iFeat = (Feature) model.queryArraGridQuads(featClass, p2);
               if(iFeat != null){
                    // Fill in the grid and quadrant values if there are any.
                    String gridl = (String) iFeat.getValue(iFeat.getFields().findField("GRID_L"));
                    String gridn = (String) iFeat.getValue(iFeat.getFields().findField("GRID_N"));
                    String quadrant = (String) iFeat.getValue(iFeat.getFields().findField("QUADRANT"));
                    arraView.getTxtGridL().setText(gridl);
                    arraView.getTxtGridN().setText(gridn);
                    arraView.getTxtQuadrant().setText(quadrant);
               } else {
                    // Revert values back to empty string after clicking in the grid, then outside of the grid.
                    arraView.getTxtGridL().setText("");
                    arraView.getTxtGridN().setText("");
                    arraView.getTxtQuadrant().setText("");
               // Set the Field Team Readings form to visible, or redisplay it if it is already visible
               arraView.setBounds(left, top, 325, 675);
               arraView.setVisible(true);
          } catch (Exception e) {
               e.printStackTrace();
     @Override
     public boolean deactivate() throws IOException, AutomationException {
          return super.deactivate();
     @Override
     public boolean isChecked() throws IOException, AutomationException {
          return super.isChecked();
     public FTView getArraView() {
          return arraView;
     public IApplication getApp() {
          return app;
     public Map getMap() {
          return map;
     public FTModel getModel() {
          return model;
}Is this enough?
Edited by: user10199598 on Jan 9, 2012 3:20 PM