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

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

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

1. PHP (5703)
2. C++ (2740)
3. Куча (2337)
4. JavaScript (2030)
5. C# (1929)
6. Java (1503)
7. Си (1103)
8. Pascal (649)
9. Python (585)
10. 1C (536)
11. SQL (434)
12. ActionScript (294)
13. Objective C (204)
14. bash (200)
15. Perl (196)
16. Assembler (148)
17. Ruby (144)
18. VisualBasic (140)
19. Lua (51)
20. Go (31)
21. Swift (29)
22. Haskell (28)
23. Kotlin (15)

1. Куча / Говнокод #3761

   +129

   1. 1
   2. 2

   {{ object.xml_data.file.read|safe }}
   {% if object.xml_data.file.close %}{% endif %}

   Мой говнокод. Django template. Не хотелось писать отдельный view, забацал все в шаблоне + generic views. Лень страшная сила)
   object.xml_data обычный FileField

   sbb, 22 Июля 2010

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

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

   +144

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

   void get_tomorrow_date( struct timeval *date ) 
   { 
   sleep( 86400 ); // 60 * 60 * 24 
   gettimeofday( date, 0 ); 
   }

   Программистский шедевр на индусском форуме на тему «как узнать завтрашнюю дату» остался непревзойденным.

   REDNES, 22 Июля 2010

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

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

   +168

   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

   function endPanel()
   	{
   		if($this->type=="common" || $this->type=="common_blank")
   		{
   				?>
   								</div>
   						</div>
   				</div>
   				<?php
   		}
   		elseif($this->type=="common_short")
   		{
   				?>
   								</div>
   						</div>
   				</div>
   				<?php
   		}
   		elseif($this->type=="main" || $this->type=="main_small")
   		{
   			?>
   					</div>
   				</div>
   				<div class="bottom">
   					<div class="bottom-right"></div>
   				</div>
   			</div>
   			<?php
   		}
   		elseif($this->type=="light" || $this->type=="light_blank" || $this->type=="light_narrow")
   		{
   			?>
   					</div>
   				</div>
   				<div class="bottom">
   					<div class="bottom-right"></div>
   				</div>
   			</div>
   			<?php
   		}
   	}

   Прислали на работе проект на доработку, мало того предыдущий программер не знает про разделение логики и представления, дак еще и такие штуки на каждом шагу встречаются

   xoros, 22 Июля 2010

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

4. PHP / Говнокод #3758

   +156

   1. 1
   2. 2
   3. 3
   4. 4
   5. 5
   6. 6
   7. 7
   8. 8

   <?php
   
   if(true) {
   ...
   }
   else $blacklist_numbers_error="неверный тип файла";
   
   ?>

   разбираю на работе код ><

   DmitryDick, 22 Июля 2010

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

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

   +167

   1. 1
   2. 2
   3. 3
   4. 4
   5. 5
   6. 6
   7. 7
   8. 8
   9. 9

   <?php
   // Парсируем установочный файл
   if(@parse_ini_file("../../phpshop/inc/config.ini",1))
   $SysValue=parse_ini_file("../../phpshop/inc/config.ini",1);
    elseif(@parse_ini_file("../../../phpshop/inc/config.ini",1))
       $SysValue=parse_ini_file("../../../phpshop/inc/config.ini",1);
   	  elseif(@parse_ini_file("../../../../phpshop/inc/config.ini",1))
   	      $SysValue=parse_ini_file("../../../../phpshop/inc/config.ini",1);
   	     else $SysValue=@parse_ini_file("../../../../../phpshop/inc/config.ini",1);

   PHPShop

   Under, 22 Июля 2010

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

6. C# / Говнокод #3756

   +118

   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

   void Say(bool work)
           {
               if (work)
               {
                   r = new Random();
                   i = r.Next(100);
                   textBlock1.Text = i.ToString();
   
                   textBlock1.Dispatcher.BeginInvoke(new AsyncSay(this.Say), DispatcherPriority.SystemIdle, null);
               }
           }
   
   void BtnSay_Click(object sender, RoutedEventArgs e)
           {
               Say(true);
           }
   
           private void BtnStopSay_Click(object sender, RoutedEventArgs e)
           {
               Say(false);
           }

   sergylens, 22 Июля 2010

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

7. C# / Говнокод #3755

   +144

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

   // Assembly2.cs
   // Compile with: /target:library
   public class BaseClass 
   {
      infernal static int intM = 0;
   }

   Это пример из MSDN.
   На самом деле там был internal static int. Просто я сегодня много работал. Мало ли что покажется. :)

   anmiles, 21 Июля 2010

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

