PHP — Говнокод.ру

PHP / Говнокод #16086

+154

01
02
03
04
05
06
07
08
09
10
11
12
13
14
15
16
17
18
19
20

if ($_SERVER["REQUEST_METHOD"] == "POST" && strlen($_REQUEST["backButton"]) > 0 && ($arParams["DELIVERY_NO_SESSION"] == "N" || check_bitrix_sessid()))
{
	if($arResult["POST"]["CurrentStep"] == 6 && $arResult["SKIP_FORTH_STEP"] == "Y")
		$arResult["CurrentStepTmp"] = 3;

	if($arResult["POST"]["CurrentStepTmp"] <= 5 && $arResult["SKIP_THIRD_STEP"] == "Y")
		$arResult["CurrentStepTmp"] = 2;

	if($arResult["POST"]["CurrentStepTmp"] <= 3 && $arResult["SKIP_SECOND_STEP"] == "Y")
		$arResult["CurrentStepTmp"] = 1;

	if(IntVal($arResult["CurrentStepTmp"])>0)
		$arResult["CurrentStep"] = $arResult["CurrentStepTmp"];
	else
		$arResult["CurrentStep"] = $arResult["CurrentStep"] - 2;
	$arResult["BACK"] = "Y";
}
if ($arResult["CurrentStep"] <= 0)
	$arResult["CurrentStep"] = 1;
$arResult["ERROR_MESSAGE"] = "";

Компонент страницы заказа в магазине Битрикса, 14я версия.
Тут вам и отсутствие констант для осмысленных шагов, и отсутствие конфигурирования этих шагов, как они i18n-ятся, вообще не представляю.
КАК ЭТО ВСЁ РАБОТАЕТ??!!!

ЗЫ мало того, тут же в компонент вкопипащен (или переговнокоден заново, не сравнивал) процесс залогинивания/регистрации нового юзера, елси он не залогинен. Не смог перекопипастить эту копипасту лишь потому, что она содержит 310 строк. Неслабый DRY.

velosipedistorg, 30 Мая 2014

Комментарии (0)

PHP / Говнокод #16066

+156

1

$result['BIRTHDATE'] = (($date['day'] < 10) ? '0'.$date['day'] : $date['day']).'.'.(($date['month'] < 10) ? '0'.$date['month'] : $date['month.'] ).'.'.$date['year'];

Пых и даты.

bormand, 28 Мая 2014

Комментарии (5)

PHP / Говнокод #16064

+154

1
2

foreach($arr as $val)
  $data[uniqid()] = $val;

Ну для ооочень медленных серверов =)

lisiy50, 27 Мая 2014

Комментарии (0)

PHP / Говнокод #16062

+162

01
02
03
04
05
06
07
08
09
10
11
12
13
14
15
16
17
18
19
20
21
22

switch ($Age){
	case "30-35 лет" : $Age="30-35 лет";	break;
	case "35-40 лет" : $Age="35-40 лет";	break;
	case "40-45 лет" : $Age="40-45 лет";	break;
	case "45-50 лет" : $Age="45-50 лет";	break;
	case "50-55 лет" : $Age="50-55 лет";	break;
	}
switch ($Driving){
	case "10-15 лет" : $Driving="10-15 лет";	break;
	case "20-25 лет" : $Driving="20-25 лет";	break;
	case "25-30 лет" : $Driving="25-30 лет";	break;
	case "30-35 лет" : $Driving="30-35 лет";	break;
	}
	
switch ($Register){
	case "не в Москве" : $Register="не в Москве";	break;
	case "Авиамоторная" : $Register="Авиамоторная";	break;
	case "Автозаводская" : $Register="Автозаводская";	break;
	case "Академическая" : $Register="Академическая";	break;
	case "Александровский сад" : $Register="Александровский сад";	break;
	case "Алексеевская" : $Register="Алексеевская";	break;
	case "Алтуфьево" : $Register="Алтуфьево";	break;

И так далее все станции...
Кто-нить понимает зачем? Программисту платили за количество строк?

aaparin, 27 Мая 2014

Комментарии (11)

PHP / Говнокод #16060

+155

1
2
3
4
5
6
7
8
9

/**
 * Returns the number of rows affected by the last query
 *
 * @return int
 */
public function getAffectedRowCount($result)
{
	return mysqli_affected_rows($this->getDatabase());
}

SugarCRM. Стоит от $35/месяц на одного пользователя.

Понимаю когда такое встречатеся в стартапах, но когда ты просишь за свой продукт деньги и деньги не малые, то выпускать такое в продакшен... Лично я бы постеснялся.

VanSanblch, 27 Мая 2014

Комментарии (26)

PHP / Говнокод #16059

+151

01
02
03
04
05
06
07
08
09
10
11
12
13

for($page = 0 ; $page <= $maxPage; $page ++) {
	$result = LoadPage($page);
	if(!$result) {
		if($try < 5) {
			$page --;
			$try ++;
		} else
			$try = 0;
	} else {
		$try = 0;
		ProcessPage($page, $result);
	}
}

два цикла по цене одного

clauclauclau, 26 Мая 2014

Комментарии (10)

PHP / Говнокод #16039

+158

1

<<?='b'.'o'.'d'.'y'?> class="p0 m0">

Чье-то из какой-то самописной CMS в шаблоне..) таким способом выводятся многие теги))

