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

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

В номинации:

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

За время:

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

+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
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72

namespace detail {
    template<entity_event_t Event>
    struct EventHasEntityStateConstructor {
        // Sanity check
        static_assert(static_cast<int32_t>(Event) >= 0
                      && static_cast<int32_t>(Event) < ENTITY_EVENTS_COUNT);

    private:
        struct TwoChar {
            char a, b;
        };

        template<typename T>
        constexpr static TwoChar _check(
            decltype(
                T(std::declval<const entityState_t &>())
            )*
        );

        template<typename T>
        constexpr static char _check(...);

    public:
        constexpr inline static bool value = (sizeof(_check<EntityEvent<Event>>(nullptr)) == sizeof(TwoChar));
    };

    template<typename BusT, entity_event_t Event>
    bool defaultEntityEventPublisher(const BusT & bus, const entityState_t & eventEntity)
    {
        static_assert(EventHasEntityStateConstructor<Event>::value,
                      "defaultEntityEventFactory<Event>() instantiated for a custom\n"
                      "event that does not have a (const entityState_t & eventEntity) constructor.\n"
                      "This should not happen (you'll get a more obscured compiler error anyway)!");
        return bus.publishEmplace<EntityEvent<Event>>(eventEntity);
    }

    template<typename BusT, typename T, T... Is>
    constexpr std::array<EntityEventPublisherPtr<BusT>, sizeof...(Is)>
        createDefaultEntityEventFactories(std::integer_sequence<T, Is...>)
    {
        return {
            [](auto i) -> EntityEventPublisherPtr<BusT> {
                if constexpr (EventHasEntityStateConstructor<static_cast<entity_event_t>(i.value)>::value) {
                    return &defaultEntityEventPublisher<BusT, static_cast<entity_event_t>(i.value)>;
                } else {
                    return nullptr;
                }
            }(std::integral_constant<T, Is>{})...
        };
    }
}

// An (event_number -> EntityState<event_number> 'publishing factory' function) mapping;
// if event N could not be constructed from a single entityState_t reference
// then this table would contain nullptr at the index N
template<typename BusT>
const std::array<EntityEventPublisherPtr<BusT>, ENTITY_EVENTS_COUNT> & getDefaultEntityEventsPublishers() noexcept
{
    static auto factories = detail::createDefaultEntityEventFactories<BusT>(std::make_integer_sequence<int32_t, ENTITY_EVENTS_COUNT>());
    return factories;
}

template<typename BusT>
EntityEventPublisherPtr<BusT> tryGetDefaultEntityEventPublisher(entity_event_t event) noexcept
{
    auto eventNum = static_cast<int32_t>(event);
    if (eventNum >= 0 && eventNum < ENTITY_EVENTS_COUNT) {
        return getDefaultEntityEventsPublishers<BusT>()[eventNum];
    } else {
        return nullptr;
    }
}

PolinaAksenova, 21 Марта 2021

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

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

−2

1
2
3
4
5
6

This would raise the true nightmare. A type variable is a different beast than the actual type of a concrete instance. 
A type variable could resolve to a, e.g. ? extends Comparator<? super Number> to name one (rather simple) example. 
Providing the necessary meta information would imply that not only object allocation becomes much more expensive, 
every single method invocation could impose these additional cost, to an even bigger extend as we are now not only 
talking about the combination of generic classes with actual classes, but also every possible wildcarded combination, 
even of nested generic types.

https://stackoverflow.com/a/38060012

Джавист-долбоеб с пеной у рта защищает type erasure, задавая вопросы "Does it [c#] have an equivalent of Function.identity()? " в комментариях и собирая плюсы таких же поехавших.
В качестве аргументов он предлагает:

1) сложна
2) хранить информацию о типах в рантайме означает что в рантайме придется хранить информацию о типах!!!
3) [s]ма-те-ма-ти-ка[/x] реф-лек-си-я

Причем ведь наверняка знает и про темплейты в крестах, и про то что шарп такой хуйней не страдает.

Fike, 05 Января 2021

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

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

0

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

