Kurze IF-Form

Horatio · 08.11.2011, 21:35

Hallo! Ich habe eine kleine Frage:
Warum funktioniert dieses Ausgeschriebene IF:

PHP-Code:

  if ($tab == "intro")

    $progress1 = $active;

else

{

    if ($progress > 1)

        $progress1 = $completed;

    else

        $progress1 = "";

}

aber diese Kurze IF-Form nicht:

PHP-Code:

  $progress1 = ($tab == "intro") ? $active : ($progress > 1) ? $completed : "";

Was mache ich an der kurzen falsch?

ChrisB · 08.11.2011, 21:49

Zitat:

Zitat von Horatio

Was mache ich an der kurzen falsch?

Du beachtest Operator-Rangfolge und -Assoziativität nicht.
http://www.php.net/manual/en/languag...precedence.php

Horatio · 08.11.2011, 22:09

So, ich habe mir das jetzt durchgelsen. Ich verstehe allerdings kaum etwas davon. Kann mir das jemand mal bitte erklären?

fab · 08.11.2011, 22:12

Hast du denn gesehen dass genau die Verkettung von ?: dort als Beispiel auftaucht? Achte dabei auf die Klammern, da hast du die Links-Assoziativität von ?: direkt für deinen Fall anschaulich gemacht.

Die Lösung liegt auf der Hand: Selber Klammern setzen.

Horatio · 08.11.2011, 22:19

Ist das so richtig:

PHP-Code:

  ($tab == "intro") ? ($progress > 1) ? $completed : "" : $active

ChrisB · 08.11.2011, 22:27

Nein, das ist totaler Nonsense.

Chriz · 08.11.2011, 23:34

a ? b : (c ? d : e)

mquadrat · 09.11.2011, 09:39

Und nach meiner bescheidenen Meinung ist das Verwenden von verschachtelten ternären Operatoren tödlich für die Wartung.

jspit · 09.11.2011, 09:57

..lag mir auch auf der Zunge. Wenn dann der/die Ersteller noch Probleme mit diesen Operatoren haben, sollten Sie erst recht beim if else bleiben.

tr0y · 09.11.2011, 11:49

Ich persönlich würde ternary operators auch nicht bis ins Wartungsnirvana nutzen, sondern nur soweit es die Lesbarkeit duldet. Gerne z.b. Nutz ich ternary operators um Daten aus dem Web vorzubereiten und "interne" Variablen damit zu befüllen:

Initialisierend...

PHP-Code:

  $page = isset($_GET['page']) ? $_GET['page'] : false;

Auch komplexer...

PHP-Code:

 
<!DOCTYPE html !>
<html lang="de">
<head>
    <title>ternary test page for forms</title>
    <meta charset="utf-8" />
</head>
<body>

<div id="status">
<?php

    $requireFields = array('a','e');
    
    // transport POST-Data
    
    $formData['a'] = isset($_POST['a']) ? $_POST['a'] : null;
    $formData['b'] = isset($_POST['b']) ? $_POST['b'] : null;
    $formData['c'] = isset($_POST['c']) ? $_POST['c'] : null;
    $formData['d'] = isset($_POST['d']) ? $_POST['d'] : null;
    $formData['e'] = isset($_POST['e']) ? $_POST['e'] : null;
    
    // identify form
    
    $formData = count( 
        array_intersect_key( 
            array_flip(array_keys($formData, null)), 
            array_flip($requireFields) 
        ) 
    ) == 0 ? $formData : false;
    
    // go
    
    if ( $formData ) {
    
        echo "Das Formular wurde ausreichend ausgefüllt, vielen Dank für Ihre Daten.";
    
    }
    
?></div>
<div>
<form method="post" action="">
    <input type="text" name="a" />
    <input type="text" name="b" />
    <input type="text" name="c" />
    <input type="text" name="d" />
    <input type="text" name="e" />
    <input type="submit" name="sender" />
</form>
</div>
</body>
</html>

Aber da hats irgendwo seine grenzen der Lesbarkeit.

Vorteilhaft, weil sehr kurz definiert sind auch Instanziierungsmethoden des Singleton-Patterns mit ternary operators:

