[Orolhawion]

Hallo liebe Leute,

folgender code bereitet mir kopfzerbrechen, möglicherweise steh ich aber auch grad voll auf dem schlauch..

PHP-Code:

if (isset($_POST['sendEmail'])){
    echo "valueIs " . $_POST['recipient'];
    if ($_POST['recipient'] != 0){
        echo "if wahr";
        $email = new Email($config);
        $index['message'] = $email->sendMailCms($_POST['recipient'], $_POST['subject'], $_POST['mailBody'], $_SESSION['email']);
    }else{
        echo "if falsch";
        $error = new Template($config['cmsTplPath'] . 'messages/error/noRecipient.tpl');
        $index['message'] =  $error->getParsedCode();
    }
} 


meinem verständnis nach überprüft der code, ob der button "sendMail" geklickt wurde, falls ja, wird geprüft, ob ein empfänger aus der selectliste "recipient" gewählt wurde. der standardwert ist "0" und soll anzeigen, daß eben kein empfänger gewählt wurde. das echo "valueIs.....; gibt mir keine "0" sondern beispielsweise eine "2", folglich müßte die bedingung wahr werden, sie wird aber falsch und der else-zweig wird ausgeführt.. wieso des? wenn ich in der bedinung $_POST['recipient'] == 0 schreibe wirds interessanterweise wahr, obwohl ne 2 drin steht..

gruß Orolhawion

[Ben]

Hm, komisch.

Mach mal aus dem

PHP-Code:

echo "valueIs " . $_POST['recipient']; 


ein

PHP-Code:

var_dump($_POST['recipient']); 


, damit du den Typ mal herausfindest. Sollte ja ein String sein.

Eventuell liegt da irgendwie der Hund begraben?
Kann ich mir aber eigentlich nicht vorstellen .. aber das heißt ja bekanntlich nichts ...

[Orolhawion]

Zitat von Ben

Mach mal aus dem

PHP-Code:

echo "valueIs " . $_POST['recipient']; 


ein

PHP-Code:

var_dump($_POST['recipient']); 


, damit du den Typ mal herausfindest. Sollte ja ein String sein.

Eventuell liegt da irgendwie der Hund begraben?
Kann ich mir aber eigentlich nicht vorstellen .. aber das heißt ja bekanntlich nichts ...

ein

PHP-Code:

var_dump($_POST['recipient']) 


ergibt

PHP-Code:

string(1) "2" 


folglich lagst du mit deiner vermutung richtig.

[Ben]

Hat das jetzt das Problem gelöst?

[Orolhawion]

Zitat von Ben

Hat das jetzt das Problem gelöst?

nein, leider nicht.. sorry, hättsch ja ma sagen können...

edit: äh, bzw. doch! die bedingung muß heißen if ($_POST['recipient'] != "0") so rennt's.
danke für den tip!

[Ben]

Machmal das hier

PHP-Code:

$recipient = (int)$_POST['recipient'];

if($recipient == 0) {
    // ...
}
else {
    // ...
} 


[Basti]

Wozu überhaupt einen Standardwert? Kannst du doch einfach weglassen und mit if(empty($_POST['recipient'])) testen, oder?

Übrigens enthält $_POST immer nur Strings oder Arrays von Strings (...).

Und, ich empfehle immer die typensicheren Vergleichsoperatoren zu benutzen. Lieber einmal mehr explizit umwandeln, als nicht zu wissen, ob z.B. der Zahlenwert einer Variable als string, integer, float oder gar boolean gespeichert ist!

Basti

[Ben]

Zitat von Basti

Übrigens enthält $_POST immer nur Strings oder Arrays von Strings (...).

Und, ich empfehle immer die typensicheren Vergleichsoperatoren zu benutzen. Lieber einmal mehr explizit umwandeln, als nicht zu wissen, ob z.B. der Zahlenwert einer Variable als string, integer, float oder gar boolean gespeichert ist!

Gerade beim Vergleich auf 1 oder 0 (was ja auch boolean-Darstellungen sind) kann es da mal fix zu Problemen führen.

[Chr!s]

Zitat von Ben

Gerade beim Vergleich auf 1 oder 0 (was ja auch boolean-Darstellungen sind) kann es da mal fix zu Problemen führen.

.. Darum sollte man vielleicht mal über die Verwendung von Variablen-Prefixen nachdenken, so wie ich es tu. Also b für boolean, s für string, i für int usw..

Bei mir im Framework gibt es dann eine Klasse genannt PostWrapper, die auf Wunsch mit ::getPostData() ein gesäubertes ARray zurückgibt, in der alle Post-Werte nach Prefix aufgeschlüsselt worden, sprich ein bReceiveNewsletter ist nicht "0" oder "1", sondern durch die Klasse entweder true oder false, Zahlen werden ins Format int bzw float konvertiert.

In meinen Augen sehr praktisch und vermeidet Fehler.

[Ben]

Ja, jetzt kommt hier wieder die "Präfix-Fraktion" .. *gg*

Mach einen entsprechenden Thread auf, wenn du darüber diskutieren willst!