По колено в коде — Говнокод.ру

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

+177

1
2
3
4

if (!($user_info['room'] == 900002) || $user_info['room'] != 900002) { 
      header("Location: main.php");  
      die(); 
  }

Разгребаю чужой код и нарыл.
Чтобы условие выглядело посолидней ... или я чего-нить не понимаю.

AndryG, 28 Октября 2010

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

VisualBasic / Говнокод #4474

−84

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

If Not CheckX Then Goto 1
    For i = 0 to USecX
	Goto 2
1:
    For i2 = 0 to USecY
2:
  	...
	If CheckX Then Goto 3
    Next
    If Not CheckX then Goto 4
3:
    Next
4:

Как-то потребовалось выполнить немалый участок кода, но в двух разных случаях нужно было использовать два разных фора. В отдельную процедуру было очень неудобно выносить (пришлось бы передавать слишком много переменных), так что пришлось сделать этот... свичджампинг форов :)

DragDen, 28 Октября 2010

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

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

+163

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

LPBYTE OUTBUFF = NULL;
	DWORD BUFFSIZE = 0;
...
		int pgp_res = m_pgp.EncodeBuff2Buff(
			( LPCVOID ) ( LPCTSTR ) post_data.c_str(),
			( DWORD ) post_data.length(),
			OUTBUFF,
			BUFFSIZE,
			( LPCTSTR ) query_mts_cfg().query_crypto_public_key().c_str() ); 
...
			std::string s;
			// мы будем рассматривать буфер как строку, а PGPEncode
			// естественно не ограничивает буфер нулевым
			// символом, поэтому строку из буфера надо
			// ограничивать насильно
			s = std::string( ( char* ) OUTBUFF );
			s = s.substr( 0, BUFFSIZE );

1) Проблема проявилась после 5 лет боевой эксплуатации.
2) Все загодногожено было именно так. Т.е. автор понимал \0, сделал что-то, но все равно с ошибкой.

bsivko, 28 Октября 2010

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

Куча / Говнокод #4472

+125

1
2
3
4
5

<form ...>
...
<input type="text"  id="fname" name="fname"  method="post" maxlength=4></input>
...
</form>

fddpro, 28 Октября 2010

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

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

+122

1
2
3
4
5
6
7
8

Код из нового  курса  Е.Попова по JS и JQuery

