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

    +68

    1. 1
    2. 2
    3. 3
    4. 4
    5. 5
    6. 6
    boolean occured = serverStats.getSwitchoverOccured();    
     
  if (occured)     
   return true;
  else  
   return false;

    Запостил: dzagy, 04 Сентября 2012

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

    • ava Steve_Brown 04.09.2012 12:14 # +1

      Случилось страшное.
      Ответить

    • ava bormand 04.09.2012 13:04 # 0

      Без первой строки минусанул бы за очередной бульшит. Но название переменной occured порадовало, поэтому плюс ;)
      Ответить

    • ava eth0 04.09.2012 18:06 # +4

      Не хватает таких простых и обыденных вещей: тернарного оператора, приведения к строке, инту и возвращения false эксепшном вверх.
      Ответить

      • ava bormand 04.09.2012 19:16 # +3

        Так?
        try {
    string occurred = new Boolean(serverStats.getSwitchoverOccured()).toString();
    throw "false".equals(occurred) ? new RuntimeException() : new Exception();
} catch (RuntimeException e) {
    return false;
} catch (Exception e) {
    return true;
}
        Ответить

        • ava 3.14159265 04.09.2012 19:26 # 0 

          finally{
     return serverStats.getSwitchoverOccured();
}
          Ответить

        • ava eth0 04.09.2012 20:39 # +1

          Вариант хороший, но я бы предполагал именно передачу эксепшона наверх, в вызывающую функцию.
          Ответить

          • ava roman-kashitsyn 04.09.2012 21:02 # 0 

            if (serverStats.getSwitchoverOccured())
    return new RuntimeException("Switchover occured");
else
    return new RuntimeException("Ok");
            Ответить

            • ava bormand 04.09.2012 21:24 # +2 

              if (serverStats.getSwitchoverOccured())
    return new TrueException();
else
    return new FalseException();
// ... и пример вызова
try {
    throw someFunction();
}
catch (TrueException e) {
    // все ок
}
catch (FalseException e) {
    // было переключение
}
              Ответить

              • ava rat4 04.09.2012 21:39 # +1 

                if (serverStats.getSwitchoverOccured())
  return new ReturnExceptionFactory.createBoolean(true);
else
  return new ReturnExceptionFactory.createBoolean(false);
// ... и пример вызова
try {
    throw someFunction();
}
catch (ReturnException e) {
  if (e.containsBoolean() && e.getBoolean() == true)
    // всё ок
  else
    // было переключение
}
                Ответить

            • ava Steve_Brown 05.09.2012 14:45 # +1 

              ...
catch (Exception e)
{
  if (e.Message == "Switchover occured")
    ...
}
              Ответить

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