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

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

В номинации:

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

За время:

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

+317

001
002
003
004
005
006
007
008
009
010
011
012
013
014
015
016
017
018
019
020
021
022
023
024
025
026
027
028
029
030
031
032
033
034
035
036
037
038
039
040
041
042
043
044
045
046
047
048
049
050
051
052
053
054
055
056
057
058
059
060
061
062
063
064
065
066
067
068
069
070
071
072
073
074
075
076
077
078
079
080
081
082
083
084
085
086
087
088
089
090
091
092
093
094
095
096
097
098
099
100

/**
     * 连接指定的手环
     *
     * @param callback
     */
    public void connect(BluetoothDevice device, final ActionCallback callback) {
        this.io.connect(context, device, callback);
    }

    public void setDisconnectedListener(NotifyListener disconnectedListener) {
        this.io.setDisconnectedListener(disconnectedListener);
    }

    /**
     * 和手环配对, 实际用途未知, 不配对也可以做其他的操作
     *
     * @return data = null
     */
    public void pair(final ActionCallback callback) {
        ActionCallback ioCallback = new ActionCallback() {

            @Override
            public void onSuccess(Object data) {
                BluetoothGattCharacteristic characteristic = (BluetoothGattCharacteristic) data;
                Log.d(TAG, "pair result " + Arrays.toString(characteristic.getValue()));
                if (characteristic.getValue().length == 1 && characteristic.getValue()[0] == 2) {
                    callback.onSuccess(null);
                } else {
                    callback.onFail(-1, "respone values no succ!");
                }
            }

            @Override
            public void onFail(int errorCode, String msg) {
                callback.onFail(errorCode, msg);
            }
        };
        this.io.writeAndRead(Profile.UUID_CHAR_PAIR, Protocol.PAIR, ioCallback);
    }

    public BluetoothDevice getDevice() {
        return this.io.getDevice();
    }

    /**
     * 读取和连接设备的信号强度RSSI值
     *
     * @param callback
     * @return data : int, rssi值
     */
    public void readRssi(ActionCallback callback) {
        this.io.readRssi(callback);
    }

    /**
     * 读取手环电池信息
     *
     * @return {@link BatteryInfo}
     */
    public void getBatteryInfo(final ActionCallback callback) {
        ActionCallback ioCallback = new ActionCallback() {

            @Override
            public void onSuccess(Object data) {
                BluetoothGattCharacteristic characteristic = (BluetoothGattCharacteristic) data;
                Log.d(TAG, "getBatteryInfo result " + Arrays.toString(characteristic.getValue()));
                if (characteristic.getValue().length == 10) {
                    BatteryInfo info = BatteryInfo.fromByteData(characteristic.getValue());
                    callback.onSuccess(info);
                } else {
                    callback.onFail(-1, "result format wrong!");
                }
            }

            @Override
            public void onFail(int errorCode, String msg) {
                callback.onFail(errorCode, msg);
            }
        };
        this.io.readCharacteristic(Profile.UUID_CHAR_BATTERY, ioCallback);
    }

    /**
     * 让手环震动
     */
    public void startVibration(VibrationMode mode) {
        byte[] protocal;
        switch (mode) {
            case VIBRATION_WITH_LED:
                protocal = Protocol.VIBRATION_WITH_LED;
                break;
            case VIBRATION_10_TIMES_WITH_LED:
                protocal = Protocol.VIBRATION_10_TIMES_WITH_LED;
                break;
            case VIBRATION_WITHOUT_LED:
                protocal = Protocol.VIBRATION_WITHOUT_LED;
                break;
            default:
                return;
        }

@chineesedoc

iramailforspam, 15 Августа 2016

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

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

−47

1
2
3
4
5
6
7
8
9

Запись = РегистрыСведений.СтатусыОповещенийПредстоящихПлатежей.СоздатьМенеджерЗаписи();
		Запись.Период = Выборка.Период;
		Запись.Контрагент = Выборка.Контрагент;
		Запись.Прочитать();
		Запись.ДатаПолучения = ТекущаяДата();
		
		НачатьТранзакцию();
		Запись.Записать(Истина);
		ЗафиксироватьТранзакцию();

Дополнительная транзакция - лишняя.

ilya2184, 09 Августа 2016

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

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

+2

1
2
3
4
5

template<typename V>
        constexpr Vector(V&& x, V&& y, V&& z) noexcept(std::is_lvalue_reference<V>::value ?
                std::is_nothrow_move_constructible<V>::value  :
                std::is_nothrow_copy_constructible<V>::value) :
            data{std::forward<V>(x), std::forward<V>(y), std::forward<V>(z)} {}

jangolare, 07 Августа 2016

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

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

+3

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

@PostLoad
    protected void initStatus() {
        anState status = null;
        if (isDeleted()) {
            status = anState.S_DELETED;
        } else if (actState != null) {
            if (actState.equals(anState.S_NOT_FILLED)) {
                status = anState.S_NOT_FILLED;
            } else if (actState.equals(anState.S_FORMED)) {
                status = anState.S_FORMED;
            } else if (actState.equals(anState.S_SENT) &&
                    (Document != null && Document.getState().equals(DocumentState.SENT))) {
                status = anState.S_SENT;
            } else if (actState.equals(anState.S_SENT) &&
                    (Document == null ||
                            (Document != null && !Document.getState().equals(DocumentState.SENT)))) {
                status = anState.S_SENDING;
            } else if (actState.equals(anState.S_CANCELLATION_REQUEST)) {
                status = anState.S_CANCELLATION_REQUEST;
            } else if (actState.equals(anState.S_CANCELED)) {
                status = anState.S_CANCELED;
            }
        }
        this.status = status;
    }

> кстати, валится же в методе

private boolean isReadonly(anState status) {
return !status.equals(anState.ACT_FORMED) &&
!status.equals(anState.ACT_NOT_FILLED);
}

> но мы же не знаем, что енумы можно сравнивать и по == (т.к. поля енума - константы), и привычки сравнивать equals на константном выражении тем более не имеем :)

