Writing to a File

[Holly Leery]

Hello there! I posted this code a few days ago, but I really shouldn�t have at the time because it was not near complete. But, I have made a lot changes and added a few things to the application. So now basically I am just wondering if someone could just let me know if I am on the right track with the switch statement and the action listeners.

The application I am building will calculate the total sales for any number of salesmen. The application will allow the salesman to enter their name, the product (numbered 1-5) they sold as well as the amount of products they sold. They would then hit done to tell the application that the salesman is done entering his information. Once all of the salesmen have entered their information they can hit Grand Total and the grand total of those sales will display in the text area.

TOTAL SALES

NameXXXXXX
NameXXXXXX

TotalXXXXXX

I put together a GUI in flow layout, but the text fields are not allowing me to type in the field, is there a particular reason that might happen? Maybe I just missed something.

Also, the final information will be printed in alphabetical order, which I haven't done yet, but I think I will use sort to do that. This will all be sent to a .txt file as well by the way.

So anyway that is a brief run down of what I am trying to do. Any little tip just letting me know I am on the right track or whatever would be appreciated!! Thanks so much everyone!!

import java.awt.*;
import javax.swing.*;
import java.awt.event.*;
import java.util.ArrayList;
import java.io.*;
 
public class Sales extends JFrame
{
static JTextArea output;
static ArrayList<Salesman> myList = new ArrayList<Salesman>();
static JLabel salesman;
static JTextField name;
static JLabel amtsold;
static JTextField amount;
static JLabel productname;
static JComboBox product;
static String [] productName = {"1", "2", "3", "4", "5"};
static JButton done;
static JButton total;
static FileWriter file;
static BufferedWriter buff;
static String p;
static Handler handler = new Handler();

public Sales()
{
super("Monthly Sales");
Container c = getContentPane();
c.setLayout(new FlowLayout());

try
{
file = new FileWriter("SalesData.txt");
buff = new BufferedWriter(file);
}
catch(IOException ioe)
{
ioe.printStackTrace();
}
 
output = new JTextArea(400, 200);
salesman = new JLabel("Salesman");
name = new JTextField(400);
amtsold = new JLabel("Amount Sold");
amount = new JTextField(400);
productname = new JLabel("Product");
product = new JComboBox(productName);
done = new JButton("Salesmen Entry Done");
total = new JButton("Grand Total");

c.add(salesman);
c.add(name);
c.add(amtsold);
c.add(amount);
c.add(productname);
c.add(product);
c.add(done);
c.add(total);
c.add(output);

done.addActionListener(handler);
total.addActionListener(handler);
product.addActionListener(new ActionListener(){
public void actionPerformed(ActionEvent e)
{
if (e.getSource() == done)
{
JComboBox product = (JComboBox)e.getSource();
        String prod = (String)product.getSelectedItem();
        p = prod;
}
}});
}
 
public static void main (String args[])
{
Sales sale = new Sales();
sale.setSize(400,400);
sale.setVisible(true);
 
sale.addWindowListener(new WindowAdapter(){
public void windowClosing(WindowEvent e)
{
try
{
buff.close();
}
catch(IOException ioe)
{
ioe.printStackTrace();
}
System.exit(0);
}});
}

public static class Handler implements ActionListener
{
public void actionPerformed(ActionEvent e)
{
String dname;
String dp;
double damount;
Salesman s1;

dname = name.getText();
dp = Integer.parseInt(p.getText());
damount = Double.parseDouble(amount.getText());
s1 = new Salesman(dname);

 
switch(dp)
{
case 1:   s1.setSales(damount * 2.98);
break;
 
case 2:   s1.setSales(damount * 4.50);
break;

case 3:   s1.setSales(damount * 9.98);
break;

case 4:   s1.setSales(damount * 4.49);
break;

case 5:   s1.setSales(damount * 6.87);
break;
 
default: System.out.println("Invalid - please choose a product");
}
myList.add(s1);
output.append("\n" + s1.toString());
output.setText(s1.toString());
 
try
{
buff.write(s1.toString());
}
 
catch (IOException ioe)
{
ioe.printStackTrace();
}
}
}
}

Class Salesman:

import java.text.NumberFormat;
 
public class Salesman
{
private String name;
private double sales;
 
NumberFormat currencyFormat = NumberFormat.getCurrencyInstance();
 
public Salesman()
{
setName("");
setSales(0);
}
 
public Salesman(String name)
{
setName(name);
setSales(0);
}
 
public void setName(String name)
{
this.name = name;
}
 
public void setSales(double sales)
{
this.sales = sales;
}

public String getName()
{
return name;
}
 
public double getSales()
{
return sales;
}

public String toString()
{
return (name + "\t" + currencyFormat.format(sales));

}
}
 

[Holly Leery]

