стиль кода )

[Andreika]

стиль кода )

а насколько страшно выглядит подобный код(если его много такого будет) и насколько сложно его будет понять, если кому-то захочется самостоятельно с ним разобраться

if (($a==$b++ && $c==$d) || ($a==$x && $b=-1)) {
	
  // do something	
}

или вот такие хитрые нагромождения

$a = str_replace(array('[b]','[c]'),array('<B>','<C>'),htmlspechalchars(stripslashes(isset($_GET['a'])?$_GET['a']:$_CFG['default_string'])));

?
> какой тут смысл использовать функции htmlspechalchars и stripslashes
поменял строку )

 

[SID]

imho код вполне понятный.
вот тока вопросик есть, часть из второго примера:

htmlspechalchars( stripslashes( isset($_GET['a'])? $_GET['a'] : 'hello world') );

Если переменная $_GET['a'] существует то смысл в функциях htmlspechalchars и stripslashes есть, а вот если "дефолтной" строкой является 'hello world' то какой тут смысл использовать функции htmlspechalchars и stripslashes ? вроде как лишние действия появляются.

 

[crocodile2u]

Andreika
Разбираться в таком коде, тем более, если его много - занятие как минимум не из приятных...

 

[tf]

Andreika
2) так оптимизировать не надо. потом самому будут трудно разбиватся

 

[SiMM]

> imho код вполне понятный.

if (($a==$b++ && $c==$d) || ($a==$x && $b=-1)) {

Манипуляции с $b скорее похожи на шаманство.

 

[kruglov]

может && $b-=1 ?

А то что-то непонятное... Сначала b увеличили на 1, потом, если неверно, пошли вычислять второе выражение - и сделали b=-1

 

[SiMM]

> пошли вычислять второе выражение - и сделали b=-1
Не факт.

$b = 0;
false && $b = -1;
echo $b;

Вот поэтому код скорее напоминает танцы с бубном, ИМХО.