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

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

В номинации:

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

За время:

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

+158

1
2
3
4
5
6
7
8
9

scope = new Object();
scope = document.getElementById("scope");
…
var block = document.getElementById("scope");
var els = block.getElementsByClassName("noactive");
for(var n in els)
{
els[n].className = "active";
}

Проклятие, этот человек хоть что-нибудь думает, когда кодит? Хочется взять — и… Потому что мне с этим потом работать.

torbasow, 31 Июля 2014

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

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

+159

1
2
3
4
5
6
7

<?
$arr = array(2,1,1,1,1);
echo $arr[rand(0,count($arr)-1)];
/*вероятность двойки 20%, 5 чисел = 100%, 1 число = 20%. Так и делается, ничего не говнокод.
И в других языках по этому же принципу.
*/
?>

Человек спросить, как сделать псевдорандом с вероятностью выпадения числа в 20 процентов. Вот что ему ответили.

timurkin, 12 Июня 2014

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

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

+9

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

// fib.h
#pragma once
using ull = unsigned long long;

inline constexpr ull fib(size_t n, ull first, ull second) {
  return n == 0 ? first : fib(n - 1, second, first + second);
}

inline constexpr ull fib(size_t n) {
  return fib(n, 0, 1);
}
// fibs.h
#pragma once
#include "fib.h"
#include <array>

enum class fibs : ull {
#define FIB(i, val) fib##i = val,
#define COUNT(val) COUNT = val
#include "fibs.inl"
#undef COUNT
#undef FIB
};

std::array<ull, static_cast<size_t>(fibs::COUNT)> const & fibs_values() {
  static std::array<ull, static_cast<size_t>(fibs::COUNT)> values = {
#define FIB(i, val) val,
#define COUNT(val)
#include "fibs.inl"
#undef COUNT
#undef FIB
  };
  return values;
}
// main.cpp
#include "fibs.h"
#include <iostream>

int main() {
  using namespace std;
  for (auto fib : fibs_values()) {
    cout << fib << "," << endl;
  }
  return 0;
}
// fibs.inl = gen.exe > fibs.inl
// gen.cpp
#include "fib.h"
#include <iostream>

int main() {
  using namespace std;
  size_t i;
  for (i = 0; i < 94; i++) {
    cout << "FIB(" << i << ", " << fib(i) << ")" << endl;
  }
  cout << "COUNT(" << i << ")";
  return 0;
}

LispGovno, 04 Декабря 2013

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

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

+23

01
02
03
04
05
06
07
08
09
10
11
12
13
14
15
16
17
18
19
20
21

#ifndef SAFE_RELEASE
#define SAFE_RELEASE(x) \
   if(x != NULL)        \
   {                    \
      x->Release();     \
      x = NULL;         \
   }
#endif

#define SAFE_DELETE(a) if( (a) != NULL ) delete (a); (a) = NULL;

#ifndef SAFE_ARRAY_DELETE
#define SAFE_ARRAY_DELETE(x) \
   if(x != NULL)             \
   {                         \
      delete[] x;            \
      x = NULL;              \
   }
#endif

#define SAFE_FREE( p )      if( p ) { free( p ) ; p=NULL ; }

Я вот все никак не могу забыть старый код из доков макрософт по COM, а также из книги Андре Ла Мота.

Два макроса до сих пор висят среди доков на сайте мс (по коду догадаетесь какие):

http://msdn.microsoft.com/ru-RU/library/windows/desktop/dd743946(v=vs.85).aspx

LispGovno, 01 Июня 2013

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

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

+142

1
2

if(!true) // это - не говнокод. это - заглушка.
                return false;

Оправдывается ещё )

pitch, 21 Февраля 2013

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

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

+67

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

public final class SomeActivity extends Activity {
       @Override
	protected Dialog onCreateDialog(int id) {
		Dialog dialog = null;
		if (id == DialogGenerator.SETTINGS_DIALOG) {
			dialog = mDialogGenerator.createSettingsDialog();
		}  {
			dialog = super.onCreateDialog(id);
		}
		return dialog;
	}

	@Override
	protected void onPrepareDialog(int id, Dialog dialog) {
		super.onPrepareDialog(id, dialog);
		if (id == DialogGenerator.SETTINGS_DIALOG) {
			mDialogGenerator.prepareSettingsDialog((AlertDialog) dialog, someBoolValue, someObjectValue);
		} 
	}
}

public final class DialogGenerator {
	public Dialog createSettingsDialog() {
		int dialogId = SETTINGS_DIALOG;
		int titleId = R.string.settingsTitle;
		String[] itemsArray = getStringArray(R.array.settings);
		ThreeTypeOptionsAdapter adapter = new ThreeTypeOptionsAdapter(mControllerAsActivity, itemsArray,
				mCheckableOptions, mTwoTextOptions);
		AlertDialogCallback dialogCallback = new AlertDialogCallback(mControllerAsDialogHost, dialogId);
		AlertDialog.Builder dialogBuilder = new AlertDialog.Builder(mControllerAsActivity);
		dialogBuilder.setTitle(titleId);
		// костыль
		dialogBuilder.setSingleChoiceItems(itemsArray, 0, null);
		// конец костыля
		dialogBuilder.setAdapter(adapter, dialogCallback);
		AlertDialog dialog = dialogBuilder.create();
		setMainListenersOnDialog(dialog);
		dialog.getListView().setChoiceMode(ListView.CHOICE_MODE_MULTIPLE);
		return dialog;
	}
	
