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

SQL / Говнокод #2978

−180.2

1

SELECT ((SELECT COUNT(*) FROM `books` WHERE `author` = '?') + (SELECT COUNT(*) FROM `magazins` WHERE `author` = '?')));

не многовато ли селектов?:))
вместо '?' пхпшный код ставит значение.

litrumb, 09 Апреля 2010

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

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

+173.4

01
02
03
04
05
06
07
08
09
10
11

function p_id($p_id)
{
	global $product_id;
	
	foreach($product_id as $k=>$v)if($p_id==$k)
	{
		$sel_r=$k;
		$sel_t=$v;
	}
	return $sel_t;
}

Реальная функция и даже реально используется.
Сотрудник, далекий от кодинга, иногда чтобы не отвлекать меня разные мелочи кодит сам... Ну там по образцам типа...
Функцию эту кстати я трогать не стал исходя из золотого правила - "работает - не трогай!", так и используется и по сей день...

larrygingras2, 09 Апреля 2010

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

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

+146.4

1

$where = $this->getAdapter()->quoteInto('idUser=?', (int)$this->idUser);

надёжная защита от инъекций )))

paranoid, 09 Апреля 2010

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

C# / Говнокод #2975

+117.4

1

int? ipLong = ip != null ? (int?)ip.Address : null;

Вот так взялись отрицательные IP в базе. А главное-то, правильно переменную назвать!

Rom@nych, 09 Апреля 2010

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

Pascal / Говнокод #2974

+106.2

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

/////////////////////////// удаление спецсимволов
Prbl:=' ';
WS_h:=AnsiReplaceText(WS_h, Chr(13), Prbl);
WS_h:=AnsiReplaceText(WS_h, Chr(10), Prbl);
WS_h:=AnsiReplaceText(WS_h, ' ', Prbl);
WS_h:=AnsiReplaceText(WS_h, '           ', Prbl);
WS_h:=AnsiReplaceText(WS_h, '          ', Prbl);
WS_h:=AnsiReplaceText(WS_h, '         ', Prbl);
WS_h:=AnsiReplaceText(WS_h, '        ', Prbl);
WS_h:=AnsiReplaceText(WS_h, '       ', Prbl);
WS_h:=AnsiReplaceText(WS_h, '      ', Prbl);
WS_h:=AnsiReplaceText(WS_h, '     ', Prbl);
WS_h:=AnsiReplaceText(WS_h, '    ', Prbl);
WS_h:=AnsiReplaceText(WS_h, '   ', Prbl);
WS_h:=AnsiReplaceText(WS_h, '  ', Prbl);
/////////////////////////////////////////////////
WS_h:=AnsiReplaceText(WS_h, '<span class=SpellE>', '');
/////////////////////////// удаление спецсимволов

Красиво оформленный код - залог успеха!

Temnenkov, 09 Апреля 2010

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

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

+156.2

1
2
3
4
5
6
7
8
9

switch ($result['state']) 
{
    case 'Oregon':
	$row->state = 'OR';
    break;
   case $row->state=='Washington':
	$row->state = 'WA';
   break;
}

нашел у своего потрудника, долго не мог понять что второй кейс делает и как это вообще работало

quard, 09 Апреля 2010

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

Pascal / Говнокод #2972

+110

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 IsBigLet(Let: String): Boolean;
var r: bool;
begin
r:=false;
if pos('Й',Let)>0 then r:=true;
if pos('Ц',Let)>0 then r:=true;
if pos('У',Let)>0 then r:=true;
if pos('К',Let)>0 then r:=true;
if pos('Е',Let)>0 then r:=true;
if pos('Н',Let)>0 then r:=true;
if pos('Г',Let)>0 then r:=true;
if pos('Ш',Let)>0 then r:=true;
if pos('Щ',Let)>0 then r:=true;
if pos('З',Let)>0 then r:=true;
if pos('Х',Let)>0 then r:=true;
if pos('Ф',Let)>0 then r:=true;
if pos('Ы',Let)>0 then r:=true;
if pos('В',Let)>0 then r:=true;
if pos('А',Let)>0 then r:=true;
if pos('П',Let)>0 then r:=true;
if pos('Р',Let)>0 then r:=true;
if pos('О',Let)>0 then r:=true;
if pos('Л',Let)>0 then r:=true;
if pos('Д',Let)>0 then r:=true;
if pos('Ж',Let)>0 then r:=true;
if pos('Э',Let)>0 then r:=true;
if pos('Я',Let)>0 then r:=true;
if pos('Ч',Let)>0 then r:=true;
if pos('С',Let)>0 then r:=true;
if pos('М',Let)>0 then r:=true;
if pos('И',Let)>0 then r:=true;
if pos('Т',Let)>0 then r:=true;
if pos('Б',Let)>0 then r:=true;
if pos('Ю',Let)>0 then r:=true;
Result:=r;
end;

