LINUX.ORG.RU

[PHP] Быдло код?

 


0

1

Быдло код это или нет?

 if (in_array($countryname, $countryarray)) {
		$praleisti=true;
		$i=$i-1;
	}
	$countryarray[] = $countryname;
	$regionnames = $data->getElementsByTagName( "REGIONNAME" );
	$regionname = $regionnames->item(0)->nodeValue;
	
	$citynames = $data->getElementsByTagName( "CITYNAME" );
	$cityname = $citynames->item(0)->nodeValue;
  
	$PRICE_FOR_PERSONs = $data->getElementsByTagName( "PRICE_FOR_PERSON" );
	$PRICE_FOR_PERSON = $PRICE_FOR_PERSONs->item(0)->nodeValue;
  
	$dates = $data->getElementsByTagName( "date" );
	$date = $dates->item(0)->nodeValue;
	
	$durations = $data->getElementsByTagName( "duration" );
	$duration = $durations->item(0)->nodeValue;
	
	$DEPARTUREAIRPORTs = $data->getElementsByTagName( "DEPARTUREAIRPORT" );
	$DEPARTUREAIRPORT = $DEPARTUREAIRPORTs->item(0)->nodeValue;
	
	$currencys = $data->getElementsByTagName( "currency" );
	$currency = $currencys->item(0)->nodeValue;
	
	$HOTELURLs = $data->getElementsByTagName( "HOTELURL" );
	$HOTELURL = $HOTELURLs->item(0)->nodeValue;
	
	$HOTELNAMEs = $data->getElementsByTagName( "HOTELNAME" );
	$HOTELNAME = $HOTELNAMEs->item(0)->nodeValue;
	
	$HOTELCATEGORYs = $data->getElementsByTagName( "HOTELCATEGORY" );
	$HOTELCATEGORY = $HOTELCATEGORYs->item(0)->nodeValue;
	if ($HOTELCATEGORY == '2*') $stars = '2s';
	if ($HOTELCATEGORY == '3*') $stars = '3s';
	if ($HOTELCATEGORY == '4*') $stars = '4s';
	if ($HOTELCATEGORY == '5*') $stars = '5s';
	
	$HOTELIMAGEs = $data->getElementsByTagName( "HOTELIMAGE" );
	$HOTELIMAGE = $HOTELIMAGEs->item(0)->nodeValue;
  
	$urls = $data->getElementsByTagName( "URL" );
	$url = $urls->item(0)->nodeValue;[/code}


Быдло не код, быдло его автор

anonymous
()

копипаста в окружении странностей. наверно он самый.

VladimirMalyk ★★★★★
()
in_array($countryname, $countryarray)

А теперь сходи на php.net и скажи, какой параметр ты забыл :)

helios ★★★★★
()

Я надеюсь, «быдло» это не обращение. Код плох излишним дублированием (можно было использовать циклы и массивы, а не ctrl+c/ctrl+v), непонятным принципом именования переменных, лишними временными переменными.

amomymous ★★★
()

Велосипед не мой, тема на спор :)

a931bw
() автор топика
Ответ на: РАБОТАЕТ - НЕ ТРОГАЙ! от visual

Поэтому это еще и на месте, было бы для себя а не для турагенства уже всю CMS снес бы.

a931bw
() автор топика

Ну нельзя так вот сразу и назвать это быдлокод.

Вопервых возможно человек не помнил наизусть как что доставать из XML и потом просто сделал одну копию. Потом вторую.... Конечно надо 2* делать заменой * на s и это уберет много. Потом ->item(0)->nodeValue; тоже надо вынести либо в функцию либо вообще сразу так и писать. Но возможно в отладке тут удобннее видно что взорвалось именно на этом вызове. Требует рефакторинга да. Быдлокод? нет. все аккуратно.

demmsnt
()

Быдло!
PHP код это или нет?

obv fix



а по теме: это не быдлокод, это калокод

q11q11 ★★★★★
()
Ответ на: комментарий от k_andy

in_array($countryname, $countryarray, true)
Там по умолчанию false. А это чревато...

helios ★★★★★
()

Тут скорее всего человек делал на скорую руку, лишь бы работало. А потом просто переделать было лень) Но по существу не аккуратно)

spiritkhl
()

if ($HOTELCATEGORY == '2*') $stars = '2s';

А я всегда фигурные скобки ставлю, даже в случае одной команды внутри. Что об этом говорят coding standarts всякие?

anonymous
()
Вы не можете добавлять комментарии в эту тему. Тема перемещена в архив.