Лучший говнокод за всё время

Лучший говнокод

В номинации:

Лучший рейтинг
Самый комментируемый

За время:

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

+163

1
2
3

function paging($pageNum,$rowsPerPage)
{
	GLOBAL $_GET, $_POST, $_SESSION;

На всякий случай. Чтоб точно глобальные были. А то всякое бывает ...

_tL, 06 Мая 2011

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

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

+169

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

bool NSFileExists(const char * FileName)
{
  struct _stat fStats;
  return (_stat(FileName, &fStats) == 0);
}

#if 0
bool NSFileExists(const char * FileName)
{
  WIN32_FIND_DATA fd;
  HANDLE hFF;
  bool bExist(true);
  hFF = FindFirstFile(FileName, &fd);
  if (hFF == INVALID_HANDLE_VALUE) bExist = false;
  else FindClose(hFF);
  return bExist;
}
#endif

#if 0
bool NSFileExists(const char * FileName)
{
  HANDLE hFile = ::CreateFile(FileName, 0, 0, 0, OPEN_EXISTING, 0, 0);
  if (hFile != INVALID_HANDLE_VALUE)
  {
    CloseHandle(hFile);
    return true;
  }
  return false;
}
#endif

Эволюция!
Без комментариев...

JeremyW, 21 Апреля 2011

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

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

+179

1
2
3

// Count rows in news table
$rows = $db -> query('SELECT * FROM news');
$count = count($rows)

Увидел в одном самопальном движке.

Shitafaka, 14 Апреля 2011

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

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

+980

1
2
3
4

if (ch=='k' && ch!='t')
{
     something
}

Найдено в довольно серьезном проекте.

timurets, 05 Апреля 2011

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

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

+172

1
2
3
4
5
6
7
8

//Взятые из разных файлов варианты обращений к файлу конфигураций характеризуют о гибкости подхода к разработке приложения
require_once '../../../config.php';
require_once("../config.php");
require_once("../../../../config.php");
require_once(dirname(dirname(__FILE__)) . '/config.php');
require('../config.php');
require_once(dirname(__FILE__) . '/../../config.php');
require_once(dirname(dirname(dirname(dirname(__FILE__)))) . '/config.php'); //included from messagelib (how to fix?)

Предлагаю обратить внимание на http://moodle.org/ -- это Система Управления Обучением.

Продукт поражает своей архитектурой. Предлагаю взглянуть одним глазком.

sectus, 28 Марта 2011

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

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

+152

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

BOOL NAKED CompareCurrentProcess(PCHAR pszName)
{
    /*
        1. Found PEB
        2. Found _RTL_USER_PROCESS_PARAMETERS
        3. Found _RTL_USER_PROCESS_PARAMETERS->ImagePathName
        4. Convert UNICODE to ANSI
        5. compare strings
    */
    __asm {
        MOV        ESI,ESP                            // SAVE STACK PTR

        MOV        EAX,DWORD PTR FS:[0x30]            // eax == PEB
        MOV        EAX,DWORD PTR DS:[EAX+0x10]        // eax == _RTL_USER_PROCESS_PARAMETERS
        ADD        EAX,0x38                        // eax == _RTL_USER_PROCESS_PARAMETERS->ImagePathName
        
        XOR        EDX,EDX                            //
        XOR        ECX,ECX                            //
        XOR        EDI,EDI                            //
                
        MOV        CL, BYTE PTR [EAX]                // CL = UNICODE_STRING.Length in bytes
        
        SUB        ESP,ECX                            // reserve in stack CL bytes 
                    
        ADD        EAX,4                            // EAX ptr to WCHAR ImagePathName
        MOV        EAX,DWORD PTR [EAX]                // EAX = PWCHAR

next_char:
        CMP        CL,0                            // WCHAR end
        JZ        end;

        MOV        DL,BYTE PTR [EAX]                // DL == *(PCHAR)WCHAR
        ADD        EAX,2                            // GOTO next WIDEchar
        MOV        BYTE PTR [ESP],DL                // SAVE char in memory reserved in stack
        INC        ESP                                // pStr++
        INC        EDI                                // pStrLen++
        SUB        CL,2                            // Length--;
        jmp        next_char                        // goto_next;
end:
        MOV        BYTE PTR [ESP],0                // *pStr = 0; null terminated
        SUB        ESP,EDI                            // pStr = (PCHAR)(pStr - pStrLen)
        
        XOR        EDX,EDX
        XOR        ECX,ECX

        mov        ecx,esp                            // ecx = pStr
        mov        edx,esp                            // edx = pStr

        //
        // HERE FOUND image file name 
m_loop:
        cmp        edi,0
        jz        file_founded

        cmp        byte ptr [edx],0x5C                // '\'
        JZ        founded;
        inc        edx
        dec        edi
        jmp        m_loop

founded:
        dec        edi
        mov        ecx,edx
        inc        edx
        jmp        m_loop

file_founded:
        push    esi
        inc        ecx
        push    ecx
        push    dword ptr [esi+4]
        call    my_strcmp

        pop        esi
        mov        esp,esi

        ret        
    }
}

Источник: http://www.wasm.ru/forum/viewtopic.php?id=40652
Что может быть плохого в функции кроме того, что эту задачу можно было бы решить тонной адекватных вариантов?

ReL, 23 Марта 2011

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

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

+134

01
02
03
04
05
06
07
08
09
10

$i=10000;

while(true)
{
$i--;

if ($i==0) break;

// Какойто код с $i
}

Выкопал в одном серьезном проекте. Так там что не цикл так while(true) c точками выхода по телу цикла..

Vasiliy, 12 Марта 2011

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

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

+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

<?php
class Cached_Query
{

 function __construct()
 { 
  $mem_cache = new Memcache;
  $mem_cache->connect('localhost', 11211); 
   mysql_connect("localhost", "root", "");
  }

 function Query($query_text)
 {
  $ms_query = mysql_query("SELECT * FROM sometable WHERE somedata = `$query_text`");
  $query_data = mysql_fetch_array($ms_query);
  return $query_data;
 }

 function QueryCache($query_text, $seconds)
 {
  $query_data = $mem_cache->fetch("somedata", $query_text);
  if(!$query_data)
  {
   $ms_query = mysql_query("SELECT * FROM sometable WHERE somedata = `$query_text`");
   $query_data = mysql_fetch_array($ms_query);
   $mem_cache->add("somedata", $query_text, false, $seconds)
  }
  
  return $query_data;
 }

 function FetchArray($result)
 {
  foreach($result as $key)
  {
    echo $key;
  }
 }

}
?>

Напишите класс-обертку для работы с php_mysql (либо mysqli), реализуйщий функции кеширования с помощью memcached. Достаточно трех методов:
Query($query_text);//обычный запрос
QueryCache($query_text,$seconds);//кеширование запроса
FetchArray($result);//вывод результатов

Да говно-вопрос.

varg242, 05 Марта 2011

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

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