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

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

+153

1
2
3
4
5
6
7
8
9

if ((this.formType.urlParam === 'sell1') || (this.formType.urlParam === 'supply1')) {
                                url = 'sell-return-list';
                            } else if ((this.formType.urlParam === 'sell2') || (this.formType.urlParam === 'supply2')) {
                                url = 'buy-return-list';
                            } else if ((this.formType.urlParam === 'service1') || (this.formType.urlParam === 'service2')) {
                                url = 'get-service-list';
                            } else {
                                url = 'provide-service-list';
                            }

Ниньзя стайл

nur, 13 Августа 2012

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

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

+18

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

template <typename TYPE> class Ptr
{
public:
    Ptr():
        Pointer_(0),
        IsValid_(false)
    {
    }
    Ptr( const Ptr<TYPE> &other )
    {
        this->Pointer_ = other.Pointer_;
        this->IsValid_ = other.IsValid_;
    }
    Ptr( TYPE* &ptr ):
        IsValid_(true)
    {
        if ( std::find( Ptr<TYPE>::List_.begin(), Ptr<TYPE>::List_.end(), ptr ) == Ptr<TYPE>::List_.end() )
            Ptr<TYPE>::List_.push_back( ptr );
        this->Pointer_ = ptr;
    }
    ~Ptr()
    {
    }

    inline Ptr<TYPE>& operator = ( const Ptr<TYPE> &other )
    {
        this->Pointer_ = other.Pointer_;
        this->IsValid_ = other.IsValid_;

        return *this;
    }

    inline Ptr<TYPE>& operator = ( TYPE* &ptr )
    {
        if ( std::find( Ptr<TYPE>::List_.begin(), Ptr<TYPE>::List_.end(), ptr ) == Ptr<TYPE>::List_.end() )
            Ptr<TYPE>::List_.push_back( ptr );

        this->Pointer_ = ptr;
        this->IsValid_ = true;

        return *this;
    }

    inline bool operator == ( const Ptr<TYPE> &other )
    {
        return (this->Pointer_ == other.Pointer_) ? true:false;
    }

    inline bool operator != ( const Ptr<TYPE> &other )
    {
        return (this->Pointer_ != other.Pointer_) ? true:false;
    }

    inline TYPE* operator -> ()
    {
        return this->Pointer_;
    }

    inline bool isValid() const
    {
        if (!this->IsValid_)
            return false;
        return this->IsValid_ = ( (std::find( Ptr<TYPE>::List_.begin(), Ptr<TYPE>::List_.end(), this->Pointer_ ) == Ptr<TYPE>::List_.end() ) ? false:true );
    }

    inline void release()
    {
        if ( this->isValid() )
        {
            Ptr<TYPE>::List_.erase( std::find( Ptr<TYPE>::List_.begin(), Ptr<TYPE>::List_.end(), this->Pointer_ ) );
            delete this->Pointer_;
        }

        this->Pointer_ = 0;
        this->IsValid_ = false;
    }

    inline TYPE* get()
    {
        return this->Pointer_;
    }
private:
    TYPE* Pointer_;
    mutable bool IsValid_;

    static std::list < TYPE* > List_;
};

template <typename TYPE> std::list < TYPE* > Ptr<TYPE>::List_;

HaskellGovno, 12 Августа 2012

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

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

+107

001
002
003
004
005
006
007
008
009
010
011
012
013
014
015
016
017
018
019
020
021
022
023
024
025
026
027
028
029
030
031
032
033
034
035
036
037
038
039
040
041
042
043
044
045
046
047
048
049
050
051
052
053
054
055
056
057
058
059
060
061
062
063
064
065
066
067
068
069
070
071
072
073
074
075
076
077
078
079
080
081
082
083
084
085
086
087
088
089
090
091
092
093
094
095
096
097
098
099
100

