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

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

+162

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

private function _moveIndexes($filters,$int){
		$count = count($filters);
		for($i=2;$i<$count;$i++ ){
			$filters[$i]["filter"]["index"] += $int;
		}
		return $filters;
	}

	private function _removeDefaultFilters(){
		$this->data["Data"]["filters"] = $this->_moveIndexes($this->data["Data"]["filters"],-2);

		$count = count($this->data["Data"]["filters"]);
		for($i=0;$i<$count-2;$i++){
			$this->data["Data"]["filters"][$i] = $this->data["Data"]["filters"][$i+2];
		}
		array_pop($this->data["Data"]["filters"]);
		array_pop($this->data["Data"]["filters"]);
	}

Такой вот код... Ни строчки пояснений к нему не прилагается.
Метод "_removeDefaultFilters()", а в коде сплошная магия.

Почему в for() $count-2??? А внутри цикла [$i+2]...
В другом же for() $i=2???
В вызове _moveIndexes() второй параметр -2 (минус 2) и это значение внутри фнкции плюсуется!
Вдовершение всего двойной вызов array_pop()...
И всё это покрыто тайной мрака.

zabuhailo, 08 Июля 2011

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

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

+168

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

if	($page	!=	1)		$pervpage		=	'<a href="?emw;News;1" rel="history2">'.first.'</a> | <a href="?emw;News;'.($page - 1).'" rel="history2">'.previous.'</a> | ';
if	($page	!=	$total)	$nextpage		=	' | <a href="?emw;News;'.($page + 1).'" rel="history2">'.next.'</a> | <a href="?emw;News;'.$total.'" rel="history2">'.last.'</a>';
if	($page - 5	>	0)	$page5left		=	'<a href="?emw;News;'.($page - 5).'" rel="history2">'.($page - 5).'</a> | ';
if	($page - 4	>	0)	$page4left		=	'<a href="?emw;News;'.($page - 4).'" rel="history2">'.($page - 4).'</a> | ';
if	($page - 3	>	0)	$page3left		=	'<a href="?emw;News;'.($page - 3).'" rel="history2">'.($page - 3).'</a> | ';
if	($page - 2	>	0)	$page2left		=	'<a href="?emw;News;'.($page - 2).'" rel="history2">'.($page - 2).'</a> | ';
if	($page - 1	>	0)	$page1left		=	'<a href="?emw;News;'.($page - 1).'" rel="history2">'.($page - 1).'</a> | ';
if	($page + 5 <= $total) $page5right	=	' | <a href="?emw;News;'. ($page + 5) .'" rel="history2">'.($page + 5).'</a>';
if	($page + 4 <= $total) $page4right	=	' | <a href="?emw;News;'. ($page + 4) .'" rel="history2">'.($page + 4).'</a>';
if	($page + 3 <= $total) $page3right	=	' | <a href="?emw;News;'. ($page + 3) .'" rel="history2">'.($page + 3).'</a>';
if	($page + 2 <= $total) $page2right	=	' | <a href="?emw;News;'. ($page + 2) .'" rel="history2">'.($page + 2).'</a>';
if	($page + 1 <= $total) $page1right	=	' | <a href="?emw;News;'. ($page + 1) .'" rel="history2">'.($page + 1).'</a>';
if	($total	>	1)
{
	Error_Reporting(E_ALL & ~E_NOTICE);
	echo	"<center><div class=\"pstrnav\">";
	echo	$pervpage.$page5left.$page4left.$page3left.$page2left.$page1left.'<b>'.$page.'</b>'.$page1right.$page2right.$page3right.$page4right.$page5right.$nextpage;
	echo	"</div></center>";
}

Навигация.

theaqua, 07 Июля 2011

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

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

+165

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

function f_includes(&$text){

		//REPLACE FILE INCLUDES LINKS
 		$regex = '/{(ФАЙЛ=)\s*(.*?)}/i';
		$matches = array();
		preg_match_all( $regex, $text, $matches, PREG_SET_ORDER );		
		foreach ($matches as $elm) {	
			$elm[0] = str_replace('{', '', $elm[0]);
			$elm[0] = str_replace('}', '', $elm[0]);			
			parse_str( $elm[0], $args );
			$file=@$args['ФАЙЛ'];	
			if ($file){		
				$output = getLink($file);			
			} else { $output = ''; }
			$text = str_replace('{ФАЙЛ='.$file.'}', $output, $text );	
		}

		return true;
}

Что за п##дец. InstantCMS.

invision70, 07 Июля 2011

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

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

+164

1
2