8. Java / Говнокод #3754

   +75

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

   public class SomeClass
   {
   	@SuppressWarnings("unused")
   	private Validator validator;
   
   	... 
   }

   Эта переменная реально не используется. Удалить бы - ан нет, автор почему-то считает, что лучше пометить её как неиспользуемую, чтобы компилятор и Эклипс не ругались.
   Чем только народ думает...

   asolntsev, 21 Июля 2010

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

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

   +82

   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

   @ValidationExpression(value = "if (inn != null && !\"\".equals(inn)) {\n" +
                                 "char[] st = inn.toCharArray();\n" +
                                 "for(int i=0; i < st.length; i++) {\n" +
                                 "if (!Character.isDigit(st[i])) { return false; }\n" +
                                 "}\n" +
                                 "}" +
                                 "return true;", message = Messages.WRONG_INN_FORMAT)
   @Expression(value = "if (inn != null && !\"\".equals(inn)) {\n" +
                       "char[] st = inn.toCharArray();\n" +
                       "for(int i=0; i < st.length; i++) {\n" +
                       "if (!Character.isDigit(st[i])) { return false; }\n" +
                       "}\n" +
                       "}" +
                       "return true;", messageKey = Messages.WRONG_INN_FORMAT)
   public String getinn() {
   	return getPerson().getinn();
   }
   
   @Required(messageKey = Messages.REQUIRED)
   @DictEntityExists(dictId = MDictionaries.DICT_CURRENCY, message = Messages.NO_DICT_ENTITY)
   @Expression(value = "SystemCounter.DICTCURRENCY__RUR_CURRENCY_TYPE.equals(agentStatementCurrencyId) ? " +
                       "   (" +
                       "       SystemCounter.DICTCURRENCY__RUR_CURRENCY_TYPE.equals(currency_id) ? " +
                       "           true : " +
                       "           SystemCounter.DICTCONTRACTCLASS__CONTRACT_CLASS_XXX.equals(contract_class_id) ? " +
                       "               false : " +
                       "               is_currency_equivalent == 1" +
                       "   ) : (" +
                       "       SystemCounter.DICTCONTRACTCLASS__CONTRACT_CLASS_GENERIC.equals(contract_class_id) &&" +
                       "       is_currency_equivalent == 0 &&" +
                       "       agentStatementCurrencyId.equals(currency_id)" +
                       "   )", messageKey = Messages.CURRENCY_NOT_EQUIVALENT_WITH_AGENT_STATEMENT)
   public String getcurrency_id() {return getContract().getcurrency_id();}

   нет слов

   borka, 21 Июля 2010

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

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

    +143

    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

    $db = new sql_db($server_db, $user_db, $pass_db, $name_db);
    $db->sql_query("SELECT * FROM `".$prefix."_global` WHERE `name`='global';");
    $row = $db->sql_fetchrow();
    
    define("LANGUAGE_ADMIN","{$row['language_admin']}");
    define("MODUL_HOME_ADMIN","{$row['home_modul_adm']}");
    define("THEMA_ADMIN","{$row['thema_admin']}");
    
    require_once ("language/".LANGUAGE_ADMIN."/ind.php");
    
    if(IN_ADMIN){
    define("IN_ADMIN", TRUE);
    
    	if (!file_exists ("modules/".MODUL_HOME_ADMIN."/index.php"))
    	die (UNIT_NO_DEFAULT);
    	
    	if (isset($_GET ['am'])){ $MODULES = $_GET ['am']; }
    	else { $MODULES = MODUL_HOME_ADMIN; }
    
    	if (file_exists ("modules/".$MODULES."/index.php")) { 
    		require ("modules/".$MODULES."/index.php");
    	}
    	else{ echo UNIT_IS_NOT_FOUND; }
    	
    	if (file_exists (Subject_file_path)) {
    		$TEMPLATES_SITE = file_get_contents(Subject_file_path);
    	}
    	else{
    		echo SUBJECTS_NOT;
    	}
    
    	require(Connect_funct_temp);
    
    	echo $COD_TEMP;
    
    if(!check_user($UserID)) logout();
    
    } else { require(Authorisation_file_path); }

    Пишу CMS и просто интересно мнение... Код короткий, так как многое через define()
    И очень интересно Ваше мнение по нему, зачем define(), чтобы человек первый встречный не наизменял там чего, чтото типа защита от ламера, НО от ЛАМЕРА только) Просто программер так всё поймёт...
    
    Кароче я тут в первые, и поэтому я ХЗ чё вам сказать ещё))

    nethak, 21 Июля 2010

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

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

• 2103
• …
• 370
• 369
• 368
• 367
• 366
• 365
• 364
• 363
• 362
• …
• 1