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

• Все
• Лучший
• Сток
• Глупый поиск
• Наговнокодить!
• Oтзывы

Нашли или выдавили из себя код, который нельзя назвать нормальным, на который без улыбки не взглянешь? Не торопитесь его удалять или рефакторить, — запостите его на говнокод.ру, посмеёмся вместе!

1. PHP (5706)
2. C++ (2739)
3. Куча (2374)
4. JavaScript (2032)
5. C# (1929)
6. Java (1502)
7. Си (1117)
8. Pascal (648)
9. Python (588)
10. 1C (539)
11. SQL (432)
12. ActionScript (292)
13. Objective C (202)
14. bash (198)
15. Perl (194)
16. Assembler (146)
17. Ruby (145)
18. VisualBasic (138)
19. Lua (49)
20. Go (31)
21. Haskell (30)
22. Swift (27)
23. Kotlin (14)

1. C# / Говнокод #13949

   +131

   1. 1
   2. 2
   3. 3
   4. 4
   5. 5

   private unsafe bool IsOptionalOutParamSet(out Guid param)
   {
       fixed (Guid* guidPtr = &param)
           return (IntPtr) guidPtr != IntPtr.Zero;
   }

   Ccik, 15 Октября 2013

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

2. C++ / Говнокод #13948

   +4

   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

   int DataBase::garbageCollector(string filename){ 
       fstream from(filename.c_str());
       fstream from0((filename+".index").c_str());
       Index x;
       Data d;
       char *a/*,*b*/;
       int oldpos = 0 ;
       bool done = false;
       from.seekg(0,ios::end);
       int dbend = static_cast<int>(from.tellp());
       from0.seekg(0,ios::end);
       int indexend = static_cast<int>(from0.tellp());
       if(dbend == BlockSize || indexend == IndexSize){
   	from.close();
   	from0.close();
   	return 0;
       }
       from.seekg(0,ios::beg);
       from0.seekg(0,ios::beg);
       while(true){ 
           from0 >> x;
           if(from0.eof())break;
           if(!x.Num()){ 
               int count = dbend - x.Pointer() - BlockSize;
               int count0 = indexend - (static_cast<int>(from0.tellp())+1);
               from.seekg(x.Pointer()+BlockSize);
               a = new char[count]; 
               from.read(a,count);
               from.seekp(x.Pointer());
               from.write(a,count);
               delete a;
               a = new char[count0];
               from0.seekp(1,ios::cur);
               int test = from0.tellp();
               from0.read(a,count0);
               from0.clear();
               from0.seekp(indexend - count0 - IndexSize,ios::beg);
               test = from0.tellp();
               from0.write(a,count0);
               test = from0.tellp();
               delete a;
               ofstream to("tmp.base",ios::trunc),to0("tmp.index",ios::trunc);
               a = new char[dbend - BlockSize];
               from.seekg(0,ios::beg);
               from.read(a,dbend - BlockSize);
               to.write(a,dbend - BlockSize);
               delete a;
               a = new char[indexend - IndexSize];
               from0.seekg(0,ios::beg);
               test = from0.tellp();
               from0.read(a,indexend - IndexSize);
               to0.write(a,indexend - IndexSize);
               delete a;
               from.close();
               from0.close();
               if(remove(filename.c_str()) || rename("tmp.base",filename.c_str())) return -1;
               if(remove((filename+".index").c_str()) || rename("tmp.index",(filename+".index").c_str())) return -1;
               oldpos = indexend - count0 - IndexSize;
               done = true;
               break;
           }
       }
       if(done){
           from0.open((filename+".index").c_str());
           while(true){ 
               from0 >> x;
               if(from0.eof())break;
               if(x.Pointer() > static_cast<unsigned>((oldpos/23)*BlockSize) ){ 
                   x.Pointer(x.Pointer()-BlockSize);
                   from0.seekp(-(IndexSize-1),ios::cur);
                   from0 << x;
                   int test = from0.tellp();
               }
               else{ 
                   from0.seekp(-(IndexSize-1),ios::cur);
                   from0 << x;
                   int test = from0.tellp();
                   from0.clear();
               }
           }
       }
       return 0;
   }

   Собираем пустые блоки

   Abbath, 15 Октября 2013

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

