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

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

+82.8

1
2
3
4
5
6
7

for (int i = 0; i < list.size(); i++) {
        SpecialItem item = list.get(i);
        if (item != null) {
          item = null;
        }
}
list.clear();

Неиндийский способ очистить java.util.List от вместимого.

xrobak, 23 Марта 2010

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

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

+168.2

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

<?php
$log = $_POST['login']; //экранируем специальные символы (обратной косой чертой) для безопасности 
$pas = md5($_POST['password']); //хэшируем ключевое слово для безопасности

$nKc6g = mysql_connect('localhost', 'root', '') or die ("<strong>Подключения к серверу баз данных не произошло.<br/>Причина: </strong> " . mysql_error()); //подключаемся к серверу баз данных
$BblgeveHue_db = mysql_select_db('my_DB', $nKc6g) or die ("<strong>Ошибка при выборе базы данных:<br/>Причина: </strong> " . mysql_error()); //выбираем базу данных

$zanpoc = "SELECT `id` FROM `novb3oBamevu` WHERE `log`='{$log}' AND `pas`='{$pas}' LIMIT 1"; //готовим запрос
$omnpaBKa_3anpoca = mysql_query($zanpoc) or die ("<strong>Ошибка при запросе. Причина:</strong>" . mysql_error()); //готовим команду отправки запроса

if (mysql_num_rows($omnpaBKa_3anpoca) == 1){ //если такой пользователь нашёлся в базе данных
 $nove = mysql_fetch_assoc($omnpaBKa_3anpoca); //возвращаем ассоциативный массив, соответствующий извлечённому из базы данных полю
 $_SESSION['id_novb3oBamev9'] = $nove['id']; //начинаем сессию со значением "user_id" равным значению "id" извлеченного выше поля
 
 include "cmpaHuLja.php"; //запускаем скрипт "личного кабинета"
}
else{
 die('Такое сочетание login-password не найдены в базе данных. [И даём ссылку на повторную авторизацию].'); //выдаём сообщение в случае не нахлждения такого пользователя в базе данных...
}

mysql_close($nKc6g); //закрываем базу данных
?>

Не говнокод, но переменные доставили

v00d00, 23 Марта 2010

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

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

+152.6

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

function str_replace ( search, replace, subject ) {	// Replace all occurrences of the search string with the replacement string
	// 
	// +   original by: Kevin van Zonneveld (http://kevin.vanzonneveld.net)
	// +   improved by: Gabriel Paderni

	if(!(replace instanceof Array)){
		replace=new Array(replace);
		if(search instanceof Array){//If search	is an array and replace	is a string, then this replacement string is used for every value of search
			while(search.length>replace.length){
				replace[replace.length]=replace[0];
			}
		}
	}

	if(!(search instanceof Array))search=new Array(search);
	while(search.length>replace.length){//If replace	has fewer values than search , then an empty string is used for the rest of replacement values
		replace[replace.length]='';
	}

	if(subject instanceof Array){//If subject is an array, then the search and replace is performed with every entry of subject , and the return value is an array as well.
		for(k in subject){
			subject[k]=str_replace(search,replace,subject[k]);
		}
		return subject;
	}

	for(var k=0; k<search.length; k++){
		var i = subject.indexOf(search[k]);
		while(i>-1){
			subject = subject.replace(search[k], replace[k]);
			i = subject.indexOf(search[k],i);
		}
	}

	return subject;

}

function str_replace(search, replace, subject) { return subject.split(search).join(replace);}

DrFreez, 22 Марта 2010

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

bash / Говнокод #2842

−139

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

#!/bin/bash

  upSeconds=`cat /proc/uptime`;
  upSeconds=${upSeconds%%.*};
  let secs=$((${upSeconds}%60))
  let mins=$((${upSeconds}/60%60))
  let hours=$((${upSeconds}/3600%24))
  let days=$((${upSeconds}/86400))
  echo -n 'Время работы: '
  if [ "${days}" -ne "0" ]
  then
   echo -n "${days} дней"
  fi
  echo -n "${hours} часов ${mins} минут"

Кавайненько показываем uptime компьютера вместо унылого вывода $uptime *_*

snoopcatt, 22 Марта 2010

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

Pascal / Говнокод #2841

+104.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
33
34
35
36
37
38
39
40
41
42
43
44
45
46

procedure TForm1.vibor_chiselClick(Sender: TObject);
begin
case vibor_chisel.ItemIndex of
   0: begin
      procedure TForm1.But_sortClick(Sender: TObject);
       begin
       n:=vib_el_v.value;
       setlength(massiv,n);
        for i:=0 to n-1 do
        massiv[i]:=strtoint(stringgrid1.Cells[i,0]);
         case vibor_sortirovki.itemindex of
         0: begin
             sortpuzirek(n,massiv);
             for i:=1 to n do
             stringgrid1.Cells[i,0]:=inttostr(massiv[i]);
            end;
         1: begin
            end;
         2: begin
            end;
         end;
        end;
       end;
   1: begin
       procedure TForm1.But_sortClick(Sender: TObject);
       begin
       n:=vib_el_v.value;
       setlength(mas_siv,n);
        for i:=0 to n-1 do
        mas_siv[i]:=strtofloat(stringgrid1.Cells[i,0]);
         case vibor_sortirovki.itemindex of
         0: begin
             sort_puzirek(n,mas_siv);
             for i:=1 to n do
             stringgrid1.Cells[i,0]:=floattostr(mas_siv[i]);
            end;
         1: begin
            end;
         2: begin
            end;
         end;
        end;
       end;
      end;
  end;
