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

    +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
    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
    35. 35
    36. 36
    37. 37
    38. 38
    39. 39
    if (myTransactions
        .get(i)
        .getTransactionType()
        .name()
        .compareToIgnoreCase(
                com.mycompany.myproject.common.TransactionType.ENTER
                        .name()) == 0
        || myTransactions
                .get(i)
                .getTransactionType()
                .name()
                .compareToIgnoreCase(
                            com.mycompany.myproject.common.TransactionType.ENTER_AGAIN
                                    .name()) == 0) {

    BigDecimal fee = myTransactions.get(i).
            .getAmount().subtract(
                    myTransactions.get(i)
                            .getContribution());
// ...
}

// Notes by Wissenstein
//
// package com.mycompany.myproject.common;
// public enum TransactionType {
//     ENTER, ENTER_AGAIN, GET, GIVE
// }
// ...
// public class Transaction {
// ...
//     public TransactionType getTransactionType() {
// ...
//     }
// }
// ...
// List<Transaction> myTransactions;
// ...
// P.S. Идентификаторы изменены.

    К тому, что myTransactions.get(i) повторяется, я уже привык и рефакторю это одним махом.
    Однако использованный здесь метод сравнения значений типа enum меня озадачил…

    Запостил: wissenstein, 23 Января 2014

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

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

    Ошибка компиляции комментария:
    1. Гости могут высказаться только во вторник, пятницу или субботу
    ava Семь раз отмерь — один отрежь, guest!
    А не использовать ли нам bbcode?
    • [b]жирный[/b] — жирный
    • [i]курсив[/i] — курсив
    • [u]подчеркнутый[/u] — подчеркнутый
    • [s]перечеркнутый[/s] — перечеркнутый
    • [blink]мигающий[/blink] — мигающий
    • [color=red]цвет[/color] — цвет (подробнее)
    • [size=20]размер[/size] — размер (подробнее)
    • [code=<language>]some code[/code] (подробнее)
    Проверочный код