	public void prepareSettingsDialog(AlertDialog dialog, boolean someBoolValue, String someStringValue) {
		//костыль
		ListView dialogList = dialog.getListView();
		dialogList.clearChoices();
		dialogList.setItemChecked(POSITION_FOR_BOOL, someBoolValue);
		//конец костыля
		ThreeTypeOptionsAdapter adapter = (ThreeTypeOptionsAdapter) dialogList.getAdapter();
		adapter.setAdditionalValue(POSITION_FOR_STRING, someStringValue);	
	}
}

Задача: показать диалог, внутри которого есть 3 типа ячеек: выделяемые с чекбоксом справа, невыделяемые с дополнительной надписью справа, обычные невыделяемые.
Примерно так:
Use GPS checkbox
Selected country Russia
Launch some activity

В комментах к методам я описал костыль:
Здравствуйте, дорогие друзья. Сегодня мы с вами поговорим об уникальном виде животных, которых открыли только в конце XX века. Это, дорогие друзья, Ява-обезьяны. Давайте дружно откроем сырцы Андроид-СДК, а именно - класс com.android.internal.app.AlertController .AlertParams - и метод createListView(AlertController). Видите, одна обезьяна решила, что нам будет удобней, если диалог автоматом задисмиссится, когда мы щёлкнем по элементу списка в CHOICE_MODE_NONE, и OnClickListener будет не null? А теперь давайте перейдём в android.app.AlertDialog.Builder. Видите, там другая обезьяна решила, что для multi-choice списка нам не потребуется хитроумный адаптер? Поэтому, дорогие друзья, когда мы поставили кастомный адаптер (setAdapter()) и сопроводили его слушателем, то какой бы мы choice mode не ставили после AlertDialog.Builder.create(), диалог дисмиссился. Поэтому, дорогие друзья, пришлось пойти на хитрость и вначале установить single choice (чтобы флаг mIsSingleChoice стал true), а потом уже ставить адаптер.

QuickNick, 20 Февраля 2013

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

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

+16

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

#include <iostream>
using namespace std;

struct T{int a;T(){cout<<"T"<<this<<endl;}~T(){cout<<"~T"<<this<<endl;}};
struct D:public T{int b;D(){cout<<"D"<<this<<endl;}~D(){cout<<"~D"<<this<<endl;}};

int main() {
   T* a = new D[2];
   cout<<"mission failure"<<endl;
   delete[] a;   
   return 0;
}

Писал тест специально для Тараса:
Виртуальный деструктор в базовых классах нужен:
*Для вызова деструкторов всех потомков. Иногда это не нужно, тк поля структуры тривиальны. Согласен с Тарасом.
*Для правильного выбора перегруженного оператора delete. Создал одним менеджером памяти, а удалил в другой - это плохо. Но также бывает редко и только в специфичных проектах.
Но вот то, что демонстрируется в этом примере - думаю чего-нибудь такого создатели той библиотеки, что ковырял Тарас, и опасались.
Не просто было вызван деструктор только для предков, забыв о деструкторах потомков, но и:
*Деструктор предков был вызван для this не по тем адресам, где расположены объекты.
Не удивлюсь, если:
*В некоторых кулхацкерских реализациях менеджеров памяти будет удален буфер меньшего размера и куча попортится или оставшаяся не удаленная часть массива в куче просто утекет.
http://liveworkspace.org/code/2X3MR0$8

LispGovno, 22 Декабря 2012

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

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

−100

1

(lambda x:request.profile.role if request.profile.role!=5 else 4)(request.profile.role)

хуита, 11 Декабря 2012

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

Си / Говнокод #12168

+140

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

#include <stdio.h>
#include <conio.h>
#include <stdlib.h>
#include <string.h>
#define MAX 100
int main()
{
	do
	{
		int i,f,kol,z=1;
		char s1[MAX],s2[MAX],*ch=" .!?",*s3[MAX];
		fflush(stdin);
		do{puts("Vvedite vash text.");
		gets(s1);}while((s1[0]==' ')||(s1[1]=='\n')||(s1[0]=='\0'));
		strcpy(s2,s1);
		s3[0]=strtok(s2,ch);
		for(i=1;s3[i]=strtok(NULL,ch);i++)
		    {
		    }
		f=i;
		puts("Otsortirovannue slova.");
		for(i=0,kol=0;i<f;i++)
			{
				if((*(s3[i])=='A')||(*(s3[i])=='a'))
							{
								printf("%s ",s3[i]);
								kol++;
								z=0;
				            }
			}
		if(z)puts("slov nety. =(");
		printf("\nKollichestvo slov na 'A' ili 'a': %d",kol);
		puts("\nvash text.");
		puts(s1);
		printf("\n");
		printf("dlya povtora nathmite lubyi klavishu. dlya vuhoda ESC.\n\n");
	}
	while(getch()!=27);
}

Моя первая работа) Оцените) Правда я не помню что она делает)

RaSta-Lion, 21 Ноября 2012

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

Си / Говнокод #12133

+132

1
2
3
4
5
6
7

#define LengthOfArrayInternal(a)(sizeof(a)/sizeof(a[0]))
 
#define is_array(x) _Generic((x), typeof((x)[0])[LengthOfArrayInternal(x)]:1, default: 0)
 
#define COMPILE_TIME_ASSERT(expr) char constraint[expr]
 
#define length_of_array(a) ({COMPILE_TIME_ASSERT(is_array(a)); LengthOfArrayInternal(a)})

Определение длины массива. gcc и с11 only.
Говно как известно рождается в споре:
http://govnokod.ru/12108#comment160631

LispGovno, 15 Ноября 2012

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

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