php-syntax für die if-Anweisung mit mehreren Bedingungen ( `||` als auch als `&&` )

Kontext: Auf unserer website, berechnen wir, ob ein Element/Auftrag erfüllt die Kriterien für kostenlosen Versand mit einer if-Anweisung auswerten, wenn eine Wert - 'ff' - wahr ist. Je nach den Bedingungen erfüllt, legt die Versandkosten entsprechend. Wir benötigen nun die Notwendigkeit zu bewerten, wenn ein Artikel versandkostenfrei ist, mit der Bestellung von _. Um das ganze abzurunden, für andere Zwecke, z.B. externe feeds, eine andere Bedingung muss gelten (Gewicht = 0 oder kein Wert), für fw frei Fracht. Hier ist der code, dass ich kann nicht herausfinden, warum es NICHT funktioniert:

if($r['freeFreight'] == 'ff' || ($r['freeFreight'] == 'fw' && ($r['weight'] == 0 || $r['weight'] == '') ) ) {
    $r['shippingStandard'] = 0;
}

Sind die Bedingungen übermäßig getrennt in der if-Anweisung mit den Sätzen von ()? Oder sind Sie einfach nur falsch platziert. Zum Beispiel sollte es sein:

if(($r['freeFreight'] == 'ff') || ($r['freeFreight'] == 'fw' && $r['weight'] == 0 || $r['weight'] == '') ) {
    $r['shippingStandard'] = 0;                     
}

Den zweiten der beiden scheint mehr logisch für mich, weil: wenn ($r['freeFreight'] == 'ff') - dann unabhängig von den folgenden Bedingungen sollte die Aussage true zurück, und setzen Sie die variable auf 0. Ist meine Logik richtig? Sorry für diese Anfänger Frage...

Vielen Dank für jede Hilfe, die Sie anbieten können.

Also ich denke vielleicht, basiert auf den Antworten so weit (Dank an alle, hat in mischte, um zu helfen amateur) - ich denke, ich werde tun, ein Versuch mit:

if( ($r['freeFreight'] == 'ff') || ( $r['freeFreight'] == 'fw' ) && empty( $r['weight'] ) ) { 
    $r['shippingStandard'] = 0;
}

Planung bis zur Ausführung-Studie mit dieser, wenn es grundsätzlich falsch ist, bitte beraten.