end;

Орали всей группой. А вы часто описываете процедуры в case of ?)
PS
Работает.)

Grivus, 22 Марта 2010

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

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

+170.2

1

if(!($var==false))

насколько это говнокод?=)

nur, 22 Марта 2010

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

ActionScript / Говнокод #2839

−88

01
02
03
04
05
06
07
08
09
10
11

if (_root["currKeyState"] == 3 and _root["currKeyStateCaps"] == "off") {
			_root["currKeyState"] = 1;
			if ("titleNewRec" == _root["currentInput"] or "msgText" == _root["currentInput"] or "prvSearchStr" == _root["currentInput"] or "mdengiId" == _root["currentInput"] or "fio" == _root["currentInput"] or "alfabplace" == _root["currentInput"] or "commut" == _root["currentInput"]) {
			} else {
			}
		} else if (_root["currKeyState"] == 4 and _root["currKeyStateCaps"] == "off") {
			_root["currKeyState"] = 2;
			if ("titleNewRec" == _root["currentInput"] or "msgText" == _root["currentInput"] or "prvSearchStr" == _root["currentInput"] or "mdengiId" == _root["currentInput"] or "fio" == _root["currentInput"] or "alfabplace" == _root["currentInput"] or "commut" == _root["currentInput"]) {
			} else {
			}
		}

Хотел посмотреть, как реализована логика в старом коде и нашел вот это...

-=Deus=-, 22 Марта 2010

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

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

+103.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

using System;

namespace Laba2
{
	class Program
	{
		public static void Main(string[] args)
		{
			
			int max, a ;
			string c;
			Random rnd=new Random();
			Console.WriteLine("Количество элементов в массиве");
			c=Console.ReadLine();
			a=Convert.ToInt32(c);
			int[] nums = new int[a];
			for (int i=0; i<a; i++)
			{nums [i]= rnd.Next(-100,99);}
			Console.Write("Массив   ");
			foreach (int i in nums)
			Console.Write(i + " "); Console.WriteLine();
			max=nums[0];
			foreach (int i in nums)
			{a=1;    if (max<a) max=a;}
			Console.WriteLine(max);
			Console.Read();


		}
	}
}

Оставлено оригинальное оформление кода. Особое внимание заслуживает куча лишних переменных и циклов....

ZiS, 22 Марта 2010

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

bash / Говнокод #2837

−136.4

1
2
3
4
5

FREEKB=$[$(sed -n 2p /proc/meminfo  |  awk '{ print $2 }')+$(sed -n 3p /proc/meminfo  |  awk '{ print $2 }')+$(sed -n 4p /proc/meminfo  |  awk '{ print $2 }') ]
USEDKB=$[$(head -n 1 /proc/meminfo | awk '{ print $2 }')-$FREEKB ]
 
echo "Свободно: $[$FREEKB/1024] Мегабайт ($FREEKB килобайт)"
echo "Занято: $[$USEDKB/1024] Мегабайт ($USEDKB килобайт)"

Получение свободной и использованной оперативной памяти (;

snoopcatt, 22 Марта 2010

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

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

+63.6

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

D3DXINLINE
D3DXVECTOR2::D3DXVECTOR2( CONST FLOAT *pf )
{
#ifdef D3DX_DEBUG
    if(!pf)
        return;
#endif

    x = pf[0];
    y = pf[1];
}

D3DXINLINE
D3DXVECTOR2::D3DXVECTOR2( CONST D3DXFLOAT16 *pf )
{
#ifdef D3DX_DEBUG
    if(!pf)
        return;
#endif

    D3DXFloat16To32Array(&x, pf, 2);
}

D3DXINLINE BOOL
D3DXVECTOR3::operator == ( CONST D3DXVECTOR3& v ) const
{
    return x == v.x && y == v.y && z == v.z;
}

D3DXINLINE BOOL
D3DXVECTOR3::operator != ( CONST D3DXVECTOR3& v ) const
{
    return x != v.x || y != v.y || z != v.z;
}

Во-первых, классный "режим отладки". Если D3DX_DEBUG определён, то при нулевом указателе pf происходит тихий возврат. Совершенно, так сказать, по-английски. А вот анализ второй функции показывает, что данный исходник вообще не получится собрать с D3DX_DEBUG. Наглядная иллюстрация вреда тупого копипаста. Определения операторов == и != демонстрируют, как не надо сравнивать вещественные числа.
Кто же автор этого безобразия? Имя не известно, зато известно место работы. Ибо этот "код" полностью представлен в файле d3dx9math.inl от компании Microsoft.

whiskey, 22 Марта 2010

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

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

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

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

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

bash / Говнокод #2842

Pascal / Говнокод #2841

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

ActionScript / Говнокод #2839

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

bash / Говнокод #2837

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