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

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

В номинации:

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

За время:

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

+3

01
02
03
04
05
06
07
08
09
10

function getLastItemName( $items ) {

  $lastItemName = '';

  foreach( $items as $item ) {
    $lastItemName = $item['name'];
  }

  return $lastItemName;
}

Имеем в проекте кучу россыпей подобного вида.
Собственно, существует ли эффективный способ с этим бороться?

bot, 18 Января 2016

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

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

0

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

// channel.h

#pragma once

namespace AMQP {

class Channel
{
private:
    std::shared_ptr<ChannelImpl> _implementation;

public:
    Channel(Connection *connection) : _implementation(new ChannelImpl()) 
    {
        // attach the connection to the channel
        _implementation->attach(connection);
    }
    
    Channel(const Channel &channel) = delete;

    virtual ~Channel() 
    {
        // close the channel (this will eventually destruct the channel)
        _implementation->close();
    }

    //...
};

// ---------------------------------------------------------
// amqpcpp.h

/**
 *  AMQP.h
 *
 *  Starting point for all includes of the Copernica AMQP library
 *
 *  @documentation public
 */

#pragma once

// base C++ include files
#include <vector>
#include <string>
#include <memory>
#include <map>
//...

// base C include files
#include <stdint.h>
#include <math.h>

// forward declarations
#include <amqpcpp/classes.h>

// utility classes
#include <amqpcpp/endian.h>
#include <amqpcpp/buffer.h>
#include <amqpcpp/bytebuffer.h>
//...

// amqp types
#include <amqpcpp/field.h>
#include <amqpcpp/numericfield.h>
#include <amqpcpp/decimalfield.h>
#include <amqpcpp/stringfield.h>
//...

// envelope for publishing and consuming
#include <amqpcpp/metadata.h>
#include <amqpcpp/envelope.h>
#include <amqpcpp/message.h>

// mid level includes
#include <amqpcpp/exchangetype.h>
#include <amqpcpp/flags.h>
#include <amqpcpp/callbacks.h>
#include <amqpcpp/deferred.h>
#include <amqpcpp/deferredconsumer.h>
#include <amqpcpp/deferredqueue.h>
#include <amqpcpp/deferreddelete.h>
#include <amqpcpp/deferredcancel.h>
#include <amqpcpp/deferredget.h>
#include <amqpcpp/channelimpl.h>
#include <amqpcpp/channel.h>
//...

https://github.com/CopernicaMarketingSoftware/AMQP-CPP

scp, 23 Декабря 2015

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

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

+2

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

class Order_OrderProcessorBuilder
{
    /**
     * @param $processorName
     * @return Order_Abstract_OrderAbstract
     */
    private static function createOrderProcessor($processorName)
    {
        $processorClassName = 'Order_Types_'.ucfirst($processorName);
        return new $processorClassName;
    }

    /**
     * @param $builderName
     * @param Order_Abstract_OrderAbstract $orderProcessor
     * @return Order_Abstract_BuilderAbstract
     */
    private static function createOrderBuilder($builderName, Order_Abstract_OrderAbstract $orderProcessor)
    {
        $builderClassName = 'Order_Builders_'.ucfirst($builderName);
        return new $builderClassName($orderProcessor);
    }

    /**
     * @param Order_Abstract_BuilderAbstract $orderBuilder
     * @return Order_Director
     */
    private static function createDirectorClass(Order_Abstract_BuilderAbstract $orderBuilder)
    {
        return new Order_Director($orderBuilder);
    }

