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

0

  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 mrFoxs, (Updated )

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

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

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

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

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

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

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

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

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

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

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

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

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

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

Где здесь C++, guest?!

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

    ↑
    8
    ↓