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

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

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

1. PHP (5703)
2. C++ (2740)
3. Куча (2338)
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 (32)
21. Swift (29)
22. Haskell (28)
23. Kotlin (15)

1. Java / Говнокод #7525

   +71

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

   if (wdContext.nodeTable().size() <= 5)
   	wdContext.currentContextElement().setRowsIn(-1);
   else
   	wdContext.currentContextElement().setRowsIn(-1);

   foGa, 12 Августа 2011

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

2. Java / Говнокод #7524

   +76

   1. 01
   2. 02
   3. 03
   4. 04
   5. 05
   6. 06
   7. 07
   8. 08
   9. 09
   10. 10
   11. 11
   12. 12

   public static java.sql.Date StringToDate(String sDate) {
   	java.sql.Date Date = null;
   	if (!sDate.startsWith("00000000")) {
   		DateFormat dateFormat = new SimpleDateFormat("yyyy-MM-dd");
   		try {
   			Date rdate = dateFormat.parse(sDate);
   			Date = new java.sql.Date(rdate.getTime());
   		} catch (Exception e) {
   		}
   	}
   	return Date;
   }

   foGa, 12 Августа 2011

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

3. Java / Говнокод #7523

   +80

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

   try {
   ...
   } catch (Exception ex) {
   	wdComponentAPI.getMessageManager().reportException("ERROR", true);
   }

   Информативно так

   foGa, 12 Августа 2011

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

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

   +78

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

   public class Settings {
   
   	public static String CURRENCY = "руб.";
   	
   	public static void setCurrency(String currency) {
   		CURRENCY = currency != null ? currency : "руб.";
   	}
   	
   }

   Мой проект. Можно ли считать это ГК?

   tir, 11 Августа 2011

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

5. Java / Говнокод #7512

   +84

   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

   search = 0;
   try {
   	search = search + wdContext.currentSearchElement().getLifnr().length();
   } catch (Exception ex) {
   //	ex.printStackTrace();
   }
   try {
   	search = search + wdContext.currentSearchElement().getBahns().length();
   } catch (Exception ex) {
   //	ex.printStackTrace();
   }
   try {
   	search = search + wdContext.currentSearchElement().getINN().length();
   } catch (Exception ex) {
   //	ex.printStackTrace();
   }
   try {
   	search = search + wdContext.currentSearchElement().getKPP().length();
   } catch (Exception ex) {
   //	ex.printStackTrace();
   }
   try {
   	search = search + wdContext.currentSearchElement().getKtokk().length();
   } catch (Exception ex) {
   //	ex.printStackTrace();
   }
   try {
   	search = search + wdContext.currentSearchElement().getLand1().length();
   } catch (Exception ex) {
   //	ex.printStackTrace();
   }
   try {
   	search = search + wdContext.currentSearchElement().getBanks().length();
   } catch (Exception e) {
   	// TODO: handle exception
   }
   try {
   	search = search + wdContext.currentSearchElement().getBankl().length();
   } catch (Exception ex) {
   	// TODO: handle exception
   }
   try {
   	search = search + wdContext.currentSearchElement().getBankn().length();
   } catch (Exception ex) {
   //	TODO: handle exception
   }
   try {
   	search = search + wdContext.currentSearchElement().getLocco().length();
   } catch (Exception ex) {
   //	ex.printStackTrace();
   }
   try {
   	//	search = search + wdContext.currentSearchElement().getLoevm().length();
   	search = search + wdContext.currentSearchElement().getName().length();
   } catch (Exception ex) {
   //	ex.printStackTrace();
   }
   try {
   	search = search + wdContext.currentSearchElement().getOKPO().length();
   } catch (Exception ex) {
   //	ex.printStackTrace();
   }
   try {
   	search = search + wdContext.currentSearchElement().getOrt01().length();
   } catch (Exception ex) {
   //	ex.printStackTrace();
   }
   try {
   	search = search + wdContext.currentSearchElement().getSortl().length();
   } catch (Exception ex) {
   //	ex.printStackTrace();
   }
   if (search == 0)
   	wdComponentAPI.getMessageManager().reportException("Введите критерии поиска", true);
   else {
   	//Далее идет поиск
   }

   Вот такой вот была забабахана проверка входных параметров перед вызовом функции поиска (мною добавлен только последний комментарий, все остальное оригинальное)

   foGa, 11 Августа 2011

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

6. Java / Говнокод #7505

   +75

   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

   public static boolean checkInt(String value, SocketMessage message,
   			int min, int max, boolean isNullCheck, Settings settings) {
   		if (value == null) {
   			if (!isNullCheck)
   				return true;
   			if (message != null && settings != null)
   				message.addSubMessage(ErrorSubElement.createErrorSubMessage(
   						ErrorSubElement.INVALID_ARGUMENT, "Invalid Int value",
   						settings));
   			return false;
   		}
   
   		if (value.length() > 9) {
   			if (message != null && settings != null)
   				message.addSubMessage(ErrorSubElement.createErrorSubMessage(
   						ErrorSubElement.INVALID_ARGUMENT, "Invalid Int value: "
   								+ value, settings));
   			return false;
   		}
   		try {
   			int val = Integer.parseInt(value);
   			MyLog.d(LOG_TAG, "Check Val:" + val);
   			if (val < min) {
   				if (message != null && settings != null)
   					message.addSubMessage(ErrorSubElement
   							.createErrorSubMessage(
   									ErrorSubElement.INVALID_ARGUMENT,
   									"Invalid Int value: " + value
   											+ " lower than " + min, settings));
   				return false;
   			}
   
   			if (val > max) {
   				if (message != null && settings != null)
   					message.addSubMessage(ErrorSubElement
   							.createErrorSubMessage(
   									ErrorSubElement.INVALID_ARGUMENT,
   									"Invalid Int value: " + value
   											+ " bigger than " + max, settings));
   				return false;
   			}
   			return true;
   		} catch (NumberFormatException e) {
   			if (message != null && settings != null)
   				message.addSubMessage(ErrorSubElement.createErrorSubMessage(
   						ErrorSubElement.INVALID_ARGUMENT, "Invalid Int value: "
   								+ value, settings));
   			return false;
   		}
   	}

   xitx, 10 Августа 2011

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

