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

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

+155

1
2
3

if(file_exists("templates/".$config->design."/amxbans.css")) {
        $smarty->assign("design",$config->design);
}

Ненавижу женскую логику.

theaqua, 18 Декабря 2010

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

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

+109

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

procedure RemoveDir(path: string);
var
  sr: TSearchRec;
begin
  if FindFirst(path + '\*.*', faAnyFile, sr) = 0 then
  begin
    repeat     
      if sr.Attr and faDirectory = 0 then
        DeleteFile(path + '\' + sr.name);
      else
          RemoveDir(path + '\' + sr.name);
    until
      FindNext(sr) <> 0;
  end;
  FindClose(sr);
  RemoveDirectory(PChar(path));
end;

История такова, писал я как-то программу по курсовому. (Это было еще года два-три назад, когда я про юнит тесты и различные технологии проектирования и разработки ничего не знал.) По задумки программа распаковывала некие файлы во временный каталог рядом с приложением и по завершению работы с ними удаляла эту папку вот этим методом. Итак, запустил я эту подпрограмму на выполнение для проверки ее работоспособности и смотрю в файловом менеджере, что папка успешно удалилась и все ок. Но вдруг студия начала жаловаться что нет каких-то файлов, я смотрю в файловый менеджер и вижу что файлы проекта программы исчезают буквально на глазах! Естественно я резко убиваю программу и далее начинаю восстанавливать исходники. Благо, что делал бэкапы и эта подпрограмма проработала все ничего, а ведь последствия могли быть еще больше. А мораль такая, нужно быть предельно внимательным, когда работаешь с удалением какой-либо информации.

P.S. Нужно добавить проверку между 10 и 11 строкой:
if (sr.Name <> '..') and (sr.Name <> '.') then

vanished, 18 Декабря 2010

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

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

+113

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

function FUNC(XD: TXMLDocument; L: ShortString): IXMLNode;
var
  c: integer;
  ln: IXMLNode;
  s: ShortString;
begin
  c:=0;
  while c <> XD.DocumentElement.ChildNodes.Count-1 do
  begin
    if VarToStr(XD.DocumentElement.ChildNodes[c].Attributes['id']) = L then
    begin
      ln:=XD.DocumentElement.ChildNodes[c];
      c:=XD.DocumentElement.ChildNodes.Count-1
    end else
    begin
      c:=c+1; {DEBUG(VarToStr(XD.DocumentElement.ChildNodes[c].Attributes['id'],L) } 
    end
  end;
  result:=ln
end;

К слову о невнимательности и "непримеченных слонах". Итак задачка: перебором найти в наборе данных нужную строку и вернуть в итоговый результате элемент набора данных. Используем код (выше). По невнимательности своей я допустил в алгоритме ошибку и функция работала несколько некорректно. Перед Вами её изначальный вариант, до исправления. Если вкратце, то баг заключался в том, что функция не читала последний элемент XML-документа. Называется, нефиг было мудрить с алгоритмом. По сути там надо убрать из двух мест "-1". Но пока я докапывался до сути ошибки, я написал тестовую функцию, сравнивающую две строки и выводящую результат сравнения в окно сообщения. Назовём её DEBUG. И... Началось непонятное. Функция выдавала сообщение, значит результаты, как видно по алгоритму, не равны, но она выдавала РАВЕНСТВО! Мозг взрывается, ломаются представления о математике и рациональности вообще!

Xander_Bass, 18 Декабря 2010

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

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

+127

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

//TODO: Delete the comment after a successful test.

var Client = new TcpClient();

int a = num._.i.i.i.i.i;
int b = num._.i.i.i.u.d;
int c = (((num)(a + b)).l++.u+b).u.i.i;

// port = 726
int port = a + b + c;

// ips = "127.0.0.1"
string ips = ip._.i.u.i.i.i.u.d.d.d.s['.'].s['.'].s['.'].i.s;

Client.Connect(ips, port);

Very hard-hard code!

vanished, 18 Декабря 2010

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

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

+154

1
2
3
4
5
6

$src = fopen("0.txt", 'r');
while(!feof($src))
{
        $line[] = fgets($src);
        $line++;
}

Интересно чему в конце будет равно $line? :)

Xander_Bass, 17 Декабря 2010

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

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

+77

1
2
3
4
5
6
7
8
9

private String get4DigitRandom() {
        final int length = 4;
        int number = (int) (Math.random() * Math.pow(10, length));
        StringBuilder sb = new StringBuilder();
        for (int i = length; i > 0; i--)
            sb.append("" + (int) (number / Math.pow(10, i - 1)) % 10);

        return sb.toString();
    }

yozh, 17 Декабря 2010

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

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

+119

1

public string flag = "no";

Golovastick, 17 Декабря 2010

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

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

+173

1

AX=pow(A[i].r.get_x()*A[i].r.get_x(),0.5);

Берем модуль числа.

Taus, 17 Декабря 2010

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

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

+163

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

<?php
// Заменить значния, выделенные жирным на действительные
// Соединяемся с "Мускулом"
 define("MYSQL_HOST","myhost.ru",true);
 define("MYSQL_USER","username",true);
 define("MYSQL_PASS","userpass",true);
 define("MYSQL_DB_N","userdb",true);
 define("MYSQL_DB_N","userstable",true);
 $_MYSQL_CONNECT = false; $_MYSQL_ERROR = -1;
 if ($_MYSQL_CONNECT = @mysql_connect(MYSQL_HOST,MYSQL_USER,MYSQL_PASS))
 {
  if (mysql_select_db(MYSQL_DB_N,$_connect))
  {
   $_MYSQL_ERROR = 0; define("MYSQL_CONNECTED",true,true);
  } else { $_MYSQL_ERROR = 2; define("MYSQL_CONNECTED",false,true); }
 } else { $_MYSQL_ERROR = 1; define("MYSQL_CONNECTED",false,true); }
// Проверяем значение
// Определяем список разрешённых неалфавитноцифровых символов
 $_SRCH[0] = '.'; $_SRCH[1] = '_'; $_SRCH[2] = '-'; $_SRCH[3] = ' ';
 if (isset($_POST['user']))
 {
  $_tmp = iconv("UTF-8","CP1251//IGNORE",urldecode($_POST['user']));
  if (ctype_alnum(str_replace($_SRCH,'',$_tmp)) && (strlen($_tmp) < 33))
  {
   if (MYSQL_CONNECTED)
   {
    $_query = 'SELECT * FROM '.MYSQL_US_N." WHERE nick='".$_tmp."'";
    if ($_result = mysql_query($_query,$_MYSQL_CONNECT))
    {
     if ($_row = mysql_fetch_assoc($_result))
     {
      define("_RESULT","RES_EX",true);
     } else { define("_RESULT","RES_OK",true); }
    } else { define("_RESULT","RES_MYSQL_BQ",true); }
   } else { define("_RESULT","RES_MYSQL_NC",true); }
  } else { define("_RESULT","POST_IN",true); }
 } else { define("_RESULT","POST_NS",true); }

qbasic, 17 Декабря 2010

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

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

+160

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

var _nickObj = document.getElementById('userNick');
var _indNick = document.getElementById('userNickInd');
 
var regVerReq;
try { regVerReq = new ActiveXObject("Msxml2.XMLHTTP"); } catch(e)
     { try { regVerReq = new ActiveXObject("Microsoft.XMLHTTP"); } catch(e)
           { regVerReq = false; }
     }
if (!regVerReq && typeof(XMLHttpRequest)!='undefined') { regVerReq = new XMLHttpRequest(); }
 
function verifyNick()
{
 if (regVerReq)
 {
  regVerReq.open("post","/php/veruser.php",true);
  regVerReq.setRequestHeader("Content-Type", "application/x-www-form-urlencoded");
  var regVerHB = 'user='+encodeURIComponent(_nickObj.value);
  regVerReq.onreadystatechange = function()
  {
   if (regVerReq.status == 200)
   {
    if (regVerReq.readyState == 4)
    {
     switch ( regVerReq.responseText )
     {
      case 'RES_OK':
        _indNick.innerHTML = 'Свободен';
        _indNick.style.color = 'green';
        break;       
      case 'RES_EX':
        _indNick.innerHTML = 'Занят';
        _indNick.style.color = 'red';
        break;       
      default:
        _indNick.innerHTML = 'Некорректно';
        _indNick.style.color = 'red';
        break;       
     }
    }
   } else
   {
    _indNick.innerHTML = 'Ошибка сервиса';
    _indNick.style.color = 'red';
   }
  }
  _indNick.innerHTML = 'Проверка';
  _indNick.style.color = 'gray';
  regVerReq.send(regVerHB);
 }
}