PHP-Code:

  class SingletonObject {

   private static $_instance;

   protected function __construct() {}
   protected function __clone() {}

   public static function getInstance() {
      return is_null(self::$_instance) ? self::$_instance = new self : self::$_instance;
   }

}

anstelle von:

PHP-Code:

  class SingletonObject {

   private static $_instance;

   protected function __construct() {}
   protected function __clone() {}

   public static function getInstance() {
      if ( self::$_instance === NULL ) {
         self::$_instance = new self;
      }
      return self::$_instance;
   }

}

Themen-Optionen
Druckbare Version zeigen Jemanden per E-Mail auf dieses Thema hinweisen
Thema bewerten
Thema bewerten:

Ähnliche Themen
Thema	Autor	Forum	Antworten	Letzter Beitrag
Problem mit form action	arreis	JavaScript, Ajax und mehr	5	19.07.2011 19:23
Kontaktformular Problem ....	root-2k10	PHP Einsteiger	5	16.03.2011 08:15
Dynamisches laden Ajax + Form verschicken	starchild	JavaScript, Ajax und mehr	10	18.02.2011 22:45
mit form buttom unterschiedliche variabeln übergeben	kyle2010	PHP Tipps 2010	7	31.03.2010 16:36
POST form self PHP und direkt form Daten in URL übergeben	Rutor	PHP Tipps 2010	0	20.02.2010 17:56
[Erledigt] mauscursor ändern wenn man über link fährt	Philipp6020	PHP Tipps 2009	17	07.01.2010 15:14
[Erledigt] Werte aus Textfeld an form action übergeben	KevinS	PHP Tipps 2009	8	09.03.2009 14:31
[Erledigt] Form Action	turntab	HTML, Usability und Barrierefreiheit	2	21.01.2009 13:49
[Erledigt] form speichert nicht alle felder auf einmal	taurus	Datenbanken	4	05.11.2008 19:51
[Erledigt] form in db schreiben...	taurus	PHP Tipps 2008	13	02.11.2008 12:01
Mail Form an verschiedene Empfänger	donjoh	PHP Tipps 2008	4	18.06.2008 15:38
ein form mit 2 versch. button - actions		PHP Tipps 2005-2	3	27.08.2005 21:26
Daten in HTML form aus einer Query anzeigen		PHP Tipps 2005	1	15.02.2005 21:03
Greenhorn: Reload einer IMG-Datei nach SUBMIT einer Form		PHP Tipps 2004	3	20.09.2004 21:19
Mail bei Form Abschicken.	JEGO	PHP Tipps 2004	9	10.06.2004 17:18

08.11.2011, 21:35
Horatio Neuer Benutzer Registriert seit: 20.10.2011 Beiträge: 10 PHP-Kenntnisse: Anfänger	Kurze IF-Form Hallo! Ich habe eine kleine Frage: Warum funktioniert dieses Ausgeschriebene IF: PHP-Code: `if ($tab == "intro") $progress1 = $active; else { if ($progress > 1) $progress1 = $completed; else $progress1 = ""; }` aber diese Kurze IF-Form nicht: PHP-Code: `$progress1 = ($tab == "intro") ? $active : ($progress > 1) ? $completed : "";` Was mache ich an der kurzen falsch?


Mister Ad PHP Code Flüsterer Registriert seit: 21.08.2005 Beiträge: 4682 PHP-Kenntnisse: Fortgeschritten

08.11.2011, 22:09
Horatio Neuer Benutzer Registriert seit: 20.10.2011 Beiträge: 10 PHP-Kenntnisse: Anfänger	So, ich habe mir das jetzt durchgelsen. Ich verstehe allerdings kaum etwas davon. Kann mir das jemand mal bitte erklären?

08.11.2011, 22:12
fab Erfahrener Benutzer Registriert seit: 28.07.2010 Beiträge: 2.308 PHP-Kenntnisse: Fortgeschritten	Hast du denn gesehen dass genau die Verkettung von ?: dort als Beispiel auftaucht? Achte dabei auf die Klammern, da hast du die Links-Assoziativität von ?: direkt für deinen Fall anschaulich gemacht. Die Lösung liegt auf der Hand: Selber Klammern setzen. __________________ PHP Blog + Library CSS Effects + Tools

