Forum PHP.pl > [PHP] Funkcja warunkowa

Pomoc - Szukaj - Użytkownicy - Kalendarz

sebekzosw

31.08.2009, 08:07:18

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

nospor

31.08.2009, 08:15:06

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

vokiel

31.08.2009, 08:26:12

Ten sposób nie zadziała

Cytat(sebekzosw @ 31.08.2009, 09:07:18 )

[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

31.08.2009, 08:28:54

@vokiel to sprawdź a nie się błednie "raczujesz"

sebekzosw

31.08.2009, 08:32:31

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

nospor

31.08.2009, 08:35:20

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

vokiel

31.08.2009, 13:10:22

Cytat(nospor @ 31.08.2009, 09:28:54 )

@vokiel to sprawdź a nie się błednie "raczujesz"

Rzeczywiście, racja, działa

.
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

1.09.2009, 06:05:13

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

1.09.2009, 20:43:26

zbędne... chyba, że warunek jest złożony

nospor

1.09.2009, 20:51:15

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

1.09.2009, 22:33:03

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

2.09.2009, 06:08:53

tez nie musisz tego brac w nawiasy

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

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

To jest wersja lo-fi głównej zawartości. Aby zobaczyć pełną wersję z większą zawartością, obrazkami i formatowaniem proszę kliknij tutaj.