I'd agree with what Richard is alluding to: turn your two dates into timestamps, and then compare those. mktime() or strtotime() should help you out. HTH, John W On 4/18/06, Richard Lynch <ceo@xxxxxxxxx> wrote: > http://php.net/mktime may be more suitable, depending on the date > range of the input. > > That said, as far as I can tell, your $formated_expiry_date is the > SAME as your $expiry_date, except possibly for some separation > characters. > > If the separation characters are ALWAYS the same, you could just do: > > $current_date = date('Y/m/d H:i:s'); //match formatting of expiry date. > return $current_date < $expiry_date; > > > On Tue, April 18, 2006 5:02 pm, Murtaza Chang wrote: > > Hi everyone, > > this is the function I have written for comparing a date time please > > tell me > > if my logic is correct ? and if there's a better alternative please > > let me > > know of that as well. > > // This function will return 1 if supplied date is expired > > function is_expire($expiry_date){ > > $current_date=date('YmdHis'); > > $year=substr($expiry_date,0,4); > > $month=substr($expiry_date,5,2); > > $day=substr($expiry_date,8,2); > > $hour=substr($expiry_date,11,2); > > $min=substr($expiry_date,14,2); > > $sec=substr($expiry_date,17,2); > > $formated_expiry_date=$year.$month.$day.$hour.$min.$sec; > > if ($current_date>=$formated_expiry_date) > > return 1; > > else > > return 0; > > } > > -- > > Murtaza Chang > > > > > -- > Like Music? > http://l-i-e.com/artists.htm > > -- > PHP General Mailing List (http://www.php.net/) > To unsubscribe, visit: http://www.php.net/unsub.php > > -- PHP General Mailing List (http://www.php.net/) To unsubscribe, visit: http://www.php.net/unsub.php