Printable View
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";
}
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. :pQuote:
Posted by marnitzg
You need double equal signs to compare. Otherwise you're asigning a value to a variable
In the if statements, not in the blocks:
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).Code:else if (intSetPieces = 200 /* wrong */ || intSetPieces <= 399)
{
dblPay = .55D; // ok
}
Oh yeah. right. I just took a quick look at the the first conditional statement sorry. :D 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.
Confused :confused::confused: :confused:Quote:
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. :p
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; :)
:rolleyes: