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

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

В номинации:

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

За время:

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)

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

+4

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

<div class="col-sm-3">
	<select class="form-control" id="year">
		<option <?php if($explode[0] == 2002) Echo "selected"; ?> value="2002">2002</option>
		<option <?php if($explode[0] == 2001) Echo "selected"; ?> value="2001">2001</option>
		<option <?php if($explode[0] == 2000) Echo "selected"; ?> value="2000">2000</option>
		<option <?php if($explode[0] == 1999) Echo "selected"; ?> value="1999">1999</option>
		<option <?php if($explode[0] == 1998) Echo "selected"; ?> value="1998">1998</option>
		<option <?php if($explode[0] == 1997) Echo "selected"; ?> value="1997">1997</option>
		<option <?php if($explode[0] == 1996) Echo "selected"; ?> value="1996">1996</option>
		<option <?php if($explode[0] == 1995) Echo "selected"; ?> value="1995">1995</option>
		<option <?php if($explode[0] == 1994) Echo "selected"; ?> value="1994">1994</option>
		<option <?php if($explode[0] == 1993) Echo "selected"; ?> value="1993">1993</option>
		<option <?php if($explode[0] == 1992) Echo "selected"; ?> value="1992">1992</option>
		<option <?php if($explode[0] == 1991) Echo "selected"; ?> value="1991">1991</option>
		<option <?php if($explode[0] == 1990) Echo "selected"; ?> value="1990">1990</option>
		<option <?php if($explode[0] == 1989) Echo "selected"; ?> value="1989">1989</option>
		<option <?php if($explode[0] == 1988) Echo "selected"; ?> value="1988">1988</option>
		<option <?php if($explode[0] == 1987) Echo "selected"; ?> value="1987">1987</option>
		<option <?php if($explode[0] == 1986) Echo "selected"; ?> value="1986">1986</option>
		<option <?php if($explode[0] == 1985) Echo "selected"; ?> value="1985">1985</option>
		<option <?php if($explode[0] == 1984) Echo "selected"; ?> value="1984">1984</option>
		<option <?php if($explode[0] == 1983) Echo "selected"; ?> value="1983">1983</option>
		<option <?php if($explode[0] == 1982) Echo "selected"; ?> value="1982">1982</option>
		<option <?php if($explode[0] == 1981) Echo "selected"; ?> value="1981">1981</option>
		<option <?php if($explode[0] == 1980) Echo "selected"; ?> value="1980">1980</option>
		<option <?php if($explode[0] == 1979) Echo "selected"; ?> value="1979">1979</option>
		<option <?php if($explode[0] == 1978) Echo "selected"; ?> value="1978">1978</option>
		<option <?php if($explode[0] == 1977) Echo "selected"; ?> value="1977">1977</option>
		<option <?php if($explode[0] == 1976) Echo "selected"; ?> value="1976">1976</option>
		<option <?php if($explode[0] == 1975) Echo "selected"; ?> value="1975">1975</option>
	</select>
</div>

Суровые челябинские программисты продолжают кодить :)

by133312, 13 Марта 2016

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

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

+6

1
2
3
4
5
6

void MultMatrix(const GLfloat m1[16], const GLfloat m2[16], GLfloat result[16])
{
  glLoadMatrixf(m1);
  glMultMatrixf(m2);
  glGetFloatv(GL_MODELVIEW_MATRIX, result);
}

LispGovno, 12 Марта 2016

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

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

+5

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

catch(...)
{
	static int i = 0;                     
	//if we enter this catch clause more than 1 time
	//it is very likely that the RestartSystem() command
	//did not succeed. If this is the case we just exit.
	if(i>0)
		exit(0);
	else
		MonitorT::GetInstance()->RestartSystem();
	i++;
	throw;
}

Навеяло...

blackhearted, 11 Марта 2016

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

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