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

    +139

    1. 1
    2. 2
    3. 3
    4. 4
    5. 5
    это твой щенок. теперь он твой навсегда.
и ты не хочешь, но считаешь его дни и года.
твоя любовь к нему огромна, с каждым днём она растёт.
она станет бесконечной,
когда он умрёт.

    для виолет_салмон

    guest, 09 Февраля 2009

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

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

    +81.3

    1. 1
    2. 2
    private final String zeroValueString="0";
private final String sevenValueString="7";

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

    guest, 27 Января 2009

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

  3. 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)

  4. 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)

  5. 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)

  6. 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)

  7. 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)

  8. 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)

  9. 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)

  10. 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)