[AdamSpeight2008]

Some Reading Material

[macosxnerd101]

AdamSpeight2008, on 24 July 2010 - 07:56 PM, said:

Some Reading Material

It's a very well-written tutorial, if I do say so myself.

[macosxnerd101]

The whole point of classes and Objects is to not use parallel arrays at all. So rather than having 2 arrays for customer name and tip, create a Customer class to encapsulate those attributes, and store a Customer[] in your Stylist class.

As for your problem, you invoke addCustomer() twice, which places the customer in the array twice. No need for the first addCustomer() invocation if you have one in the if statement.

one.addCustomer(cName, tip);

if (!one.addCustomer(cName, tip))
      JOptionPane.showMessageDialog(null, "No tip!!!");

[pbl]

Look at macosxnerd101 tutorial. It is well written and you won't need to use parallel arrays anymore

[baavgai]

Kathllen, on 24 July 2010 - 10:36 PM, said:

I HAVE to use parallel arrays. Its part of the requirments...

Pity.

This is your big error: "for(int x =0;x < tips.length; x++)".

Also, you need to be careful about initializing and checking your vars. Be careful they don't have zero customers on the toString. Given your unfortunate array requirement, I'd do it like this:

public class Stylist {
	private static final int MAX_CUSTOMERS = 20;
	private final int MINWAGE = 11, MAXWAGE = 25, MINHOURS = 1, MAXHOURS = 60;
	private final double MINTIP = 0.1, MAXTIP =100;
	private String name;
	//private double totalTip;
	private double hours, wage;
	
	private int numCustomers;
	private String[] cusName = new String[MAX_CUSTOMERS];
	private double[] tips = new double[MAX_CUSTOMERS];

	public Stylist (String name, double wage) {
		this.numCustomers = 0;
		setName(name);
		this.wage = MINWAGE;
		setWage(wage);
	}

	public Stylist () { this(null, MINWAGE); }
	
	public Stylist (String name) { this(name, MINWAGE); }
	
	private boolean valid(double value, double min, double max) {
		return (value >= min && value <= max);
	}
	
	
	public boolean addCustomer(String customerName, double tip) {
		if (!valid(tip,MINTIP, MAXTIP)) { return false; }
		if (customerName==null) { return false; }
		cusName[numCustomers] = customerName;
		tips[numCustomers] = tip;
		numCustomers++;
		return true;
	}
	


	public boolean setHours (double hours) {
		if (!valid(hours, MINHOURS, MAXHOURS)) { return false; }
		this.hours = hours;
		return true;
	}
	
	public boolean setWage(double wage) {
		if (!valid (w, MINWAGE,MAXWAGE)) { return false; }
		this.wage = wage;
		return true;
	}
	
	public void setName(String name) { this.name = name==null ? "Uknown" : name; }

	public double getSalary() { return hours*wage; }
	
	public double getWage() { return wage; }
	
	public int getTotalTips() {
		int totalTip = 0;
		for(int x =0; x < numCustomers.length; x++) {
			totalTip += tips[x];
		}
		return totalTip;
	}
	
	public String getName(){ return name; }
	
	public int getNumCustomer(){ return numCustomers; }
	
	public double getHours() { return hours; }
	
	public String toString() {
		double salary = getSalary();
		double tips = getTotalTips();
		String s = "Stylist:  "+ name + " earned a salary of $" + salary;
		if (numCustomers>0)
			for(int i = 0; i < numCustomers; i++) { 
				// this is a bad check, you shouldn't let them enter a null name to start with
				// if(cusName[i] == null) { break; }
				s += "\nCustomer: " + cusName[i] + "\nTip :" + tips[i];
			}
			s += "\nTotal Pay:  $" + (salary + tips) + "\nAverage tip: $" + (tips/numCustomers);
		}
		return s;
	}
}

This post has been edited by baavgai: 25 July 2010 - 11:01 AM

[pbl]

Kathllen, on 24 July 2010 - 10:36 PM, said:

I HAVE to use parallel arrays.

If it is for school assignment and you haven't seen Objects yet
sins forgiven