3. C++ / Говнокод #13947

   +6

   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

   int DataBase::split(fstream& from,fstream& from0,unsigned int p){
       int pos0 = static_cast<int>(from.tellp());
       char *a,*b;
       Data d,d0;
       Index x,x0;
       from.clear();
       from.seekp(0,ios::end);
       int PosEnd = static_cast<int>(from.tellp());
       PosEnd -=static_cast<int>(p);
       a = new char[PosEnd];
       from.seekp(p,ios::beg);
       from.read(a,PosEnd); 
       from.seekp(p+BlockSize,ios::beg);
       from.write(a,PosEnd); 
       from.seekp(p+BlockSize,ios::beg);
       b = a+5*RecordSize;
       from.write(b,RecordSize*5);
       for (int i = 0; i < 5; ++i){ 
           from << d;
       }
       from.seekp(p+RecordSize*5,ios::beg);
       for (int i = 0; i < 5; ++i){ 
           from << d;
       }
       from.seekg(p+RecordSize*4);
       from >> d;
       int newmax = d.Id(); 
       from0.seekp(0,ios::end);
       int indexend =static_cast<int>(from0.tellp());
       from0.seekg(0,ios::beg);
       while(true){ 
           from0 >> x;
           if(from0.eof())break;
           if(x.Pointer() == p){ 
               int oldpos = static_cast<int>(from0.tellp());
               oldpos++;
               int count = indexend - oldpos;
               char *c = new char[count];
               from0.seekg(1,ios::cur);
               from0.read(c,count);
               from0.seekp(oldpos,ios::beg);
               x0.Max(x.Max()); 
               x0.Num(5);
               x0.Pointer(p + BlockSize);
               from0 << x0;
               from0.write(c,count);
               from0.seekp(oldpos - IndexSize,ios::beg);
               x.Max(newmax); 
               x.Num(5);
               from0 << x;
               delete c;
               from.clear();
               from0.seekp(0,ios::beg);
               break;
           }
       }
       from.clear();
       bool first = false;
       while(true){ 
           from0 >> x;
           if(from0.eof())break;
           if(x.Pointer() > p ){
               if(!first){ 
                   from0.seekp(-(IndexSize-1),ios::cur);
                   from0 << x;
                   int test = from0.tellp();
                   from0.clear();
                   first = true;
                   continue;
               }
               x.Pointer(x.Pointer()+BlockSize);
               from0.seekp(-(IndexSize-1),ios::cur);
               from0 << x;
               int test = from0.tellp();
           }
           else{ 
               from0.seekp(-(IndexSize-1),ios::cur);
               from0 << x;
               int test = from0.tellp();
               from0.clear();
           }
       }
       from.clear();
       from.seekp(pos0);
       delete a;
       //delete b;
       return 0;
   }

   Нарезаем

   Abbath, 15 Октября 2013

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

4. Java / Говнокод #13945

   +81

   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

   public boolean hasNext() {
           if (vectorIndex  == vector.size()) 
               return false;
           return true;
       }
       
       public Object next() {
           try {
               return new Double(vector.getElement(vectorIndex));
           }
           catch(OutOfBoundsException e) {
               throw new RuntimeException(e);
           }
           finally {
               vectorIndex++ ;
           }
       }

   Такой вот итератор у начинающего жавера...

   d3m0n1c, 14 Октября 2013

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

