Kurcze, wlasnie staram sie edytnac skrypt pewnego forum i napotkalem na pewien problem.
Otoz jest tam pare IFów bez klamer otwierajacych/zamykajacych instrukcje....jak mam rozumiec taki zapis?
Tzn. skad mam wiedziec, gdzie ten IF sie konczy?

Przyklad:

[PHP] pobierz, plaintext 
if ( strlen($uid) == BBCODE_UID_LEN )
		$text = str_replace( ':' . $uid, '', $text );
		//$text = preg_replace('#\[(.*?)\=\"(.*?)\"\](.*?)\[\/(.*?)\]#si', '[\\1=\\2]\\3[/\\4]', $text);
 
	$text = str_replace( array('\'', '"', '\\' ), array( ''', '"', '\' ), $text );
	$text = preg_replace('#\[(.*?)\="(.*?)"\](.*?)\[\/(.*?)\]#si', '[\\1=\\2]\\3[/\\4]', $text);
	preg_match_all('#\[/((.*?)(:[0-9a-f]+))\]#', $text, $arr);
	foreach( $arr[1] as $id => $value )
....
[PHP] pobierz, plaintext 

drugi przyklad:

[PHP] pobierz, plaintext 
function costam()
{
	if ( $is_sig || $postdata['user_id'] != $userdata['user_id'] )
	$text = preg_replace("#(\[|\&\#91;)((.(?<!\])(?<!\&\#93;))+?):[0-9]+(\]|\&\#93;)#i", '', $text);
	$text = preg_replace("#(\[|\&\#91;)((.(?<!\])(?<!\&\#93;))+?):[0-9]+(\]|\&\#93;)#i", '\\1\\2\\4', $text);
	return $text;
}
// return $text - nie wiem teraz czy ten return jest od funkcji (w ktorej jest ten if), czy od tego ifa....dodam, ze cala funkcja operuje na zmiennej $text, wiec te eregi nie sa wskazowka, ze ten return jest od ifa :)
[PHP] pobierz, plaintext 

Skopiowalem z oryginalnymi tabami, zeby bylo widac dokladnie o co chodzi.
No i teraz zagadka, ktore linijki naleza pod tego ifa, a ktore juz nie?

Ktos zalowal tych klamerek jak widac, przez co kod jest nieczytelny

Wykona się tylko pierwsza instrukcja jeżeli warunek jest spełniony

Taka konstrukcja sprawia, że if działa tylko do najbliższego średnika, czyli wykonuje 1 instrukcję występującą zaraz po sobie.

czyli ten drugi preg_match sie nie wykona w instrukcji, tzn. nie nalezy do niej?
Kurcze to po co ktos tak to "formatowal" tabami, ze sa dokladnie pod soba, a reszta kodu nie :/
Ale w sumie, tego to juz nikt nie wie - chyba jedynie autor tego kodu.
Dzieki za opdowiedzi, bo sie pogubilem w tym...

[PHP] pobierz, plaintext 
if($test)
 echo 'jest';
 echo 'nie ma';
[PHP] pobierz, plaintext 

Działa tak samo jak

[PHP] pobierz, plaintext 
if($test)
{
 echo 'jest';
}
else
{
 echo 'nie ma';
}
[PHP] pobierz, plaintext 

Autor najwyraźniej czerpał nawyk np. z JavaScriptu, gdzie nie używa się klamer ze względu na "objętość" kodu... Co nie jest mile widziane w php.

No nie bardzo !*!, raczej to:

[PHP] pobierz, plaintext 
if($test)
 echo 'jest';
else
 echo 'nie ma';
[PHP] pobierz, plaintext 

!*!, Daiquiri obydwoje źle mówicie...
kod:

[PHP] pobierz, plaintext 
if($test)
 echo 'jest';
 echo 'nie ma';
[PHP] pobierz, plaintext 

działa tak samo jak:

[PHP] pobierz, plaintext 
if($test)
{
 echo 'jest';
}
 echo 'nie ma';
[PHP] pobierz, plaintext 

nie wprowadzajcie ludzi w błąd.

Odnosiłam się do tego co napisał !*!, pisząc że działa tak samo jak "to" poniżej.

No wlasnie tez mi sie cos nie podobalo, jak patrzylem na ten drugi przyklad, ktory podalem.
Wedlug Twojego przykladu, co bedzie wynikiem, jesli $test == true?
1. "jest" i "nie ma"
2. samo "jest"

Bo juz sie zakolowalem
Kurcze juz sam nie wiem, po co ten return $text jest dokladnie pod tymi preg_matchami - wyglada to tak jakby autor celowo uzyl tabow, aby wskazac ze ten return jest pod if'a

jeśli $test = true wynikiem będzie "jest nie ma", jeśli $test = false, wynikiem będzie samo "nie ma".

!*!, zgubił else w swoim przykładzie (stąd poprawka poniżej). I żeby było jasne: nie, nie jest to kod o który pytałeś (bo jest konstrukcją if/else a nie samym if).


@down:
To dobrze .

Ja Cie zrozumialem - napisalas, ze gdyby mialo to miec taki sens jak pokazal !*!, to by musialo wygladac tak, jak Ty napisalas

Wersja prawidłowa

[PHP] pobierz, plaintext 
if($test)
{
 echo 'jest';
}
 echo 'nie ma';
 
[PHP] pobierz, plaintext 

Ale TAKIEJ konstrukcji

[PHP] pobierz, plaintext 
if ($test)
   echo 'cos';
[PHP] pobierz, plaintext 

NIE UŻYWAMY! Oczywiście zadziała, ale xdebug podczas generowanie pokrycia kodu FAŁSZYWIE oznaczy linijkę z "echo 'cos';" na wykonaną odrazu gdy tylko dojdzie do sprawdzenia warunku w IF.
Jest to fatalny błąd, jeżeli chcecie robić wszelkiego rodzaju testy jednostkowe z raportem pokrycia kodu.
Błąd jest NIENAPRAWIALNY i wynika z ułomności parsera PHP (a nie XDebuga). Dlatego zawsze stosujemy klamry w If-ach, while, foreach, itd itd

Tak informacyjnie, to jest jeszcze krótka wersja if/else. Czyli to:

[PHP] pobierz, plaintext 
if($user=='Zenek'){
  echo 'Czesc Zenek';
}
else{
  echo 'Nie znam cie';
}
[PHP] pobierz, plaintext 

Można zapisać tak:

[PHP] pobierz, plaintext 
($user=='Zenek') ? 'Czesc Zenek' : 'Nie znam cie';
[PHP] pobierz, plaintext