    /**
     * @param $orderTypeName
     * @return Order_Abstract_OrderAbstract
     */
    public static function build($orderTypeName)
    {
        /** @var Order_Abstract_OrderAbstract $orderProcessor */
        $orderProcessor = self::createOrderProcessor($orderTypeName);
        /** @var Order_Abstract_BuilderAbstract $orderBuilder */
        $orderBuilder = self::createOrderBuilder($orderTypeName, $orderProcessor);
        $director = self::createDirectorClass($orderBuilder);

        $director->construct();
        return $orderBuilder->getResult();
    }
}

Вот такая у нас есть прослойка, которая работает с паттероном "строитель", честно говоря недавно понял зачем она нужна, не знаю.. Как вам решение? Код не мой

proweber1, 16 Сентября 2015

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

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

+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
38
39
40
41
42
43
44
45
46
47

#include <bits/stdc++.h>
#define MP make_pair
#define PB push_back
#define int long long
#define st first
#define nd second
#define rd third
#define FOR(i, a, b) for(int i =(a); i <=(b); ++i)
#define RE(i, n) FOR(i, 1, n)
#define FORD(i, a, b) for(int i = (a); i >= (b); --i)
#define REP(i, n) for(int i = 0;i <(n); ++i)
#define VAR(v, i) __typeof(i) v=(i)
#define FORE(i, c) for(VAR(i, (c).begin()); i != (c).end(); ++i)
#define ALL(x) (x).begin(), (x).end()
#define SZ(x) ((int)(x).size())
#ifdef LOCAL
#define debug(x) {cerr <<#x<<" = " <<x<<"\n"; }
#define debug2(x, y) {cerr <<#x<<" = " <<x<<", "<<#y <<" = " <<y <<"\n";}
#define debug3(x, y, z) {cerr <<#x<<" = " <<x<<", "<<#y <<" = " <<y <<", "<<#z<<" = "<<z<<"\n";}
#define debug4(x, y, z, t) {cerr <<#x<<" = " <<x<<", "<<#y <<" = " <<y <<", "<<#z<<" = "<<z<<", "<<#t <<" = " <<t<<"\n";}
#define debugv(x) {{cerr <<#x <<" = "; FORE(itt, (x)) cerr <<*itt <<", "; cerr <<"\n"; }}
#else
#define debug(x)
#define debug2(x, y)
#define debug3(x, y, z)
#define debug4(x,y,z,t)
#define debugv(x)
#define cerr if(0)cout
#endif
#define make(type, x) type x; cin>>x;
#define make2(type, x, y) type x, y; cin>>x>>y;
#define make3(type, x, y, z) type x, y, z; cin>>x>>y>>z;
#define make4(type, x, y, z, t) type x, y, z, t; cin>>x>>y>>z>>t;
#define next ____next
#define prev ____prev
#define left ____left
#define hash ____hash
using namespace std;
typedef long long ll;
typedef long double LD;
typedef pair<int, int> PII;
typedef pair<ll, ll> PLL;
typedef vector<int> VI;
typedef vector<VI> VVI;
typedef vector<ll> VLL;
typedef vector<pair<int, int> > VPII;
typedef vector<pair<ll, ll> > VPLL;

Даже #define int long long есть
http://codeforces.com/contest/575/submission/12867420

3_dar, 15 Сентября 2015

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

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

+142

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

<?php

$self=$_SERVER['PHP_SELF'];

if (isset($_GET['page'])) { $page=$_GET['page'];} else {$page=1;}

$pages=25;
$page_offset=5;
$unique_links=9;

if ($page != 1) { // Don't show back link if current page is first page.
$back_page = $page-1;
echo("<a href=\"$self?page=$back_page\">prev</a>    \n");}

for ($i=1; $i <= $pages; $i++)  {

if ($i == $page) {
 echo("<b>$i</b>\n");// If current page don't give link, just text.
 }  else {

if (($i>$page-$page_offset) && ($i<$page+$page_offset)  && ($page>$page_offset)) {	
echo("<a href=\"$self?page=$i\">$i</a> \n");
} 
elseif (($i<=$unique_links) && ($page<=$page_offset)) {
	echo("<a href=\"$self?page=$i\">$i</a> \n");
}
elseif (($i>$pages-$unique_links) && ($page>$pages-$page_offset)) {
echo("<a href=\"$self?page=$i\">$i</a> \n");	
}

}

}

if (!($page >= $pages) && $pages != 1) { // If last page don't give next link.
$next_page = $page + 1;
echo("<a href=\"$self?page=$next_page\">next</a>");}


?>

Адовый аромат (хоть и рабочий), понимаю, но помогите упростить, пожалуйста. Условия внутри цикла скорее всего избыточны.
Вкратце: "плавающий" пагинатор, 9 уникальных ссылок.

nurfed362, 30 Июня 2015

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

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

+56

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

std::map<int, int> aSummator; //Массив частичных сумм
std::vector<int> v; //Исходный массив

void InitSummator()
{
    aSummator[0] = v[0];
    aSummator[-1] = 0;

    for(int i = 1; i < int(v.size()); i++)
    {
        aSummator[i] = aSummator[i - 1] + v[i];
    }
}

int GetSum(int l, int r)
{
    return aSummator[r] - aSummator[l - 1]; 
}

Как я писал сумматор 0.1 года назад. Вместо того, чтобы написать один if, я использовал std::map, что увеличило ассимптотику алгоритма на запрос с O(1) до O(log(n)). Но задачу при тех ограничениях (в массиве до 100000 элементов, запросов не более 100000) алгоритм решил. Преподу, естественно, показывать забоялся.

Janycz, 03 Апреля 2015

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

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

+67

1
2
3

public boolean isUsing(AttributeType type) {
        return (this.attributeTypes.indexOf(type) >= 0);
}

низкоуровневое программирование

dmli, 24 Марта 2015

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

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

+74

1
2
3
4
5
6
7

public void setSkyboxName(String skyboxName) {
    if (skyboxName != null && skyboxName.length() > 2) {
      this.setSkybox(new SkyBox(skyboxName));
    } else {
      this.setSkybox(null);
    }
  }

если имя неба длинна > 2 значит создать новое небо,
а иначе неба не будет вообще

dmli, 18 Марта 2015

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

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

+145

1
2
3
4
5
6
7

/* */
        int fooBar() {
            /* do something */

/* - */     return NULL;
/* + */     return 0ULL;
        }

> src/foobar.c:42:3: warning: return makes integer from pointer without a cast
> return NULL;

П О Ч И Н Е Н О

Elvenfighter, 11 Марта 2015

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

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

+103

1

if (openFileDialog1.ShowDialog().ToString() == "OK")

не хватает еще

if(1.ToString() == "1")

Lokich, 13 Февраля 2015

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

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