Witam,
mam nadzieję, że temat wątku mówi to co chcę. Czy poniższy skrypt jest poprawny, tzn. działa, ale czy jest sposób, aby zrobić to krócej, wydajniej. Chodzi mi o radę, gdyż dopiero się uczę i chciałbym wyrobić sobie dobre nawyki.

[PHP] pobierz, plaintext 
 
if(!isset($_GET['newsy']) && $_GET['newsy'] != 'otworz' && !isset($_GET['id']))
{
 
	echo '<a href="index.php?newsy=otworz&id=' . $wiersze['id'] . '">' . $wiersze['tytul'] . '</a>';
}
 
elseif(isset($_GET['newsy']) && $_GET['newsy'] == 'otworz' && isset($_GET['id']))
{
 
}
[PHP] pobierz, plaintext 

Da się krócej/poprawniej napisać warunek w if()?

[PHP] pobierz, plaintext 
f(!isset($_GET['newsy']) && $_GET['newsy'] != 'otworz' && !isset($_GET['id']))
[PHP] pobierz, plaintext 

skoro $_GET['newsy '] nie istnieje to dlaczego dalej sprawdza wartość dla klucza 'newsy' ?

Pamiętaj... Jeśli używasz && to każdy kolejny warunek jest sprawdzany gdy poprzednie dają TRUE. Tylko taki układ daje optymalność. If bowiem czeka na true lub false a logiczne AND ma prawdę tylko gdy wszystkie części warunku zwracają prawdę. Wystarczy, że jeden po drodze da false a if przeskoczy do else ( lub do sprawdzanie ifelse jeśli go posiadasz ). Stąd bardzo sensowne jest

[PHP] pobierz, plaintext 
isset($_GET['newsy']) && $_GET['newsy'] == 'otworz' && isset($_GET['id'])
[PHP] pobierz, plaintext 

bo idzie to łańcuchem. Ja bym nawet już zrobił na tym etapie na końcu jeszcze walidację, jeśli id jest liczbą dodatnią w stylu:

[PHP] pobierz, plaintext 
ctype_digit( $_GET['id'] ) && $_GET['id'] > 0
[PHP] pobierz, plaintext 

Ale to co masz w if to pomyłka... Skoro $_GET['newsy'] nawet nie istnieją, to po co jeszcze sprawdzać je? Jeśli akurat nie będzie ich, to skrypt przejdzie dalej i wywali warning, że porównujesz do nie istniejącej zmiennej (utworzy ją w locie i ustawi na ciąg pusty -> '') i tak czy siak wywali false, czyli zakończy sprawdzanie warunku w if, przechodząc do ifelse. Logika matematyczna się kłania.