Некорректное закрытие QSqlDatabase, при использовании QSqlTableModel Опции

[acen83]

Скорее всего, у тебя остаются висеть незакрытые запросы

Remove database у меня есть, но он закрывает только ругательства о незакрытых запросах, а сообщения о закрытии старой БД все равно остаются

Плохо что нет какой-нибудь функции наподобие clear() и соответственно isNull(), для того чтобы определять была инициализирована база или нет. Так получить базу через QSqlDatabase::database() конечно можно, но как проверить была она уже настроена или нет

[Litkevich Yuriy]

во-первых, в программе работай по порядку, и давай имена соединениям. И по этим именам обращайся к соединению.
во-вторых, в Qt'ях работа идет с именоваными соединениями,

Используйте contains(), чтобы видеть если заданное имя соединения в списке соединений

если нет то надо создать, а вообще удалять соединение, все-таки нужды нет.

функции наподобие clear() и соответственно isNull()

что должны были бы делать эти функции, еслиб они были?

[acen83]

2 Litkevich Yuriy:

clear() - возвращало бы все в начальное состояние, как при запуске программы.

Дело в том что после QSqlDatabase::removeDatabase("qt_default_connection_что_то_типа_этого") в списке соединений ничего нет. Т.е. базу мы закрыли, удалили соединение, а при запуске новой базы ругательство все равно остается - непорядок.

У меня при нажатии на кнопку происходит инициализация БД, настройка БД, выполнение запросов, закрытие всего этого хозяйства. Получается в разных частях программы надо создавать именованные соединения, причем вынесенные в конструктор? че то некрасиво как то, тем более вероятность нажатия кнопки - 1 к 100.

[Litkevich Yuriy]

у меня в классе главного окна создается экземпляр "одиночки", а внем соединение с БД настроек приложения (это я так решил вместо всяких ини-конфигов сделать). далее после конструктора, т.е. после того как окно программы отобразится создается соединение основной БД.

исходник

MainWindow::MainWindow()
{
    mdiArea = new QMdiArea;
    setCentralWidget(mdiArea);
    connect(mdiArea, SIGNAL(subWindowActivated(QMdiSubWindow *)), this, SLOT(updateMenus()));
    
    windowMapper = new QSignalMapper(this);
    connect(windowMapper, SIGNAL(mapped(QWidget *)), this, SLOT(setActiveSubWindow(QWidget *)));

    createActions();    // Создаем действия
    createMenus();        // Создаем меню
    createToolBars();    // Создаем панель инструментов
    createStatusBar();    // Создаем панель статуса
    readWindowSettings();// Читаем настройки окна
    // Создаем экземпляр класса сесии
    auth = Session::InitInstance();    

    
    connect(this, SIGNAL(SigShow()), SLOT(SlotOnShow()));
    
    
}

MainWindow::~MainWindow()
{
    auth->FreeInstance();
}

bool MainWindow::event(QEvent *p_event)
{
    if( p_event->type() == QEvent::ShowToParent)
        emit SigShow();

    return QWidget::event(p_event);
}

/*=====================================================================
                Постинициализация
  ===================================================================*/