#include <iostream>
#include <string>
using namespace std;
class Govnokod {
    bool _flag_dot;
    bool _flag_mant;
    int _index_mant;
    bool vetka1(int i, const string stroka) {
        for (int j = i++; j < stroka.length(); j++) {
            switch (stroka[j]) {
            case '.':  
                if (_flag_dot) return false;
                _flag_dot = true;
                break;     
            case '0' ... '9': break; 
            default:
                return false;
                break; }}
        return true;}
    bool vetka2_dalshe(const string stroka) {
        for (int j = 1; j < stroka.length(); j++) {
            switch (stroka[j]) {
            case '0' ... '9': break;
            default:
                return false;
                break; }} 
        return true; }
    bool vetka2(const string stroka) {
        switch (stroka[0]) {
        case '+':
        case '-':
            if (stroka.length() < 2) return false;  
            return vetka2_dalshe(stroka);
            break;   
        case '0' ... '9':
            return vetka2_dalshe(stroka);
            break;
        default:
            return false;
            break; }}
    bool mantissa(const string stroka) {
       for (int j = 0; j < stroka.length(); j++) {
           switch (stroka[j]) {
           case 'e':
           case 'E':            
               if ((_flag_mant) or (j == (stroka.length() - 1))) return false;
               _flag_mant = true;
               _index_mant = j;
               break; }}
       return true; }
    bool Dalshe(int i, const string stroka) {
        _flag_dot = false;
        _flag_mant = false;
        if (not mantissa(stroka)) return false;
        else if (_flag_mant) {
            string sub1 = stroka.substr(0, _index_mant);
            string sub2 = stroka.substr(_index_mant+1);
            return (vetka1(i, sub1) and vetka2(sub2)); }
        else return vetka1(i, stroka); }
    bool proverka(const string stroka) {
        switch (stroka[1]) {
        case '0' ... '9':
            return Dalshe(1, stroka);
            break;   
        default:
            return false;
            break; }}
    bool general_proverka(const string stroka) {
        switch (stroka[0]) {
        case '-':
        case '+':
            if (stroka.length() > 1) return proverka(stroka);
            else return false;
            break;  
        case '0' ... '9':
            return Dalshe(0, stroka);
            break; 
        default:
            return false;
            break; }}
    public:
    long double opros(char s) {
        string argument;
        while (true) {
            cout << "Введите значение " << s << ": ";
            getline(cin, argument);
            if (argument.length() == 0)
                cout << "Вы не ввели значение!" << endl;
            else if (not general_proverka(argument))
                cout << "Некорректное значение!" << endl;
            else 
                return strtold(argument.c_str(), nullptr); }
    }
    } obj;
int main() {
    for (char c = 'a'; c < 'd'; c++) {
        long double result = obj.opros(c);
        cout << "Значение: " << result << " -- корректное!" << endl;
    }
}

Решил попробовать в стиле ООП переписать.

Westnik_Govnokoda, 25 Декабря 2020

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

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

0

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

#include <iostream>
#include <string>
using namespace std;
bool vetka1(bool &flag, int i, const string stroka) {    
    int j = 1;
    for (j += i; j < stroka.length(); j++) {            
            switch (stroka[j]) {              
            case '.':              
                if (flag) return false;
                flag = true;
                break;                        
       case '1' ... '9': break;    
            default: return false; } }    
    return true; }
bool vetka2_dalshe(const string stroka) {
    for (int j = 1; j < stroka.length(); j++) {          
            switch (stroka[j]) {               
            case '0' ... '9': break;   
            default: return false; } }
return true; }
bool vetka2(const string stroka) {
    switch (stroka[0]) {       
    case '+':
    case '-':
        if (stroka.length() < 2) return false;
        return vetka2_dalshe(stroka);
        break;     
    case '0' ... '9': return vetka2_dalshe(stroka); break;        
    default: return false; break; } }
bool mantissa(const string stroka, bool &flag, int &index) {
    for (int j = 0; j < stroka.length(); j++) {       
        switch (stroka[j]) {         
        case 'e':
        case 'E':        
            if (flag) return false;
            if (j == (stroka.length() - 1)) return false;
            flag = true;
            index = j;
            break; } }
    return true; }
