Broken conditional (!)

Cerulean

I just can't figure out what's wrong with this loop. I was hoping that maybe you fine folks could help me out. This "if" line is the one that's tossing the error. I've got this big array, and data[6] is going to be x:xx format, string to time. I've tried removing that, but i get nowhere:


if ($data[4] >= 15) and ($data[5] <= 10) and ($data[6] >= mktime(0, 3, 10, 0, 0, 0)) {
	continue;

} else {

basically, it should just skip this record if those conditions are met, otherwise hit the else. It's just not happening.

Thanks so much
-Marty

tekky

what error is it giving and if data[6] is x:xx format, mktime will never compare properly to it... you'd have to do your strtotime before the comparison.....

Cerulean

Parse error. That's all I'm getting. I'll try chaging that to use strtotime. So that conditional doesnt look messed up? I thought it looked OK...

-m

tekky

you may have to enclose the whole thing in ( ) just to ensure they get parsed as one conditional.... but other than that... it looks alright....

[edit-- on a side note, if you are getting a parse error on THAT line #, then post the 5 lines above it, generally a parse error reports a line or 2 AFTER it occurs --]

laserlight

//yeah, I prefer using && to and to save space
if ($data[4] >= 15 && $data[5] <= 10 && $data[6] >= mktime(0, 3, 10, 0, 0, 0)) {
	continue;
} else {

the problem being that the condition was written as:
if ($data[4] >= 15)
followed by a mysterious string of undefined constants, some of which didnt even qualify as constants.