Kommentare zu: Kürzer als Kurz: Der Ternär Operator für Geizige

Von: Patrik

Patrik — Wed, 11 Mar 2015 08:00:54 +0000

nun ja… schlecht lesbar ist der Code wohl nur, wenn die Schreibweise (a<4)?true:false; oder ähnlich nicht bekannt sind. Bei vermehrter Verwendung, lernt man dies aber schnell und wird dies dann auch immer problemlos verstehen.

Von: Andreas

Andreas — Wed, 17 Sep 2014 10:10:28 +0000

Moin,

diese Schreibweise bringt durchaus Vorteile und kann bei mehreren Zuweisungen nacheinander die Übersichtlichkeit durchaus erhöhen. Man muss zuvor natürlich verstehen wie es funktioniert sonst bleibt es unübersichtlich.

Der Aussage das kurzer code gleich schlecher code ist würde ich mich pauschal nicht anschließen.

Gruß
Andreas

Von: Cognac

Cognac — Mon, 19 Dec 2011 13:08:08 +0000

Moin,

ich bin mittlerweile ein Fan dieser Kurzschreibweise, insbesondere wenn es um Variablenprüfung geht:

$var = empty (‘var’, $_POST) ? true : false;

Oder

empty (‘var’, $_POST) ? $var = $_POST['var'] : “”;

Rein inhaltlich geht es hier für mich nicht um Fallunterscheidungen sondern um eine Prüfung und da ist der kurze Code schön übersichtlich…

Von: Daniel

Daniel — Thu, 16 Sep 2010 22:10:20 +0000

Ich finde es immer sehr interessant, wenn viele verschiedene Meinungen aufeinander treffen. Jeder hat andere Erfahrungen und eine andere Vorstellung von gutem Code.

Von: nik

nik — Thu, 16 Sep 2010 19:49:29 +0000

@Fabian: Bin ja auch Stamm-PHPler. Mag aber trotzdem sprechenden Code.

Von: Manuel Grundner

Manuel Grundner — Thu, 16 Sep 2010 19:15:18 +0000

Fabian muss ich solange recht geben solange es in einer Methode gekapselt ist:

public function GetComparator()
{
return $this->comparator ? new ReverseComparator($this->getComparator()) : $this->getComparator();
}

macht schon Sinn, wobei jedoch ein Backstore mit:

public function GetComparator()
{
if($this->comparator === null)
$this->comparator = new ReverseComparator($this->getComparator());
return $this->comparator;
}

auch lesbar ist. man sollte ja die Object-Erzeugung von Objekt-Gebrauch entkapseln.
Somit ist der Ternäre Operator in 99.9% der fällen ein klarer WFT.

Hilfreich ist er in Helper methoden, oder in vielen fällen von Optionalen Object Parametern.

public function __construct(MyObject $obj = null)
{
$this->myObj = $obj new MyObject();
}

hierbei muss man jedoch 2 sachen beachten:
1: ClassHint sollte gegen InterfaceHint getauscht werden
2: Objekte soll man nicht im Ctor erzeugen.
(3: Man sollte sowieso nichts im Ctor machen/erzeugen/…, außer übergabeparameter zu speichern, und gegen invalid arguments prüfen)

so genug senf ;p

Von: Fabian

Fabian — Tue, 14 Sep 2010 16:50:13 +0000

@Patrick: Das ist wohl Geschmackssache, allerdings ist das Beispiel auch etwas unglücklich, für 0 und 1 würde ich eigentlich auch keine neuen Zeilen anfangen, das sollte sich anders lösen lassen. Aber hier mal ein konkreter Fall aus meinem aktuellen Code (übrigens inspiriert vom Java Interfaces Artikel in diesem Blog )
$comparator = $this->reverse ? new ReverseComparator($this->getComparator()) // decorate : $this->getComparator();

@nik: Das ist wohl eine PHP-Krankheit, ich sage nur “or die()”. Verlockend ist es aber schon manchmal