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

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

+58

1
2
3

$attr_id = json_decode($_GET['attr_id']);
$obj_id = json_decode($_GET['obj_id']);
$a_href = $_GET['file_id'];

Разгребаю что осталось от недавно уволенного джуниора.
2 json_decoda толкают в уныние. При том что джсон туда вообще не приходит =\

Жаль вот уже, не сохранился код, в к-м он пришедший на _клиент в браузер_ json парсил руками(JSON.parse() - для слабых)

vitaly, 29 Июня 2012

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

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

+65

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
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94

$quGroup = mysql_query($x = "
        SELECT
            a.latitude, a.longitude, a.catid, a.id, a.title, a.arttype, a.userid, b.firstname, b.lastname, b.usertype,
            a.latitude2, a.longitude2, a.latitude3, a.longitude3, a.latitude4, a.longitude4, a.latitude5, a.longitude5,
            a.latitude6, a.longitude6, a.latitude7, a.longitude7, a.latitude8, a.longitude8, a.latitude9, a.longitude9,
            a.latitude10, a.longitude10, a.latitude11, a.longitude11
        from materials a, users b
        where a.status='1' and a.userid=b.id and add_date > $day)
        $sql_cond");
    while ($flGroup = mysql_fetch_row($quGroup)) {
        $j++;
        $a1 = $a2 = array ();
        $l1 = $flGroup[0];
        $l2 = $flGroup[1];
        if ($l1 && $l2) {
            $a1[] = $l1;
            $a2[] = $l2;
        }

        $latitude2 = $flGroup[10];
        $longitude2 = $flGroup[11];
        if ($latitude2 && $longitude2) {
            $a1[] = $latitude2;
            $a2[] = $longitude2;
        }

        $latitude3 = $flGroup[12];
        $longitude3 = $flGroup[13];
        if ($latitude3 && $longitude3) {
            $a1[] = $latitude3;
            $a2[] = $longitude3;
        }

        $latitude4 = $flGroup[14];
        $longitude4 = $flGroup[15];
        if ($latitude4 && $longitude4) {
            $a1[] = $latitude4;
            $a2[] = $longitude4;
        }

        $latitude5 = $flGroup[16];
        $longitude5 = $flGroup[17];
        if ($latitude5 && $longitude5) {
            $a1[] = $latitude5;
            $a2[] = $longitude5;
        }

        $latitude6 = $flGroup[18];
        $longitude6 = $flGroup[19];
        if ($latitude6 && $longitude6) {
            $a1[] = $latitude6;
            $a2[] = $longitude6;
        }

        $latitude7 = $flGroup[20];
        $longitude7 = $flGroup[21];
        if ($latitude7 && $longitude7) {
            $a1[] = $latitude7;
            $a2[] = $longitude7;
        }

        $latitude8 = $flGroup[22];
        $longitude8 = $flGroup[23];
        if ($latitude8 && $longitude8) {
            $a1[] = $latitude8;
            $a2[] = $longitude8;
        }

        $latitude9 = $flGroup[24];
        $longitude9 = $flGroup[25];
        if ($latitude9 && $longitude9) {
            $a1[] = $latitude9;
            $a2[] = $longitude9;
        }

        $latitude10 = $flGroup[26];
        $longitude10 = $flGroup[27];
        if ($latitude10 && $longitude10) {
            $a1[] = $latitude10;
            $a2[] = $longitude10;
        }

        $latitude11 = $flGroup[28];
        $longitude11 = $flGroup[29];
        if ($latitude11 && $longitude11) {
            $a1[] = $latitude11;
            $a2[] = $longitude11;
        }

        if (!$l1 && !$l2 && !$latitude2 && !$longitude2 && !$latitude3 && !$longitude3 && !$latitude4 && !$longitude4 && !$latitude5 && !$longitude5 && !$latitude6 && !$longitude6 && !$latitude7 && !$longitude7 && !$latitude8 && !$longitude8 && !$latitude9 && !$longitude9 && !$latitude10 && !$longitude10 && !$latitude11 && !$longitude11)
            continue;

...
}

как изящно

shmaltorhbooks, 28 Июня 2012

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

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

+65

1
2
3
4
5
6
7
8
9

