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

    +139.8

    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
    public enum TimePeriod {
    	
    	NONE,
    	DAILY,
    	WEEKLY,
    	MONTHLY,
    	YEARLY;
    	
    	public static TimePeriod getTimePeriod(String periodStr) {
    		if (null == periodStr) {
    			return null;
    		}
    		if (0 == periodStr.length()) {
    			return NONE;
    		}
    		if (0 == periodStr.compareToIgnoreCase("daily")) {
    			return DAILY;
    		}
    		if (0 == periodStr.compareToIgnoreCase("weekly")) {
    			return WEEKLY;
    		}
    		if (0 == periodStr.compareToIgnoreCase("monthly")) {
    			return MONTHLY;
    		}
    		if (0 == periodStr.compareToIgnoreCase("yearly")) {
    			return YEARLY;
    		}
    		return null;
    	}
    }

    а правильно будет так
    public static TimePeriod parse(String type) {
    return valueOf(type.toUpperCase());
    }

    guest, 20 Февраля 2009

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

    +140.8

    1. 01
    2. 02
    3. 03
    4. 04
    5. 05
    6. 06
    7. 07
    8. 08
    9. 09
    10. 10
    11. 11
    12. 12
    private String mPrimaryCorrelationKey;
     public String getPrimaryCorrelationKey()
      {
    	long current = System.currentTimeMillis();
    	current++;
    
    	return new Long(current).toString();
      }
    
      public void setPrimaryCorrelationKey(String mPrimaryCorrelationKey) {
    	  this.mPrimaryCorrelationKey = mPrimaryCorrelationKey;	  
      }

    соответствующий геттер для этой проперти должен возвращать всегда уникальный id

    guest, 19 Февраля 2009

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

    +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
    public final class SqliteWrapper {
        private static final String SQLITE_EXCEPTION_DETAIL_MESSAGE
                    = "unable to open database file";
    
        ...
    
        // FIXME: need to optimize this method.
        private static boolean isLowMemory(SQLiteException e) {
            return e.getMessage().equals(SQLITE_EXCEPTION_DETAIL_MESSAGE);
        }
    
        ...
    }

    Android source

    guest, 19 Февраля 2009

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

    +145

    1. 1
    2. 2
    Field maxx, minx, maxy, miny = row.getFields().toArray(new Field[0])[0];
    minx = maxx = maxy = miny;

    Адская комбинация.

    guest, 10 Февраля 2009

    Комментарии (0)
  5. 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)
  6. Java / Говнокод #520

    +139

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

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

    guest, 09 Февраля 2009

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

    +81.3

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

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

    guest, 27 Января 2009

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