void MainWindow::SlotOnShow()
{
  QString    errmsg;
  QString    title = tr("Ошибка программы");

qDebug() << "SlotOnShow: " << QDateTime::currentDateTime() << "\n\r";
  
    // Вход от имени анонимного пользователя 
    auth->LoginAnonymous();
    
    // Обновляем меню
    updateMenus();
    
    // Проверяем все ли хорошо с настройками
    if (!auth->isGood(errmsg))
    {
        QMessageBox::critical(this, title, tr("Ошибка настроек, обратитесь к специалисту. Сообщение: \n\r") + errmsg);
        return;
    }
    
    // Здесь проверяется и устанавливается соединение с БД.
  bool ok;
  QSqlDatabase app_db = QSqlDatabase::database("appdb");    // создано в Session::InitInstance();
  QSqlQuery query(app_db);
  QString    querystr;
  QString    qerr;
  
  QString    db_driver;
  QString    db_name;
  QString    db_user_name;
  QString    db_user_paswd;
  
  
    querystr="SELECT F_VALUE FROM APPSETTINGS \
    WHERE ((F_TYPE='db_driver') OR (F_TYPE='db_name') OR \
    (F_TYPE='db_user_name') OR (F_TYPE='db_user_paswd')) \
    ORDER BY F_TYPE ASC";
    ok = query.exec(querystr);
qDebug() << "select DB settings - " << ok << "\n\r";
    
    if (!ok)
    {
        qerr = query.lastError().text();
        QMessageBox::critical(0, title,
                                 tr("Невозможно прочитать настройки БД.\n\r Причина: ")+qerr,
                                QMessageBox::Ok);
    }
    
    query.first();
    if (!query.isValid())
    {
        qerr = query.lastError().text();
        QMessageBox::critical(0, title,
                                 tr("Невозможно прочитать настройки БД.\n\r Причина: ")+qerr,
                                QMessageBox::Ok);
    }
    
    db_driver = query.value(0).toString();
    query.next();
    db_name = query.value(0).toString();
    query.next();
    db_user_name = query.value(0).toString();
    query.next();
    db_user_paswd = query.value(0).toString();
    
    db_main = QSqlDatabase::addDatabase(db_driver,"maindb");
    db_main.setDatabaseName(db_name);
    db_main.setUserName(db_user_name);
    db_main.setPassword(db_user_paswd);
qDebug() << "db_main.open: " << QDateTime::currentDateTime() << "\n\r";
    ok = db_main.open();
    
    if (!ok)
    {
        qerr = db_main.lastError().text();
        QMessageBox::critical(0, title,
                                 QObject::tr("Невозможно установить соединение c БД.\n\r")+qerr,
                                QMessageBox::Ok);
    }
qDebug() << "db_main.open: " << QDateTime::currentDateTime() << "\n\r";
qDebug() << db_main << "\n\r";
}

[Tonal]

2 Litkevich Yuriy
Нет нужды делать соединение с БД одиночкой, т.к. оно само только удобный интерфейс к реальному соединению, которое уже одиночка.
Вроде с тобой же обсуждали в "переводах" значение value type в данном контексте.

[Litkevich Yuriy]

у меня в классе главного окна создается экземпляр "одиночки"

Да, надо было полнее прокоментировать. Это у меня для авторизации пользователей программы, но там же и соединение с БД хранящих настройки пользователей.

А соединение с основной БД так:

далее после конструктора, т.е. после того как окно программы отобразится создается соединение основной БД.

[silver]

Remove database у меня есть, но он закрывает только ругательства о незакрытых запросах, а сообщения о закрытии старой БД все равно остаются

Может я чего-то пропустил? Что ещё за сообщение о закрытии старой БД? В первом посте ты указал только одно сообщение: о том, что на момент закрытия соединения остались запросы, которые это соединение используют.

Плохо что нет какой-нибудь функции наподобие clear() и соответственно isNull(), для того чтобы определять была инициализирована база или нет. Так получить базу через QSqlDatabase::database() конечно можно, но как проверить была она уже настроена или нет

Есть функции QSqlDatabase::isOpen() и QSqlDatabase::isValid(), которые именно это и делают.

[Litkevich Yuriy]

Что ещё за сообщение о закрытии старой БД?

да он дважды одно и тоже соединение создает:

QSqlDataBase* bd=QSqlDataBase::addConnection("QMYSQL");
// настройка базы, логина и т.п.
bd->open();
// запросы
bd->close();

затем в другом месте программы повторяя этот код появляется предупреждение

QSqlDatabasePrivate::removeDatabase: connection 'qt_sql_default_connection' is still in use, all queries will cease to work.

[acen83]

Есть функции QSqlDatabase::isOpen() и QSqlDatabase::isValid(), которые именно это и делают.

Так чтоли?

QSqlDatabase db=QSqlDatabase::database();
if (!db.isValid())
{
db.addConnection("QMYSQL");
// добавление логинов, пассов и т.п.
}
return db;

Сообщение отредактировал acen83 - 7.8.2008, 18:47

[silver]

Так чтоли?

Ну, как один из вариантов:

    QSqlDatabase db = QSqlDatabase::database("my_very_rarely_used_db");
    if (!db.isValid()) {
            // выполнится только в первый раз, когда соединения ещё нет
        db = QSqlDatabase::addDatabase("QMYSQL", "my_very_rarely_used_db");
        // Заполняем параметры соединения
    }
    db.open();
    // работаем
    db.close();
    // всё, никаких removeDatabase() делать не надо

Хотя я бы всё-таки создал именованное соединение в конструкторе, и не открывал его. Эта операция практически не займёт ни времени, ни ресурсов. Зато когда БД понадобится, можно сразу открывать её и работать, без всяких дополнительных проверок.