[PHP] Funkcja warunkowa, Który sposób jest w miarę lepszy? Opcje

[sebekzosw]

Który sposób na sprawdzanie danych jest szybszy:

[PHP] pobierz, plaintext 
<?php
    $LOL = 4;
 
    $wynik = ($LOL == 5 ? 'Prawda<br />' : 'Fałsz<br />');
    echo $wynik;
?>
[PHP] pobierz, plaintext 

czy

[PHP] pobierz, plaintext 
<?php 
    $LOL = 4;   
    if($LOL == 5) {
        $wynik2 = "Prawda<br />";
    } else {
        $wynik2 = "Fałsz<br />";
    }
    echo $wynik2;
?>
[PHP] pobierz, plaintext 

I które sprawdzanie będzie w miarę dobre dla moich potrzeb:

[PHP] pobierz, plaintext 
<?php
        if(empty($_POST["login"])) { $registration_error .= "Wpisz login<br />"; } else { $registration_error .= SprawdzLogin($_POST["login"]); }
        if(empty($_POST["name"])) { $registration_error .= "Nie podałeś imienia i nazwiska!<br />"; }
        if(empty($_POST["mail1"])) { $registration_error .= "Wpisz Adres E-Mail!<br />"; }
        if(empty($_POST["mail2"])) { $registration_error .= "Powtórz adres E-Mail<br />"; }
?>
[PHP] pobierz, plaintext 

Ten post edytował sebekzosw 31.08.2009, 08:07:54

[nospor]

a nie mozesz zrobic petli i porownacz czasy wykonania? Naprawde trzeba z tym na forum leciec?

[vokiel]

Ten sposób nie zadziała

[PHP] pobierz, plaintext 
<?php
    $LOL = 4;
    $wynik = ($LOL == 5 ? 'Prawda<br />' : 'Fałsz<br />');
    echo $wynik;
?>
[PHP] pobierz, plaintext 

Raczej tak:

[PHP] pobierz, plaintext 
<?php
    $LOL = 4;
    $wynik = ($LOL == 5) ? 'Prawda<br />' : 'Fałsz<br />';
    echo $wynik;
?>
[PHP] pobierz, plaintext 

[nospor]

@vokiel to sprawdź a nie się błednie "raczujesz" (IMG:style_emoticons/default/winksmiley.jpg)

[sebekzosw]

Niby pierwszy sposób jest szybszy, ale czy warto to używać? Jakie jest wasze zdanie?

[nospor]

a dlaczego ma nie byc warto? Przeciez jest to krotszy zapis. Ja go czesto stosuje wlasnie przy tak prostych warunkach.

[vokiel]

@vokiel to sprawdź a nie się błednie "raczujesz" (IMG:style_emoticons/default/winksmiley.jpg)

Rzeczywiście, racja, działa (IMG:style_emoticons/default/sciana.gif) .
IMHO taki zapis jest nielogiczny (może czegoś nie rozumiem). Skrót ten ma zastepować użycie if (){}else{}. Sprawdzanie warunku w if występuje w zwykłym nawiasie, tak samo, wg mnie tutaj.

Co daje włożenie całości w nawias?

Wracając do wyboru. Ja osobiście stosuję zamiennie, jednak w prostych przypadkach (zwykłego przypisania wartości do zmiennych) preferuję krótszy zapis.

[nospor]

konstrukcja ta ma inną składnię niż IF. tutaj ten nawias jest wogole zbędny. Kod powinien wyglądac tak:

[PHP] pobierz, plaintext 
  $wynik = $LOL == 5 ? 'Prawda<br />' : 'Fałsz<br />';
[PHP] pobierz, plaintext 

[sebekzosw]

zbędne... chyba, że warunek jest złożony (IMG:style_emoticons/default/smile.gif)

[nospor]

co masz na mysli mowiac zlozony? Bo nadal jest zbedny. Co najwyzej moze byc przydatny dla lepszej czytelnosci, choć nie wiem w czym ma to pomoc objecie wszystkiego w nawias jak ty to zrobiles

[sebekzosw]

np takie coś (złożone)

[PHP] pobierz, plaintext 
<?php 
$wynik = (($LOL == 5) AND ($LOL != 0)) ? 'Prawda<br />' : 'Fałsz<br />';
?>
[PHP] pobierz, plaintext 

[nospor]

tez nie musisz tego brac w nawiasy (IMG:style_emoticons/default/smile.gif) A juz szczegolnie w podwoje jak ty to zrobiles.
Ale jak to ma sie do tego co na początku zrobiles:

[PHP] pobierz, plaintext 
$wynik = ($LOL == 5 AND $LOL != 0 ? 'Prawda<br />' : 'Fałsz<br />');
[PHP] pobierz, plaintext 

?
Przeciez o tym rozmawialismy (IMG:style_emoticons/default/tongue.gif)

Podsumowując: nawiasy tu są potrzebne tylko i wyłącznie dla czytelności kodu.