Немножко индусского кода!

Temnenkov, 09 Апреля 2010

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

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

+60.2

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

bool CMLayer::operator==(CMLayer ^a, CMLayer ^b)
{
	CLayer *pA;
	CLayer *pB;
	bool bA = false;
	bool bB = false;

	// Проверка в катче
	try
	{
		pA = a->m_pelMain;
	}
	catch (System::NullReferenceException ^)
	{
		pA = NULL;
		bA = true;
	}

	// Проверка в катче
	try
	{
		pB = b->m_pelMain;
	}
	catch (System::NullReferenceException ^)
	{
		pB = NULL;
		bB = true;
	}

	if ((bA == true) && (bB == true))
		return true;
	else if ((bA == true) || (bB == true))
		return false;

	return pA == pB;
}

Код на Management C++, тот кто его писал был явно в ажуре))))
Проверяются указатели на внутрении данные, а штоб System::NullReferenceException не выскакивал при сравнее на null все сделанно в катче

Snake2101, 09 Апреля 2010

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

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

+144.8

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
95
96
97
98
99

bool PathProcessor::Process(PPProcess *pProcess)
{
	std::vector<int> viRoute; int nRouteCount; bool bRetVal = false;
	for (int i = 0; i < m_viExcludeStop.size(); i++){
		if (m_viExcludeStop[i] == pProcess->nStartStopId) return false;
	}
	int bbb = 0;
	if (pProcess->nRouteId < 0){
		viRoute = GetRoutesID(pProcess->nStartStopId);
		nRouteCount = viRoute.size();
		if (nRouteCount <= 0) return false;
	}
	else{
		viRoute = GetRoutesID(pProcess->nStartStopId, pProcess->nRouteId);
		nRouteCount = viRoute.size();
		if (nRouteCount <= 0) return false;
		nRouteCount = 1;
	}
	for (int i = 0; i < nRouteCount; i++){
		RouteDataInfo rdCur = GetRoute(viRoute[i]);
		pProcess->nRoutePartId = rdCur.nId;
		PPResult *pRes;
		bool bCurRetVal = false;
		if (pProcess->bIsFirstStop == true){
			pRes = new PPResult();
			pRes->bFinished = false;
			pRes->nFirstStopId = pProcess->nFirstId;
			pRes->nLastStopId = -1;
			pRes->nRouteId = rdCur.nRouteId;
			m_viResult.push_back(pRes);
		}
		else pRes = pProcess->pLastResult;
		if (pRes != NULL){
			PartInfo piTemp(rdCur.nId, rdCur.nLastStopId);
			pRes->viPartsId.push_back(piTemp);
		}
		if (IsEndStop(rdCur.nLastStopId) == true){
			if (pRes != NULL){
				pRes->bFinished = true;
				pRes->nLastStopId = rdCur.nLastStopId;
			}
			bRetVal = true;	bCurRetVal = true;
		}
		if ((pProcess->IsStopExist(rdCur.nLastStopId) == false) && (bCurRetVal == false)){
			if ((pProcess->nCurStep < m_nMaxStep) && (pProcess->nRouteId >= 0)){
				PPProcess ppProc;
				ppProc.nCurStep = pProcess->nCurStep;
				ppProc.nRouteId = pProcess->nRouteId;
				ppProc.nStartStopId = rdCur.nLastStopId;
				ppProc.pPrevProc = pProcess;
				ppProc.bIsFirstStop = false;
				ppProc.nFirstId = pProcess->nFirstId;
				if (pProcess->pLastResult == NULL) ppProc.pLastResult = pRes;
				else ppProc.pLastResult = pProcess->pLastResult;
				if (Process(&ppProc) == true) bRetVal = true;
			}
			else if (pProcess->nCurStep < m_nMaxStep){
				PPProcess ppProc;
				ppProc.nCurStep = pProcess->nCurStep;
				ppProc.nRouteId = rdCur.nRouteId;
				ppProc.nStartStopId = rdCur.nLastStopId;
				ppProc.pPrevProc = pProcess;
				ppProc.bIsFirstStop = false;
				ppProc.nFirstId = pProcess->nFirstId;
				if (pProcess->pLastResult == NULL) ppProc.pLastResult = pRes;
				else ppProc.pLastResult = pProcess->pLastResult;
				if (Process(&ppProc) == true) bRetVal = true;
			}
			if (pProcess->nCurStep+1 < m_nMaxStep){
				PPProcess ppProc2;
				ppProc2.nCurStep = pProcess->nCurStep+1;
				ppProc2.nRouteId = -1;
				ppProc2.nStartStopId = rdCur.nLastStopId;
				ppProc2.pPrevProc = pProcess;
				ppProc2.bIsFirstStop = true;
				ppProc2.nFirstId = rdCur.nLastStopId;
				ppProc2.pLastResult = NULL;
				if (Process(&ppProc2) == true){
					PPResult *pRes2;
					pRes2 = new PPResult();
					pRes2->bFinished = true;
					pRes2->nFirstStopId = pRes->nFirstStopId;
					pRes2->nLastStopId = rdCur.nLastStopId;
					pRes2->nRouteId = pRes->nRouteId;
					PPProcess *pProc = pProcess;
					while (pProc != NULL){
						PartInfo piTemp(pProc->nRoutePartId, pProc->nStartStopId);
						pRes2->viPartsId.push_back(piTemp);
						pProc = pProc->pPrevProc;
					}
					m_viResult.push_back(pRes2); bRetVal = true;
				}
			}
		}
	}
	if ((bRetVal == false) && (pProcess->nRouteId < 0))
		m_viExcludeStop.push_back(pProcess->nStartStopId);
	return bRetVal;
}

