First Japplet and having issue refreshing output

[Joseph Cho]

Excuse the ignorance if there is a better way to do things, I'm a student and we are supposed to reuse an old project and convert it to an applet so I'm trying to do the best I can.

My issues is when I submit data, the applet starts looking jarbled, and looks like it starts double printing things. Can anyone spot the error? Here is the link to the actual applet to see for yourself

http://students.cs.niu.edu/~z151674/VerOne.html

and here is the code:

/****************************************************************
  ****************************************************************/
import javax.swing.*; // swing components
import java.awt.*;
import java.awt.event.*; //events


import java.io.*;
import java.util.Scanner;
import java.util.ArrayList;
import java.util.InputMismatchException;
import java.text.DecimalFormat;

class Trip
{
  private double  airCost       = 0,   //total air fair per person
                  lodgingCst    = 0,   //lodging cost per day
                  mealPerDay    = 0,   //cost of meals per day
                  miscCost      = 0;   //any miscellaneous cost per day
  
  private int     numOfDays     = 0,   //number of days user will be on trip
                  numOfTrvlrs   = 0;   //number of travelers on trip

  private String  destination = "default";  //area of interest for trip
  
  /****************************************************************
  getDestination

  Access to private instance variable
****************************************************************/ 
  public String getDestination()
  {
    return (destination);
  }
  /****************************************************************
  setDestination

  Access to set private instance variable
****************************************************************/ 
  public void setDestination(String s)
  {
    destination = s;
  }
  /****************************************************************
  getAirCost
  
  Access to private instance variable
****************************************************************/
  public double getAirCost()
  {
    return (airCost);
  }
  /****************************************************************
  setAirCost

  Access to set private instance variable
****************************************************************/ 
  public void setAirCost(double setV)
  {
    airCost = setV;
  }

/****************************************************************
  getMealPerDay

  Access to private instance variable
****************************************************************/ 
  public double getMealPerDay()
  {
    return (mealPerDay);
  }
/****************************************************************
  setMealPerDay

  Access to set private instance variable
****************************************************************/ 
  public void setMealPerDay(double setV)
  {
    mealPerDay = setV;
  }
  /****************************************************************
  getLodgingCst
 
  Access to private instance variable
****************************************************************/ 
  public double getLodgingCst()
  {
    return (lodgingCst);
  }
  
  /****************************************************************
  setLodgingCst

  Access to set private instance variable
****************************************************************/ 
  public void setLodgingCst(double setV)
  {
    lodgingCst = setV;
  }
  
/****************************************************************
  getMiscCost

  Access to private instance variable
****************************************************************/ 
  public double getMiscCost()
  {
    return (miscCost);
  }
  /****************************************************************
  setMiscCost

  Access to set private instance variable
****************************************************************/ 
  public void setMiscCost(double setV)
  {
    miscCost = setV;
  }
  
/****************************************************************
  setNumOfDays

  Access to set private instance variable
****************************************************************/ 
  public void setNumOfDays(int setV)
  {
    numOfDays = setV;
  }
  
/****************************************************************
  getNumOfDays

  Access to private instance variable
****************************************************************/ 
  public int getNumOfDays()
  {
    return (numOfDays);
  }
  /****************************************************************
  getNumOfTrvlrs

  Access to private instance variable
****************************************************************/ 
  public int getNumOfTrvlrs()
  {
    return (numOfTrvlrs);
  }
  /****************************************************************
  setNumOfTrvlrs

  Access to set private instance variable
****************************************************************/ 
  public void setNumOfTrvlrs(int setV)
  {
    numOfTrvlrs = setV;
  }
  /****************************************************************
  dailyCost

  Calculates sum of daily cost per traveler.  
****************************************************************/ 
  double dailyCost()
  {
    double sum = (lodgingCst +
                  mealPerDay +
                  miscCost);
   
    return (sum);
  
  }
}//end class Trip





public class TripPlanner extends JApplet implements ActionListener 
{
  private boolean notFirstRun = false;    //Used to clear input of scanner on second
                                          //run of do while loop. 
  
  String input;   // used with scanner object to get user input
  //String temp = "test";
  Scanner sc = new Scanner(System.in);
  JButton b = new JButton("Calculate");
  JButton b2 = new JButton("Clear");
  Container contentPane = this.getContentPane();
  Trip trips = new Trip();
  
   JTextField inField1 = new JTextField(5);
   JTextField inField2 = new JTextField(5);
   JTextField inField3 = new JTextField(5);
   JTextField inField4 = new JTextField(5);
   JTextField inField5 = new JTextField(5);
   JTextField inField6 = new JTextField(5);
   JTextField inField7 = new JTextField(5);
   JTextField inField8 = new JTextField(5);
   JTextField inField9 = new JTextField(5);
   
   JLabel lodgeCost = new JLabel("0.0");
  
