Job Class

[kizilbas1]

I have been trying to run the following code and it compiles but somehow it does not show the cost as stored. So here is the code below;

public Job(int ref, String nam, int pago, int copo, int hour, int minute, boolean doubleSide)
{

reference = ref;
name = nam;
pageno = pago;
copyno = copo;
doubleSides = doubleSide;


setTime = new Time(hour, minute);
Date = null;

}

/**
*
*/

public void calCost(boolean doubleSide, int pago, int copo)
{
if (doubleSides == true)
{
int c = pago * copo * 2;


}
else
{
int c = pago * copo * 3;
}
}


and this is what i have been requiring to do;

The Job class must store the following information about a job: the job's reference number as an int, the name of the member of staff submitting the job as a String, the number of pages and the number of copies to be made as int, whether the copies are to be double sided or not.
&
Write an accessor getCost which returns the cost of the job as specified above

[ShoeNinja]

I think this might be what you are looking for.

Code:

public Job(int ref, String nam, int pago, int copo, int hour, int minute, boolean doubleSide)
{

reference = ref;
name = nam;
pageno = pago;
copyno = copo;
doubleSides = doubleSide;

setTime = new Time(hour, minute);
Date = null;

//add cost field.  May need to be something besides an int.
int cost = this.calCost(doubleSides, pageno, copyno);  
}

/**
*
*/

//change this to return cost.
public int calCost(boolean doubleSide, int pago, int copo)
{
if (doubleSide == true)
{
int c = pago * copo * 2;


}
else
{
int c = pago * copo * 3;
}

return c;
}

Disclaimer: I didn't compile it. There is a chance I mistyped something.

[kizilbas1]

Quote:

Originally Posted by ShoeNinja

I think this might be what you are looking for.

Code:

public Job(int ref, String nam, int pago, int copo, int hour, int minute, boolean doubleSide)
{

reference = ref;
name = nam;
pageno = pago;
copyno = copo;
doubleSides = doubleSide;

setTime = new Time(hour, minute);
Date = null;

//add cost field.  May need to be something besides an int.
int cost = this.calCost(doubleSides, pageno, copyno);  
}

/**
*
*/

//change this to return cost.
public int calCost(boolean doubleSide, int pago, int copo)
{
if (doubleSides == true)
{
int c = pago * copo * 2;


}
else
{
int c = pago * copo * 3;
}

return c;
}

Disclaimer: I didn't compile it. There is a chance I mistyped something.

I have corrected the mistypes, but still it does not store the cost

[ShoeNinja]

I don't know what you mean by store the cost? In the above code, there is a field in the job class where the cost is stored.

Write an accessor getCost which returns the cost of the job as specified above.


Is this what you need?

If so:

Code:

public int getCost(){
return this.cost;
}

[kizilbas1]

Quote:

Originally Posted by ShoeNinja

I don't know what you mean by store the cost? In the above code, there is a field in the job class where the cost is stored.

Write an accessor getCost which returns the cost of the job as specified above.


Is this what you need?

If so:

Code:

public int getCost(){
return this.cost;
}

ye later on i will return the cost, but before that it should calculate the cost. As a result, when i return the cost, it must show me the cost which i have entred not as 0.

[ShoeNinja]

That should be what it's doing.

Code:

this.cost = this.calCost(doubleSides, pageno, copyno);

Here the calCost method uses the data passed to the constructor to calculate and return the cost. Therefore, the cost is stored in cost (imagine that?).

Again I didn't run this, but I am confident that the concept is correct. A little troubleshooting on your part and you should be there.

[ShoeNinja]

Is that all of your code up top? I assume you have something like this before that.

Code:

public class Job{

int reference, pageno, copyno, cost;
String name;
boolean doubleSides;

[kizilbas1]

Quote:

Originally Posted by ShoeNinja

That should be what it's doing.

Code:

this.cost = this.calCost(doubleSides, pageno, copyno);

Here the calCost method uses the data passed to the constructor to calculate and return the cost. Therefore, the cost is stored in cost (imagine that?).

Again I didn't run this, but I am confident that the concept is correct. A little troubleshooting on your part and you should be there.

it compiles, but still shows the cost as 0.

[ShoeNinja]

Did you make the changes to the calCost method that I posted?

Code:

public int calCost(boolean doubleSide, int pago, int copo)
{
if (doubleSide == true)
{
int c = pago * copo * 2;


}
else
{
int c = pago * copo * 3;
}

return c;
}

How are you instantiating your Job object?

[kizilbas1]

Quote:

Originally Posted by ShoeNinja

Did you make the changes to the calCost method that I posted?

Code:

public int calCost(boolean doubleSide, int pago, int copo)
{
if (doubleSide == true)
{
int c = pago * copo * 2;


}
else
{
int c = pago * copo * 3;
}

return c;
}

How are you instantiating your Job object?

Finally. i have just got it worked. thanks a lot buddy

[ShoeNinja]

Sweet man. What was wrong?

[kizilbas1]

Quote:

Originally Posted by ShoeNinja

Sweet man. What was wrong?

I should have put the if statement into constructor not making it as a method