Witajcie

Pracuje nad pewnym kodem i na razie wyrzucalem sobie exity tam gdzie byly bledy lub cos takiego np:

[PHP] pobierz, plaintext 
$this->x = (1-1) ? $x : exit('error');
[PHP] pobierz, plaintext 

ale jednak przestalo mi to wystarczac i zreszta predzej czy pozniej i tak bym musial wiec przeszedlem sobie na wyjatki, no i problem w tym ze nie moge uzyc konstrukcji

[PHP] pobierz, plaintext 
$this->x = (1-1) ? $x : throw new Exception('err');
[PHP] pobierz, plaintext 

gdyz nie poprawna skladnia, wiec obszedlem to w taki sposob, ze stworzylem metode throwException() ktora wyrzuca mi Exceptiona a ja robie tak:

[PHP] pobierz, plaintext 
$this->x = (1-1) ? $x : $this->throwException('err');
[PHP] pobierz, plaintext 

Dziala tak jak ma dzialac, tyle ze stack jest o poziom wiecej z racji extra metody, ale tu jest moje pytanie, czy jest to poprawne czy nie, dziekuje

[PHP] pobierz, plaintext 
     if(1==1)
        $this->x = 1;
    else
        throw new Exceptrion();
[PHP] pobierz, plaintext 

Witaj, dzieki za zainteresowanie, ale wiem ze tak mozna, moje pytanie brzmialo czy to co ja zrobilem jest poprawne?

Korzystam ze skroconego ifa bardzo czesto i jest jak dla mnie o wiele lepszy - kod wyglada o wiele lepiej i smuklej niz z pelna konstrukcja
Poza tym to chyba logiczne ze to byl przyklad dokladnie mam takie cos:

[PHP] pobierz, plaintext 
$new = (1+1) 
                ? $a
                : ((1+1) 
                    ? $x
                    : $this->throwException('err');
        }
[PHP] pobierz, plaintext 

Te ify sa czesto wielo poziomowe i nie chcialbym z nich rezygnowac, gdyz lepiej mi sie je czyta, oraz kod metody jest krotszy,
wiec ponawiam pytanie, czy to jest poprawnie (wiem ze moglbym uzyc pelnej konstrukcji if), czy tez moze nie.

pozdrawiam

Najwidoczniej się nie da

Jeżeli uważasz że to się lepiej czyta to gratuluję.

A czy poprawne? Nie do końca, bo oczekujesz zwrotu z funkcji a tymczasem dostajesz exception

Poprawne, jeżeli chodzi o działanie. Bo przecież działa.

Nigdy jednak nie widziałem, by ktoś używał tego operatora, żeby w pełni zastąpić ify i elsy. Podejrzewam, że znaczna większość innych programistów nie będzie chciała się przyzwyczaić do Twojego stylu pisania. Dodatkowo jak sam zauważyłeś, ta składnia jest "upośledzona" ze względu na to, że musisz podawać w niej tylko wyrażenia (http://php.net/expressions).


To, co się lepiej czyta to subiektywna opinia. Choć zgadzam się z tym, że jest to okropnie nieczytelne, to jednak rozumiem, że są osoby, które mogą chcieć tak pisać. To tak jak dyskusja z 2 spacje vs 4 spacje vs taby.

otoz to. (oprocz tego ze okropnie nieczytelne )

Rozumiem juz, nie chodzi ze chce, a raczej chcialem zastapic tego wszedzie, ale mam takiego dlugiego ifa z kilkoma elsifami (lol) i wygodnie bylo miec to zapisane w wlasnie taki sposob a na samym koncu mialem exit(),
jak to zmienilem na

[PHP] pobierz, plaintext 
if () {
} elseif () {
}
[PHP] pobierz, plaintext 

to zrobilo sie po prostu 2x wiecej linijek, wiec stad bylo pytanie czy mozna tak to rozwiazac, jak widac, nie.
Dzieki i pozdrawiam.

edit:
O i jeszcze to: + tej skroconej skladni jest taki, ze jak cos przypisujemy do zmiennej to jednak fajnie to napisac raz

[PHP] pobierz, plaintext 
 $x = () ? true : false;
[PHP] pobierz, plaintext 

niz, klepac to kilka razy:

[PHP] pobierz, plaintext 
if () {
$x = 1;
} else { 
$x = 0
}
[PHP] pobierz, plaintext 

To byly glowne przyczyny, pozdrawiam

Po prostu w skróconym ifie, oczekuje się, że podasz konkretną wartosc ktora na podstawie warunku zostanie przypisana, wiec nie mozesz tam wywołać void'a

Wyjątek nie zwraca żadnej wartości więc nawet próba przypisania do $this->x można powiedzieć, zostawiła by ten kod w jakimś nieokreślonym stanie. Chyba w żadnym języku to nie zadziała.

Teoretycznie mozesz, bo dziala - ale wiem juz, jest to zle.

Pozdrawiam

Jeśli wziąć pod uwagę kod np

[PHP] pobierz, plaintext 
//gdzieś tam $this->x =1
 
try {
    $this->x = (1-1) ? $x : throw new Exception('err');
} catch (Exception $e) {
    var_dump($this->x);
    //co powinno się tu wyświetlić? 1, null, false, coś innego
}
[PHP] pobierz, plaintext