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

    +77

    1. 1
    2. 2
    3. 3
    4. 4
    5. 5
    6. 6
    Set<String> allDocs = new HashSet<String>();
    allDocs.addAll(getQuery(app).execute(query.toString()));
    ArrayList<String> result = new ArrayList<String>();
    if (allDocs !=null){ 
      result.addAll(allDocs);
    }

    Самое хорошее в строках 1 и 4, потому что хз, че там у этой джавы на уме. Лишняя проверка никогда не лишняя.

    croacker, 16 Марта 2013

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

    +80

    1. 1
    2. 2
    3. 3
    4. 4
    5. 5
    public static java.sql.Date currentSQLDate() {
            java.sql.Date result = null;
            Date date = new Date();
            return result;
        }

    ох, ёптеть...

    grobotron, 12 Марта 2013

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

    +76

    1. 1
    public static final String RANDOM_VALUE_QUERY = "select to_number(to_char(dbms_random.value(100,999),'999'),'999') from dual";

    Прогрессивный способ генерации случайных чисел=.

    nafania217518, 11 Марта 2013

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

    +75

    1. 1
    2. 2
    3. 3
    4. 4
    5. 5
    public Double toDouble(String str){
    	if ((str==null)||(str.equals(""))) str = "0.0";
    	if ((str.contains(","))&&(!str.contains("."))) str = str.replace(",", ".");
    	return new Double(str);
    }

    Лишняя защита никогда не бывает лишней

    nafania217518, 06 Марта 2013

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

    +114

    1. 1
    2. 2
    3. 3
    4. 4
    5. 5
    6. 6
    7. 7
    8. 8
    9. 9
    private List<PalletePath> killDupes(List<PalletePath> pathesNew) {
    	List<PalletePath> noDupes = new ArrayList<PalletePath>();
    	for (PalletePath tp : pathesNew) {
    		if (!noDupes.contains(tp)) {
    			noDupes.add(tp);
    		}
    	}
    	return noDupes;
    }

    Set? Не, не слышали.

    someone, 05 Марта 2013

    Комментарии (42)
  6. Java / Говнокод #12686

    +74

    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
    private boolean userInOneRegistrationNode() throws DfException {
    
            String uname = OrganizationStaffStructureHelper.getCurrentUserName();
    
            int i = 0;
            IDfCollection NodesCol = DQLHelper.getCollection(DQL_GET_REGISTRATOR_DIV, new String[]{uname});
            while (NodesCol.next()) {
                if (!(NodesCol == null)) {
                    String group_name = NodesCol.getString(GROUP_NAME);
                    i = i + 1;
                }
            }
    
            if (i == 1) {
                return true;
            }
            return false;
    }

    Заменяется 2мя строками один - select count(*), вторая - полученный результат Integer.ValueOf(...).

    Landing, 04 Марта 2013

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

    +71

    1. 1
    2. 2
    3. 3
    4. 4
    5. 5
    6. 6
    7. 7
    String currentDate = new SimpleDateFormat("MM/dd/yyyy HH:mm:ss").format(new Date());
     accountNumber.append(String.valueOf(1900 + new SimpleDateFormat("MM/dd/yyyy HH:mm:ss").parse(currentDate).getYear()));
     accountNumber.append(String.valueOf(new SimpleDateFormat("MM/dd/yyyy HH:mm:ss").parse(currentDate).getMonth()));
     accountNumber.append(String.valueOf(new SimpleDateFormat("MM/dd/yyyy HH:mm:ss").parse(currentDate).getDate()));
     accountNumber.append(String.valueOf(new SimpleDateFormat("MM/dd/yyyy HH:mm:ss").parse(currentDate).getHours()));
     accountNumber.append(String.valueOf(new SimpleDateFormat("MM/dd/yyyy HH:mm:ss").parse(currentDate).getMinutes()));
     accountNumber.append(String.valueOf(new SimpleDateFormat("MM/dd/yyyy HH:mm:ss").parse(currentDate).getSeconds()));

    ну а чо, зато внушительно выглядит

    nafania217518, 01 Марта 2013

    Комментарии (13)
  8. Java / Говнокод #12665

    +81

    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
    public String isThisScreenWithHdpiDensity() {
        DisplayMetrics metrics = new DisplayMetrics();
        getWindowManager().getDefaultDisplay().getMetrics(metrics);
        switch (metrics.densityDpi) {
            case DisplayMetrics.DENSITY_LOW:
                return "ldpi";
            case DisplayMetrics.DENSITY_MEDIUM:
                return "mdpi";
            case DisplayMetrics.DENSITY_HIGH:
                return "hdpi";
        }
        return "xhdpi";
    }
    
    ...
    if (isThisScreenWithHdpiDensity().equals("hdpi")) {
        lpTabPane.height = 65;
    }
    if (isThisScreenWithHdpiDensity().equals("mdpi")) {
        lpUpperPane.height = 70;
    }
    if (isThisScreenWithHdpiDensity().equals("ldpi")) {
        lpTabPane.height = 32;
    }
    if (isThisScreenWithHdpiDensity().equals("xhdpi")) {
    
    }

    Перегоним константу в стрингу, а потом проверим ее equals. Отлично!

    danikula, 28 Февраля 2013

    Комментарии (10)
  9. Java / Говнокод #12663

    +113

    1. 1
    2. 2
    3. 3
    public static void assertAlmostEquals(final double actual, final double expected) {
    	Assert.assertTrue(Math.abs(actual - expected) < 0.000001);
    }

    someone, 28 Февраля 2013

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

    +142

    1. 1
    2. 2
    if(!true) // это - не говнокод. это - заглушка.
                    return false;

    Оправдывается ещё )

    pitch, 21 Февраля 2013

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