Cyril, 22 Мая 2014

Комментарии (117)

PHP / Говнокод #16037

+158

1
2
3
4
5
6

$inst = $db->prepare("insert into year_$year (user_name,type_of_day,month_row,day_1,day_2,day_3,day_4,day_5,day_6,day_7,day_8,day_9,
day_10,day_11,day_12,day_13,day_14,day_15,day_16,day_17,day_18,day_19,day_20,day_21,day_22,day_23,day_24,day_25,day_26,day_27,day_28,day_29,day_30,day_31)
values
(?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?)");

$inst->execute($chunk_post[$init]);

Оригинал тут http://hashcode.ru/questions/321962/php-mysql-%D0%B2%D1%81%D1%82%D0%B0%D0%B2%D0%BA%D0% B0-%D1%80%D0%B0%D0%B4%D0%BE%D0%BC%D0%BD%D0% BE%D0%B3%D0%BE-%D0%BA%D0%BE%D0%BB%D0%B8%D1%87%D0%B5%D1% 81%D1%82%D0%B2%D0%B0-%D0%B4%D0%B0%D0%BD%D0%BD%D1%8B%D1%85

dimka3210, 22 Мая 2014

Комментарии (10)

PHP / Говнокод #16036

+157

1
2
3
4
5
6
7
8

echo '
<font class="phonetop">
  +7 (495) 
  <font style="font-size:28px">
    99-888-77
  </font>
</font>
';

ну как так можно?
использовать в теге font class, и в то же время, умудриться непонятно для чего изменять размер шрифта в теге font через style?
даже если нравится тег font, так у него есть атрибут size=""
...

govnokoder2, 22 Мая 2014

Комментарии (2)

PHP / Говнокод #16035

+154

01
02
03
04
05
06
07
08
09
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57

function box_navigation($gc, $num, $id, $function, $act){
	global $tpl, $page;
	$gcount = $gc;
	$cnt = $num;
	$items_count = $cnt;
	$items_per_page = $gcount;
	$page_refers_per_page = 5;
	$pages = '';		
	$pages_count = ( ( $items_count % $items_per_page != 0 ) ) ? floor( $items_count / $items_per_page ) + 1 : floor( $items_count / $items_per_page );
	$start_page = ( $page - $page_refers_per_page <= 0  ) ? 1 : $page - $page_refers_per_page + 1;
	$page_refers_per_page_count = ( ( $page - $page_refers_per_page < 0 ) ? $page : $page_refers_per_page ) + ( ( $page + $page_refers_per_page > $pages_count ) ? ( $pages_count - $page )  :  $page_refers_per_page - 1 );
	
	if(!$act)
		$act = "''";
	else
		$act = "'{$act}'";
			
	if($page > 1)
		$pages .= '<a href="" onClick="'.$function.'('.$id.', '.($page-1).', '.$act.'); return false">«</a>';
	else
		$pages .= '';
				
	if ( $start_page > 1 ) {
		$pages .= '<a href="" onClick="'.$function.'('.$id.', 1, '.$act.'); return false">1</a>';
		$pages .= '<a href="" onClick="'.$function.'('.$id.', '.($start_page-1).', '.$act.'); return false">...</a>';
			
	}
					
	for ( $index = -1; ++$index <= $page_refers_per_page_count-1; ) {
		if ( $index + $start_page == $page )
			$pages .= '<span>' . ( $start_page + $index ) . '</span>';
		else 
			$pages .= '<a href="" onClick="'.$function.'('.$id.', '.($start_page+$index).', '.$act.'); return false">'.($start_page+$index).'</a>';
	} 
			
	if ( $page + $page_refers_per_page <= $pages_count ) { 
		$pages .= '<a href="" onClick="'.$function.'('.$id.', '.($start_page + $page_refers_per_page_count).', '.$act.'); return false">...</a>';
		$pages .= '<a href="" onClick="'.$function.'('.$id.', '.$pages_count.', '.$act.'); return false">'.$pages_count.'</a>';	
	} 
				
	$resif = $cnt/$gcount;
	if(ceil($resif) == $page)
		$pages .= '';
	else
		$pages .= '<a href="/" onClick="'.$function.'('.$id.', '.($page+1).', '.$act.'); return false">»</a>';

	if ( $pages_count <= 1 )
		$pages = '';

	$tpl_2 = new mozg_template();
	$tpl_2->dir = TEMPLATE_DIR;
	$tpl_2->load_template('nav.tpl');
	$tpl_2->set('{pages}', $pages);
	$tpl_2->compile('content');
	$tpl_2->clear();
	$tpl->result['content'] .= $tpl_2->result['content'];
}

дали проект чтобы разобрался)

progsmile, 21 Мая 2014

Комментарии (3)

Говнокод: по колено в коде.

PHP / Говнокод #16086

PHP / Говнокод #16066

PHP / Говнокод #16064

PHP / Говнокод #16062

PHP / Говнокод #16060

PHP / Говнокод #16059

PHP / Говнокод #16039

PHP / Говнокод #16037

PHP / Говнокод #16036

PHP / Говнокод #16035