public Quote[] ParseQuotesFromFile(string p){
            HtmlDocument d = new HtmlDocument();
            d.Load(p, new UTF8Encoding());
#if !DEBUG
            return
#endif
#if DEBUG
            var v1 =
#endif
 d.GetElementbyId("body")//main div
#if DEBUG
; var v2 = v1
#endif
.ChildNodes//subnodes
#if DEBUG
; var v3 = v2
#endif
.Where(a => a.Attributes.Contains("class"))//nodes with classes
#if DEBUG
; var v4 = v3
#endif
.Where(a => a.Attributes["class"].Value == "quote")//quotes
#if DEBUG
; var v5 = v4
#endif
.Where(a => a.ChildNodes.Any(b => b.Attributes.Contains("class")))//only real quotes
#if DEBUG
; var v6 = v5
#endif
.Select(a => a.ChildNodes.Where(b => b.Attributes.Contains("class")).ToArray())//only usable
#if DEBUG
; var v7 = v6
#endif
.Where(a => a.Length == 2)//drop ads
#if DEBUG
; var v8 = v7
#endif
.Select(a => new Quote()
{
    Text = conv.HtmlSpecialCharsDecode(a.First(b => b.Attributes["class"].Value == "text").InnerHtml.Replace("<br>", "\r\n"),StringTools.ReplaceType.Both),
    ID=BadParse(GetIdString(a)),
    Score=BadParse(GetScoreString(a))})
#if DEBUG
; var v9 = v8
#endif
.ToArray();
#if DEBUG
            return v9;
#endif
        } private static string GetScoreString(HtmlNode[] a){
#if !DEBUG
            return
#endif
#if DEBUG
            var v1 =
#endif
 a.First(b => b.Attributes["class"].Value == "actions")
#if DEBUG
; var v2 = v1
#endif
.ChildNodes.Where(b => b.Name != "#text")
#if DEBUG
; var v3 = v2
#endif
.First(b => b.Attributes["class"].Value == "rating-o")
#if DEBUG
; var v4 = v3
#endif
.FirstChild
#if DEBUG
; var v5 = v4
#endif
.InnerText;
#if DEBUG
            return v5;
#endif
        } private static string GetIdString(HtmlNode[] a) {
#if !DEBUG
            return
#endif
#if DEBUG
            var v1=
#endif
            a.First(b => b.Attributes["class"].Value == "actions")
#if DEBUG
; var v2 = v1
#endif
.ChildNodes.Where(b => b.Name != "#text")
#if DEBUG
; var v3 = v2
#endif
.First(b => b.Attributes["class"].Value == "id")
#if DEBUG
; var v4 = v3
#endif
                .InnerText.Replace("#", "");
#if DEBUG
            return v4;
#endif
}

Это должен был быть парсер bash.org.ru
Самоое странное, что код даже работает

kasthack, 12 Августа 2012

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

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

+8

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

bool ASN1Parser::ConvertOID(const ASN1Block &blk, char *oid, unsigned int bufSize) {
    int a = 0;
    bool first = true;
    char tmp[32];
    oid[0] = 0;
    for (unsigned int i=0;i<blk.size;i++) {
        unsigned char c = data[blk.offset+i];
        if (c & 0x80) {
            a = (a << 7) | (c & 0x7F);
        } else {
            a = (a << 7) | (c & 0x7F);
            if (!first) {
                sprintf(tmp,".%d",a);
            } else {
                sprintf(tmp,"%d.%d",a/40,a%40);
            }
            a=0;
            first = false;
            if (strlen(tmp) >= bufSize) return false;
            strcat(oid, tmp);
        }
    }
    return true;
}

И еще один говнокодец на тему ASN.1 - распаковка OID'а.
Кто найдет ошибку - получит пирожок с полочки ;)

bormand, 12 Августа 2012

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

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

+24

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

#include <stdio.h>
#include "gost.h"

int main() {
    MemoryFile mf("root.cer");
    MemoryFile mf2("test.cer");

    ASN1Parser parser(mf);
    ASN1Parser parser2(mf2);

    ASN1Block e1[2];
    parser.Split("30{30{A0#,02$1,30#,30#,30#,30#,30{30#,03$2},A3#},30#,03#}", e1);

    unsigned char px[32], py[32], r[32], s[32], h[32];
    for (int i=0;i<32;i++) {
        px[i] = mf.data[e1[1].offset+i+3];
        py[i] = mf.data[e1[1].offset+i+35];
    }

    parser2.Split("30{30$1,30#,03$2}", e1);
    for (int i=0;i<32;i++) {
        r[i] = mf2.data[e1[1].offset+64-i];
        s[i] = mf2.data[e1[1].offset+32-i];
    }

    Gost3411 hash;
    hash.AddData(mf2.data+e1[0].offset-4, e1[0].size+4);
    hash.Finish(h);

    Gost3410 g;
    if (!g.VerifySignature(h, px, py, r, s)) {
        printf("Invalid!");
    } else {
        printf("Valid!");
    }
    return 0;
}

Продолжение http://govnokod.ru/11528. Вот так я проверял валидность сертификата...
Прекрасные говорящие имена переменных...
Удобный парсер ASN.1...
Отличная инкапсуляция...
Ни одного магического числа...

bormand, 12 Августа 2012

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

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

+20

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

//Вколите мне каплю никотина, а когда меня разорвет - сожгите ошметки напалмом.
//Только это смоет мой позор.

//Примерно так у меня выглядит вся обработка исключений:
int main(int, char**) //или wWinMain, не важно
{
	try
	{
		//all stuff here
	}
	catch(std::exception e)
	{
		MessageBoxA(NULL, e.what(), "oops", MB_OK | MB_ICONERROR);
	}
	catch(...)
	{
		MessageBoxA(NULL, "Unknown error", "oops", MB_OK | MB_ICONERROR);
	}
	return 0;
}

//А еще у меня есть свой класс строки:
class MyString
{
	wchar_t * data;
	size_t length;
	MyString(wchar_t * _data, size_t _length) : data(_data), length(_length) { }
public:
	//...
	static MyString MyString::fromNumber(int num) 
	{ //Грешен, определяю функции в заголовочном, хотя это и не тот случай.
		wchar_t * data = new wchar_t[16];
		HRESULT hr = StringCchPrintfW(data, 16, L"%d", num);
		if(FAILED(hr))
		{
			throw std::exception("shit happend inside MyString::fromNumber(int)");
		}
		size_t len = 0;
		hr = StringCchLengthW(data, 16, &len);
		if(FAILED(hr))
		{
			throw std::exception("shit happend inside MyString::fromNumber(int)");
		}
		return MyString(data, len);
	}
};

