[RESOLVED] Select to print elements from loop...

gszauer

$today = date("m d y");
$month = date("m");
$day = date("d");
$year = date("y");
// --
$query = @mysql_query("SELECT * FROM $table");
$total_rows = @mysql_num_rows($query); 
$counter = 0; 
while($ourData = @mysql_fetch_array($query)){ 
     $adUrl = $ourData["adUrl"]; 
     $adName   = $ourData["adName"]; 
     $exMonth   = $ourData["exMonth"]; 
     $exDay   = $ourData["exDay"]; 
     $exYear   = $ourData["exYear"]; 
     $impressions   = $ourData["impressions"]; 
     $adAt  

	 $counter++; 
    print ("&adUrl$counter=$adUrl");
}

My basic code is above,
I would like to know how i could make

print ("&adUrl$counter=$adUrl");

only print IF the ad is not expired, so i put

$month = date("m");
$day = date("d");
$year = date("y");
//...
 $exMonth   = $ourData["exMonth"]; 
 $exDay   = $ourData["exDay"]; 
 $exYear   = $ourData["exYear"];

I am a n00b at php, and have no clue how to write an appropriate if statement, so if the ad has expired it wont play....
Can anyone PLEASE help me out?
I really appritiate your time.
~Gabor Szauer

Weedpacket

First, fix your database; MySQL has date/time field types; use them.

You can then have the select query return the expiry date as a Unix timestamp (if I recall correctly the MySQL function is UNIX_TIMESTAMP(datefield)), and your code can check to see if that is less than the current [man]time[/man].

bradgrafelman

In fact, if you're only worried about dealing with current ads, you don't even need to SELECT them in your query, do you? Modify your table as Weedpacket suggested, and then use a query like so:

$query = @mysql_query("SELECT * FROM $table WHERE datefield >= CURDATE()");

That way, you can simply print() the information you need since you know that MySQL is only returning rows that have a date in the future (or today).

Note that MySQL's CURDATE() function returns dates in the format of 'YYYY-MM-DD', or the 'DATE' column type. You may need to pick a different Date and Time Column Type and function to suit your needs (i.e. if you specify the time as well).

gszauer

Thank you bouth wery much, that was exactly what i was looking for