Ok I have even bigger changes than before. I am sorry I keep posting this, I am just hoping to get a little feedback on how I am doing. Everything seems to be working great (it compiles) except for when it should be reading and writing the data to the textarea and to the .txt file. I'm wondering if the problem might have to do with myList. Could this be the case?

import java.awt.*;
import javax.swing.*;
import java.awt.event.*;
import java.util.ArrayList;
import java.io.*;
import java.util.*;
import java.text.NumberFormat;
 
public class Sales extends JFrame
{
static JLabel salesmanlabel;
static JTextField name;
static JLabel amtsold;
static JTextField amount;
static JLabel productname;
static JComboBox product;
static JButton salesmandone;
static JButton total;
static JTextArea output;
static FileWriter file;
static BufferedWriter buff;
static String outp;
static Handler handler = new Handler();
static ArrayList<Salesman> myList = new ArrayList<Salesman>();
static NumberFormat currencyFormat = NumberFormat.getCurrencyInstance();

public Sales()
{
super("Monthly Sales");
Container c = getContentPane();
c.setLayout(new FlowLayout());

try
{
file = new FileWriter("SalesData.txt");
buff = new BufferedWriter(file);
}
catch(IOException ioe)
{
ioe.printStackTrace();
}
 
salesmanlabel = new JLabel("Salesman:");
name = new JTextField(20);
amtsold = new JLabel("Amount Sold:");
amount = new JTextField(20);
productname = new JLabel("Product Sold:");
product = new JComboBox();
salesmandone = new JButton("Salesman Entry Done");
total = new JButton("Grand Total");
output = new JTextArea(400, 200);

product.addItem("");
product.addItem("Product 1");
product.addItem("Product 2");
product.addItem("Product 3");
product.addItem("Product 4");
product.addItem("Product 5");

c.add(salesmanlabel);
c.add(name);
c.add(amtsold);
c.add(amount);
c.add(productname);
c.add(product);
c.add(salesmandone);
c.add(total);
c.add(output);

salesmandone.addActionListener(handler);
total.addActionListener(handler);
}
 
public static void main (String args[])
{
Sales sale = new Sales();
sale.setSize(300,300);
sale.setVisible(true);
 
sale.addWindowListener(new WindowAdapter(){
public void windowClosing(WindowEvent e)
{
try
{
buff.close();
}
catch(IOException ioe)
{
ioe.printStackTrace();
}
System.exit(0);
}});
}

public static class Handler implements ActionListener
{
public void actionPerformed(ActionEvent e)
{
if(e.getSource() == salesmandone)
{
addSalesman();
}
else if(e.getSource() == total)
{
calculateTotal();
}
}
}

public static void addSalesman()
{
double sales = calculateSales(product.getSelectedIndex(), Integer.parseInt(amount.getText()));

myList.add(new Salesman(name.getText(), sales));
 
name.setText("");
product.setSelectedIndex(0);
amount.setText("");
name.requestFocus();
}

public static double calculateSales(int sproduct,int samount)
{
double sales;

sproduct++;

switch(sproduct)
{
case 1:   sales = (samount * 2.98);
break;
 
case 2:   sales = (samount * 4.50);
break;

case 3:   sales = (samount * 9.98);
break;

case 4:   sales = (samount * 4.49);
break;

case 5:   sales = (samount * 6.87);
break;
 
default: sales = (0.00);
}

return sales;
}
public static void calculateTotal()
{
double total = 0.00;

Collections.sort(myList, new Comparator<Salesman>(){
public int compare(Salesman s1, Salesman s2) {
return s1.getName().compareTo(s2.getName());}});
 
outp = "\tTOTAL SALES\n\n";

 
for (Salesman salesm : myList)
{
outp += salesm.toString() +"\n";
total += salesm.getSales();
}
 
outp += "\nTotal\t\t" + currencyFormat.format(total);
output.setText(outp);

try
{
buff.write(outp);
}
 
catch (IOException ioe)
{
ioe.printStackTrace();
}
}
}

public class Salesman
{
private String name;
private double sales;
 
public Salesman()
{
setName("");
setSales(0);
}
 
public Salesman(String name, double sales)
{
setName(name);
setSales(0);
}
 
public void setName(String name)
{
this.name = name;
}
 
public void setSales(double sales)
{
this.sales = sales;
}

public String getName()
{
return name;
}
 
public double getSales()
{
return sales;
}

public String toString()
{
return (name + "\t\t" + sales);
}
}

[Matti Poro]

You should open a file stream only if you need to and close it right after you don't need it anymore. From Sun's tutorial: "A program should close a stream as soon as it is done with it, in order to free up system resources." Also your members are static which isn't necessary.

[Holly Leery]

Are you saying I might need to move the file writer and buffer to open just before I run the calculations and close after I print the information to the output? Everything seems to be running properly but the output is not being sent to the text area and the calculations are not taking place. The text is being sent to the .txt file. The calculations just are not happening.