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

    +74

    1. 1
    2. 2
    3. 3
    4. 4
    5. 5
    6. 6
    synchronized public void refreshConversionTable() {
        btnClick = true;
        if (dataModel != null)
            dataModel.reset();
        dataModel = null;
    }

    если модель данных существует, сначала сбросить все данные, а потом занулим....

    Запостил: mrFoxs, 31 Января 2012

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

    • ava QuickNick 31.01.2012 10:44 # 0

      Если за такое не убивать, то за что вообще тогда убивать? (с)
      Ответить

    • ava guest 31.01.2012 11:35 # 0

      А в чем тут ошибка (я не профессиональный программист, но интересно)?
      Ответить

    • ava absolut 31.01.2012 12:07 # 0

      Ну забыли 4-5 в блок взять. Подумаешь, лишний раз занулили.
      Ответить

      • ava guest 31.01.2012 12:11 # 0

        Если проблема только в этом, тогда это вроде и не проблема?
        Ответить

        • ava mrFoxs 31.01.2012 13:33 # 0

          dataModel.reset - запрос большого набора табличных данных
          а дальше мы убиваем переменную, в которую записали кучу данных
          тут и тормоза для системы и лишнее обращение в БД
          Ответить

          • ava guest 31.01.2012 13:37 # +1

            аа. Ясно, спасибо. Как-то сложно догадаться по названию метода reset() что внутри него выполняется тяжелый запрос. Да еще и с выборкой данных.
            Ответить

            • ava mrFoxs 31.01.2012 14:17 # 0

              да, виноват, нужно было подробнее расписать
              Ответить

            • ava guest 07.02.2012 04:44 # 0

              Автор этого метода не слышал о нормальных именах
              Ответить

      • ava mrFoxs 31.01.2012 13:24 # 0

        Эх если бы, это логика такая
        Ответить

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