Java

Java RSS

Why won't this compile?

•

Join Date: Mar 2006

Posts: 131

Reputation: degamer106 is an unknown quantity at this point

Solved Threads: 0

degamer106 degamer106 is offline

Offline

Junior Poster

Why won't this compile?

Dec 3rd, 2007

This is a relatively simple program but for some reason, I can't compile it. I'm pretty sure the code is correct and working.

My compiler keeps saying that it cannot find a constructor matching a HourlyWorker constructor when I instantiate a HourlyWorker object. I've checked and rechecked the code (in WorkerTester and HourlyWorker classes) to make sure everything matches, but I can't find anything wrong.

Worker.java

 Help with Code Tags 
 Java Syntax (Toggle Plain Text) 
public class Worker
{
	public Worker(String n, int r)
	{
		name = n;
		rate = r;
	}
	public double computePay(int hours)
	{
		return (rate * hours);
	}
	public int getRate()
	{
		return rate;
	}
	private String name;
	private int rate;
}
public class Worker
{
	public Worker(String n, int r)
	{
		name = n;
		rate = r;
	}
	public double computePay(int hours)
	{
		return (rate * hours);
	}
	public int getRate()
	{
		return rate;
	}
	private String name;
	private int rate;
}

SalariedWorker

 Help with Code Tags 
 Java Syntax (Toggle Plain Text) 
public class SalariedWorker extends Worker
{
	public SalariedWorker(String n, int r)
	{
		super(n, r);
	}
	public double computePay(int hours)
	{
		double totalPay = 0;
 
		totalPay = this.getRate() * 40;
 
		return totalPay;
	}
}
public class SalariedWorker extends Worker
{
	public SalariedWorker(String n, int r)
	{
		super(n, r);
	}
	public double computePay(int hours)
	{
		double totalPay = 0;

		totalPay = this.getRate() * 40;

		return totalPay;
	}
}

HourlyWorker.java

 Help with Code Tags 
 Java Syntax (Toggle Plain Text) 
public class HourlyWorker extends Worker
{
	public HourlyWorker(String n, int r)
	{
		super(n, r);
	}
	public double computePay(int hours)
	{
		double totalPay = 0;
 
		if (hours >= 0 && hours <= 40)
			totalPay = this.getRate() * hours;
		else
			totalPay = this.getRate() * ((hours - 40) * 1.5 + 40);
 
		return totalPay;
	}
}
public class HourlyWorker extends Worker
{
	public HourlyWorker(String n, int r)
	{
		super(n, r);
	}
	public double computePay(int hours)
	{
		double totalPay = 0;

		if (hours >= 0 && hours <= 40)
			totalPay = this.getRate() * hours;
		else
			totalPay = this.getRate() * ((hours - 40) * 1.5 + 40);

		return totalPay;
	}
}

/**
This class tests class Worker and its subclasses.
*/
WorkerTester.java

 Help with Code Tags 
 Java Syntax (Toggle Plain Text) 
public class WorkerTester
{
   public static void main(String[] args)
   {
      Worker s = new SalariedWorker("Sally", 40);
      Worker h = new HourlyWorker("Harry", 40);
 
      System.out.println(s.computePay(30));
      System.out.println("Expected: 1600");
      System.out.println(h.computePay(30));
      System.out.println("Expected: 1200");
      System.out.println(s.computePay(50));
      System.out.println("Expected: 1600");
      System.out.println(h.computePay(50));
      System.out.println("Expected: 2200");
   }
}
public class WorkerTester
{
   public static void main(String[] args)
   {
      Worker s = new SalariedWorker("Sally", 40);
      Worker h = new HourlyWorker("Harry", 40);

      System.out.println(s.computePay(30));
      System.out.println("Expected: 1600");
      System.out.println(h.computePay(30));
      System.out.println("Expected: 1200");
      System.out.println(s.computePay(50));
      System.out.println("Expected: 1600");
      System.out.println(h.computePay(50));
      System.out.println("Expected: 2200");
   }
}

Last edited by degamer106; Dec 3rd, 2007 at 4:07 am.

•

Join Date: Feb 2006

Posts: 2,388

Reputation: masijade has much to be proud of

Solved Threads: 254

masijade

Offline

Nearly a Posting Maven

Re: Why won't this compile?

Dec 3rd, 2007

compile with

Help with Code Tags

Java Syntax (Toggle Plain Text)

javac -classpath . <Classname>.java