5. C++ / Говнокод #13944

   +10

   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

   void ImageArea::saveConf(bool def)
   {
       static bool fp = true;
       static QString filename = QString("default.conf");
       if(!def){
           filename = QFileDialog::getSaveFileName(this, tr("Save config"), "", tr("Config files (*.conf)"));
           if(fp){
               QFile file(filename);
               if(file.open(QFile::WriteOnly)){
                   QTextStream str(&file);
                   str << crop[0].x() << " " << crop[0].y() << "\n";
                   str << crop[1].x() << " " << crop[1].y() << "\n";
                   str << square[0].x() << " " << square[0].y() << "\n";
                   str << square[1].x() << " " << square[1].y() << "\n";
                   str << square[2].x() << " " << square[2].y() << "\n";
                   QMessageBox::information(this, tr("Next step"), tr("Put 3 points then press Save again"));
               }else{
                   QMessageBox::warning(this, tr("Error"), tr("Can not open a file"));
               }
               fp = false;
           }else{
               QFile file(filename);
               if(file.open(QFile::Append)){
                   QTextStream str(&file);
                   str << square[0].x() << " " << square[0].y() << "\n";
                   str << square[1].x() << " " << square[1].y() << "\n";
                   str << square[2].x() << " " << square[2].y() << "\n";
               }else{
                   QMessageBox::warning(this, tr("Error"), tr("Can not open a file"));
               }
               fp = true;
           }
       }else{
           QFile file(filename);
           if(file.open(QFile::WriteOnly)){
               QTextStream str(&file);
               str << conf.crop[0].x() << " " << conf.crop[0].y() << "\n";
               str << conf.crop[1].x() << " " << conf.crop[1].y() << "\n";
               str << conf.square[0].x() << " " << conf.square[0].y() << "\n";
               str << conf.square[1].x() << " " << conf.square[1].y() << "\n";
               str << conf.square[2].x() << " " << conf.square[2].y() << "\n";
               str << conf.square0[0].x() << " " << conf.square0[0].y() << "\n";
               str << conf.square0[1].x() << " " << conf.square0[1].y() << "\n";
               str << conf.square0[2].x() << " " << conf.square0[2].y() << "\n";
           }
       }
   }

   Сохраняем конфиг в джва захода.

   Abbath, 14 Октября 2013

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

6. PHP / Говнокод #13943

   +160

   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

   function casinoGamesParser($body, $infoBoxyLink)
   {
   	// try ruletka
   	$ruletkaReplaceCode = read_file('system/application/views/site/boxy-rul.php');
   	$ruletkaReplaceCode = str_replace('<?php echo $boxylink;?>', $infoBoxyLink, $ruletkaReplaceCode);
   	$ruletkaReplaceCode = str_replace('<?php echo site_url();?>', site_url(), $ruletkaReplaceCode);
   	$delimiterFirst = "<p>фун_рулетка(";
   	$delimiterLast = ")енд</p>";
   	$keyword = between2finer($body, $delimiterFirst, $delimiterLast);
   	if (!empty($keyword)) $ruletkaReplaceCode = str_replace('<?php echo $keyword ?>', $keyword, $ruletkaReplaceCode);
   	if (!empty($keyword)) return str_replace($delimiterFirst.$keyword.$delimiterLast, $ruletkaReplaceCode.$keyword, $body);
   	// try poker
   	$pokerReplaceCode = read_file('system/application/views/site/boxy-poker.php');
   	$pokerReplaceCode = str_replace('<?php echo $boxylink;?>', $infoBoxyLink, $pokerReplaceCode);
   	$pokerReplaceCode = str_replace('<?php echo site_url();?>', site_url(), $pokerReplaceCode);
   	$delimiterFirst = "фун_покер(";
   	$delimiterLast = ")енд";
   	$keyword = between2finer($body, $delimiterFirst, $delimiterLast);
   	if (!empty($keyword)) $pokerReplaceCode = str_replace('<?php echo $keyword ?>', $keyword, $pokerReplaceCode);
   	if (!empty($keyword)) return str_replace($delimiterFirst.$keyword.$delimiterLast, $pokerReplaceCode.$keyword, $body);
   	// if nothing found - return original body
   	return $body;
   }

   Берут РНР файл как текстовый в переменную и заменяют в нем вставки РНР кода <?php echo $keyword ?>. Получается РНР парсит свой код str_replace*ом. Ну вобщем неплохо получилось запутать китайских шпионов!

   increazon, 14 Октября 2013

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