//А вкупе с обработкой исключений чуть выше...
//Чуть не забыл про auto_ptr. Укорочено до сути.
class Base
{
protected:
	std::auto_ptr<SomeType> data; //А где он инициализируется?...
public:
	virtual ~Base() { }
};
class Deriv : public Base
{
public:
	Deriv() : Base() 
	{ 
		data = std::auto_ptr<SomeType>(new SomeType(args)); //...а вот тут он инициализируется!
	}
	virtual ~Deriv() { }
};

//Паттерны!
class Single
{
	Single(); //Без реализации.
	template<typename T> Single(T); //Без реализации.
	template<typename T1, typename T2> T1 operator = (T2); //Без реализации.
	class Ton
	{
		//...
	};
	static Ton object;
	static Ton initObject();
public:
	Ton & instance() { return object; }
};


//Мне стыдно.

Xom94ok, 11 Августа 2012

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

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

+83

1
2
3
4
5

void somefunc(bool val)
{
    assert(0 == val || 1 == val);
...
}

Многоуровневый говнокод. Во первых, индусское сравнение задом наперед. Во вторых, выражение в ассерте всегда истенно. Когда GCC возбухнул по этому поводу, то код был за-#ifdef-ен, но оставлен для других менее умных компиляторов.

alexzak, 11 Августа 2012

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

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

−101

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

def comment_absolute_url(request, pid):
    """
    This is a function which return paginated absolute URL for comment.
    It replaces default comment.get_absolute_url()
    """
    comment = Comment.objects.get(id=pid)
    pagination_interval = 25 # will be defined by user settings in future
    all_comments = Comment.objects.filter(content_type=comment.content_type, object_pk=comment.object_pk,
        is_public=True, is_removed=False)
    # I will post it to govnokod.ru, okay? ;)
    dummy_counter = int(0)
    needed_page = int(dummy_counter / pagination_interval) + 1
    for dummy_comment in all_comments:
        dummy_counter = dummy_counter + 1
        if dummy_comment.id == comment.id:
            needed_page = int(dummy_counter / pagination_interval) + 1
        # making an URL
    if comment.content_type.name == "forum post":
        item = ForumPost.objects.get(id=comment.object_pk)
        url_prefix = "forum/%s/%i" % (item.forum.url, item.id)
    elif comment.content_type.name == "new":
        item = New.objects.get(id=comment.object_pk)
        url_prefix = "news/%i" % item.id
    fixed_url = "/%s/?page=%i#%i" % (url_prefix, needed_page, int(pid))
    return fixed_url

В Django есть достаточно няшный comments framework. А ещё там есть такой же няшный paginator. А вот вместе их подружить разработчики django почему-то постеснялись, и поэтому comment.get_absolute_url() нихрена не знает о существовании paginator-а. Эта функция призвана помочь безумному автору сослаться на коммент.

Веселье начинается от dummy_counter-а.

fat0troll, 11 Августа 2012

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

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

+75

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

private void viewField(int id) {
        try {
            List<Fieldmaterials> materials = materialsSession.FieldmaterialsBy(id);
            int size = materials.size() - 1;
            int i = 0;
            JSONWriter json = new JSONStringer().object();
            json.key("namefield").array();

            while (i <= size) {
                Fieldmaterials get = materials.get(i);
                json.value(get.getNamefield());
                i++;
            }
            json.endArray();
            json.key("typefield").array();

            i = 0;
            while (i <= size) {
                Fieldmaterials get = materials.get(i);
                json.value(get.getTypefield());
                i++;
            }

            json.endArray();
            json.key("orderfield").array();

            i = 0;
            while (i <= size) {
                Fieldmaterials get = materials.get(i);
                json.value(get.getOrderfieldmaterials());
                i++;
            }
            json.endArray().endObject();

        } catch (Exception e) {
            e.printStackTrace();
        }

Не нашел более хорошего способа сериализовать в JSON. Нужно было оформить поля объектов в массивы, что-бы потом удобно разобрать в javascript

Kerny, 11 Августа 2012

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

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

+52

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

$hash = md5($eshopId."::".
                        $orderId."::".
                	    $_REQUEST["serviceName"]."::".
                        $_REQUEST["eshopAccount"]."::".
                        $_REQUEST["recipientAmount"]."::".
            			$_REQUEST["recipientCurrency"]."::".
            			$_REQUEST["paymentStatus"]."::".
            			$_REQUEST["userName"]."::".
                        $_REQUEST["userEmail"]."::".
            			$_REQUEST["paymentData"]."::".
            			$secretKey);
            if (strtoupper($_REQUEST["hash"]) != strtoupper($hash)) {
                die('Err: wrong hash.');
            }
......

я конечно уважаю modx но ето через чур
http://bezumkin.ru/modx/minishop/extra/intellectmoney.html

Sulik78, 11 Августа 2012