Список говнокодов пользователя quall

Всего: 9

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

+156

1
2
3
4
5
6

if ($end < $start) {
	$errorLimit = '<p><u><b>End limit</b></u> should be bigger number than <u><b>Start limit</b></u></p>';
	//echo $errorLimit;
	throw new \grdprc\Exception(strip_tags($errorLimit));
	return;
}

<p><u><b> тоже ок.

quall, 25 Июня 2014

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

Куча / Говнокод #13874

+126

1
2
3
4
5
6

<link rel="stylesheet" href="/libraries/mediaelement/mediaelementplayer.min.css">
<link href="http://code.jquery.com/ui/1.10.0/themes/base/jquery-ui.css" rel="stylesheet">
<script type="text/javascript" src="/js/jquery/jquery-latest.js"></script>
<script src="//ajax.googleapis.com/ajax/libs/jqueryui/1.9.2/jquery-ui.min.js"></script>
<script type="text/javascript" src="/libraries/mediaelement/andplayer.min.js"></script>
<script type="text/javascript" src="/js/jquery_tools.js"></script>

Все это, пара десятков строк мутного джаваскрипта и столько же HTMLя и PHP потребовалось опытному филиппинскому сотруднику, чтобы добавить видеоролик на страницу.

quall, 30 Сентября 2013

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

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

+167

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

/**
 * Return data
 * 
 * @return array 
 */
public function __toString()
{
    return $this->toArray();
}

/**
 * Return Data
 * 
 * @return array
 */
public function toArray()
{
    return $this->_data;
}

No comments

quall, 10 Июля 2013

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

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

+156

1
2
3
4
5
6

extract($params);
        foreach (array("subject", "title", "content") as $par) {
            if (empty($$par)) {
                return false;
            }
        }

Валидация входных параметров. Каждая строка тут прекрасна

quall, 10 Июля 2013

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

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

+160

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

<?php

class UserSessionComponent extends Component {
	//...
	protected function loginByUserId($user_id, $is_autologin = false) {
		$list = $this->_loadComponents($this->components, array('UserAuth'));
		$userAuth = $list['UserAuth'];
		$userAuth->startup($this->controller); # !!! WORKFLOW VIOLATION !!!
		$method = $is_autologin ? 'autologin' : 'login';
		if ($userAuth->$method($user_id)) {
			return $this->controller->Session->read('User.data');
		}
	}
	//...
}

class UserAuthComponent extends BaseMagicComponent {
	protected $loginType = 'explicit';

	//...

	public function autologin($params = array()) {
		$this->loginType = 'implicit';
		return $this->_login($params);
	}

	public function login($params = array()) {
		$this->loginType = 'explicit';
		return $this->_login($params);
	}

	//...
	protected function _login($params = array()) {
		switch (true) {
			case (is_array($params) && isset($params['user']) && isset($params['password'])):
				$result = $this->loginByUserAndPassword($params['user'], $params['password']);
				break;
			case (is_numeric($params) && $params > 0):
				$result = $this->loginUser($params);
				break;
			default:
				if (empty($params)) {
					$result = false;
				} else {
					$result = $this->loginUser($params);
				}
		}
		// ...
	}

	// ...
	public function loginUser($user) {
		switch (true) {
			case is_object($user):
				if (isset($user->data)) {
					$a_user = $user->data;
				} else {
					return false;
				}
				break;
			case (is_numeric($user) && ($user > 0)):
				if (!($a_user = $this->User->findById($user))) {
					return false;
				}
				break;
			case (is_array($user) && (isset($user['User']['id']))):
				$a_user = $user;
				break;
			default:
				return false;
		}
		// ...
		CFL::using('log');
		if (empty($GLOBALS['loginIsLogged'])) {
			$loginLog = new cflLoginTrackingLogger();
			if ($this->loginType == 'implicit') {
				$loginLog->implicit_login(array('user_id' => $a_user['User']['id']));
			} else {
				$loginLog->explicit_login(array('user_id' => $a_user['User']['id']));
			}
			$GLOBALS['loginIsLogged'] = true;
		}
		// ...
	}
}

Сорри за много-много кода, еле-еле вписался в лимит. Но пришлось запостить все. Логическая цепочка совершенно потрясающая (смотрим с самого верхнего метода и идем дальше к самому нижнему методу). Тут у нас кода просто по колено: сумасшедший пахлава-код, использование глобальной переменной флага, особое извращенное использование конструкции switch, фактическое дублирование этого самого switch в двух последних методах, нейминг переменных и методов...
Казалось бы, как все это вообще могло быть написано? Но использование компонента с говорящим именем BaseMagic все ставит на свои места.

quall, 09 Февраля 2011

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

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

+160

01
02
03
04
05
06
07
08
09
10

if ($this->Session->check('_userJustRegistered')) {
	switch (($tmpJR = $this->Session->read('_userJustRegistered'))) {
		case 'confirmed':
			$this->Session->delete('_userJustRegistered');
			break;
		case 'not_confirmed_first':
			$this->Session->write('_userJustRegistered', 'not_confirmed');
			break;
	}
}

Кусочек многоступенчатой системы по определению статуса юзера.

quall, 08 Февраля 2011

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

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

+157

01
02
03
04
05
06
07
08
09
10
11

public function getUserDeliveryList($deliverySettings) {
	$obligatoryFieldsList = array('type', 'periods', 'freq');
	foreach ($obligatoryFieldsList as $v){
		if (!isset($deliverySettings[$v])){
			return false;
		} else {
			$$v = $deliverySettings[$v];
		}
	}
//	...
}

Обратил внимание после того, как в тексте метода обнаружил некоторое количество undefined (по мнению IDE) переменных.

quall, 08 Февраля 2011

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

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