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

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

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

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

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

   +81.3

   1. 1
   2. 2

   private final String zeroValueString="0";
   private final String sevenValueString="7";

   Если б можно, то ещё б и hundredElevenValueString = "111"; впиндюрили бы

   guest, 27 Января 2009

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

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

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

   // parse from the uninstall, the actuall installation path
   		try {
   			int jarLoc = uninstallDir.indexOf("-jar");
   			String s = uninstallDir.substring(jarLoc + 5);
   			// turn it into a normalized file
   			int uniLoc = s.indexOf("uninstaller");
   			s = s.substring(1,uniLoc);
   			s = s.replace("\\", "/");
   			if(s.endsWith("/"))
   				s = s.substring(0,s.length()-1);
   
   			File f = new File(s);
   			if(!f.exists()) {
   				throw new Exception("installation directory does not exist: " + s);
   			}
   			return f;
   		}
   		catch(Exception e) {
               throw new Exception("installation directory could not be parsed from: " + uninstallDir);		    
   		}

   Typical code of one highly paid "tech expert" from USA
   Exception handling makes me cry, this guy doesn't even now about any other exception classes

   guest, 26 Января 2009

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

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

   +89.9

   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

   static char DigitToChar(
       int _Digit
     )
   {
     switch (_Digit)
     {
       case 0: return '0';
       case 1: return '1';
       case 2: return '2';
       case 3: return '3';
       case 4: return '4';
       case 5: return '5';
       case 6: return '6';
       case 7: return '7';
       case 8: return '8';
       case 9: return '9';
       default: return ':';
     }
   }

   в результате долгих размышлений всетаки заменили на return '0' + _Digit;

   guest, 26 Января 2009

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

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

   −50

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

   String[] search_array;
   int index = -1;
   for (int y = 0; y < search_array.length; y++) {
       if (search_array[y].equals("SEARCH_PATTERN") {
           index = y;
           y = 100;
       }
   }

   guest, 20 Января 2009

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

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

   −55.5

   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

   for (ComplectationItemTO complectation : previewFullTO.getItemComplectations()) {
               for (Map.Entry entry : complectation.getInstances().entrySet()) {
                   if (entry.getKey().equals(SoundData.SOUND_COMPLECTATION_NAME)) {
   
                       SoundData soundData = getComplectationSoundData(complectation.getId(), previewFullTO);
                       soundsMap.put(complectation.getId(), soundData);
                   }
               }
           }
   
   
    private SoundData getComplectationSoundData(Long complectatiomId, PreviewFullTO previewFullTO) {
           for (ComplectationItemTO complectationSound : previewFullTO.getItemComplectations()) {
               if (complectationSound.getId().equals(complectatiomId)) {
                   for (Map.Entry entrySound : complectationSound.getInstances().entrySet()) {
                       if (entrySound.getKey().equals(SoundData.SOUND_LENGTH)) {
                           String soundLength = complectationSound.getInstances().get(SoundData.SOUND_LENGTH);
                           String soundData = complectationSound.getInstances().get(SoundData.SOUND_COMPLECTATION_NAME);
                           String productNumber = previewFullTO.getItemTO().getFeatureInstances().get("Product number");
                           String productRewritedName = previewFullTO.getItemTO().getProductTypeRewrited();
                           return new SoundData(soundData, productRewritedName, Long.valueOf(productNumber), soundLength);
                       }
                   }
               }
           }
           return null;
       }

   цикл в цикле и циклом погоняет

   guest, 20 Января 2009

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

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

   −45.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

   if ((solution.indexOf("filter") != -1)
                       || (solution.indexOf("Filter") != -1)
                       || (solution.indexOf("FILTER") != -1)) {
                   solution = "Filter incoming traffic to port/service \""
                           + serviceName + "\" if the service is not used";
               } else if ((solution.indexOf("package") != -1)
                       || (solution.indexOf("Package") != -1)
                       || (solution.indexOf("PACKAGE") != -1)
                       || (solution.indexOf("vendor") != -1)
                       || (solution.indexOf("Vendor") != -1)
                       || (solution.indexOf("VENDOR") != -1)
                       || (solution.indexOf("service") != -1)
                       || (solution.indexOf("Service") != -1)
                       || (solution.indexOf("SERVICE") != -1)) {
                   solution = "AMBIQUITY_SOLUTION_START port/service \""
                           + item.getServiceName() + "\'} " + solution;
               }
   
   //...............................................etc

   Развиваю один старый проект. Не буду говорить о всех перлах того, что я там увидел, про великую архитектуру проекта, но кое-что хотелось бы показать.
   З.Ы. Я бы хотя бы сделал так
   solution.toLowerCase().indexOf("package" ) != -1
   З.З.Ы Этот ужас таки порефакторил и его убрали :)

   guest, 19 Января 2009

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

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

   −19

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

   public Calendar getTomorrow() {
      Thread.sleep(1000*60*60*24);
      return Calendar.getInstance();
   }

   как узнать дату завтрашнего дня :

   guest, 16 Января 2009

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

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

   +17.1

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

   public boolean isSocialNWportal(){
   		int accountId = getCurrentAccountID();
   		if(accountId == XXX.ACCOUNT_ID_1 ||
   		   accountId == XXX.ACCOUNT_ID_2 ||
   		   accountId == XXX.ACCOUNT_ID_3
   		)
   			return true;
   		else
   			return false;
   	}

   Ну и конечно же смотрим как был реализован метод isSocialNWportal() ранее. Тут все как всегда: хиндогавнологика одна...

   guest, 14 Января 2009

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

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

   +29

   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

   if (searchBox.isFocused()) {
   			addressBookPopupMenu.enableItem(0, false);//view
   			addressBookPopupMenu.enableItem(1, true);//search
   			addressBookPopupMenu.enableItem(2, true);//add contact
   			addressBookPopupMenu.enableItem(3, false);//edit contact
   			addressBookPopupMenu.enableItem(4, false);//delete contact
   			addressBookPopupMenu.enableItem(5, false);            
   		}
   		else {
   			addressBookPopupMenu.enableItem(0,true);
   			addressBookPopupMenu.enableItem(1,false);
   			addressBookPopupMenu.enableItem(2,true);
   			addressBookPopupMenu.enableItem(3,true);
   			addressBookPopupMenu.enableItem(4,true);
   			addressBookPopupMenu.enableItem(5, true);           
   		}
   
   		if(isSocialNWportal()) {
   			addressBookPopupMenu.enableItem(2, false);//add contact
   			addressBookPopupMenu.enableItem(3, false);//edit contact
   			addressBookPopupMenu.enableItem(4, false);//delete contact
   		}

   индусская логика vs Российская. Чуть ни упал пацтол когда соптимизячил и сравнил со своим кодом:
   
   boolean allowFlag = !(searchBox.isFocused() || contactListBox.isEmpty());
   boolean portalFlag = !isSocialNWportal();
   addressBookPopupMenu.enableItem(0, allowFlag);
   addressBookPopupMenu.enableItem(1, !allowFlag);
   addressBookPopupMenu.enableItem(2, portalFlag);//add contact
   addressBookPopupMenu.enableItem(3, allowFlag && portalFlag);
   addressBookPopupMenu.enableItem(4, allowFlag && portalFlag);
   addressBookPopupMenu.enableItem(5, !searchBox.isFocused());

   guest, 14 Января 2009

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

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

    −56.7

    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

    private List<WorkOrderWrapper> copy(List<WorkOrderWrapper> src) {
    		List<WorkOrderWrapper> ret = Collections.emptyList();
    
    		if (src != null) {
    			ret = new ArrayList<WorkOrderWrapper>(src.size());
    
    			for (int i = src.size() - 1; i >= 0; i--) {
    				ret.add(src.get(i));
    			}
    		}
    
    		return ret;
    	}

    Мы легких путей не ищем!

    guest, 29 Декабря 2008

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

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

• 151
• …
• 8
• 7
• 6
• 5
• 4
• 3
• 2
• 1