if ($user_id){ $user_sql = "(c.user_id=$user_id OR c.session_id='$sid')"; } else { $user_sql = "(c.user_id=0 AND c.session_id='$sid')"; }
$sql = "SELECT i.title as title, i.price as price, c.itemscount as itemscount, i.category_id as category_id FROM cms_uc_cart c, cms_uc_items i WHERE c.item_id = i.id AND $user_sql";

Все гениально и просто б#я! InstantCMS. Пол движка с подобными условиями. Писали школьники?! Видимо да ;(
P.S Соре просто с г#вном пришлось работать. Чувствую себя фермером 0_0

invision70, 07 Июля 2011

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

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

+160

01
02
03
04
05
06
07
08
09
10

<?php
//...
public function saveTask($task){
			if(is_numeric($task->id)){ //How do I check whether a member exists in php?
				//update
			}else{
				// insert new
			}
		}
//...

Комментарий на 4-й строке убивает.
Снова код из vTiger CRM.

mr.The, 07 Июля 2011

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

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

+161

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

const LinkText = 'Разместить заявку'; // текст по умолчанию для ссылки в шаблонах

// Применение

		$sql = 'SELECT
					ContestID,
					DateStart,
					DateEnd,
					Name,
					ThumbImg,
					EntryText,
					ResultText,
					LinkText,
					"' . self::LinkText . '" AS LinkTextByDefault,
					VoteDateStart,

Действительно, почему бы не отправить мускулу константу, чтобы потом получить ее и подставить в шаблон?

lorwe, 07 Июля 2011

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

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

+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

function GetStyleList(){
	$Styles = array();

	$Styles['a']['class'] = 'a';
    $Styles['a']['name'] = 'Просто ссылка';
    $Styles['a']['style'] = array('font-family'=>'','font-size'=>'','color'=>'','text-decoration'=>'','font-weight'=>'','font-style'=>'');
    $Styles['a']['hover'] = array('font-family'=>'','font-size'=>'','color'=>'','text-decoration'=>'','font-weight'=>'','font-style'=>'');

    // menu
    $Styles['menu']['class'] = 'a.menu';
    $Styles['menu']['name'] = 'Вертикальное меню';
    $Styles['menu']['style'] = array('font-family'=>'','font-size'=>'','color'=>'','text-decoration'=>'','font-weight'=>'','font-style'=>'');
    $Styles['menu']['hover'] = array('font-family'=>'','font-size'=>'','color'=>'','text-decoration'=>'','font-weight'=>'','font-style'=>'');

    $Styles['menu2']['class'] = 'a.menu2';
    $Styles['menu2']['name'] = 'Вертикальное подменю';
    $Styles['menu2']['style'] = array('font-family'=>'','font-size'=>'','color'=>'','text-decoration'=>'','font-weight'=>'','font-style'=>'');
    $Styles['menu2']['hover'] = array('font-family'=>'','font-size'=>'','color'=>'','text-decoration'=>'','font-weight'=>'','font-style'=>'');

    
  ......................................./*длины гениталий слона код*/...................................................... 

    $Styles['block_text2_page']['class'] = '.block_text2_page';
    $Styles['block_text2_page']['name'] = 'Контент блока меню';
    $Styles['block_text2_page']['style'] = array('text-align'=>'');
    $Styles['block_text2_page']['hover'] = array();

    $Styles['page_name']['class'] = '.page_name';
    $Styles['page_name']['name'] = 'Название страницы';
    $Styles['page_name']['style'] = array('font-family'=>'','font-size'=>'','color'=>'','text-decoration'=>'','font-weight'=>'','font-style'=>'', 'text-align'=>'');
    $Styles['page_name']['hover'] = array();

    $Styles['block_text_center']['class'] = '.block_text_center';
    $Styles['block_text_center']['name'] = 'Контент польз. центрального блока';
    $Styles['block_text_center']['style'] = array('font-family'=>'','font-size'=>'','color'=>'','text-decoration'=>'','font-weight'=>'','font-style'=>'', 'text-align'=>'');
    $Styles['block_text_center']['hover'] = array();

Цмс одной компании. На данный момент она банкрот

warider, 07 Июля 2011

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

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

+158

01
02
03
04
05
06
07
08
09
10
11

if ($category) {
        $api_categories = $category['api_categories'];
        $params['categories'] = $category['api_categories'];
} else {
    $api_categories = array();
    foreach ($categories as $category_data) {
        $api_categories = array_merge($api_categories, $category_data['api_categories']);
    }
}
        
$params['categories'] = $api_categories;

Вот нафигачил то

striker, 07 Июля 2011

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

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

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

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