Temat: Porównanie dat.

Tomasz Kiełbowski:
$data_dzis = mktime($dzis[1], $dzis[2], $dzis[0]);
[...]
$data_zwrotu2 = mktime($zwrot[1], $zwrot[2], $zwrot[0]);

Późno jest i oczy mi się kleją, ale te mktime-y wyglądają mi jakoś podejrzanie.
Wg dokumentacji http://www.php.net/manual/pl/function.mktime.php wpisałeś miesiąc w miejsce godziny, dzień w miejsce minut a rok w miejsce sekund. Funkcja mktime przyjęła ponadto domyślnie dzień, miesiąc i rok z aktualnego czasu (argumenty pomija się od prawej do lewej a właśnie z "prawej" strony zapisywana jest data). Pomijam oczywiście oznaczenie czasu letniego i zimowego.

Wg mnie powinieneś spróbować zapisać te mktime-y jako:

mktime(0, 0, 0, $zwrot[1], $zwrot[2], $zwrot[0]);	

Ponadto mam jeszcze taką drobną uwagę - zapisywanie warunku typu

$sortowanie_lista_wirt == 'ORDER BY data_zwrotu ASC'

jest moim zdaniem bardzo ryzykowne. Teraz wszystko działa pewnie pięknie, ale w którymś momencie zmienisz gdzieś zapytanie i np. dla czytelności dodasz jakąś spację, a następnie spędzisz kupę czasu na szukaniu błędu. Lepiej jest moim zdaniem używać dodatkowych flag.

Temat: Porównanie dat.

Po co tak kombinować?

$today = new DateTime();

/* domyślam się z przykładu, że data zwrotu jest w formacie Y-m-d */
$zwrot = DateTime::createFromFormat("Y-m-d", "2012-02-05");

$interval = $zwrot->diff($today);

if ($interval->format('%R') === "+" && (int) $interval->format('%a') > 0) {
echo 'Data zwrotu jest starsza od daty dzisiejszej';
} else {
echo 'Data zwrotu nie jest starsza od daty dzisiejszej';
}

EDIT: po krótkim przemyśleniu tematu. Da się jeszcze bardziej uprościć: diffa wywalamy, a warunek zapisujemy w postaci:

if ($zwrot->getTimestamp() < $dzis->getTimestamp()) etc.Paweł K. edytował(a) ten post dnia 04.02.12 o godzinie 23:16