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

    −45.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
    if ((solution.indexOf("filter") != -1)
                    || (solution.indexOf("Filter") != -1)
                    || (solution.indexOf("FILTER") != -1)) {
                solution = "Filter incoming traffic to port/service \""
                        + serviceName + "\" if the service is not used";
            } else if ((solution.indexOf("package") != -1)
                    || (solution.indexOf("Package") != -1)
                    || (solution.indexOf("PACKAGE") != -1)
                    || (solution.indexOf("vendor") != -1)
                    || (solution.indexOf("Vendor") != -1)
                    || (solution.indexOf("VENDOR") != -1)
                    || (solution.indexOf("service") != -1)
                    || (solution.indexOf("Service") != -1)
                    || (solution.indexOf("SERVICE") != -1)) {
                solution = "AMBIQUITY_SOLUTION_START port/service \""
                        + item.getServiceName() + "\'} " + solution;
            }

//...............................................etc

    Развиваю один старый проект. Не буду говорить о всех перлах того, что я там увидел, про великую архитектуру проекта, но кое-что хотелось бы показать.
    З.Ы. Я бы хотя бы сделал так
    solution.toLowerCase().indexOf("package" ) != -1
    З.З.Ы Этот ужас таки порефакторил и его убрали :)

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

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

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