  /****************************************************************
  plan

  prompts for inputs, instantiates Trip instances, calls methods 
  of the Trip class and prints out outputs. Utilizes ArrayList
  to store objects of class Trip to print a list of total cost
  per each trip after user acknowledges they are finished planning
  all trips.  
  ****************************************************************/
  public void plan()
  { 
   
    contentPane.setLayout(new GridLayout(9,2,1,1));
    contentPane.add(new JLabel("Destination"));
    inField1.setText(trips.getDestination());
    contentPane.add(inField1);
    
    contentPane.add(new JLabel("Air Cost"));
    inField2.setText(Double.toString(trips.getAirCost()));
    contentPane.add(inField2);
    
    contentPane.add(new JLabel("Meal Cost per Day"));
    inField3.setText(Double.toString(trips.getMealPerDay()));
    contentPane.add(inField3);

    contentPane.add(new JLabel("Lodging Cost Per Day"));
    inField4.setText(Double.toString(trips.getLodgingCst()));
    contentPane.add(inField4);
    

    contentPane.add(new JLabel("Misc cost per day"));
    inField5.setText(Double.toString(trips.getMiscCost()));
    contentPane.add(inField5);    
    
    contentPane.add(new JLabel("Number of Days"));
    inField6.setText(Integer.toString(trips.getNumOfDays()));
    contentPane.add(inField6); 
    

    contentPane.add(new JLabel("Number of Travellers"));
    inField7.setText(Integer.toString(trips.getNumOfTrvlrs()));
    contentPane.add(inField7); 
    
    contentPane.add(new JLabel("Daily Cost"));
    lodgeCost.setText(Double.toString(trips.dailyCost()));
    contentPane.add(lodgeCost);
    
    
    contentPane.add(b);
    b.addActionListener(this);
    b.setActionCommand ("calculate");
    
    contentPane.add(b2);
    b2.addActionListener(this);
    b2.setActionCommand ("clear");
    }//end plan()
  public void actionPerformed(ActionEvent ev)
  {
    String Action;
    Action = ev.getActionCommand ();
    if (Action.equals("calculate"))
    {  input = inField1.getText();
       input = input.trim();
       trips.setDestination(input);
       
       input = inField2.getText();
       input = input.trim();
       trips.setAirCost(Double.parseDouble(input));
       
       input = inField3.getText();
       input = input.trim();
       trips.setMealPerDay(Double.parseDouble(input));
       
       input = inField4.getText();
       input = input.trim();
       trips.setLodgingCst(Double.parseDouble(input));
     
       input = inField5.getText();
       input = input.trim();
       trips.setMiscCost(Double.parseDouble(input));
       
       input = inField6.getText();
       input = input.trim();
       trips.setNumOfDays(Integer.parseInt(input));
       
       input = inField7.getText();
       input = input.trim();
       trips.setNumOfTrvlrs(Integer.parseInt(input));
      
       
       plan();
    }
    else if (Action.equals("clear"))
    {
       trips.setDestination("");
       
       trips.setAirCost(0);

       trips.setMealPerDay(0);
       
       trips.setLodgingCst(0);
     
       trips.setMiscCost(0);
       
       trips.setNumOfDays(0);
       
       trips.setNumOfTrvlrs(0);
       
       plan();

    }
  }
  public void init()
  {
    plan();
    
  }



}//end class TripPlanner

[Paul Clapham]

Well, first of all, you're confusing an applet with a console application. If you're going to write a console application, then System.in is a reasonable place to get input from. But if you're writing an applet, then it should get its input from components in the GUI. So there's no point in continuing with what you have there; get rid of the Scanner first.

[Joseph Cho]

Sorry that was left in there from the original assignment.. I actually don't actually use it. I forgot to remove it.

[Joseph Cho]

The clear method works and the calculations work.. but everything just gets scrambled on the calculate run..

[Joseph Cho]

Figured it out.. I had an object Label being created each pass..

new code:

 public void plan()
  { 
    
    contentPane.setLayout(new GridLayout(9,2,1,1));
      
    //title.setText("Destination");
    contentPane.add(title1);
    inField1.setText(trips.getDestination());
    contentPane.add(inField1);
    
   // title.setText("Air Cost");
    contentPane.add(title2);
    inField2.setText(Double.toString(trips.getAirCost()));
    contentPane.add(inField2);
    
    //title.setText("Meal Cost per Day");
    contentPane.add(title3);    
    inField3.setText(Double.toString(trips.getMealPerDay()));
    contentPane.add(inField3);

    //title.setText("Lodging Cost Per Day");
    contentPane.add(title4);
    inField4.setText(Double.toString(trips.getLodgingCst()));
    contentPane.add(inField4);
    
    //title.setText("Misc cost per day");
    contentPane.add(title5);
    inField5.setText(Double.toString(trips.getMiscCost()));
    contentPane.add(inField5);    
    
    //title.setText("Number of Days");
    contentPane.add(title6);
    inField6.setText(Integer.toString(trips.getNumOfDays()));
    contentPane.add(inField6); 
    
    //title.setText("Number of Travellers");
    contentPane.add(title7);  
    inField7.setText(Integer.toString(trips.getNumOfTrvlrs()));
    contentPane.add(inField7); 
    
   //title.setText("Daily Cost");
    contentPane.add(title8);
    lodgeCost.setText(Double.toString(trips.dailyCost()));
    contentPane.add(lodgeCost);
    
      
    contentPane.add(b);
    b.addActionListener(this);  
    b.setActionCommand ("calculate");
    
    contentPane.add(b2);
    b2.addActionListener(this);
    b2.setActionCommand ("clear");
    }//end plan()