Jawa / Говнокод #11699 Ссылка на оригинал

0

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

Запостил: dzagy dzagy, (Updated )

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

  • ava bormand # Ссылка на оригинал 0

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

  • ava eth0 # Ссылка на оригинал 0

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

    • ava bormand # Ссылка на оригинал 0 ⇈

      Так?
      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 eth0 # Ссылка на оригинал 0 ⇈

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

        • ava roman-kashitsyn # Ссылка на оригинал 0 ⇈ 

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

          • ava bormand # Ссылка на оригинал 0 ⇈ 

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

            • ava rat4 # Ссылка на оригинал 0 ⇈ 

              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
    // было переключение
}
              Ответить

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

Я, guest, находясь в здравом уме и твердой памяти, торжественно заявляю:

    А не использовать ли нам bbcode?

    ↑
    8
    ↓