JOptionPane doesn't show

[Chris Mattmiller]

The program gets to the JOptionPane part, reads it, then returns to the calling method without reading the rest of the recTrip method. If I comment the JOptionPane out the program runs through recTrip with no problems.
I put the JOptionPane in a try/catch hoping to catch an error but to no avail.

Here is the code:

    
public void recTrip(Driver dri, Object oldVal){
    log.debug("In recTrip");
    try{
int result = JOptionPane.showConfirmDialog(this, "This trip has been reconciled.  Do you want to change it?");
    }catch(HeadlessException he){
    log.debug("Headless " + he);
    }
    
    log.debug("Past JOptionPane");
    
    try {
  ejbtm.setValue(ejbtm.getCurrentRowValue("main_dri_uid"), row, ejbtm.getRealColumnName(jt.getSelectedColumn()));
 updateWeekWithNewResource("Driver", "DRI", dri, (EjbTableModel)triptm.getValue(0, "DRI"), (ArrayList)triptm.getValue(0, "DRI_AVL_AL"), "driver_nbr", oldVal);
} catch (Exception e) {
// TODO Auto-generated catch block
e.printStackTrace();
}
    

[Campbell Ritchie]

Welcome to JavaRanch

What sort of class are you inside? I presume the "this" passed to the showConfirmDialog() method is an appropriate type.

What actually happens? Your thread title says "JOptionPane doesn't show," but I can't imagine that dialogue not showing unless there is a HeadlessException.
What are you doing with the result from the dialogue? It returns either JOptionPane.YES_OPTION or JOptionPane.NO_OPTION and you are supposed to test for those.

if (JOptionPane.showConfirmDialog(this, "This trip has been reconciled etc."
         == JOptionPane.YES_OPTION)
{
   doSomething();
}
else
{
   dontDoIt();
}//  etc etc

I think you can also have CANCEL_OPTION as a return type: look here.

Can't see how that method would return without recording the ejbtm values. Try some System.out.println() statements in that method to show the old and new values.

[G Estes]

On the confirm dialog, you get (by default) Yes, No and Cancel buttons. You will need to query the returned value as follows and then take the appropriate action:

  
      JButton button = createButton("Flash Colors");
      button.addActionListener(new ActionListener()
      {
         public void actionPerformed(ActionEvent e)
         {
            //doColorPanel();
            String msg = "Are you sure you want to do this?";
 
            int result = JOptionPane.showConfirmDialog(null, msg);
            
            // the YES button was selected...
            if(result == JOptionPane.YES_OPTION)
            {
               System.out.println("YES was selected");
            }
            else
            {
               // the NO or Cancel options were selected...
               System.out.println("Something else was selected...");
               // so do not proceed
               return;
            }
 
            doTest();
         }
      });

[ August 21, 2008: Message edited by: gil_e ]

[Chris Mattmiller]

I modified the code a bit. Now instead of just leaving this method, it goes back to the "log.debug("setCurrentDriver begin again");" at the top. Comment out the JOptionPane and it will run the program as expected.

I copied the JOptionPane from another method and only changed the message.

    
protected void setCurrentDriver(Driver dri, Object oldVal) throws Exception {
    log.debug("setCurrentDriver begin again");
    
    if(triptm.getCurrentRowValue("trp_status_code").toString().equals("RR")){
log.info("this trip has been reconciled ");
int result = JOptionPane.showConfirmDialog(this, "This trip has been reconciled.  Do you want to change it?", "Reconciled Trip", JOptionPane.YES_NO_OPTION);

if (result == JOptionPane.YES_OPTION){
log.info("Set Current Driver " + result);
    log.debug("Driver: " + ejbtm.getCurrentRowValue("main_dri_uid") + " Selected row: " + row + " Select column: " + (jt.getSelectedColumn() + 1) + " Driver: " + jt.getValueAt(jt.getSelectedRow(), (jt.getSelectedColumn() + 1)));
    setDriver(jt.getSelectedRow(), ejbtm.getRealColumnName(jt.getSelectedColumn()), dri);
    
        updateWeekWithNewResource("Driver", "DRI", dri, (EjbTableModel)triptm.getValue(0, "DRI"),
                (ArrayList)triptm.getValue(0, "DRI_AVL_AL"), "driver_nbr", oldVal);
} else {
log.info("Set Current Driver " + result);
    ejbtm.setValue(ejbtm.getCurrentRowValue("main_dri_uid"), row, ejbtm.getRealColumnName(jt.getSelectedColumn()));
        
        updateWeekWithNewResource("Driver", "DRI", dri, (EjbTableModel)triptm.getValue(0, "DRI"), (ArrayList)triptm.getValue(0, "DRI_AVL_AL"), "driver_nbr", oldVal);
}
log.info("this trip has been reconciled ");
} else {
log.info("Not Reconciled");
setDriver(jt.getSelectedRow(), ejbtm.getRealColumnName(jt.getSelectedColumn()), dri);

    updateWeekWithNewResource("Driver", "DRI", dri, (EjbTableModel)triptm.getValue(0, "DRI"), (ArrayList)triptm.getValue(0, "DRI_AVL_AL"), "driver_nbr", oldVal);
}   

[Campbell Ritchie]

Wherever you copied it from, they might have a mistake there too
I presume you actually see the option pane dialogue window?
Suggest you insert a System.out.println() statement after "if (result == JOptionPane.YES_OPTION) {"
and before "log.info(. . ." to tell you where you have gone (or even
"JOptionPane.showMessageDialog(this, "In Yes block", "OK");").
Similar print or showMessageDialog statements will let you follow the execution through the whole method.

Or you can use the debugging function of an IDE.
[ August 21, 2008: Message edited by: Campbell Ritchie ]