По колено в коде — Говнокод.ру

Java / Говнокод #5584

+71

1
2

//rapeint
this.repaint();

Выеби интегер!

danilissimus, 08 Февраля 2011

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

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

+163

1
2
3
4
5
6
7
8

function onsub(text){
	if (confirm(text)) { 
		return true;
	}
	else {
		return false;
	}
}

Копаюсь в системе биллинга...
Убило!

Gogogo, 08 Февраля 2011

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

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

+161

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

// Функция для проверки входнях данных и присвоения входных данных переменым
// $pattern задаёт регулярное выражение
// $key ключ в массивах $_POST[] или $_GET[]
// $firts и $second порядок их просмотра
// $second не обязательный параметр
// $die_if_not_match задаёт поведение при не соответствии входных данных регулярному выражению
// $die_if_not_set задаёт поведение при отсутствии входных данных
// по умолчания при не совпаденини или отсутствии входных данных функция не возвращаяет ничего
function inchk($pattern, $key, $first, $second = '' ,$die_if_not_match = '0',$die_if_not_set = '0' ){
	if(!$pattern){
		die("Шаблон для inchk() не задан");
	}elseif(!$key){
		die("Ключ для inchk() не задан");
	}
	elseif(!$first){
		die("Источник данных inchk() не задан");
	}elseif($first and $second){
		if($first == "post" or $first == "p"){
			if($_POST[$key] != ''){
				if(preg_match($pattern, $_POST[$key])){
					return $_POST[$key];
				}else{
					if($die_if_not_match){ die('недопустимый формат $_POST[\''.$key.'\']'); }
				}
			}elseif($_GET[$key] != ''){
				if(preg_match($pattern, $_GET[$key])){
					return $_GET[$key];
				}else{
					if($die_if_not_match){ die('недопустимый формат $_GET[\''.$key.'\']'); }
				}
			}else{
				if($die_if_not_set){ die('$_POST[\''.$key.'\'] или $_GET[\''.$key.'\'] не задан'); }
			}
		}
		elseif($first == "get" or $first == "g"){
			if($_GET[$key] != ''){
				if(preg_match($pattern, $_GET[$key])){
					return $_GET[$key];
				}else{
					if($die_if_not_match){ die('недопустимый формат $_GET[\''.$key.'\']'); }
				}
			}elseif($_POST[$key] != ''){
				if(preg_match($pattern, $_POST[$key])){
					return $_POST[$key];
				}else{
					if($die_if_not_match){ die('недопустимый формат $_POST[\''.$key.'\']'); }
				}			
			}else{
				if($die_if_not_set){ die('$_GET[\''.$key.'\'] или $_POST[\''.$key.'\'] не задан'); }
			}
		}else{
			die("Источник данных inchk() задан не верно");
		}

	}elseif($first == "get" or $first == "g"){
		if($_GET[$key] != ''){
			if(preg_match($pattern, $_GET[$key])){
				return $_GET[$key];
			}else{
				if($die_if_not_match){ die('недопустимый формат $_GET[\''.$key.'\']'); }
			}
		}else{
			if($die_if_not_set){ die('$_GET[\''.$key.'\'] не задан'); }		
		}
	}elseif($first == "post" or $first == "p"){
		if($_POST[$key] != ''){
			if(preg_match($pattern, $_POST[$key])){
				return $_POST[$key];
			}else{
				if($die_if_not_match){ die('недопустимый формат $_POST[\''.$key.'\']'); }
			}
		}else{
			if($die_if_not_set){ die('$_POST[\''.$key.'\'] не задан'); }		
		}
	}elseif($first){
		die("Источник данных inchk() задан не верно");
	}else{
		die("Источник данных inchk() не задан");
	}

}

OH, SH*T! Казалось бы, зачем?

Gogogo, 08 Февраля 2011

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

C++ / Говнокод #5581

+163

1

throw new TSilentException("");

Говногость, 08 Февраля 2011

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

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

+159

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

