If-Anweisung liefert verkehrte Ausgabe

[Peter_AM]

Hallo Zusammen,

in meinem Script wird immer nur der erste Teil der If Anweisung
ausgegeben, sollte aber aber bei Datum älter als 90 Tagen
den else-Teil ausgeben.

Bin wenig erfahren mit PHP

Mein QT

----------------------------------------------

$date = date_create($datensatz[datum]);
date_sub($date, date_interval_create_from_date_string('90 days'));
$abfragedatum = date_format($date, 'Y-m-d');                                                                
if ($datensatz[datum] >= $abfragedatum)                                                                
{
echo"<td>$datensatz[datum]</td></br>"; //Dieser Teil wird immer ausgegeben                                                            
}else 
{                                                                        
echo "<td><form> </form></td>";                                                                         
} 

-----------------------------------------------------

Danke für Eure Hilfe
Peter Heldmann

[wahsaga]

Was hat deine Kontrollausgabe der beiden Werte ergeben?

[Peter_AM]

Hallo wahsaga,

Danke für deine Hilfe

$datensatz[datum] = gibt das Datum aus der DB zurück
$abfragedatum = gibt den Wert $datensatz[datum] - 90 Tage zurück

ich bin mir nicht sicher ob hier alles passt:

if ($datensatz[datum] >= $abfragedatum) // gibt nur $datensatz[datum] aus
if ($datensatz[datum] < $abfragedatum) // gibt nur <form> aus

Gruß Peter

[wahsaga]

Zeig mal konkrete Werte bitte.

[Peter_AM]

if ($datensatz[datum] >= $abfragedatum) // gibt 2015-05-11 aus
if ($datensatz[datum] < $abfragedatum) // gibt 2015-02-11 aus

[Peter_AM]

Also $abfragedatum gibt imm 3 Monate kleiner aus

[fireweasel]

Zweiter Versuch. Den ersten hat das Security-Token-Monster gefressen.

in meinem Script wird immer nur der erste Teil der If Anweisung
ausgegeben, sollte aber aber bei Datum älter als 90 Tagen
den else-Teil ausgeben.

Punkt Eins: DateTime-Objekte lassen sich direkt vergleichen. Vergleiche mit formatierten Datumsstrings sind nicht nötig und verursachen nur Probleme. Folgendes reicht also:

$date = date_create($datensatz['datum']);
$abfragedatum = clone $date;
date_sub($abfragedatum, date_interval_create_from_date_string('90 days'));
if ($date >= $abfragedatum) {
}
else {
} 

Punkt zwei: Wie dir an dem vereinfachten Code jetzt vielleicht auffällt, ist deine Vergleichsbasis falsch gewählt: $date ist immer 90 Tage aktueller als $abfragedatum. Deswegen kommt immer der If-Zweig zum Zug.

Vergleichst du stattdessen mit einem aktuellen Zeitpunkt, wie beispielsweise "Jetzt" ("now"), klappts auch mit der Else:

$date = date_create($datensatz['datum']);
$abfragedatum = date_create('now');
$abfragedatum = date_sub($abfragedatum, date_interval_create_from_date_string('90 days'));
if ($date >= $abfragedatum) {
}
else {
} 

P.S.: Deine Variablen-Benennung ist grausam. Lege dich auf eine Sprache fest (Englisch oder Deutsch). Wähle aussagekräftige Namen. Dann wird dein Code lesbarer und man findet solche (eigentlich recht einfachen) Logikfehler auch schneller.

[Peter_AM]

Riesen Dank

Hallo fireweasel,

danke Dir.
Hab`s nun verstanden und so umgesetzt.
Es funzt einwandfrei.

Super erklärt mit Beispiel

Manchmal denkt man einfach zu kompliziert.....

Hast was gut bei mir...

Danke Peter