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

    Всего: 101

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

    +148

    1. 1
    2. 2
    3. 3
    4. 4
    5. 5
    6. 6
    public List<OrderEntity> getOrders() {
    if (orders == null) {
        orders = new ArrayList<OrderEntity>();
    }
    return orders;
}

    Потокобезопасность? Не, не слышал.

    roman-kashitsyn, 23 Сентября 2011

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

  3. Java / Говнокод #7931

    +77

    1. 1
    2. 2
    3. 3
    4. 4
    5. 5
    6. 6
    7. 7
    try {
    order.setAddress(address.clone());
} catch (CloneNotSupportedException e) {
    // We will not get here, because priorAddressDtoToAdd.address is of Address type.
    // Moreover - none of the subclasses throws this exception
    LOG.error("Address clone is not supported? Nonsense...", e);
}

    Это было бы даже весело, если бы не повторялось вместе с коментами 3 раза в одном классе.

    roman-kashitsyn, 22 Сентября 2011

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

  4. Java / Говнокод #7840

    +77

    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
    public class SomeServlet extends HttpServlet {
    //...
    private static final int ERROR_NOT_FOUND = 404;
    private static final int ERROR_INTERNAL = 503;
    //...
    protected void doPost(HttpServletRequest req, HttpSerletResponse resp) {
        //...
        if (buff != null) {
            if (buff.length == 0) {
                resp.sendError(ERROR_INTERNAL);
            }
            //...
        } else {
            resp.sendError(ERROR_NOT_FOUND);
        }
    }
}

    Велосипедисты взялись за сервлеты.

    roman-kashitsyn, 12 Сентября 2011

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

  5. Java / Говнокод #7626

    +71

    1. 1
    2. 2
    3. 3
    4. 4
    5. 5
    6. 6
    7. 7
    8. 8
    @Test
 public void testExecuteNoBean1() {   	
    	try {
                //...
		} catch (Exception e) {
			assertEquals("ActionHandlerDefineDueDate: Error, failed to retrieve ... <over 9000 chars>", e.getMessage());
		}
 }

    У кого-то явно проблемы с TDD.

    roman-kashitsyn, 23 Августа 2011

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

  6. Java / Говнокод #7371

    +147

    1. 1
    2. 2
    3. 3
    4. 4
    5. 5
    6. 6
    7. 7
    8. 8
    9. 9
    try {
			final SecurityClientInternalLocal scil =
				(SecurityClientInternalLocal) EJBLocator.getEJBLocal(
						SecurityClientInternalLocal.class);
			final String result = scil.getPath2ChipherTables(login);
			return result;
		} catch (Exception ex) {			
			throw ex;
		}

    Промышленная обработка исключений такая промышленная...

    roman-kashitsyn, 28 Июля 2011

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

  7. Java / Говнокод #7229

    +78

    1. 1
    2. 2
    3. 3
    private String createJndiName(Class homeClass) {
        return "ejb/" + homeClass.getName().replace(".".charAt(0), "/".charAt(0));
}

    no comments

    roman-kashitsyn, 12 Июля 2011

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

  8. C# / Говнокод #7211

    +954

    1. 1
    System.ComponentModel.Win32Exception: The operation completed successfully

    Из стек трейса. Не совсем говнокод, но ОЧЕНЬ смешно.

    roman-kashitsyn, 08 Июля 2011

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

  9. Java / Говнокод #7083

    +78

    1. 1
    public Boolean checkPermission(String login, Integer nIdObject, String sObjectSSCode, Integer nIdOperation, String sOperationSSCode, String sSubsystemCode)

    Вот так Windows разъедает мозг разработчиков. Венгерская запись прокралась в Java-код. Да и Boolean с Integer не совсем в тему.
    И, конечно, метод нужно было назвать 

    hasPermission
    .

    roman-kashitsyn, 27 Июня 2011

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

  10. Java / Говнокод #6969

    +77

    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 class SomeEntityBean implements javax.ejb.EntityBean {
    private boolean loadMember;
    private Wrapper list;

    /* ... */

    public void ejbLoad() {
        this.loadMember = false;
        load();
    }

    private void load() {
        /* a LOT of code */
        String[] attrList = this.loadMember ?  a.attrListMember() : a.attrList();
        this.list = new Wrapper(attrList);
        /* a LOT of code */
    }

    public Something getSomething() {
        /* a LOT of code */
        this.loadMember = true;
        load();
        /* do something with this.list */
        this.loadMember = false;
        return something;
    }
}

    Имена персонажей были умышленно изменены.
    Разумеется, loadMember больше нигде не используется.
    Яркий пример повторного использования кода.

    roman-kashitsyn, 16 Июня 2011

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

  11. Java / Говнокод #6953

    +92

    1. 1
    2. 2
    3. 3
    4. 4
    5. 5
    public Boolean checkPermission(String login) {
    if (login == null && login.trim().length() == 0)
        throw new Exception("Не указан логин пользователя.");
    /* ... */
}

    Чудесное условие. Есть в этом Exception что-то неуловимое - его никто никогда не поймает.

    roman-kashitsyn, 14 Июня 2011

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