7. Java / Говнокод #7486

   +89

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

   while (m.find()) {
       String sentence = m.group(0);
       setterMethodName = "setElement" + sentence;
       break;
   }

   Чем автора не устроил оператор if остается загадкой.

   askell, 08 Августа 2011

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

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

   +147

   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

   public class Person {
   	
   	private final int id;
   	private final String name;
   	private final int age;
   	
   	public Person(int i, String s, int a)
   	{
   		id = i;
   		name = s;
   		age = a;
   	}
   	
   	public int getId()
   	{
   		return id;
   	}
   	
   	public String getName()
   	{
   		return name;
   	}
   	
   	public int getAge()
   	{
   		return age;
   	}
   }

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

   redenemy, 05 Августа 2011

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

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

   +147

   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

   String classToType(Class cls) {
       if (cls == Point.class) {
         return "int2";
       } else if ((cls == Integer.TYPE) || (cls == Integer.class)) {
         return "int";
       } else if ((cls == Double.TYPE) || (cls == Double.class)) {
         return "double";
       } else if (cls == String.class) {
         return "String";
       } else if ((cls == Boolean.TYPE) || (cls == Boolean.class)) {
         return "boolean";
       } else {
         return null;
       }
     }

   Lure Of Chaos, 04 Августа 2011

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

10. Java / Говнокод #7445

    +76

    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

    public class ThreadPoolExecutor implements Executor {
    	private int maximumPoolSize; 	// ìàêñèìàëüíîå êîëè÷åñòâî ïîòîêîâ
    	private long keepAliveTime; 	// âðåìÿ îæèäàíèÿ íîâîé çàäà÷è
    	private Integer poolSize;			// êîëè÷åñòâî ñîçäàííûõ ïîòîêîâ
    	private Vector workQueue;			// î÷åðåäü çàäà÷
    	private Vector threadPool;			// ïóë ïîòîêîâ
    
    	public ThreadPoolExecutor(int maxPoolSize, long time) {
    		this.maximumPoolSize = maxPoolSize;
    		this.keepAliveTime = time;
    		this.poolSize = new Integer(0);
    		workQueue = new Vector();
    		threadPool = new Vector();
    		new Thread() {	
    			public void run() {
    				for (;;) {
    					try {
    						Thread.sleep(keepAliveTime);
    					} catch (InterruptedException e) {
    						System.out.println("interrupted");
    					}
    					interruptIfIdle();
    				}
    			}
    		}.start();
    	}
    
    	public void execute(Runnable task) {
    		if (task == null)
    			throw new NullPointerException();
    		workQueue.addElement(task);
    		for (;;) {
    			int status = addIfUnderMaximumPoolSize(task);
    			if (status > 0)
    				return;
    			if (status == 0) {
    				reject(task);
    				return;
    			}
    		}
    	}
    	
    	public void shutdown() {
    		if (threadPool.size() > 0) 
    			for (int i=0; i < threadPool.size(); i++) {
    				((Thread) threadPool.elementAt(i)).interrupt();
    			}
    	}
    	
    	public void reject(Runnable task) {
    		//
    	}
    	
    	private int addIfUnderMaximumPoolSize(Runnable task) {
    		Thread t = null;
    		int status = 0;
    		synchronized (poolSize) {
    			if (poolSize.intValue() < maximumPoolSize) {
    				Runnable next = (Runnable) workQueue.elementAt(0);
    				workQueue.removeElementAt(0);
    				if (next == task) {
    					status = 1;
    				} else
    					status = -1;
    				t = addThread(next);
    			}	
    		}
    		return status;
    	}
    	
    	private Thread addThread(Runnable task) {
    		Thread thread = new Thread(task);
    		threadPool.addElement(thread);
    		thread.run();
    		poolSize = new Integer(poolSize.intValue()+1);
    		return new Thread();
    	}
    		
    	private void interruptIfIdle() {
    		synchronized (threadPool) {
    			for (int i=0; i < threadPool.size(); i++) {
    				try {
    					((Thread) threadPool.elementAt(i)).interrupt();
    				} finally {
    					poolSize = new Integer(poolSize.intValue()-1);
    				}
    			}
    		}
    	}
    	
    }

    Junior пишет весьма упрощенный ThreadPoolExecutor для BlackBerry (сорри, не тот пост кинул в прошлый раз).

    enikey, 04 Августа 2011

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

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

• 151
• …
• 58
• 57
• 56
• 55
• 54
• 53
• 52
• 51
• 50
• …
• 1