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

    +76.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
    21. 21
    22. 22
    23. 23
    24. 24
    25. 25
    26. 26
    27. 27
    28. 28
    29. 29
    30. 30
    31. 31
    32. 32
    33. 33
    34. 34
    public String getServiceClient(final Service service) {
        SqlDao dao = (SqlDao) getBean("sqlDao");
        return (String) dao.getTemplate().query(SQL_GET_CLIENT, new Long[]{service.getClient().getId()}, new ResultSetExtractor() {
            public Object extractData(ResultSet rs) throws SQLException, DataAccessException {
                String result;
                SqlDao dao = (SqlDao) getBean("sqlDao");
                rs.next();
                if (rs.getInt("IS_ORG") == 1) {
                    result = (String) dao.getTemplate().query(SQL_GET_LEGAL_INFO, new Long[]{service.getClient().getId()}, new ResultSetExtractor() {
                        public Object extractData(ResultSet rsi) throws SQLException, DataAccessException {
                            if (rsi.next()) {
			        return rsi.getString("C_INN") + ", " + rsi.getString("C_NAME");
                            } else {
                                log.debug("====== No legal client found for service: " + service.getRegistrationNumber() + " (" + service.getId() + ")");
                                return null;
                            }
                        }
                    });
                 } else {
                     result = (String) dao.getTemplate().query(SQL_GET_PERSONINFO, new Long[]{service.getClient().getId()}, new ResultSetExtractor() {
                         public Object extractData(ResultSet rsi) throws SQLException, DataAccessException {
                             if (rsi.next()) {
                                 return rsi.getString("LASTNAME") + " " + rsi.getString("FIRSTNAME") + " " + rsi.getString("MIDDLENAME");
                             } else {
                                 log.debug("====== No phisical client found for service: " + service.getRegistrationNumber() + " (" + service.getId() + ")");
                                 return null;
                             }
                        }
                    });
                }
                return result;
            }
      });
}

    Я конечно все понимаю, deadline, все эти вещи... Но return на 20 с чем-то строк это сильно.

    Запостил: nagakhl, 19 Января 2010

    Комментарии (2) RSS

    • ava guest 19.01.2010 08:25 # 0

      Enterprise govnokod detected))) +1
      Видел такое и писал сам.
      Ответить

      • ava nagakhl 19.01.2010 23:52 # 0

        Именно что интырпрайзненький, и именно что говнокод. Хотя в том проекте этот кусок еще не самый говнокодистый. Там и больший 3.14 найти можно, если покопаться. А уж какой мороз по коже идет, когда глядишь на pl/sql внутрях базы...
        Ответить

    Добавить комментарий