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

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

В номинации:

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

За время:

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

+96

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

{$APPTYPE CONSOLE}
uses SysUtils;
function Add(const x: Integer; i: Integer): Integer; overload;
  begin
    Result := x + i
  end;

function Add(const x: PChar; i: Integer): PChar; overload;
  var
    x1: Integer absolute x;
  begin
    Integer(Result) := x1 + i
  end;


const s: PChar = 'KoKoKoBormondos';
begin
  Writeln (StrLen(Add(s, 5)))
end.

http://govnokod.ru/15510#comment221692

gost, 26 Марта 2014

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

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

+166

1
2

if (value) return (value != null)
           else return false;

Из кода проекта.... Автор уволился

alex123098, 17 Марта 2014

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

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

+155

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

<?php

class WidgetSidebarUser extends Portlet
{
    public function renderContent(){
        if(Yii::app()->user->isGuest)
            Yii::app()->controller->redirect('/logout');
        
        $model = User::model()->findByPk(Yii::app()->user->id);
        $this->render('sidebar/user', array('user'=>$model));
    }
}
?>

// sidebar/user

<aside id="sidebar1" class="span3 sidebar">
    <div class="content left ">
        <?php if(!Yii::app()->user->isGuest) if(Yii::app()->user->id === $user->id): ?>
        ...
        <?php else: ?>
            <?php
            $this->widget('application.widgets.ProfileSidebarOther', array(
                'user' => $user,
                'profile' => $user->profile,
            ));
            ?>
        <?php endif; ?>
    </div>
</aside>

Дорабатываю проект и код немного радует своими проверками. Получается мы выбираем данные пользователя, а потом проверяем "А точно это тот самый пользователь".

md5S, 03 Марта 2014

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

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

+165

1
2
3
4

if (($pos = strpos($_SERVER["REQUEST_URI"], "?")) !== false)
{
	$params = substr($_SERVER["REQUEST_URI"], $pos+1);
	parse_str($params, $_GET);

Bitrix, 404.php

Посоны не слышали про $_SERVER[QUERY_STRING], зато, сука, неймспейсы знают!

Boolean, 26 Января 2014

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

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

+66

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

@NamedQuery(
    		name = "ImageBankTag.findAllTags",
    		query = "select distinct tag from AdTag tag where tag.tag in (:tags) order by tag.tag"
    )
})

@Data
@Entity
@Table(name = "image_bank_tags",uniqueConstraints = {@UniqueConstraint(columnNames={"tag"})})
public class ImageBankTag implements Serializable{

. . .

    @NamedQuery(
    		name = "AdTag.findAllTags",
    		query = "select distinct tag from AdTag tag where tag.tag in (:tags) order by tag.tag"
    )
})

@Data
@Entity
@Table(name = "admanager2_tags",uniqueConstraints = {@UniqueConstraint(columnNames={"tag"})})
public class AdTag implements Serializable{

Такой вот своеобразный джоин, с подстраховкой.

wvxvw, 26 Января 2014

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

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

+158

1
2
3
4

$result_max = mysql_query("SELECT MAX(id)+1 as max FROM cms_competition;");
$row_max = mysql_fetch_object($result_max);
if(!$row_max->max)
      $row_max->max = 1;

Так ведь проще

kindofbear, 24 Января 2014

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

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

+165

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

/* Здесь мы имеем сраный унаследованный пиздец: количество товара передается в параметрах POST-запроса,
	имеющих имена вида productX_count, где X - ИД товара. Как предыдущие разработчики разобирали это счастье?
	Правильно, перебирали все (блядь, ВСЕ, блядь, 20000 штук!) товары и проверяли, есть ли соответствующее
	значение в запросе.
*/

  $items5 = GetIBlockElementList(5);
  while($arItem = $items5->GetNext())
    if(isset($_POST['product'.$arItem['ID']."_id"]) && is_numeric($_POST['product'.$arItem['ID']."_id"]))
    {
      $_PRODUCTS[$arItem['ID']]['count'] = $_POST['product'.$arItem['ID']."_count"];
    }

В этом коде меня утешает только боль и ненависть комментария.

clauclauclau, 20 Января 2014

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

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

−94

01
02
03
04
05
06
07
08
09
10

import time                                                                      
                                                                                 
def inttime():                                                                   
    return int(''.join(str(time.time()).split('.')))      
                                                                                 
def rand():                                                                      
    while True:                                                                  
        s = bin(inttime())[2:]                                                   
        for x in s:                                                              
            yield x

Этим "рандомом" заполняется клеточный автомат "Жизнь". Нечего было делать.

x0m9k, 08 Января 2014

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

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

+157

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

var allowed = [ 0xfe, 0xfc, 0xf8, 0xf0,
                0xe0, 0xc0, 0x80, 0x00 ];
for (var i = 0; i < parts.length; i++) {
    var part = parts[i]; 
   
    // ...

    if ($.grep(allowed, function(a) { return part == a; }).length > 0) {
        max = 0x00;
    } else {
        FocusObject(object);
        show_alert( jstextTemplate("<jstext>field_contains_bad_IP_mask</jstext>", {name: name}) );
        return false;
    }
}

Кусочек валидации для маски подсети. Мы не ищем лёгких путей.

Elvenfighter, 02 Января 2014

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

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

+137

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
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94

using System;
 using System.Collections.Generic;
 namespace Builder
 {
  public class MainApp
  {
    public static void Main()
    {
      // Create director and builders
      Director director = new Director();
 
      Builder b1 = new ConcreteBuilder1();
      Builder b2 = new ConcreteBuilder2();
 
      // Construct two products
      director.Construct(b1);
      Product p1 = b1.GetResult();
      p1.Show();
 
      director.Construct(b2);
      Product p2 = b2.GetResult();
      p2.Show();
 
      // Wait for user
      Console.Read();
    }
  }
  // "Director"
  class Director
  {
    // Builder uses a complex series of steps
    public void Construct(Builder builder)
    {
      builder.BuildPartA();
      builder.BuildPartB();
    }
  }
  // "Builder"
  abstract class Builder
  {
    public virtual void BuildPartA(){}
    public virtual void BuildPartB(){}
    public virtual Product GetResult(){}
  }
  // "ConcreteBuilder1"
  class ConcreteBuilder1 : Builder
  {
    private readonly Product product = new Product();
    public override void BuildPartA()
    {
      product.Add("PartA");
    }
    public override void BuildPartB()
    {
      product.Add("PartB");
    }
    public override Product GetResult()
    {
      return product;
    }
  }
  // "ConcreteBuilder2"
  class ConcreteBuilder2 : Builder
  {
    private readonly Product product = new Product();
    public override void BuildPartA()
    {
      product.Add("PartX");
    }
    public override void BuildPartB()
    {
      product.Add("PartY");
    }
    public override Product GetResult()
    {
      return product;
    }
  }
  // "Product"
  class Product
  {
    private readonly List<string> parts = new List<string>();
    public void Add(string part)
    {
      parts.Add(part);
    }
    public void Show()
    {
      Console.WriteLine("\nProduct Parts -------");
      foreach (string part in parts)
        Console.WriteLine(part);
    }
  }
 }

"Хороший","годный" пример паттерна билдер с википедии, не соответствующие лежащей там же Uml схеме чуть больше чем полностью

Схема
http://upload.wikimedia.org/wikipedia/ru/2/28/Builder.gif

kegdan, 29 Декабря 2013

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

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