[PHP]Glupie pyt. dot. instrukcji IF bez klamer. Opcje

[kaźmirz]

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 (IMG:style_emoticons/default/sad.gif)

Ten post edytował kaźmirz 16.03.2011, 22:53:53

[Fifi209]

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

[Blame]

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

[kaźmirz]

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...

Ten post edytował kaźmirz 16.03.2011, 23:00:40

[!*!]

[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.

Ten post edytował !*! 16.03.2011, 23:47:03

[Daiquiri]

No nie bardzo !*!, raczej to:

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

[krzywy36]

!*!, 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.

[Daiquiri]

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

[kaźmirz]

!*!, 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.

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 (IMG:style_emoticons/default/wink.gif)
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

Ten post edytował kaźmirz 17.03.2011, 00:13:22

[krzywy36]

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

[Daiquiri]

!*!, 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 (IMG:style_emoticons/default/smile.gif) .

Powód edycji: [Daiquiri]:

[kaźmirz]

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

Ja Cie zrozumialem - napisalas, ze gdyby mialo to miec taki sens jak pokazal !*!, to by musialo wygladac tak, jak Ty napisalas (IMG:style_emoticons/default/smile.gif)

Ten post edytował kaźmirz 17.03.2011, 00:15:54

[wookieb]

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

[sadistic_son]

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 

Ten post edytował sadistic_son 17.03.2011, 00:43:46