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

    −34.2

    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
    public boolean foo(...){
      ...
      try{
      ...
      }
      finally{
        if(expr_1){
          return false;
        }
        if(expr_2){
          return false;
        }
        ...
        if(expr_n){
          return false;
        }
      }
      ...
      return true;
    }

    поймал на сопровождении, долго курил (метод на 3 экранах)

    Запостил: guest, 28 Ноября 2008

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

    • Сильно круто для finally. Сам по себе блок finally несет другую семантику. Тогда уж лучше в catch-блоке делать обработку, причем по видам исключений. А еще лучше - делать несколько try-catch блоков, и в каждом проверять свое. И если много вложенностей - вынести в отдельные методы (имеющие семантический смысл!)
      Ответить
      • показать все, что скрытоvanished
        Ответить
        •                                                                   ▄                                                                                                                 
          ██   ██                                                      ▄▄▄▄█▀              ▀██                                                ▀██                        ▄▄ ▄▄          ▄██▄  
          ██  ▄██  ▄▄▄▄▄           ▄▄▄▄▄   ▄▄▄▄        ▄▄▄▄▄▄  ▄▄▄▄▄  ██       ▄▄▄▄▄▄       ██▄▄    ▄▄▄▄▄  ▄▄ ▄▄▄  ▄▄▄ ▄▄   ▄▄▄▄   ▄▄ ▄▄▄    ▄▄██       ▄▄   ▄▄ ▄▄ ▄ ▄▄  ▄▄▄▄▄     ▄▄▄▄ ████  
          ██▄█▀██ ▀▀   ██ ▄▄▄▄▄▄▄ ▀▀   ██  ▄▄▄██       ▀ ██ ▀ ██   ██ ██▀▀▀█▄ ██   ██       ██ ▀█▄ ██   ██  ██▀ ██ ██▀█▀██  ▄▄▄██   ██  ██ ▄█▀ ██       ██   ██ ██ █ ██ ██   ██  ▄█▀ ██  ██   
          ██▀  ██   ▀▀▀█▄           ▀▀▀█▄ ██  ██         ██   ██▀▀▀▀▀ ██   ██  ▀██▀██       ██  ██ ██   ██  ██     ██ █ ██ ██  ██   ██  ██ ██  ██       ██   ██ ██ █ ██ ██▀▀▀▀   ██  ██  ▀▀   
          ██   ██ ▀█▄▄▄█▀         ▀█▄▄▄█▀ ▀█▄▄▀█▄       ▄██▄  ▀█▄▄▄█▀ ▀█▄▄▄█▀ ▄██  ██       ██▄▄█▀ ▀█▄▄▄█▀ ▄██▄    ██ ▀ ██ ▀█▄▄▀█▄  ██  ██ ▀█▄▄▀█▄       ▀▀▀▀██ ██▄█▄██ ▀█▄▄▄█▀ ▄██  ██  ██   
                                                                                                                                                        ▄▄   ██                               
                                                                                                                                                         ▀▀▀▀▀                                
          Ответить
    • показать все, что скрытоvanished
      Ответить

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