7. C++ / Говнокод #13942

   +13

   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

   if(ortho){
               if( abs(e->x() - line.p1().x() ) * 2 < abs( e->y() - line.p1().y() ) ){
                   y = e->y();
                   x = line.p1().x();
               }else if(abs(e->x() - line.p1().x() ) > 2 * abs( e->y() - line.p1().y() )){
                   x = e->x();
                   y = line.p1().y();
               }else if(abs(e->x() - line.p1().x() ) * 2 > abs( e->y() - line.p1().y() ) &&
                        abs(e->x() - line.p1().x() ) < abs( e->y() - line.p1().y() )){
                   if((e->x() < line.p1().x() && e->y() < line.p1().y()) || (e->x() > line.p1().x() && e->y() < line.p1().y())){
                       x = e->x();
                       y = line.p1().y() - abs(line.p1().x() - e->x());
                   }else if((e->x() > line.p1().x() && e->y() > line.p1().y()) || (e->x() < line.p1().x() && e->y() > line.p1().y())){
                       x = e->x();
                       y = line.p1().y() + abs(line.p1().x() - e->x());
                   }
               }else if(abs(e->x() - line.p1().x() ) < 2 * abs( e->y() - line.p1().y() ) &&
                        abs(e->x() - line.p1().x() ) > abs( e->y() - line.p1().y() )){
                   if((e->x() < line.p1().x() && e->y() < line.p1().y()) || (e->x() < line.p1().x() && e->y() > line.p1().y())){
                       y = e->y();
                       x = line.p1().x() - abs(line.p1().y() - e->y());
                   }else if((e->x() > line.p1().x() && e->y() > line.p1().y()) || (e->x() > line.p1().x() && e->y() < line.p1().y())){
                       y = e->y();
                       x = line.p1().x() + abs(line.p1().y() - e->y());
                   }
               }else{
                   x = e->x();
                   y = e->y();
               }

   Abbath, 14 Октября 2013

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

8. C# / Говнокод #13941

   +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
   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

   #region Work with massive slovo
   
   public string sFileNameMassiveSlovar = "";
   public bool flChangeSlovo = false;
   public int nCountSavedSlovo { get; private set; }
   public int nCountAllSlovo { get; private set; }
   public int nCountAddSlovo { get; private set; }
   public void ClearMassiveSlovo()
   {
     slovo = slovo.Select(n => (byte)0).ToArray();
   }
   
   public int GetCountAllSlovo()
   {
     int npock = nLengthSlovo;
     return slovo.Where((n, index) => index % npock == 0 && n != 0).Count();
   }
   
   public bool WriteMassiveSlovo()
   {
     bool res = false;
     FileStream fs = null;
     try
     {
       fs = File.Open(sFileNameMassiveSlovar, FileMode.Create, FileAccess.Write);
       if (fs != null)
       {
         fs.Write(slovo, 0, nSizeMassiveSlova);
       }
       res = true;
     }
     catch (Exception e1)
     {
       res = false;
       System.Windows.Forms.MessageBox.Show(e1.Message);
     }
     finally
     {
       if (fs != null) fs.Close();
     }
     return res;
   }
   
   #endregion

   just_nameless, 14 Октября 2013

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

9. Java / Говнокод #13940

   +171

   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

   public abstract class BaseDateTime
           extends AbstractDateTime
           implements ReadableDateTime, Serializable {
   
       /** The millis from 1970-01-01T00:00:00Z */
       private volatile long iMillis;
       /** The chronology to use */
       private volatile Chronology iChronology;
   
   /////////////////////////////////////////////////////////////////	
   /*	
    * DateTime is thread-safe and immutable, provided that the Chronology is as well.
    * All standard Chronology classes supplied are thread-safe and immutable.
    *
    * @see MutableDateTime
    */
   public final class DateTime
           extends BaseDateTime

   Любителям joda-time.
   Cмущает меня этот volatile, который приходит в немутабельный класс от родителя.

   3.14159265, 14 Октября 2013

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

10. PHP / Говнокод #13939

    +150

    1. 1
    2. 2
    3. 3
    4. 4

    public static function getDeferredSubsCardExpirationDate($date) {
        $ts = strtotime($date);
        return mktime(date('H', $ts), date('i', $ts), date('s', $ts), date('m', $ts), date('d', $ts), date('Y', $ts) + Constants::GIFT_CARD_EXPIRATION_PERIOD);
    }

    ОУКЕЙ

    Sc0rp10, 14 Октября 2013

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

• ← влево
• вправо →

• 2108
• …
• 1252
• 1251
• 1250
• 1249
• 1248
• 1247
• 1246
• 1245
• 1244
• …
• 1