Sind Sie der überprüfung isset($r['weight']) ?
Ich bin nicht.. smh - so wäre es, als wenn(($r['freeFreight'] == 'ff') || (isset($r['Gewicht'] && $r['freeFreight'] == 'fw' && $r['Gewicht'] == 0 || $r['Gewicht'] == ") ) { $r['shippingStandard'] = 0; }
emtpy würde sauberer sein, als wenn alle diese Prüfungen
Das sieht OK für mich.
versuchen Sie, verlassen Sie sich nie auf eine Sprache, die operator-precedence-Regeln, wenn Sie überhaupt helfen, und vor allem, wenn Sie arbeiten mit mehreren Sprachen. es kann ein bisschen langweilig, aber immer die richtigen Belichtungsreihe durchzusetzen, die "Ihre" Regeln, anstatt zu hoffen, die Sprache zu analysieren, werden es richtig für Sie.
guter Tipp. JamesLee: ich gehe mit Ascherer Antwort.
Ich Stimme zu, guter Tipp. Es bietet die Gewissheit, dass ich bin nicht über-bracketing (aber wahrscheinlicher, dass ich mich einfach nicht bracketing richtig, lol)
Vereinbart, ich bin nur Feintuning es leicht zu Klammer die erste Bedingung, so wird es bewertet auf seinen eigenen Bezug, ohne den rest der Anweisung.
Sie nicht brauchen, um setzen Sie Klammern um die ff-Stück. Sein gehen, um gesehen zu werden als der erste Betreiber, und wenn seine wahre, das wenn der wird am Ende Recht gibt
Sie wurden sehr, sehr hilfreich. Ich werde mit genau das, was Sie empfohlen, Ihre Antwort

InformationsquelleAutor James Lee | 2013-05-17

2

Ausprobieren?
```
if( $r['freeFreight'] == 'ff' || ( $r[ 'freeFreight' ] == 'fw' && empty( $r['weight'] ) ) ) {
    $r['shippingStandard'] = 0;
}
```
mit empty könnte ein wenig sauberer zu

wenn Sie wirklich wollen, um Sie zu brechen mehr, Sie könnten dies tun
```
if( $r[ 'freeFreight' ] == 'ff' ) {
    $r[ 'shippingStandard' ] = 0;
} elseif( $r[ 'freeFreight' ] == 'fw' && empty( $r[ 'weight' ] ) ) {
    $r[ 'shippingStandard' ] = 0;
} else {
    //Everything else
}
```
Wenn Sie möchten, verkürzen Sie die Variablen, und verwenden Sie Sie später:
```
$freight  = $r[ 'freeFreight' ];
$weight   = isset( $r[ 'weight' ] ) ? $r[ 'weight' ] : null;
$shipping = $r[ 'shippingStandard' ]; //Or whatever you want the default value to be...
if( $freight == 'ff' || ( $freight == 'fw' && empty( $weight ) ) ) {
    $shipping = 0;
}

//... Later down the file
$r = array( 
    'freeFreight' => $freight,
    'weight' => $weight, 
    'shippingStandard' => $shipping
);
```
Ich wirklich nicht sagen, wie sich der rest der Datei sieht aus, wie wenn dies in eine Funktion zu bekommen, Versand, Sie konnte einfach nur zurück $shipping. Hoffe, das hilft. Sie sollten in der Lage sein zu verschieben die Konzepte rund um das zu bekommen, was Sie wollen
- wenn im Lesen der Anforderungen falsch, lass es mich wissen
- Das problem ist, wenn $r['freeFreight'] == 'ff' wir kümmern uns nicht, ob $r['Gewicht'] = null, ", 0, oder 9081239082. Sollte dies nur getan werden, mit einer if-else-Anweisung?
- Ich denke, das ist, ich werde die leere ein Schuss, aber Klammer aus der ersten Bedingung. Es kann genau das, was wir suchen.
- ja, das habe ich behoben, das war es also was Sie wollte. Man könnte es sauber mehr durch die Zuordnung $r['freeFreight'] zu $freight, so passt er besser auf der Linie, idk, wenn Sie verwenden Sie die variable später in der Funktion
- Hinzugefügt noch ein edit, @JamesLee wenn Sie wollte, es zu brechen so
- versuchen Sie verlassen Kommentare eher als die Bearbeitung der Antwort, diese wird einfach abgelehnt werden als ungültig, Versuch, Kommentar oder Antwort"
- Danke. Tut mir Leid, dass ich nicht sicher war, dass etwa. Dies ist meine erste Zeit eigentlich posting etwas auf stackoverflow.com - obwohl ich habe recherchiert viele Dinge hier, und es ist eine großartige Ressource zu haben. Unser Chefentwickler ist nicht mehr mit der Firma, und da ich geringe Kenntnisse von PHP, ich habe irgendwie angenommen, dass der interim Rolle. (Eigentlich bin ich im marketing, aber ich habe früher sehr viel schreiben von js/jquery für diese Website. Ich werde versuchen, die richtige Etikette mit Bezug auf Kommentare/änderungen/posts und so weiter. Danke nochmal für den Hinweis mich in die richtige Richtung.
- Sie können prüfen wollen, die FAQ zu.
InformationsquelleAutor Ascherer
-1

Ihre erste option sollte identisch sein mit der folgenden:
```
if ($r['freeFreight'] == 'ff') {
    $r['shippingStandard'] = 0;
}
elseif ($r['freeFreight'] == 'fw') {
    //the dual check for an empty value here is kind of redundant and unneccessary.
    //I'd probably just use "if ($r['weight'])" and be done with it
    if ($r['weight'] == 0) {
        $r['shippingStandard'] = 0;
    }
    elseif ($r['weight'] == '') {
        $r['shippingStandard'] = 0;
    }
    else {
        //$r['shippingStandard'] stays whatever it was before
    }
}
else {
    //$r['shippingStandard'] stays whatever it was before
}
```
wenn diese Logik richtig aussieht für Sie, dann sollten Sie richtig sein und ich würde print_r($r) um sicherzustellen, dass Sie halten, was Sie erwarten, es zu halten. Ihre Klammer an das erste Beispiel ist genau so wie ich es tun würde.
- if( $r[ 'weight' ] ) ist ein sehr fauler überprüfen, und lassen nicht viel zu sagen, was Sie für die überprüfung. BITTE verwenden Sie so etwas wie empty zu sagen, was genau Sie suchen. Nur weil PHP können Sie hängen selbst mit code, es bedeutet nicht, sollten Sie. if( empty( $r[ 'weight' ] ) ) Prüfungen für null -, set -, 0, und -"
- Genial! Mir war nicht bewusst empty überprüft für 0, das ist toll! Kann ich knock off-der Letzte Teil meiner if-Anweisung habe ich nur noch auf meine OP.
- Normalerweise bin ich auf der "explizite Intentionen" auf den fahrenden Zug aufspringen, und ich könnte kommen, um auf empty() schließlich aber alle es tut, ist, die zu true ausgewertet, wenn die gelieferte variable implizit zu false ausgewertet wird... ich versuche, Sie zu beseitigen Reste, wo ich kann, und ich habe nicht bekommen, über die crufty Gefühl empty(). Was es macht ist, dass ich keine Sorge um die Existenz der variable, die ich fühle ich sollte es wissen, oder mehr zu überprüfen, die explizit mit isset(). Auf jeden Fall, weiß nur, es ist ein Stil Wahl habe ich bisher gemacht mit Absicht.
InformationsquelleAutor Jason

Schreibe einen Kommentar

Du musst angemeldet sein, um einen Kommentar abzugeben.