Where's the mistake is this update statement?

jonki

Please can anyone spot what I am doing wrong.

$query6 = "UPDATE db SET tarifa = CASE(max4 < 5 THEN "2", ((5 <= max4) AND (max4 < 10)) THEN "1.8", ((10 <= max4) AND (max4 < 20)) THEN "1.5", ((20 <= max4) AND (max4 < 50)) THEN "1.2", ((50 <= max4) AND (max4 < 80)) THEN "1.1", (80 <= max4) THEN "1")"
$result6 = mysql_query($query6) or die (mysql_error() );

Thanks a lot.

I get a parse error but it doesn't tell me anything.

Jon

LordShryku

The parse errors are probably because you're not escaping your double quotes.

jonki

.... but I am still getting the same error....

&query6 = "UPDATE tenercoche_concesionarios SET tarifa = CASE(max4 < 5 THEN \"2\", (5 <= max4 AND max4 < 10) THEN \"1.8\", 
		(10 <= max4 AND max4 < 20) THEN \"1.5\", (20 <= max4 AND max4 < 50) THEN \"1.2\", (50 <= max4 AND max4 < 80) THEN \"1.1\", 
		(80 <= max4) THEN \"1\")" ;
		$result6 = mysql_query($query6);

I get
Parse error: parse error in /home/tenercoc/tenercoche-www/cronjob.php on line 13

Cheers for your help

LordShryku

&query6

Is that really an ampersand?

jonki

...er it was ..... but I copied that from further down in my script .... line 13 was ok ....... and I still get the error.

jonki

It's ok - I've got it.

I corrected the \" further down but missed off one...

I really apreciate all your help.

Thanks a lot.

jonki

I get no error now but the column "tarifa" is not updated even though I have several "test entries" in my db with varying values of max3

$query6 = "UPDATE db SET tarifa = CASE(max3 < 5 THEN \"2\", ((5 <= max3) AND (max3 < 10)) THEN \"1.8\", 
		((10 <= max3) AND (max3 < 20)) THEN \"1.5\", ((20 <= max3) AND (max3 < 50)) THEN \"1.2\", ((50 <= max3) AND (max3 < 80)) THEN \"1.1\", 
		(80 <= max3) THEN \"1\")" ;
		$result6 = mysql_query($query6);

I tried it with and without the double brackets around the ((20 <= max3) AND (max3 < 50))

Any more help will be greatly appreciated

Thanks

ahundiak

Fire up mysql or phpadmin and enter the update manually a bit at a time. It's a logic error but it's too difficult for me to debug from your post.

jonki

Hi - thanks for your help.
The problem was the commas and I think the brackets were also wrong. I removed them and it was fine.

Cheers for the suggestion.