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

    +64.6

    1. 1
    2. 2
    3. 3
    4. 4
    private static final int LENGHT255 = 255;
    private static final int LENGHT450 = 450;
    private static final int LENGHT3 = 3;
    private static final int LENGHT5 = 5;

    Константы для проставления длин в Entity.

    guest, 10 Февраля 2009

    Комментарии (4)
  2. Java / Говнокод #520

    +139

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

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

    guest, 09 Февраля 2009

    Комментарии (7)
  3. Java / Говнокод #466

    +81.3

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

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

    guest, 27 Января 2009

    Комментарии (1)
  4. 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)
  5. 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)
  6. 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)
  7. 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)
  8. 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)
  9. 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)
  10. 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)