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

Всего: 4

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

+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

// берет запрос и делает из него массив
	public function requestUri() {
		if (isset($_SERVER['REQUEST_URI'])) {
			$uri = $_SERVER['REQUEST_URI'];
		}
		else {
			if (isset($_SERVER['argv'])) {
				$uri = $_SERVER['SCRIPT_NAME'] .'?'. $_SERVER['argv'][0];
			}
			elseif (isset($_SERVER['QUERY_STRING'])) {
				$uri = $_SERVER['SCRIPT_NAME'] .'?'. $_SERVER['QUERY_STRING'];
			}
			else {
				$uri = $_SERVER['SCRIPT_NAME'];
			}
		}
		// Prevent multiple slashes to avoid cross site requests via the FAPI.
		$uri = trim($uri, '/');
		$arr = explode('/', $uri);
		$this->queryArr = array();
		foreach($arr as $val) {
			$val = trim($val);
			if($val != '') $this->queryArr[] = $val;
		}

		return $uri;
	}

Такая вот реализация разбора "красивого" uri

rudolff, 07 Августа 2010

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

JavaScript / Говнокод #3886

+157

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

switch(resp.error_code)
			{
				case 0:
				{
					new URI('/account/success/').go();
					return;
				}
				case BAD_CAPTCHA:
				{					
					var c = $('fRegistration').getElement(".captcha");
					c.innerHTML = '<br />' + resp.error_message;
					break;
				}
				case BAD_EMAIL:
				{
					var c = $('fRegistration').getElement(".email");
					c.innerHTML = '<br />' + resp.error_message;
					break;
				}
				case BAD_PASSWORD:
				{
					var c = $('fRegistration').getElement(".passwd");
					c.innerHTML = resp.error_message + '<br />';
					break;
				}
				case BAD_AGREEMENT:
				{
					var c = $('fRegistration').getElement(".agreement");
					c.innerHTML = resp.error_message + '<br />';
					break;
				}
				default:
				{
					var c = $('fRegistration').getElement(".other_errors");
					c.innerHTML = resp.error_message + '<br />';
					break;
				}
			}

Классно, да?
Особенно интересно то, что используется фреймворк mootools и не используются его функции :)

$$('#fRegistration .other_errors")[0].set('html', resp.error_message + '<br />');

Вообще считаю избыточным передавать и код и строку ошибки

rudolff, 06 Августа 2010

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

JavaScript / Говнокод #3881

+149

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

//Server errors
var BAD_REQUEST = 400; 			//Неверный запрос
var UNAUTHORIZED = 401; 			//Authentication Required
var METHOD_NOT_ALLOWED = 405; 		//Метод не поддерживается
var SERVICE_UNAVAILABLE = 503; 	//Сервис недоступен

//User errors
var UNKNOWN_ERROR = 1000; 			//Неизвестная ошибка
var BAD_PARAMETERS = 1001; 		//Неверные параметры запроса
var BAD_LOGIN = 1002; 				//Неверный логин
var BAD_VERSION = 1003; 			//Неверна задана версия
var AUTHENTICATION_ERROR = 1004; 	//Ошибка аутентификации
var NO_SUCH_RECORD = 1005; 		//Запись не найдена (например, при попытке удалить несуществующий контакт будет выдана данная ошибка)


var BAD_CONTACT_ID = 2000; 		//Неверный ID контакта
var REV_GET_FAILED = 2001; 		//Ошибка получения ревизии

var BAD_CAPTCHA = 3000;
var BAD_EMAIL = 3001;
var BAD_PASSWORD = 3002;
var BAD_AGREEMENT = 3003;
var BAD_IPPORT_ID = 3004;
var BAD_ACTIVATION_ID = 3005;

Очень здорово и удобно объявлять константы(и др. объекты) в общей области видимости!

rudolff, 05 Августа 2010

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

JavaScript / Говнокод #3879

+161

1
2
3
4

var s = '';
	s += (hours)  ? ( (hours>9)  ? hours  + ':' : '0' + hours  + ':') : '00:';
	s += (minute) ? ( (minute>9) ? minute + ':' : '0' + minute + ':') : '00:';
	s += (second) ? ( (second>9) ? second + '.' : '0' + second + '.') : '00.';

почему бы не сделать так:
var s = '';
s += (hours<10) ? '0' + hours : hours) + ':';
// и так далее...

rudolff, 05 Августа 2010

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

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