08.11.2011, 22:19
Horatio Neuer Benutzer Registriert seit: 20.10.2011 Beiträge: 10 PHP-Kenntnisse: Anfänger	Ist das so richtig: PHP-Code: `($tab == "intro") ? ($progress > 1) ? $completed : "" : $active`

08.11.2011, 22:27
ChrisB Moderator¹ Registriert seit: 28.03.2010 Beiträge: 7.470 PHP-Kenntnisse: Fortgeschritten	Nein, das ist totaler Nonsense. __________________ RGB is totally confusing - I mean, at least #C0FFEE should be brown, right?

08.11.2011, 23:34
Chriz Moderator Registriert seit: 11.05.2008 Beiträge: 6.267	a ? b : (c ? d : e) __________________ "Nuschel ich?" - "Was?"

09.11.2011, 09:39
mquadrat Erfahrener Benutzer Registriert seit: 02.09.2009 Beiträge: 1.019 PHP-Kenntnisse: Fortgeschritten	Und nach meiner bescheidenen Meinung ist das Verwenden von verschachtelten ternären Operatoren tödlich für die Wartung. __________________ Wir suchen PHP Entwickler (Vollzeit) im Raum Darmstadt / Rhein-Main. Infos via E-Mail mueller@new-frontiers.de

09.11.2011, 09:57
jspit Erfahrener Benutzer Registriert seit: 19.08.2011 Beiträge: 701 PHP-Kenntnisse: Anfänger	..lag mir auch auf der Zunge. Wenn dann der/die Ersteller noch Probleme mit diesen Operatoren haben, sollten Sie erst recht beim if else bleiben.

09.11.2011, 11:49
tr0y Erfahrener Benutzer Registriert seit: 26.07.2010 Beiträge: 4.874 PHP-Kenntnisse: Fortgeschritten	Ich persönlich würde ternary operators auch nicht bis ins Wartungsnirvana nutzen, sondern nur soweit es die Lesbarkeit duldet. Gerne z.b. Nutz ich ternary operators um Daten aus dem Web vorzubereiten und "interne" Variablen damit zu befüllen: Initialisierend... PHP-Code: `$page = isset($_GET['page']) ? $_GET['page'] : false;` Auch komplexer... PHP-Code: <!DOCTYPE html !> <html lang="de"> <head> <title>ternary test page for forms</title> <meta charset="utf-8" /> </head> <body> <div id="status"> <?php $requireFields = array('a','e'); // transport POST-Data $formData['a'] = isset($_POST['a']) ? $_POST['a'] : null; $formData['b'] = isset($_POST['b']) ? $_POST['b'] : null; $formData['c'] = isset($_POST['c']) ? $_POST['c'] : null; $formData['d'] = isset($_POST['d']) ? $_POST['d'] : null; $formData['e'] = isset($_POST['e']) ? $_POST['e'] : null; // identify form $formData = count( array_intersect_key( array_flip(array_keys($formData, null)), array_flip($requireFields) ) ) == 0 ? $formData : false; // go if ( $formData ) { echo "Das Formular wurde ausreichend ausgefüllt, vielen Dank für Ihre Daten."; } ?></div> <div> <form method="post" action=""> <input type="text" name="a" /> <input type="text" name="b" /> <input type="text" name="c" /> <input type="text" name="d" /> <input type="text" name="e" /> <input type="submit" name="sender" /> </form> </div> </body> </html> Aber da hats irgendwo seine grenzen der Lesbarkeit. Vorteilhaft, weil sehr kurz definiert sind auch Instanziierungsmethoden des Singleton-Patterns mit ternary operators: PHP-Code: `class SingletonObject { private static $_instance; protected function __construct() {} protected function __clone() {} public static function getInstance() { return is_null(self::$_instance) ? self::$_instance = new self : self::$_instance; } }` anstelle von: PHP-Code: `class SingletonObject { private static $_instance; protected function __construct() {} protected function __clone() {} public static function getInstance() { if ( self::$_instance === NULL ) { self::$_instance = new self; } return self::$_instance; } }` __________________ Lasse mir ohne Anwendung von Gewalt Dinge schenken, Amazon weiß darüber bald mehr. Geändert von tr0y (09.11.2011 um 13:21 Uhr). Grund: singleton instancing procedures hinzugefügt.