if ($_GET['where'] != '') {
	if ($_GET['where'] != '') {
		die ("<meta http-equiv='refresh' content='0; url=".$page."?where=".$_GET['where']."'>");
	} else {
		die ("<meta http-equiv='refresh' content='0; url=".$page."?pid=".$_GET['pid']."'>");
	}
} else {
	die ("<meta http-equiv='refresh' content='0; url=".$page."?pid=".$_GET['pid']."'>");
}

Интересно, в каком случае выполнится второй die()?

domaster, 27 Июня 2012

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

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

+71

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

# Разбор time()
function GetTime( $time ) {
	
	# Получение массива из элементов даты
	$time = getdate( $time );
	
	# Все элементы с ведущими нулями
	$d = ( $time['mday'] > 9 ) ? $time['mday'] : '0' . $time['mday'];
	$w = ( $time['mon'] > 9 ) ? $time['mon'] : '0' . $time['mon'];
	$y = $time['year'];
	$h = ( $time['hours'] > 9 ) ? $time['hours'] : '0' . $time['hours'];
	$m = ( $time['minutes'] > 9 ) ? $time['minutes'] : '0' . $time['minutes'];
	$s = ( $time['seconds'] > 9 ) ? $time['seconds'] : '0' . $time['seconds'];
	
	# Сборка и возврат результата
	return $d . '/' . $w . '/' . $y . '&nbsp;' . $h . ':' . $m . ':' . $s;
}

Про date() явно не в курсе...

nethak, 27 Июня 2012

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

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

+62

1
2
3
4
5
6
7
8

function find( &$db, $id) {
		$statement = Contact::findStatement() . " WHERE id = $id";
		$dbResult = ActiveRecord::abstractFind( $db, $statement);
		foreach ($dbResult as $dbRow) {
			return Contact::load( $db, $dbRow);
		}
		return null;
	}

shmaltorhbooks, 26 Июня 2012

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

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

+141

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

<?php
if(!$this->model_forum->community_once(array('id'=>$community))) {
   	 echo file_get_contents($_SERVER['DOCUMENT_ROOT'].'/style/templates/mobile/no_community.tpl');
} 

else if(isset($_GET['topic']) && is_numeric($_GET['topic']) && $topic = $this->model_forum->topic_once(array('id'=>(int)$_GET['topic']))) {
   	 $tpl =  file_get_contents($_SERVER['DOCUMENT_ROOT'].'/style/templates/mobile/add_response_topic.tpl');
	 $tpl =  str_replace('{topic_id}', (int)$_GET['topic'] ,$tpl);
	 echo $tpl;
}

else {
    $tpl = file_get_contents($_SERVER['DOCUMENT_ROOT'].'/style/templates/mobile/add_topic.tpl');
    $tpl = str_replace('{community}',$community, $tpl);
	$tpl = str_replace('{topic_date_closing}', '23', $tpl);
	echo $tpl;
	
} ?>

оцените на предмет гавнокода

nepster, 26 Июня 2012

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

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

+67

1

$this->bgcolor = $bgcolor ? ($bgcolor <= 0xFFFFFF) ? ($bgcolor > 0xFFFFFF) ? 0xFFFFFF : $bgcolor : 0x000000 : 0xFFFFFF;

FFFFFFFFFFFFFFFFFFFFFFF

Razban_Guestov, 26 Июня 2012

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

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

+72

1
2
3
4
5
6
7
8

public function isAdmin() {
        $admins = Model::factory('User')->where('access_level_id', 1)->find_many();
        $ids = array();
        foreach($admins as $admin) {
            $ids[]= $admin->id;
        }
        return in_array($this->id, $ids) ? true : false;
    }

Метод проверки на админе из модели пользователей. Не, ну а чо.

mr.The, 26 Июня 2012

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

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

+61

1
2
3
4
5
6
7
8

if (is_page('orders_item')) {
	die ("<meta http-equiv='refresh' content='0; url=".$base_url."/orders_item.php?pid=".$_GET['pid']."'>");
} elseif (is_page('orders_garant')) {
	die ("<meta http-equiv='refresh' content='0; url=".$base_url."/orders_garant.php?pid=".$_GET['pid']."'>");
} else {
	die ("<meta http-equiv='refresh' content='0; url=".$base_url."/orders_item.php?pid=".$_GET['pid']."'>");
}
exit();

Хоть опросник пиши:
- чем первый die() отличается от последнего?
- почему их 3?
- кто будет экранировать $_GET?
- зачем в конце exit() подпирает die()?

domaster, 25 Июня 2012