Java Programmer and Sun Systems Administrator

----------------------------------------------

Debugging is twice as hard as writing the code in the first place. Therefore, if you write the code as cleverly as possible, you are, by definition, not smart enough to debug it.
--Brian Kernighan

•

Join Date: Mar 2006

Posts: 131

Reputation: degamer106 is an unknown quantity at this point

Solved Threads: 0

degamer106 degamer106 is offline

Offline

Junior Poster

Re: Why won't this compile?

Dec 3rd, 2007

Still getting the same problem.

•

Join Date: Nov 2007

Posts: 62

Reputation: Black Box is on a distinguished road

Solved Threads: 7

Black Box

Offline

Junior Poster in Training

Re: Why won't this compile?

Dec 3rd, 2007

I've copied your code and it works perfectly. How do you have your files organized?

Like this?
-WorkerDirectory
+ Worker.java
+ HourlyWorker.java
+ SalariedWorker.java
+ WorkerTester.java

Some package definitions lacking maybe?

Black Box

Studio Black Box

•

Join Date: Feb 2006

Posts: 2,388

Reputation: masijade has much to be proud of

Solved Threads: 254

masijade

Offline

Nearly a Posting Maven

Re: Why won't this compile?

Dec 3rd, 2007

Show your exact directory structure, and the exact command you used to compile.

•

Join Date: Mar 2006

Posts: 131

Reputation: degamer106 is an unknown quantity at this point

Solved Threads: 0

degamer106 degamer106 is offline

Offline

Junior Poster

Re: Why won't this compile?

Dec 3rd, 2007

•

Show your exact directory structure, and the exact command you used to compile.

c:\fall2007\test\Homework\Due Dec 5> javac WorkerTester.java

The WorkerTester.java file as well as all the other files associated with it is located in the directory, so I don't think I have to specify the classpath as a parameter.

I get the same compile-time error as when I compile the file with Textpad; can't find an HourlyWorker constructor matching the declaration.

•

I've copied your code and it works perfectly. How do you have your files organized?

Like this?
-WorkerDirectory
+ Worker.java
+ HourlyWorker.java
+ SalariedWorker.java
+ WorkerTester.java

Some package definitions lacking maybe?

Black Box

Everything is in the same directory.

edit: hmm..weird. I tried to compile the files on the compiler at school and it works fine.

Last edited by degamer106; Dec 3rd, 2007 at 6:48 pm.

•

Join Date: Dec 2007

Posts: 10

Reputation: satya5321 is an unknown quantity at this point

Solved Threads: 0

satya5321

Offline

Newbie Poster

Re: Why won't this compile?

Dec 17th, 2007

hi from my understanding there is no default constructor defined in any of the Classes that you wrote.when you are overloading constructor the default constructor is not provided. externally you have to write the default constructor

tell me i am wrong.

Free EBooks Site. CLICK HERE

•

Join Date: Feb 2006

Posts: 2,388

Reputation: masijade has much to be proud of

Solved Threads: 254

masijade

Offline

Nearly a Posting Maven

Re: Why won't this compile?

Dec 17th, 2007

•

Originally Posted by satya5321

hi from my understanding there is no default constructor defined in any of the Classes that you wrote.when you are overloading constructor the default constructor is not provided. externally you have to write the default constructor

tell me i am wrong.

This has nothing to do with a default constructor. And, you never have to implement a default constructor. Only if you want to.

Edit: Well, I shouldn't really say never, but at least the cases where you might have to are not that common, and definately not a problem in this case.

Last edited by masijade; Dec 17th, 2007 at 4:15 am.

•

Join Date: Feb 2006

Posts: 2,388

Reputation: masijade has much to be proud of

Solved Threads: 254

masijade

Offline

Nearly a Posting Maven

Re: Why won't this compile?

Dec 17th, 2007

•

Originally Posted by degamer106

The WorkerTester.java file as well as all the other files associated with it is located in the directory, so I don't think I have to specify the classpath as a parameter.

"." is not necessarily on the classpath, and if it is not, then yes, you do have to add it. So did you actually try with the command I showed you? Or did you just assume that it wouldn't make a difference? Also, try to compile one of the classes that does not depend on any of the others, and then make sure that the classfile actually appears in the same directory as the java file (not that you have some alias set that automatically causes the compiler to place the class files elsewhere). If it is there, then compile them one at a time from the file with the least dependencies, to the file with the most. More than bit annoying, but hey.