> facepalm.jpg, словом :)

Lure Of Chaos, 28 Июля 2016

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

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

−4

1
2
3
4
5
6
7
8
9

public static int opposite(int number)
     {
         if(number>0){
             return number - number - number;
         } if (number<0){
         return number - number - number;
     }return opposite(number);
     }
     }

Это так я по фасту решил задачу из положительных чисел сделать отрицательные и наоборот :D

HellMoth, 07 Июля 2016

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

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

+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

#include <iostream>
using namespace std;
 
class Boolean {
public:
    Boolean() : m_flag(false), m_val(0) {}
    Boolean(bool flag) : m_flag(flag), m_val(0) {}
    Boolean(bool flag, int val) : m_flag(flag), m_val(val) {}
 
    Boolean operator || (int val) {
        return Boolean(m_flag || val == m_val, m_val);
    }
 
    operator bool() { return m_flag; }
 
private:
    bool m_flag;
    int m_val;
};
 
class Integer {
public:
    Integer() : m_val(0) {}
    Integer(int val) : m_val(val) {}
 
    operator int() { return m_val; }
    Boolean operator == (int val) { return Boolean(val == m_val, m_val); }
    Boolean operator == (const Integer & val) { return Boolean(val.m_val == m_val, m_val); }
 
private:
    int m_val;
};
 
int main() {
    Integer a(10);
 
    cout << bool(a == 15 || 10) << endl;
    cout << bool(a == 15 || 11) << endl;
    cout << bool(a == 15 || 11 || 13 || 11 || 0 || 10 || 5) << endl;
    cout << bool(a == 15 || 11 || 13 || 11 || 0 || 9 || 5) << endl;
 
    return 0;
}

https://ideone.com/xwMvx7

gost, 07 Июля 2016

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

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

+5

01
02
03
04
05
06
07
08
09
10
11

<?
if($login=='')
{
   global $APPLICATION;
   $APPLICATION->ThrowException('Имя входа должно быть заполнено.'); 
   return false;
}

// Получить последнее исключение можно методом 
$APPLICATION->GetException()
?>

Bitrix. Лучшие стандарты веб-разработки и новое ядро! Создатели логики try-catch-throw наверное перевернулись бы 100500 раз в гробу увидев такую "реализацию". Пруф как "нужно делать": https://dev.1c-bitrix.ru/api_help/main/reference/cmain/throwexception.php

zenn1989, 08 Июня 2016

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

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

+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
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49

#include <iostream>
#include <map>

template <typename T1, typename T2, typename F>
void operator ^= (std::map<T1,T2> &m, F func) {
    for (typename std::map<T1,T2>::iterator i = m.begin();  i != m.end(); ) {
        if (func(i))
            i = m.erase(i);
        else
            ++i;
    }
}

template <typename T1, typename T2>
void operator ^= (std::map<T1,T2> &m, T2 value) {
    for (typename std::map<T1,T2>::iterator i = m.begin();  i != m.end(); ) {
        if (value == i->second)
            i = m.erase(i);
        else
            ++i;
    }
}

int main() {
    std::map<int,bool> m = {
        {0,false},
        {1,true},
        {2,false},
        {3,true},
        {4,false},
        {5,true},
        {6,false},
        {7,true},
        {8,false},
        {9,true},
        {10,false}
    };

    for (auto i = m.begin(); i != m.end(); ++i)
        std::cout << "{" << i->first << "," << i->second << "} ";
    std::cout << std::endl;

    m ^= false;
    m ^= [](auto &i) { return i->first == 5; };

    for (auto i = m.begin(); i != m.end(); ++i)
        std::cout << "{" << i->first << "," << i->second << "} ";
    std::cout << std::endl;
}

Для std::vector'a запись не особо-то кошерная:

std::vector<int> v = {1,2,3,4,5};

v.erase(std::remove_if(v.begin(),v.end(),[](int &i){ return i <= 3;}), v.end());

А для std::map даже такого не придумали. Пора исправлять

Antervis, 10 Мая 2016

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

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

+7

1
2
3
4

function checkPermission(user, post) {
  return equals(post.author, user) ||
         user.role = 'admin';
}

when you see it, you'll shit bricks

thepotato, 21 Апреля 2016

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

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

−3

1
2

$this->strQueryText = $strQuery = trim($arParams["QUERY"]);
$this->strTags = $strTags  = $arParams["TAGS"];

Из ядра.
Многобуков не любят.

braun3812, 07 Апреля 2016

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

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