Когда-то давно писал алгоритм поиска маршрутов, теперь удевляюсь как это все работает да еще и не виснет

Snake2101, 09 Апреля 2010

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

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

+144.8

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

#include "stdafx.h"
#include "windows.h"

#define число int
#define если if
#define иначе else
#define начало {
#define конец }
#define НОЛЬ NULL
#define Сообщение MessageBoxA
#define КНОПКА_ОК MB_OK
#define возврат
#define присвоить =
#define равняется ==
#define больше >
#define ТочкаВхода _tmain
#define Строка _TCHAR
#define УКАЗАТЕЛЬ *
#define КонецСтроки ;
#define СлучайноеЧисло rand
#define Посев srand
#define ВзятьКоличествоТиков GetTickCount

/****************************************************************************************************/
/*																									*/
/****************************************************************************************************/
число ТочкаВхода(число КоличествоПараметров, Строка УКАЗАТЕЛЬ Параметры[])
{
	Посев(ВзятьКоличествоТиков());
	число новое присвоить СлучайноеЧисло() КонецСтроки
	если (новое больше 10000) начало
		Сообщение(НОЛЬ, "Больше 10000", НОЛЬ, КНОПКА_ОК) КонецСтроки
	конец иначе начало
		Сообщение(НОЛЬ, "Меньше 10000", НОЛЬ, КНОПКА_ОК) КонецСтроки
	конец

	возврат 0 КонецСтроки
}

Ооо могучий C++

Snake2101, 09 Апреля 2010

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

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

SQL / Говнокод #2978

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

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

C# / Говнокод #2975

Pascal / Говнокод #2974

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

Pascal / Говнокод #2972

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

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

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