1. Список говнокодов пользователя mrFoxs

    Всего: 4

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

    +142

    1. 1
    2. 2
    3. 3
    4. 4
    5. 5
    6. 6
    7. 7
    8. 8
    try {
    	connection.executeQuery("select to_date('" + this.getMonth() + "', 'yyyyMM') from dual", null);
    } catch (Exception e) {
    	logger.error(e.getMessage(), e);
    	Error = true;
    	fault.addError(8, 100);
    	return;
    }

    Лучший способ по проверке корректности даты

    this.getMonth() - возвращает дату из пришедшего запроса

    mrFoxs, 22 Апреля 2015

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

    +72

    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
    private String shutdown = "SHUTDOWN";
    
    ........
    
    // Read a set of characters from the socket
    int expected = 1024; // Cut off to avoid DoS attack
    
    <--Зачем это тут?-->
    while (expected < shutdown.length()) {
        if (random == null)
            random = new Random();
        expected += (random.nextInt() % 1024);
    }
    
    
    while (expected > 0) {
        int ch = -1;
        try {
            ch = stream.read();
        } catch (IOException e) {
            log.warn("StandardServer.await: read: ", e);
            ch = -1;
        }
        if (ch < 32)  // Control character or EOF terminates loop
            break;
        command.append((char) ch);
        expected--;
    }

    Не понимаю зачем тут while. Авторы Apache Tomcat расскажите...

    mrFoxs, 25 Ноября 2014

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

    +62

    1. 01
    2. 02
    3. 03
    4. 04
    5. 05
    6. 06
    7. 07
    8. 08
    9. 09
    10. 10
    11. 11
    12. 12
    _L1:
            if(i$ >= len$)
                break MISSING_BLOCK_LABEL_133;
            Constructor constructor = arr$[i$];
            if(Objects.equals(pattern, constructor.getParameterTypes()))
                return instantiateByConstructor(constructor, params);
            i$++;
              goto _L1
            return instantiateByClass(clazz);
            Exception e;
            e;
            Exception cause = e;

    TopS BI и где вы только находите таких программистов.......

    mrFoxs, 15 Ноября 2012

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

    +74

    1. 1
    2. 2
    3. 3
    4. 4
    5. 5
    6. 6
    synchronized public void refreshConversionTable() {
            btnClick = true;
            if (dataModel != null)
                dataModel.reset();
            dataModel = null;
        }

    если модель данных существует, сначала сбросить все данные, а потом занулим....

    mrFoxs, 31 Января 2012

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