bool Dalshe(int i, const string stroka) {    
    int index_mant;
    bool flag_dot = false;
    bool flag_mant = false;
    if (not mantissa(stroka, flag_mant, index_mant)) return false;
    else {   
        if (flag_mant)  {       
            string sub1 = stroka.substr(0, index_mant);      
            string sub2 = stroka.substr(index_mant+1);
            return (vetka1(flag_dot, i, sub1) and vetka2(sub2)); }  
        else return vetka1(flag_dot, i, stroka); } }
bool proverka(const string stroka) {
    switch (stroka[1]) {        
    case '0' ... '9': return Dalshe(1, stroka); break;       
    default: return false; break; } }
bool general_proverka(const string stroka) {
    switch (stroka[0]) {        
    case '-':
    case '+':       
        if (stroka.length() > 1) return proverka(stroka);
        else return false;
        break;     
    case '0' ... '9': return Dalshe(0, stroka); break;        
    default: return false; break; } }
long double opros(char s) {    
    string argument;    
    do {
    cout << "Введите значение " << s << ": ";
    getline(cin, argument);
    if (argument.length() == 0) cout << "Вы не ввели значение!" << endl;    
    else if (not general_proverka(argument)) cout << "Некорректное значение!" << endl;          
    else break;
    } while (true);    
    return atof(argument.c_str()); } 
int main() {    
    for (char i = 'a'; i < 'd'; i++) {       
        long double a = opros(i);
        cout << "Значение: " << a << " - корректное!" << endl; 
} 
}

В общем, частично переписал некоторые куски кода. Отчасти тут предыдущая версия (с дублированием кода),
что, конечно, не менее говно, но зато стало чуть читабельнее и работает без очевидных багов, как в исходной версии,
например: если в исходной версии ввести - "3w", то значение отображалось, как корректное, что не верно.

Westnik_Govnokoda, 22 Декабря 2020

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

Python / Говнокод #27167

+3

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

class Metapetuh(type):
    def __subclasscheck__(cls, C):
        return True
    def __instancecheck__(self, other):
        return True


class Petuh(metaclass=Metapetuh):
    pass


issubclass(object, Petuh)  # True
isinstance(42, Petuh)      # True

Мы зашкварили весь «Питон», и теперь все классы в нём — петухи.

По просьбам трудящихся: https://govnokod.ru/27166#comment602776.

gost, 19 Декабря 2020

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

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

+3

1