var DatePicker = function () {
  ...
  return {
    init: function(options) {...},
    showPicker: function() {...},
    hidePicker: function() {...},
    setDate: function(date, shiftTo) {...},
    getDate: function(formated) {...},
    clear: function() {...},
    fixLayout: function() {...}
  };
}();
$.fn.extend({
  DatePicker: DatePicker.init,
  DatePickerHide: DatePicker.hidePicker,
  DatePickerShow: DatePicker.showPicker,
  DatePickerSetDate: DatePicker.setDate,
  DatePickerGetDate: DatePicker.getDate,
  DatePickerClear: DatePicker.clear,
  DatePickerLayout: DatePicker.fixLayout
});

Взято с http://www.eyecon.ro/datepicker/

wmmorgun, 08 Февраля 2011

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

C++ / Говнокод #5579

+149

1
2
3
4
5
6

struct SomeData 
{
    ...
    double balls;
    ....
}

Interfere, 08 Февраля 2011

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

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)

C++ / Говнокод #5577

+180

01
02
03
04
05
06
07
08
09
10

...
#define EXT12 	//Не комментировать эту директиву. Иначе не работает.
...
#define EXT34 	//Эта директива почти не используется. Может перерефакторим без неё?
		//Ну убирай.
		//Давай ты.
		//Там есть пару мест, где она используется. Поэтому я не хочу.
		//Но ведь нужно. Эти директивы можно заменить на полиморфизм классов.
		//Ну займись. Я не хочу. Вдруг сломается.
...

У нас 2 человека работают над одной веткой проекта на С++. Люди работают в отделах на разных этажах (один на этаж выше другого). Им лень сходить друг к другу или поговорить по жаберу. Я уже как полгода наблюдаю за картиной: они переговариваются через комментарии в коде через систему контроля версий. Я не против переговоров в системе контроля версий, но они уже месяц осбуждают EXT34, а так ничего с ним и не сделали. Хоть бы комментарии удалили, если делать ничего не собираюются. Может с них 0.125 премии снять?
Как-нибудь ещё переговоры повеселее выложу.

Говногость, 08 Февраля 2011

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

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

+165

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

if ($arResult["CurrentStep"] == 1)
	include($_SERVER["DOCUMENT_ROOT"].$templateFolder."/step1.php");
elseif ($arResult["CurrentStep"] == 2)
	include($_SERVER["DOCUMENT_ROOT"].$templateFolder."/step2.php");
elseif ($arResult["CurrentStep"] == 3)
	include($_SERVER["DOCUMENT_ROOT"].$templateFolder."/step3.php");
elseif ($arResult["CurrentStep"] == 4)
	include($_SERVER["DOCUMENT_ROOT"].$templateFolder."/step4.php");
elseif ($arResult["CurrentStep"] == 5)
	include($_SERVER["DOCUMENT_ROOT"].$templateFolder."/step5.php");
elseif ($arResult["CurrentStep"] >= 6)
	include($_SERVER["DOCUMENT_ROOT"].$templateFolder."/step6.php");

Из проекта на битриксе.

hdkeeper, 08 Февраля 2011

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

C++ / Говнокод #5575

+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

QString convToHex( unsigned char *bytes, int size )
{
	if( size > 16 )
		size = 16;
	QString ret;
	for( int i = 0; i < size; i++ )
	{
		char buf[64];
		::snprintf( buf, sizeof(buf), "%02x", (unsigned int) bytes[i] );
		ret += buf;
	}	
	return ret;
}

QString TarCreator::generateGuid( const std::string &tDeviceSerial,
		const std::string &nDatetime, unsigned long long int id )
{
	std::ostringstream s;
	s << tDeviceSerial;
	s << nDatetime;
	s << id;
	
	QCryptographicHash hash( QCryptographicHash::Md5 );
	hash.addData( QByteArray( s.str().c_str() ) );
	QByteArray result = hash.result();
	return convToHex( (unsigned char*)result.data(), result.size() );
}

panter_dsd, 08 Февраля 2011

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

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