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

    +80

    1. 01
    2. 02
    3. 03
    4. 04
    5. 05
    6. 06
    7. 07
    8. 08
    9. 09
    10. 10
    11. 11
    if((remoteMachineName == null || remoteMachineName == "")){
      result = new Result(false,"Invalid Remote Machine Name..!!");     
    }else{
     CheckOrTerminateProcInstanceTask task = new CheckOrTerminateProcInstanceTask
     (processName, argList, oper,  username, password,remoteMachineName,chkTer,chkTerMatch);
     if (remoteMachineName != null && remoteMachineName.length() > 0) {
     ....
     }else{
     ....
     }
    }

    Суровые индийские проверки

    Запостил: Desperate, 20 Ноября 2011

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

    • ava tir 20.11.2011 20:06 # 0

      6-я строчка что ли смутила?
      Ответить

    • ava Desperate 20.11.2011 20:14 # 0

      да, поскольку выражение под 2ым if является отрицанием того, что стоит под первым if... видимо индусы еще не знают, что отрицание произведения есть сумма отрицаний. Следовательно второй else никогда не выполнится
      Ответить

      • ava tir 20.11.2011 21:18 # 0

        ммм.... пожалуй не соглашусь. Смущает вот что:
        remoteMachineName == ""

        т. о. в else ветке может оказаться пустая строка.

        В целом, код конечно же хреновый.
        Ответить

    • ava roman-kashitsyn 21.11.2011 09:34 # 0

      императивный код такой императивный
      так и тянет написать ИМПЕРАТИВОПРОБЛЕМЫ
      Ответить

    • ava guest8 09.04.2019 12:10 # −999

      показать все, что скрытоvanished
      Ответить

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