data:text/html,<body onload=z=c.getContext`2d`,setInterval(`c.width=W=150,Y<W&&P<Y&Y<P+E|9<p?z.fillText(S++${Y=`,9,9|z.fillRect(p`}*0,Y-=--M${Y+Y},P+E,9,W),P))):p=M=Y=S=6,p=p-6||(P=S%E,W)`,E=49) onclick=M=9><canvas id=c>

Энтузиаст создал Flappy Bird на минималках весом всего лишь 228 байт.

А чтобы запустить игру, достаточно скопировать этот код в адресную строку браузера.

3_dar, 05 Декабря 2020

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

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

−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
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

using System;
using System.Linq;
using System.Text.RegularExpressions;

namespace Filter
{
    class MainClass
    {
        /// <summary>
        /// Filter ver. 1.0
        /// Matching only one word
        /// <created>Aleksandr 23.10.2020</created>
        /// </summary>
        /// <param name="args"></param>
        /// 
        public static void Main(string[] args)
        {
            try
            {
                string str = "ехал грека через редиска че за редиска кек";

                //Настройки=====================================================================
            ////Уберите комментарий строки 20 если хотите ввести свое слово, и уберите строку 21
            ///
            //string str1 = Console.ReadLine();       //<==== decomment
            //==================================================================================

            string str1 = "редиска";

            //если разкомментировали строку 21, то замените слово "тварь" на слвоо которое ввели   //<==== decomment
            Regex regex = new Regex(@"редиска(\w*)");
            MatchCollection matches = regex.Matches(str);
            if (matches.Count > 0)
            {
                foreach (Match match in matches)
                {
                    Console.WriteLine(match.Value);
                }

                var arrS = str.Split(' ').Select(s => s.Trim(',', '?', '.', '!')).ToArray();

                for (var i = 0; i < arrS.Length; i++)
                {
                    if (arrS[i].ToLower() == str1)
                        arrS[i] = "***";
                }
                var newStr = string.Join(" ", arrS);

                Console.WriteLine(newStr);
                Console.ReadLine();
            }
            else
            {
                Console.WriteLine("Совпадений не найдено");
            }
            }
            catch (Exception ex)
            {
                Console.WriteLine($"Исключение: {ex.Message}");
                Console.WriteLine($"Метод: {ex.TargetSite}");
                Console.WriteLine($"Информация об исключении: { ex.InnerException}");
            }
        }
    }
}

AlexStrew, 25 Октября 2020

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

Python / Говнокод #26945

+1

1
2
3

https://docs.python.org/3.9/whatsnew/3.9.html#optimizations

См 38 -- > 3.9

MAPTbIwKA, 11 Сентября 2020

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

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

+1

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

SELECT 
    `1000, 1000000 field`
    (SELECT group_concat(band_category_languages.name SEPARATOR ', ')
    FROM calendar_entry_categories
    LEFT JOIN band_category_languages ON band_category_languages.band_category_id = calendar_entry_categories.band_category_id
    WHERE calendar_entry_categories.calendar_entry_id = calendar_entries.id
    AND band_category_languages.language_id = 1
    GROUP BY calendar_entry_categories.calendar_entry_id) AS band_category_string,
    (SELECT entry.id
    FROM calendar_entries AS entry
    WHERE entry.entry_kind_parent_id = calendar_entries.id
    AND entry.profile_type_id = 10869
    AND entry.profile_type = 1) AS move_to_personal_calendar,
    (SELECT count(moved_entry.id)
    FROM calendar_entries AS moved_entry
    WHERE moved_entry.entry_kind_parent_id = calendar_entries.id
    AND moved_entry.status = 3) AS count_moved,
    (SELECT user_images.image
    FROM calendar_entries AS calendar_entries1
    LEFT JOIN user_images ON user_images.id = calendar_entries1.image
    WHERE calendar_entries1.id = calendar_entries.entry_kind_parent_id
    AND calendar_entries.entry_kind = 4) AS festival_image,
    (SELECT countries1.code
    FROM calendar_entries AS calendar_entries1
    LEFT JOIN locations AS locations1 ON locations1.id = calendar_entries1.location_id
    LEFT JOIN countries AS countries1 ON countries1.id = locations1.country_id
    WHERE calendar_entries1.id = calendar_entries.entry_kind_parent_id
    AND calendar_entries.entry_kind = 4) AS festival_country_code,
    (SELECT count(*)
    FROM calendar_entry_likes
    WHERE calendar_entry_likes.calendar_entry_id = calendar_entries.id) AS count_of_likes,
    (SELECT count(*)
    FROM calendar_entry_likes
    WHERE calendar_entry_likes.calendar_entry_id = calendar_entries.id
    AND calendar_entry_likes.user_id = 10869 ) AS is_like,
    (SELECT group_concat(entry_type_languages.name SEPARATOR ', ')
    FROM calendar_entry_types
    LEFT JOIN entry_type_languages ON entry_type_languages.entry_type_id = calendar_entry_types.entry_type_id
    WHERE (calendar_entry_types.calendar_entry_id = calendar_entries.id
    OR calendar_entry_types.calendar_entry_id = calendar_entries.entry_kind_parent_id)
    AND entry_type_languages.language_id = 1
    GROUP BY calendar_entry_types.calendar_entry_id) AS entry_types_string,
    (SELECT band_addresses.city_name
    FROM band_addresses
    WHERE band_addresses.band_id = calendar_entries.profile_type_id) AS band_city_name,
    (SELECT countries.code
    FROM band_addresses
    LEFT JOIN countries ON countries.id = band_addresses.country_id
    WHERE band_addresses.band_id = calendar_entries.profile_type_id) AS band_country_code,
    (SELECT group_concat(genre_languages.name SEPARATOR ', ')
    FROM calendar_entry_genres
    LEFT JOIN genre_languages ON genre_languages.genre_id = calendar_entry_genres.genre_id
    WHERE calendar_entry_genres.calendar_entry_id = calendar_entries.id
    AND genre_languages.language_id = 1
    GROUP BY calendar_entry_genres.calendar_entry_id) AS genre_string
FROM `calendar_entries`
LEFT JOIN `locations` ON `locations`.`id` = `calendar_entries`.`location_id`
LEFT JOIN `calendar_entries` AS `festival` ON `festival`.`id` = `calendar_entries`.`entry_kind_parent_id`
LEFT JOIN `locations` AS `festival_location` ON `festival_location`.`id` = ( 
    SELECT calendar_entries1.id
    FROM calendar_entries AS calendar_entries1
    WHERE calendar_entries1.id = calendar_entries.entry_kind_parent_id
    AND calendar_entries.entry_kind = 4
)
LEFT JOIN `countries` ON `countries`.`id` = `locations`.`country_id`
LEFT JOIN `country_languages` ON `country_languages`.`country_id` = `locations`.`country_id`
LEFT JOIN `states` ON `states`.`id` = `locations`.`state_id`
LEFT JOIN `bands` ON `bands`.`id` = `calendar_entries`.`profile_type_id`
AND `bands`.`soft_delete` = 0
AND `bands`.`status` = 1
LEFT JOIN `band_images` ON `band_images`.`id` = `calendar_entries`.`image`
LEFT JOIN `band_privacy_settings` ON `band_privacy_settings`.`band_id` = `calendar_entries`.`profile_type_id`
WHERE (`bands`.`soft_delete` = 0 OR `bands`.`soft_delete` IS NULL)
AND (`bands`.`status` = 1 OR `bands`.`status` IS NULL)
AND ((
        (SELECT count(*)
        FROM band_administrators
        WHERE band_administrators.band_id = bands.id
        AND band_administrators.user_id = 10869 ) > ? OR `band_privacy_settings`.`band_privacy_level` != ?
    ) AND (
        (SELECT user_type
        FROM users
        WHERE users.id = 10869 ) = ? OR `band_privacy_settings`.`band_privacy_level` != ?
    ) AND (
        (SELECT count(*)
        FROM user_bands
        WHERE user_bands.band_id = bands.id
        AND user_bands.status = 1
        AND user_bands.user_id = 10869 ) > ? OR `band_privacy_settings`.`band_privacy_level` != ?
    ) OR `band_privacy_settings`.`band_privacy_level` IS NULL
) AND (`festival`.`entry_kind` = ? OR `festival`.`id` IS NULL)
AND (date(`calendar_entries`.`date_from`) >= ?)
AND `calendar_entries`.`soft_delete` = ?
AND (`country_languages`.`language_id` = ? OR `country_languages`.`language_id` IS NULL)
AND `calendar_entries`.`entry_kind` IN (?, ?)
AND `calendar_entries`.`event_class` IN (?, ?)
AND `calendar_entries`.`status` = ?
GROUP BY `1000, 1000000 field`
ORDER BY calendar_entries.date_from
LIMIT 1000, 20

Сидить замовник такий не тужить тай думку гадає, а що це я буду платити півтори куска спеціалусту за роботу, як є "профі" за 500 доларів, а тут ше і Вася програміст каже, а шо це ти будеш платити таку суму, як я зроблю дешевше за 300! Настрій хороший, угода укладена, робота кипить. Проходить не так багато часу, як замовник каже, а що це сторінка 2хв грузиться, а Вася каже, то нічо ми то все оптимізуємо! Проходить ще декілька місяців, а результату нема. Тоді замовнек розуміє, що Вася не профі і шукає профі в Х рази дорощого! Не так багато часу пройшло, як профі за 1000 каже, я в два рази оптимізував, більше не реально! Замовник пригнічений, шукає профі, і в один прекрасний момент знаходиться дійсно профі, який по домовленості в зарплату X оптимізував, повикидував, переписав, дописав і в результаті 50 мс при 500000 записів. Підсумок: 300 + 1000 + X != 300 :)

silverreve23, 12 Августа 2020

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

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