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

    +71

    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
    public static Connection getDBConnectionReader() throws Exception {
        Connection conn = null;
        try {
            conn = enrollmentsDataSource.getConnection();
        } catch (SQLException e) {
            LOGGER.warn("SQL Exception: get DB connection reader", e);
            resetConnectionReader();
            try {
                conn = enrollmentsDataSource.getConnection();
            } catch (SQLException e1) {
                throw new Exception("Exception: get DB connection reader", e1);
            }
        }
        return conn;
}

    Фрактал...

    Запостил: amarfey, 20 Марта 2013

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

    • ava bormand 20.03.2013 17:39 # 0

      Ну а что, масштабируемое решение. Надо будет сделать 3 попытки - заменим строку 11 на копию 6-12, и так далее.
      Ответить

    • ava scriptin 20.03.2013 17:58 # +5 

      throw new Inception("We need to go even deeper");
      Ответить

    • ava zummenix 20.03.2013 18:39 # +1

      А как правильно в таком случае?
      Ответить

      • ava VseGovnoOdinYaKrut 20.03.2013 19:28 # 0

        Такое чувство, что всё говно внутри метода resetConnectionReader() осталось. Или же ГК в использовании лишней переменной conn?
        Ответить

        • ava zummenix 20.03.2013 19:55 # 0

          Что-то волшебное там происходит, после чего соединение магически восстановится :)
          Ответить

      • ava bormand 20.03.2013 19:33 # +1

        Ну я бы цикл написал. И строк меньше, и число попыток легко поменять...
        Ответить

        • ava zummenix 20.03.2013 19:50 # −3

          Насколько я знаю, с ексепшинами цикла не напишешь.
          Ответить

          • ava bormand 20.03.2013 19:59 # +5

            > ексепшинами цикла не напишешь.
            Кул стори, бро.

            P.S. А из топора каши не сваришь ;)
            Ответить

          • ava VseGovnoOdinYaKrut 20.03.2013 20:01 # +1

            Как-то так это будет выглядеть в цикле
            public static Connection getDBConnectionReader() throws SQLException {
    for (int i = 0; ; i++) {
        try {
            return enrollmentsDataSource.getConnection();
        }
        catch (SQLException ex) {
            if (i >= 1) {
                throw ex;
            }
            LOGGER.warn("SQL Exception: get DB connection reader", ex);
            resetConnectionReader();
        }
    }    
}
            Ответить

            • ava VseGovnoOdinYaKrut 20.03.2013 20:10 # +2

              А вообще в данном случае и цикл не нужен, resetConnectionReader() не сработал — адью! Другое дело, что всю писанину можно сократить до:
              public static Connection getDBConnectionReader() throws SQLException {
        try {
            return enrollmentsDataSource.getConnection();
        }
        catch (SQLException ex) {
            LOGGER.warn("SQL Exception: get DB connection reader", ex);
            resetConnectionReader();
            return enrollmentsDataSource.getConnection();
    }    
}
              Ответить

              • ava bormand 20.03.2013 20:16 # 0

                > и цикл не нужен
                Не, ну так категорично не надо, не от хорошей жизни же стали вторую попытку делать... Видимо какие-то проблемы с подключением были, а лишний раз беспокоить юзера не хотят.

                Но вот две попытки вызывают в моей дурной голове стойкую мысль о магическом числе. Почему не семь? Почему не сорок две?
                Ответить

                • ava VseGovnoOdinYaKrut 20.03.2013 20:29 # +2

                  Попытка одна и это попытка реконнекта в методе resetConnectionReader(). Почему я так думаю. С наибольшей вероятностью тут используется пул коннектов, т.е. getConnection() возвращает соединение из загашника, а оно может и протухнуть, например если СУБД перезагружали. И тут на помощь приходит метод resetConnectionReader, который волшебным образом восстанавливает протухшие коннекты.
                  Ответить

                  • ava bormand 20.03.2013 20:43 # 0

                    А, вот оно как. Ну тогда это не такой уж говнокод.
                    Ответить

                    • ava 3.14159265 20.03.2013 21:06 # +1

                      Думал что очевидно, но вижу что нет. Придется отписаться.
                      >public static Connection getDBConnectionReader() throws Exception
                      Гавно это нубское, на уровне лабы. Есть один признак:

                      Нормальные, ленивые люди не получают руками соединения из датасорсов и не закрывают их и говнопулы c попытками они тоже не пишут.

                      За них это делают либы. А DataSource - это кошерная абстракция-обёртка.
                      Ответить

      • ava someone 21.03.2013 06:19 # +1

        Совсем правильно - завести пул. Есть же DBCP.
        Ответить

    • ava Lure Of Chaos 20.03.2013 23:19 # +1

      а еще говно в том, что метод статический. т.е. более одного источника ну никак не судьба (сомневаюсь, что они пулятся циклически\рандомно)
      Ответить

    • ava guest 22.03.2013 21:22 # −2 

      public static Connection getDBConnectionReader() throws Exception {
begin:        
Connection conn = null;
        try {
            conn = enrollmentsDataSource.getConnection();
        } catch (SQLException e) {
            LOGGER.warn("SQL Exception: get DB connection reader", e);
            resetConnectionReader();
           goto begin;
        }
        return conn;
}
      Ответить

      • ava roman-kashitsyn 22.03.2013 21:30 # +1

        в жабе нет goto
        Ответить

      • ava 3.14159265 22.03.2013 21:53 # +2

        Про цикл while еще учитель не рассказал? По всей видимости не зря убрали.
        Не пойму это тут много вас идиотов, или перед нами один уёбок, который толком ни знает ни о структурах, ни английского, ни явы?
        Ответить

        • ava bormand 22.03.2013 22:00 # +2

          > По всей видимости не зря отказались.
          Ну, как я когда-то писал, в Си у goto есть несколько адекватных применений:
          1) выход из нескольких циклов - в жабе есть особый бряк;
          2) прыжок в конец функции для освобождения ресурсов - в жабе есть finally;
          3) конченные автоматы, высранные кодогенераторами - вот тут фейл, придется запилить по-другому, или генерить сразу байткод;

          Поэтому goto в жабе не особо нужно.
          Ответить

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