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

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

В номинации:

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

За время:

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

+3

01
02
03
04
05
06
07
08
09
10
11

function get_user_info($client, $param)
{
global $DATABASE;
$getparam = $DATABASE()....;
if($getparam)
{
return true;
}
else
return false;
}

Друг показал в качестве "запроса в бд для получения инфы о юзерах с целью минимизировать код"

AlexBrin, 27 Января 2016

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

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

−1

1
2
3
4
5
6

if (string.IsNullOrWhiteSpace(frmDelegatorLogin.ViewModel.Username))
                throw new Exception("Не введен логин");
            if (dbEmployee == null)
                throw new Exception(string.Format("Не найден пользователь с логином \"{0}\"", frmDelegatorLogin.ViewModel.Username));
            if (dbEmployee.DelegatePassword != frmDelegatorLogin.ViewModel.Password)
                throw new Exception("Указан неверный пароль");

Вот такой вот новый метод валидации полей. Да, логин и пароль вводятся текстом, да, Exception потом ничем не ловится и пользователь получает unhandeled exception...

kerman, 20 Января 2016

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

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

+1

1
2
3

if ($registered_register != "")
    if (substr($registered_register, strlen($registered_register)-1, strlen($registered_register)) == ".")
        $registered_register = substr($registered_register, 0, strlen($registered_register) - 1);

rtrim()?
Нет, не слышали!

Arris, 20 Января 2016

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

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

+7

1
2

function api($method,$params=false) {
if (!$params) $params = array();

Олег Илларионов, Разработчик ВКонтакте
Для тех кто не врубился:
Можно написать $params = array()

d_fomenok, 28 Декабря 2015

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

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

+1

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

//  Qt QML >:D
UI_Button
{
        id: bt_connect
        height: 40
        width: 150
        radiusBorder: 20
        anchors.right: led_connection.left
        anchors.rightMargin: 10
        anchors.verticalCenter: parent.verticalCenter
        textOn: qsTr("Отключиться")
        textOff: qsTr("Подключиться")
        state: putState(QVipGUI.btConnection)
        onClicked:{
            if(QVipGUI.btConnection !== GuiDefines.DEF_STATE_DIMMED) {
                if(QVipGUI.btConnection === GuiDefines.DEF_STATE_OFF) {
                    QVipGUI.setLedConnection(GuiDefines.DEF_STATE_OFF)
                    QVipGUI.method_ActionGui(GuiDefines.ID_ACTION_CONNECT)
                    QVipGUI.method_ActionGui(GuiDefines.ID_ACTION_GETPORT) // <!>

                    var i = 0;
                    while(++i < 1000000){};
                    QVipGUI.method_ActionGui(GuiDefines.ID_ACTION_GETCONF);


                }
                else {
                    QVipGUI.method_ActionGui(GuiDefines.ID_ACTION_DISCONNECT)
                }
            }
        }
 }

Qt-среда, язык описания интерфейса - QML >:D

eremite88, 28 Декабря 2015

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

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

+1

1
2
3

function intersectRange (r1, r2) {
     return !(Math.max(r1['min'], r2['min']) > Math.min(r1['max'], r2['max']));
 }

Пересечения диапазонов.
Это то самое, которое: (r1[min] <= r2[max]) and (r1[max] >= r2[min])
Я ж джва года над этим думал, прежде чем накласть.

bot, 25 Декабря 2015

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

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

+2

1

https://toster.ru/q/274179

Ученики Mr.Shit (беднягу зачмырили и он сменил домен на php. webtm.ru) продолжают радовать

loki90, 11 Декабря 2015

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

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

+5

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

$scales = db_select('_scale', 'q')
        ->condition('q.gender', $gender)
        ->fields('q', array('id'))
        ->execute();

    $i = 1;
    while ($scale = $scales->fetchObject()) {
        $raw_score = 0;

        $query = db_select('_scale_question', 'q')
            ->condition('q.scale_id', $scale->id)
            ->fields('q')
            ->execute();

        while ($item = $query->fetchObject()) {
            $user_answer = db_select('_user_answer', 'q')
                ->condition('q.question_id', $item->question_id)
                ->condition('q.result_id', $result_id)
                ->fields('q', array('answer'))
                ->execute()
                ->fetchField();

            if ($user_answer == $item->expected_answer) {
                $raw_score++;
            }
        }


        $raw_scores_item['scale_id'] = $scale->id;
        $raw_scores_item['score'] = $raw_score;

        $raw_scores[$i] = $raw_scores_item;
        $i++;
    }

Drupal 7, SQL-запросы с DB API Drupal в двойном цикле(!).
...А заменить можно 1 нормальным SQL-запросом.

skigf, 10 Декабря 2015

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

1C / Говнокод #18923

−88

1
2
3

ЕСли ЗаявкаСсылка.таблица.Выгрузить().Колонки.Найти("СхемаЗакупки")  = Неопределено ТОгда
      ТекстЗапроса = 1/0;//вызываем Исключение
     КонецЕсли;

Пунктуация сохранена.

alexinzaz, 27 Октября 2015

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

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

+1

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

$content = array(
    'html' => '',
    'meta_title' => $sitename,
    'meta_keys' => $sitename,
    'meta_desc' => $sitename,
    'left_menu' => '',
    'path' => ''
);

if ($_GET['module'] == 'page')
    $_GET['module'] = 'site';

if (!isset($_GET['module'])) {
    $_GET['module'] = 'site';
}

l('modules');

/* подключение текущего модуля, все остальные грузятся через __autoload() в includes/function.php */
$file = ROOT_DIR . 'modules/' . $_GET['module'] . '/i.php';
if (file_exists($file)) {
    //l('load_module '. $file);
    require($file);
} else {
    echo 'Файл modules/' . $_GET['module'] . '/i.php не найден';
}


if (empty($content['path']) || $content['path'] == '')
    $site->_404();

if (!empty($content['html'])) {
    $content['html'] = fix_content($content['html']);
}

почти на уровне 1 != 1

LesterTeheMolester, 12 Октября 2015

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

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