1. SQL / Говнокод #12044

    −117

    1. 1
    2. 2
    3. 3
    4. 4
    5. 5
    6. 6
    7. 7
    8. 8
    9. 9
    Чудеса от Oracle. 
    
    --------- работает: 
    select to_number(null) from dual union all
    select to_char(null)       from dual
    
    ------------- меняем строчки местами - не работает
    select to_char(null)       from dual union all
    select to_number(null) from dual

    Null - это чудо расчудесное, что на дереве растет. Кто его увидит, тот от ужаса замрёт.

    dim1r, 02 Ноября 2012

    Комментарии (15)
  2. Куча / Говнокод #12042

    +100

    1. 1
    2. 2
    3. 3
    4. 4
    <form action="/cgi-bin/repare.mpl" method="post" enctype="multipart/form-data" class="frm">
    	<input type="hidden" name="sql" value="UPDATE  parcels p, parcels_doc d SET p.state = 'transporter_store' WHERE  d.doctype = 'confirm.L'  AND d.docid IN ($SORD)  AND p.good  IN ($ter)  AND d.docitem = p.good  AND p.state <> 'transporter_store'  AND d.parcel = p.id">
    	<input type="submit" class="butt" name="main_ok" value="Исправить">
    </form>

    Нашли тут в боевом проекте шедевральную SQL Injection "by design".

    aml, 01 Ноября 2012

    Комментарии (29)
  3. Куча / Говнокод #12041

    +129

    1. 01
    2. 02
    3. 03
    4. 04
    5. 05
    6. 06
    7. 07
    8. 08
    9. 09
    10. 10
    11. 11
    12. 12
    13. 13
    14. 14
    15. 15
    16. 16
    17. 17
    18. 18
    19. 19
    20. 20
    public Match(source:SourceLexemes, namedLinkDictionary:NamedLinkDictionary, settings:MatchSettings):MatchResult
    				def endMatchingBlockLexemesByType = settings._endMatchingBlockLexemesByType;
    				def endMatchingBlockLexemesByValue = settings._endMatchingBlockLexemesByValue;
    				assert([_beginOfMatchBlock, _endOfMatchBlock].ForAll(excludedItem => !endMatchingBlockLexemesByValue.Contains(excludedItem)));
    				def matchLexeme(amountOfEntryBlock=0, matchedLexemes=[], sourceTail=source)
    					|(0, _::_, _)                                                                    => Some(matchedLexemes, sourceTail)
    					|(_, _, lexeme_::_) when endMatchingBlockLexemesByType.Contains(lexeme_._type)   => None()
    					|(_, _, lexeme_::lexemes_)                                                       => 
    						match(lexeme_._value, amountOfEntryBlock)
    							|(value_, _) when endMatchingBlockLexemesByValue.Contains(value_)   => None()
    							|(_beginOfMatchBlock, _)                                            => matchLexeme(amountOfEntryBlock+1, lexeme_::matchedLexemes, lexemes_)
    							|(_endOfMatchBlock, 0)                                              => None()
    							|(_endOfMatchBlock, _)                                              => matchLexeme(amountOfEntryBlock-1, lexeme_::matchedLexemes, lexemes_)
    							|(_, 0)                                                             => None()
    							|(_, _)                                                             => matchLexeme(amountOfEntryBlock, lexeme_::matchedLexemes, lexemes_)
    							|_                                                                  => None()
    					|_                                                                               => None()
    				match(matchLexeme())
    					|None                                                    => MatchResult.Failure(source, namedLinkDictionary)
    					|Some((matchedLexemes_, unmatchedLexemesTail_))          => MatchResult.Success(unmatchedLexemesTail_, updateNamedLinkDictionary(matchedLexemes_, namedLinkDictionary))

    Вчера заглянул в свой код.
    Есть один язык, так вот в нем паттерн-матчинг задается на уровне сигнатур функций.
    В связи с тем, что я каждую ночь читал книгу по кулХацкелу, то то что там было написано - вьелось мне в подкоррку и я этого даже не заметил.
    По ночам когда читаешь что-то, оно проникает в незащищенное подсознание. Я даже не ожидал, что напишу такой код. Но я его написал и только опосля понял, что с кодом что-то не так.
    Беда пришла оттуда, откуда её не ждали.
    Ещё я больше не могу без таких функций, как id или ($) или (.) или стрелочка\рыбка. Меня все это конечно раздражает... Мне кажется, что меня держут в клетке и не дают пользоваться правильными функциями.
    Я всегда считал, что зашквар и профессиональная деформация может быть только от крестов.
    Похоже я очень сильно ошибался и поплатился за это.
    Что посоветуете?

    LispGovno, 01 Ноября 2012

    Комментарии (80)
  4. ActionScript / Говнокод #12040

    −115

    1. 01
    2. 02
    3. 03
    4. 04
    5. 05
    6. 06
    7. 07
    8. 08
    9. 09
    10. 10
    11. 11
    12. 12
    13. 13
    14. 14
    15. 15
    16. 16
    17. 17
    18. 18
    19. 19
    20. 20
    21. 21
    22. 22
    23. 23
    24. 24
    25. 25
    // =================================================================================================
    //
    //	Starling Framework
    //	Copyright 2011 Gamua OG. All Rights Reserved.
    //
    //	This program is free software. You can redistribute and/or modify it
    //	in accordance with the terms of the accompanying license agreement.
    //
    // =================================================================================================
    
    package starling.utils
    {
        /** Returns the next power of two that is equal to or bigger than the specified number. */
        public function getNextPowerOfTwo(number:int):int
        {
            if (number > 0 && (number & (number - 1)) == 0) // see: http://goo.gl/D9kPj
                return number;
            else
            {
                var result:int = 1;
                while (result < number) result <<= 1;
                return result;
            }
        }
    }

    Для кого-то и логарифм - высшая математика... :(

    wvxvw, 01 Ноября 2012

    Комментарии (122)
  5. C++ / Говнокод #12038

    +23

    1. 01
    2. 02
    3. 03
    4. 04
    5. 05
    6. 06
    7. 07
    8. 08
    9. 09
    10. 10
    11. 11
    12. 12
    13. 13
    14. 14
    15. 15
    16. 16
    17. 17
    18. 18
    19. 19
    20. 20
    21. 21
    22. 22
    23. 23
    24. 24
    25. 25
    26. 26
    27. 27
    28. 28
    29. 29
    30. 30
    31. 31
    32. 32
    33. 33
    34. 34
    35. 35
    36. 36
    37. 37
    38. 38
    39. 39
    40. 40
    41. 41
    42. 42
    43. 43
    44. 44
    45. 45
    46. 46
    47. 47
    48. 48
    49. 49
    50. 50
    51. 51
    52. 52
    53. 53
    54. 54
    55. 55
    56. 56
    57. 57
    58. 58
    59. 59
    60. 60
    61. 61
    62. 62
    63. 63
    64. 64
    65. 65
    66. 66
    67. 67
    68. 68
    69. 69
    70. 70
    71. 71
    72. 72
    73. 73
    74. 74
    75. 75
    76. 76
    77. 77
    78. 78
    79. 79
    80. 80
    81. 81
    82. 82
    83. 83
    84. 84
    85. 85
    86. 86
    87. 87
    88. 88
    89. 89
    90. 90
    91. 91
    92. 92
    93. 93
    94. 94
    95. 95
    96. 96
    97. 97
    98. 98
    99. 99
    #include <iostream>
    #include <stdlib.h>
    #include <typeinfo>
     
    using namespace std;
     
    #define ololo for(volatile register int i=0;i<10;++i);
     
    struct VB
    {
            virtual void f() const =0;
    };
     
    class V1: public VB
    {
            void f() const {ololo}
    };
     
    class V2: public VB
    {
            void f() const {ololo}
    };
     
    struct TU1
    {
            inline void f() const {ololo}
    };
     
    struct TU2
    {
            inline void f() const {ololo}
    };
     
    struct TUB
    {
            const type_info* type;
            union 
            {
                    TU1 tu1;
                    TU2 tu2;
            };
     
            template<class T>
            void ctor()
            {
                    this->type=&typeid(T);
            }
            
            template<class T>
            inline void call(const T& f)
            {
                    if(this->type==&typeid(TU1))
                            f(this->tu1);
                    else
                            f(this->tu2);
            }
    };
     
    enum {N=1000, N2=N*50};
     
    int main() {
            cout<<"ok"<<endl;
            {
                    VB*v[N];
                    for(int i=0;i<N;++i)
                            if(rand()%2)
                                    v[i]=new V1;
                            else
                                    v[i]=new V2;
                    volatile clock_t a=clock();
                    for(int j=0;j<N2;++j)
                            for(int i=0;i<N;++i)
                                    v[i]->f();
                    volatile clock_t b=clock();
                    cout<< (double)(b - a) / CLOCKS_PER_SEC<<endl;
            }
            cout<<"ok"<<endl;
            {
                    TUB v[N];
                    for(int i=0;i<N;++i)
                            if(rand()%2)
                                    v[i].ctor<TU1>();
                            else
                                    v[i].ctor<TU2>();
                    struct Continuation
                    {
                            inline void operator()(const TU1& a) const {a.f();}
                            inline void operator()(const TU2& a) const {a.f();}
                    } cps;
                    volatile clock_t a=clock();
                    for(int j=0;j<N2;++j)
                            for(int i=0;i<N;++i)
                                    v[i].call(cps);
                    volatile clock_t b=clock();
                    cout<< (double)(b - a) / CLOCKS_PER_SEC<<endl;
            }
            cout<<"ok"<<endl;
            return 0;
    }

    http://ideone.com/plFaLM
    Тут в соседней теме разгорелся спор, что быстрее - полиморфизм виртуальных функций или полиморфизм tagget union. По сути последнее - выбор по if нужной виртуальной функции. Говорят в Java быстрее второе.
    Тема родилась из http://govnokod.ru/12025#comment158188
    Получилось по результатам измерений:
    Виртуальные функции: 1.8 секунд.
    tagget union: 1.94 секунд.
    Притом это всего 2 полиморфных типа в tagget union, а если рост числа полиморфных классов будет расти, то разрыв между виртуальными функциями и tagget union только увеличится. Притом производительность tagget union будет только падать.
    Тема поднята ещё со взглядом на функциональные языки. Это ведь там так модны ADT с постоянным внутри ifподобным паттернматчингом по ним.
    Жду указания на косяки или способы поднять производительность tagget union.

    LispGovno, 31 Октября 2012

    Комментарии (94)
  6. Куча / Говнокод #12036

    +124

    1. 01
    2. 02
    3. 03
    4. 04
    5. 05
    6. 06
    7. 07
    8. 08
    9. 09
    10. 10
    11. 11
    12. 12
    13. 13
    14. 14
    15. 15
    16. 16
    17. 17
    18. 18
    19. 19
    20. 20
    21. 21
    22. 22
    23. 23
    24. 24
    25. 25
    26. 26
    27. 27
    28. 28
    29. 29
    30. 30
    31. 31
    Сайт ЦИК Украины, нынешние выборы народных депутатов. 
    http://cvk.gov.ua/vnd2012/wp300pt001f01=900.html
    Говно везде - html, css, дизайн, юзабилити. 
    
    Для начала цитаты из html-кода страницы. В шапке мы видим 
    заголовок "XHTML 1.0 Transitional", а в коде - конструкции 
    в стиле 90-х годов, например:
    
    <IMG src="img2012/bar-blue1.jpg" WIDTH=" 1" HEIGHT=20>
    
    <table class=t1 cellspacing=0><tr>
        <td width="50%" class="td2"><font class="f3"> © WWW ІАС "Вибори 
            народних депутатів України"
        <td align="right" class="td2"> 
    </table>
    
    <td class=td10 align=center><font color="maroon">2</font>
    <td class=td2><A class=a1 href="wp302pt001f01=900pf7171=52.html">
    політична партія Всеукраїнське об’єднання "Батьківщина"</A></td>
    
    В стилях тоже все в порядке, товарищи проявили 
    отличное знание спецификаций:
    ... height=14px; ...
    ... background-color: 666699; ...
    ... text-decoration: non ...
    ... background-color : none; ...
    ... valign: center; ...
    
    Про отсутствие смысла я даже не говорю. 
    Вот, например, такой перл:
    FONT.F1 {color: black; font-family:Arial;font-size: 14pt}

    Сайт ЦИК Украины. Не сомневаюсь, что ООО "НПП "Медирент"" и ООО "НПП "Проком" получили за свою работу сотни нефти.

    P.S. Плохо, что в самом говнокоде нельзя применять форматирование bb-тегами, для кучи было бы полезно.

    ZX_Spectrum, 31 Октября 2012

    Комментарии (17)
  7. PHP / Говнокод #12035

    +60

    1. 01
    2. 02
    3. 03
    4. 04
    5. 05
    6. 06
    7. 07
    8. 08
    9. 09
    10. 10
    11. 11
    12. 12
    13. 13
    14. 14
    15. 15
    16. 16
    17. 17
    18. 18
    19. 19
    20. 20
    21. 21
    function GetEndUTCTime($stime) {
            $ddd = split('-', $stime);
            if ($ddd[1] == '01')
                $ddd[1] = '02'; else if ($ddd[1] == '02')
                $ddd[1] = '03'; else if ($ddd[1] == '03')
                $ddd[1] = '04'; else if ($ddd[1] == '04')
                $ddd[1] = '05'; else if ($ddd[1] == '05')
                $ddd[1] = '06'; else if ($ddd[1] == '06')
                $ddd[1] = '07'; else if ($ddd[1] == '07')
                $ddd[1] = '08'; else if ($ddd[1] == '08')
                $ddd[1] = '09'; else if ($ddd[1] == '09')
                $ddd[1] = '10'; else if ($ddd[1] == '10')
                $ddd[1] = '11'; else if ($ddd[1] == '11')
                $ddd[1] = '12'; else if ($ddd[1] == '12') {
                $ddd[1] = '01';
                $ddd[0]++;
            } $result = $ddd[0] . '-' . $ddd[1] . '-' . $ddd[2];
            return $result;
        }
    
    // Returns submitted time in appropriate format (e.g. 2009-04-01T09:43:25.5244704Z)

    Документацыя по API SmartMove.

    Hits, 31 Октября 2012

    Комментарии (6)
  8. Objective C / Говнокод #12034

    −102

    1. 01
    2. 02
    3. 03
    4. 04
    5. 05
    6. 06
    7. 07
    8. 08
    9. 09
    10. 10
    11. 11
    12. 12
    13. 13
    14. 14
    15. 15
    16. 16
    - (int)someMethod
    {
        @try
        {
            // some code
            return 0;
        }
        @catch (NSException *e)
        {
            // some more code
            return -1;
        }
    
        // and some more code
        return -1;
    }

    Вот бывает же...

    glook, 31 Октября 2012

    Комментарии (38)
  9. Си / Говнокод #12033

    +137

    1. 001
    2. 002
    3. 003
    4. 004
    5. 005
    6. 006
    7. 007
    8. 008
    9. 009
    10. 010
    11. 011
    12. 012
    13. 013
    14. 014
    15. 015
    16. 016
    17. 017
    18. 018
    19. 019
    20. 020
    21. 021
    22. 022
    23. 023
    24. 024
    25. 025
    26. 026
    27. 027
    28. 028
    29. 029
    30. 030
    31. 031
    32. 032
    33. 033
    34. 034
    35. 035
    36. 036
    37. 037
    38. 038
    39. 039
    40. 040
    41. 041
    42. 042
    43. 043
    44. 044
    45. 045
    46. 046
    47. 047
    48. 048
    49. 049
    50. 050
    51. 051
    52. 052
    53. 053
    54. 054
    55. 055
    56. 056
    57. 057
    58. 058
    59. 059
    60. 060
    61. 061
    62. 062
    63. 063
    64. 064
    65. 065
    66. 066
    67. 067
    68. 068
    69. 069
    70. 070
    71. 071
    72. 072
    73. 073
    74. 074
    75. 075
    76. 076
    77. 077
    78. 078
    79. 079
    80. 080
    81. 081
    82. 082
    83. 083
    84. 084
    85. 085
    86. 086
    87. 087
    88. 088
    89. 089
    90. 090
    91. 091
    92. 092
    93. 093
    94. 094
    95. 095
    96. 096
    97. 097
    98. 098
    99. 099
    100. 100
    void riffoser_track_writeriff(struct riffoser_track * track,char * filename,riffoser_samplerate_t samplerate,riffoser_bitspersample_t bitspersample) {
    	FILE * fp;
    	unsigned long i1,i2,i3,i4,i5,i6,fpi;
    	unsigned char c1,bytespersample;
    	unsigned char * skipbuf;
    	float fret,val;
    	long ival;
    	unsigned char nomorewaves;
    	riffoser_channel_t chan;
    	bytespersample=bitspersample/8;
    	fp=fopen(filename,"wb");
    	i1=track->channels*samplerate*bytespersample*track->length;
    	i2=4+24+8+i1+(i1%2>0?1:0);
    	riffoser_writestr("RIFF");
    	riffoser_writeint(4,4+24+8+i1+(i1%2>0?1:0));
    	riffoser_writestr("WAVEfmt ");
    	riffoser_writeint(4,16);
    	riffoser_writeint(2,1);
    	riffoser_writeint(2,track->channels);
    	riffoser_writeint(4,samplerate);
    	riffoser_writeint(4,bytespersample*track->channels*samplerate);
    	riffoser_writeint(2,bytespersample*track->channels);
    	riffoser_writeint(2,bytespersample*8);
    	riffoser_writestr("data");
    	riffoser_writeint(4,i1);
    	if (i1%2>0)
    		riffoser_writeint(1,0);
    
    	i1=0;
    	i3=track->waves_count;
    	i6=0;
    	nomorewaves=0;
    	while (1) {
    		chan=i1%track->channels;
    		if (i3==track->waves_count&&!nomorewaves) {
    			i5=i1;
    			0&&printf("%lu: finding nearest wave...\n",i1);
    			for (i2=0;i2<track->waves_count;i2++){
    				if ((track->wavestates[i2]->state==RIFFOSER_WAVESTATE_IDLE)&&(((i3==track->waves_count))||(RIFFOSER_RENDER___FROM(track->wavestates[i2])<=i5))&&(RIFFOSER_RENDER___FROM(track->wavestates[i2])>=i6)) {
    					0&&printf("%lu: found wave %lu with from %f\n",i1,i2,RIFFOSER_RENDER___FROM(track->wavestates[i2]));
    					i3=i2;
    					i5=RIFFOSER_RENDER___FROM(track->wavestates[i2]);
    				}
    				else {
    					0&&printf("%lu: skipping wave %lu\n",i1,i2);
    				}
    			}
    			if (i3!=track->waves_count)
    				0&&printf("next wave will be %lu at %f\n",i3,RIFFOSER_RENDER___FROM(track->wavestates[i3]));
    			else nomorewaves=1;
    		}
    		c1=0;
    		val=0;
    		for (i2=0;i2<track->waves_count;i2++){
    			if (track->wavestates[i2]->state==RIFFOSER_WAVESTATE_RENDERING) {
    				if (RIFFOSER_RENDER___TO(track->wavestates[i2])<i1) {
    					track->wavestates[i2]->state=RIFFOSER_WAVESTATE_FINISHED;
    					0&&printf("%lu: wave %lu finished\n",i1,i2);
    				} else {
    					if (track->wavestates[i2]->channel==chan) {
    //						printf("correct channel\n");
    						track->wavestates[i2]->samplenum++;
    						if (track->wavestates[i2]->samplenum>RIFFOSER_RENDER___WSC(track->waves[i2]))
    							track->wavestates[i2]->samplenum-=RIFFOSER_RENDER___WSC(track->waves[i2]);
    						0&&printf("%lu: rendering wave %lu (%f-%f), samplenum is %f (%f%%)\n",i1,i2,RIFFOSER_RENDER___FROM(track->wavestates[i2]),RIFFOSER_RENDER___TO(track->wavestates[i2]),track->wavestates[i2]->samplenum,RIFFOSER_RENDER___WPP(track->waves[i2],track->wavestates[i2]->samplenum));
    						RIFFOSER_WAVE_FUNC(track->waves[i2],RIFFOSER_RENDER___WPP(track->waves[i2],track->wavestates[i2]->samplenum));
    						fret=fret*track->waves[i2]->amplitude/200;
    						// replace for now
    						val=fret;
    					}
    					else {
    //						printf("wrong channel ( %u / %u )\n",track->wavestates[i2]->channel,chan);
    					}
    					c1=1;
    				}
    			}
    		}
    		if (bytespersample==1) {
    			ival=round(val*2.56)-128;
    			if (ival>127)
    				ival=127;
    		}
    		else if (bytespersample==2) {
    			ival=round(val*655.36);
    			if (ival>65535)
    				ival=65535;
    		}
    		else if (bytespersample==4) {
    			ival=round(val*42949672.96);
    			if (ival>4294967295)
    				ival=4294967295;
    		}
    		riffoser_writeint(bytespersample,ival);
    		if (c1==0) {
    			if (i3!=track->waves_count) {
    				0&&printf("%lu: warping to position %lu\n",i1,i5);
    				i4=(i5-i1)*bytespersample;
    				skipbuf=malloc(i4);
    				memset(skipbuf,0,i4);
    				riffoser_writebuf(i4,skipbuf);

    Полная версия - https://github.com/anon4ik/riffoser/blob/master/libriffoser.c

    kostoprav, 31 Октября 2012

    Комментарии (21)
  10. Java / Говнокод #12032

    +69

    1. 01
    2. 02
    3. 03
    4. 04
    5. 05
    6. 06
    7. 07
    8. 08
    9. 09
    10. 10
    11. 11
    12. 12
    13. 13
    14. 14
    15. 15
    16. 16
    17. 17
    18. 18
    19. 19
    20. 20
    21. 21
    22. 22
    23. 23
    24. 24
    25. 25
    26. 26
    27. 27
    28. 28
    29. 29
    30. 30
    31. 31
    @Override
    public String toString() {
        StringBuilder sb = new StringBuilder();
        sb.append("\nAVPCODE: "+code+'<'+vendorId+'>');
        return sb.toString();
    }
    
    public String getCodeAsStringWithVendorIdAlways(){
        StringBuilder sb = new  StringBuilder();
    
        sb.append(code);
        sb.append('<');
        sb.append(vendorId);
        sb.append('>');
    
        return sb.toString();
    }
    
    public String getCodeAsStringWithVendorIdOnlyForRealVendorIds(){
        StringBuilder sb = new  StringBuilder();
    
        sb.append(code);
    
        if(vendorId > 0){
            sb.append('<');
            sb.append(vendorId);
            sb.append('>');
        }
    
        return sb.toString();
    }

    конвертим Diameter's AVP код/вендор ид в строку.

    "\n" как всегда порадовал.

    и само собой разумеется что только последний метод - с именем в 47 символов - работает именно так как все и ожидают и как всем нужно (соответствует синтаксису конфигов).

    Dummy00001, 30 Октября 2012

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