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

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

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

1. PHP (5706)
2. C++ (2739)
3. Куча (2370)
4. JavaScript (2030)
5. C# (1928)
6. Java (1502)
7. Си (1117)
8. Pascal (648)
9. Python (586)
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 (28)
22. Swift (27)
23. Kotlin (14)

1. PHP / Говнокод #15529

   +157

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

   // Чистим мусор и дубликаты
   foreach ($phones as $i => $p1){
       foreach ($phones as $j => $p2){
           if ($i = $j) continue;
           ...

   Все же и я наступил на эти грабли.

   DIX315, 19 Марта 2014

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

2. PHP / Говнокод #15528

   +154

   1. 1
   2. 2

   $parts = parse_url($url);
           foreach($parts as $key=>$val) $$key = $val;

   extract()? Нет, не слышал..

   DIX315, 19 Марта 2014

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

3. PHP / Говнокод #15526

   +154

   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

   <?
   include "../db.php";
   if (isset($_POST['auth_name'])) {
   $name=mysql_real_escape_string($_POST['auth_name']);
   $pass=mysql_real_escape_string($_POST['auth_pass']);
   $query = "SELECT * FROM skygall_opt  WHERE name='$name' AND pass='$pass'";
   $res = mysql_query($query) or trigger_error(mysql_error().$query);
   if ($row = mysql_fetch_assoc($res)) {
   session_start();
   $_SESSION['user_id'] = $row['id'];
   $_SESSION['ip'] = $_SERVER['REMOTE_ADDR'];
   }
   header("Location: http://".$_SERVER['HTTP_HOST'].$_SERVER['REQUEST_URI']);
   echo "<center><div class=alert>Введенный логин/пароль неверны.</div></center>"; 
   exit;
   }
   if (isset($_REQUEST[session_name()])) session_start();
   if (isset($_SESSION['user_id']) AND $_SESSION['ip'] == $_SERVER['REMOTE_ADDR']) return;
   else {
   
   ?>
   <meta http-equiv="Content-Type" content="text/html; charset=windows-1251" />
   <title>SkyGallery Ver 1.0 — Администрирование</title>
   <link href="../st.css" rel="stylesheet" type="text/css" />
   <script type="text/javascript" src="../scripts/form.js"></script>
   <? 
   include "../verh.php"; 
   ?>
   
   <br><br><br><br><form id="form1" name="form1" method="POST" action="">
   <table width="350" align="center" border="0" cellspacing="5">
     <tr>
       <td rowspan="3" valign="top"><img src="pic/Login.png"></td>
       <td>Логин:</td>
       <td><input  class=bginp2 onblur=inputBG(this,0)  onfocus=inputBG(this,1) type="text" name="auth_name"></td>
     </tr>
     <tr>
       <td>Пароль:</td>
       <td><input  class=bginp2 onblur=inputBG(this,0)  onfocus=inputBG(this,1) type="password" name="auth_pass"></td>
     </tr>
     <tr>
       
       <td colspan="2" align="center"><input type="submit" value=" войти "></td>
     </tr>
   </table>
   </form>
   <br><br><br><br>
   <?
   include "../niz.php";
   }
   exit;
   ?>

   Говно

   ledoburit, 19 Марта 2014

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

4. JavaScript / Говнокод #15524

   +155

   1. 1

   $(obj).parent().parent().parent().append('<div class="add_new_comment"><div class="add_comment added">'+c_area+"</div></div>");

   Очень быстрый код, оптимальный алгоритм обращения к селектору.

   Gordun, 19 Марта 2014

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

5. PHP / Говнокод #15523

   +154

   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

   <?php
   for($tb=0;$tb<=10;$tb++){
   switch($tb){
   case 0: $number_table=3; $cros_table=20;
   break;
   case 1:$number_table+=19; $cros_table+=20;
   break;
   case 2:$number_table+=20; $cros_table+=20;
   break;
   case 3:$number_table+=20; $cros_table+=20;
   break;
   case 4:$number_table+=20; $cros_table+=20;
   break;
   case 5:$number_table+=20; $cros_table+=20;
   break;
   case 6:$number_table+=20; $cros_table+=20;
   break;
   case 7:$number_table+=20; $cros_table+=20;
   break;
   case 8:$number_table+=20; $cros_table+=20;
   break;
   case 9:$number_table+=20; $cros_table+=20;
   break;
   case 10:$number_table+=20; $cros_table+=20;
   break;
   }
   
   for($i=$number_table; $i<=$cros_table; $i++){
   $name_detail = $data->sheets[$global_page]['cells'][$i][1];
   $option_detail = $data->sheets[$global_page]['cells'][$i][2];
   $mark_detail = $data->sheets[$global_page]['cells'][$i][3];
   $actions = $data->sheets[$global_page]['cells'][$i][4];
   $work_cost = $data->sheets[$global_page]['cells'][$i][5];
   $detail_cost = $data->sheets[$global_page]['cells'][$i][6];
   if($name_detail==null){break;}
   ?>

   Когда я это узрел я усомнился в адекватности человека...

   masashama, 19 Марта 2014

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

6. SQL / Говнокод #15521

   −115

   1. 1

   SELECT `1`,`4` FROM `kada-region` WHERE `2` = "SAV" ORDER BY `7`

   Комментариев нет. Кроме нескольких, типы столбцов int. Догадайся что есть что.

   Nerfair, 18 Марта 2014

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

7. PHP / Говнокод #15520

   +160

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

   foreach ($config['entries'] as $config['entry']){
       $config['entry'] = $config['entries'][0];
       $url = $config['entry']['url'];
       $entry_host = parse_url($url, PHP_URL_HOST);
      ...
   }

   Обратите внимание на вторую строчку

   DIX315, 18 Марта 2014

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

8. C++ / Говнокод #15519

   +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

   // функция квадрата расстояния на гексагональном поле
   Fixed SDist2 (Fixed dx, Fixed dy)
   {
   	return (dx*dx+dy*dy+dx*dy);
   }
   
   // а теперь типа находим ближайшее целое, ближайшее в гексагональном смысле
         const Fixed rx = int(rtx), ry = int(rty);
   			const Fixed 
   				d00 = SDist2(rx    -rtx, ry    -rty),
   				d10 = SDist2(rx+fx1-rtx, ry    -rty),
   				d01 = SDist2(rx    -rtx, ry+fx1-rty),
   				d11 = SDist2(rx+fx1-rtx, ry+fx1-rty);
   
   			int x,y;
   			if (d00<d10 && d00<d01 && d00<d11) 
   			{
   				x=int(rx); y=int(ry);
   			} else if (d10<d01 && d10<d11)
   			{
   				x=int(rx)+1; y=int(ry);
   			} else if (d01<d11)
   			{
   				x=int(rx); y=int(ry)+1;
   			} else
   			{
   				x=int(rx)+1; y=int(ry)+1;
   			}

   изящно не получилось

   TarasB, 18 Марта 2014

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

9. C++ / Говнокод #15517

   +3

   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

   #include <iostream>
   
   using namespace std;
   
   class Base {
   public:
          Base() {
                 cout << "Base construct\n";
          }
          virtual  ~Base() {
                 cout << "Base destruct\n";
          }
   };
   
   class Child: public Base {
   public:
          Child() {
                 cout << "Child construct\n";
          }
          ~Child() {
                 cout << "Child destruct\n";
          }
   };
   
   int main() {
          Base *base = new Child();
          delete base;
   }

   Hy нe oчeвиднo жe! Для тoгo, чтoб вызывaлиcь вce дecтpyктopы, нyжнo oбъявить eгo виpтyaльным. B тo вpeмя кaк кoнcтpyктop бeз мoдификaтopa virtual paбoтaeт тaк жe, кaк виpтyaльный дecтpyктop.
   И зaчeм вoбщe ocтaвлять вoзмoжнocть нe виpтyaльнoгo дecтpyктopa - нe мoгy пpeдcтaвить ceбe cлyчaя, кoгдa пpи yдaлeнии oбъeктa нyжeн вызoв дecтpyктopa тoлькo бaзoвoгo клacca.

   Pythoner, 18 Марта 2014

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

10. C# / Говнокод #15514

    +133

    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

    try
                {
                    using (var context = new vbEntities())
                    {                   
                        
                        var DBF = new DBF();
                        var dt = DBF.GetAll(@"C:\Users\.........\Documents\BNKSEEK.DBF");
    
                        int i = 0;
                        progressBar1.Maximum = dt.Rows.Count;
    
                        foreach (DataRow dr in dt.Rows)
                        {
                            var row = new Bank();
                            row.BIK = dr["RKC"].ToString();
                            row.CityName = dr["NNP"].ToString();
                            row.Address = dr["ADR"].ToString();
                            row.Name = dr["NAMEP"].ToString();
                            row.KS = dr["KSNP"].ToString();
                            row.Phone = dr["TELEF"].ToString();
                            row.OKPO = dr["OKPO"].ToString();  
                            context.Bank.Add(row);
                            progressBar1.Value = i++;
                        }
                        context.SaveChanges();
                    }
                }
                catch (Exception ex)
                {
                    MessageBox.Show(ex.Message, "Error", MessageBoxButtons.OK, MessageBoxIcon.Error);
                }
            }

    orozov, 18 Марта 2014

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

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

• 2107
• …
• 1317
• 1316
• 1315
• 1314
• 1313
• 1312
• 1311
• 1310
• 1309
• …
• 1