if help

[308holes]

ok im getting errors when i try this saying that (Boolean,int) are not allowed??? first if works fine all the else ifs are %^&*ed up ?

if (intSetPieces > 0 || intSetPieces <= 199)
{
dblPay = .50D;
}
else if (intSetPieces = 200 || intSetPieces <= 399)
{
dblPay = .55D;
}
else if (intSetPieces = 400 || intSetPieces <= 599)
{
dblPay = .60D;
}
else if (intSetPieces = 600 || intSetPieces <= 100)
{
dblPay = .65D;
}
else
{
intError = 1;
strError = "Error With Calculating Payment";
}

[marnitzg]

You need double equal signs to compare. Otherwise you're asigning a value to a variable

[Dillinger4]

Posted by marnitzg
You need double equal signs to compare. Otherwise you're asigning a value to a variable

I think that is what he wants to do. I couldnt see using the equals operator by itself. I wouldn't be a statement.

[CornedBee]

In the if statements, not in the blocks:

 else if (intSetPieces = 200 /* wrong */ || intSetPieces <= 399)
{
dblPay = .55D; // ok
}

Oh, btw you should use logical and (&&) not or (||) in the first if, else it catches everything (every number is greater 0 or less than 199).

[Dillinger4]

Oh yeah. right. I just took a quick look at the the first conditional statement sorry. Yeah 308holes CornedBee is right. Using the short circut operator || if the first condition equates to true the second condition will just be skiped so intSetPieces could end up being more than 199 clearly not what you want.

[marnitzg]

Originally posted by Dilenger4
I think that is what he wants to do. I couldnt see using the equals operator by itself. I wouldn't be a statement.

Confused

[Dillinger4]

Sorry marnitzg. I didnt see the assignment operator in the second conditional statement so i thought you were refering to the line dblPay = .50D; meaning it should be dblPay == .50D;

[marnitzg]