var name = "Иваh";
for (var i = 0; i < name.length; i++) {
    if (name.charCodeAr(i) < 1040 || name.charCodeAt(i) > 1103 {
        document.write("Найден символ: " + name.charAt(i));
    }
}

Vasiliy, 28 Октября 2010

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

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

+77

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

;
                Scanner reader = new Scanner(System.in);
                int razm = 0;
                boolean flag = false;

                int[] rez = new int[100];
                for (int i = 0; i < 100; i++)
                         rez[i] = i;

                do {
                        System.out.println("введите размерность матрицы");
                        razm = reader.nextInt();

                        for (int i = 0; i < 100; i++) {
                                if (rez[i] == razm)
                                        flag = true;
                        }

                        System.out.println("размерность не является целочисленным типом,повторите ввод");

                } while (flag == false);

                double[][] matrix = new double[razm][razm];
;

Студенческое изделие, блинННН.
Я даже не знаю, как это прокомментировать....
я не могу себе представить, КАК надо мыслить, чтоб такое учудить.
Ну а , то, что сообщение про неправильный ввод выводится
безотносительно к правильности введённого - это уже так, мелочи жизни.

minixoid, 28 Октября 2010

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

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

+166

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

// Обторное, свежее, сочнейшие гавнище. Давно такого не встречал.


curl_setopt($ch, 42, 1);
curl_setopt($ch, 47, 1);
curl_setopt($ch, 43, 1);
curl_setopt($ch, 41, 0);
curl_setopt($ch, 52, 0);
curl_setopt($ch, 81, 0);
curl_setopt($ch, 64, 0);

if($this->binary) {
       curl_setopt($ch, 19914, 1);
}

Вот так нужно задавать настройки cURL. И никакой обфуксации не надо. Суть гавна поймут только истиные ценители.

j0kz, 27 Октября 2010

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

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

+154

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

////.....
			// 
			// button2
			// 
			this->button2->Name = L"button2";   //Кнопка с номером 2,
			this->button2->Text = L"3";         //текстом "3"
			            //и вызывающая обработчик кнопки 1.
			this->button2->Click += gcnew System::EventHandler(this, &Form1::button1_Click); 
			// 
			// button3
			// 
			this->button3->Name = L"button3"; //Кнопка с номером 3,
			this->button3->Text = L"5";       //текстом "5"
			            //тоже вызывающая обработчик кнопки 1
			this->button3->Click += gcnew System::EventHandler(this, &Form1::button1_Click);
////.....
#pragma endregion
	private: System::Void button6_Click(System::Object^  sender, System::EventArgs^  e) {
			 }
private: System::Void button1_Click(System::Object^  sender, System::EventArgs^  e) {

			 if (Form1::textBox1->Text=="0")
Form1::textBox1->Text=((System::Windows::Forms::Button^ )sender)->Text;
			 else
				 Form1::textBox1->Text+=((System::Windows::Forms::Button^ )sender)->Text;
		 }
private: System::Void button17_Click(System::Object^  sender, System::EventArgs^  e) {
			 Form1::textBox1->Text="0";
			 m_Box=0;		 }
private: System::Void button15_Click(System::Object^  sender, System::EventArgs^  e) {
			 float x = float::Parse(Form1::textBox1->Text);

			 m_Box = float::Parse(Form1::textBox1->Text);
			
			  Form1::textBox1->Text="";
			 oper =((System::Windows::Forms::Button^)sender)->Text;
		 }
private: System::Void button16_Click(System::Object^  sender, System::EventArgs^  e) {
			 float x = float::Parse(Form1::textBox1->Text);
			
if (oper=="+")
m_Box+=x;
if(oper=="-")
m_Box-=x;
if(oper=="*")
m_Box*=x;
if (oper=="/")
m_Box /=x;
Form1::textBox1->Text=m_Box.ToString();
		 }
};
}

Это кусок простейшего калькулятора одной девушки. Кроме всего прочего вышеописанный код составляющий единственную логику приложения находится в "Form1.h".

nekotwi, 27 Октября 2010

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

Objective C / Говнокод #4467

−114

1
2
3
4
5
6
7
8
9

bool userNameChanged = true;
  [userName isEqualToString:oldUserName]!=YES;
  bool passwordChanged = true;
  [password isEqualToString:oldPassword]!=YES;
  bool retVal = true;
  if(userNameChanged)
  {
    retVal &= setUserNameInPreferences(userName);
  }

Чудеса сравнения

apodrugin, 27 Октября 2010

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

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

−165

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

def getrows(c):
    return \
        sum( # суммируем по категориям
            map(
                lambda category:
                    sum( # суммируем по itemid
                        map(
                            lambda itemidset:
                                sum( # суммируем по itemstring
                                    map(
                                        lambda itemstring:
                                            map(
                                                lambda info:
                                                    transaction(category, itemstring, info),
                                                itemidset[itemstring].values()
                                            ),
                                        itemidset.keys()
                                    ),
                                []),
                            c[category].values()
                        ),
                        []
                    ),
                ("completedAuctions", "failedAuctions", "completedBidsBuyouts")
            ),
            []
        )

Залез в свой старый скрипт, генерирующий отчеты, думал кое-что подправить...
Увидел одну из функций (эту) и решил лучше забить тут что-то править хД

Причины такого ужаса - наверно отсутствие явной типизации в питоне и средств рефакторинга (rename variable, extract method, ...) в "IDE"

burdakovd, 27 Октября 2010