Как в обработчике нажатия кнопки использовать локальную переменную другого метода?

@akellaV · Регистрация: 19.02.2015

Студворк — интернет-сервис помощи студентам

Нужно чтобы в методе кнопки clicked() можно было использовать переменную db из MainWindow

C++ (Qt)

#include "mainwindow.h"
#include "ui_mainwindow.h"
#include <QDebug>
#include <QtSql>
 
MainWindow::MainWindow(QWidget *parent) :
    QMainWindow(parent),
    ui(new Ui::MainWindow)
{
    ui->setupUi(this);
    QString path="C:\\Users\\akella\\Desktop\\BD\\Games.sqlite";
    QSqlDatabase db=QSqlDatabase::addDatabase("QSQLITE");
    db.setDatabaseName(path);
    db.open();
    qDebug()<<db.open();
 
}
MainWindow::~MainWindow()
{
    delete ui;
}
 
void MainWindow::on_pushButton_clicked()
{
// не видит db
}

и как можно проделать то же самое если открытие db будет в main.cpp?

@0x90h · 27.11.2015, 03:21

Объявите db в хедере MainWindow:

C++ (Qt)

1 2	#include <QtSql> QSqlDatabase db;

а в конструкторе:

C++ (Qt)

1 2	QString path="C:\\Users\\akella\\Desktop\\BD\\Games.sqlite"; db=QSqlDatabase::addDatabase("QSQLITE");

Если будете открывать соединение в main.cpp, то информацию о соединении в MainWindow можно получить с помощью QSqlDatabase db = QSqlDatabase::database();, дальше запрашиваете isOpen(), если соединение не открыто, то подробнее об ошибке можно узнать с помощью lastError().

P.S.: и пользуйтесь тегами редактора, неудобно код читать.

@OneMoreGres · 27.11.2015, 09:32

0x90h, не обязательно (я бы даже сказал вредно) делать db глобальной переменной: QSqlDatabase::database() достаточно, чтобы получить то же соединение.

QSqlDatabase is a value class. Changes made to a database connection via one instance of QSqlDatabase will affect other instances of QSqlDatabase that represent the same connection.

@0x90h · 27.11.2015, 10:15

OneMoreGres,

а вы разницу между глобальными переменными и данными-членами, видимо, не очень осознаете

@OneMoreGres · 27.11.2015, 13:51

Не по теме:

0x90h, если отвечать в таком же стиле, то подключать заголовочные файлы в теле класса нельзя :p А именно это изображено в вас в примере, если не искать глубокий смысл (а человек, спрашивающий, как сделать переменную доступной для двух функций этого делать не будет).

В любом случае, копировать экземпляр соединения с БД не имеет смысла, разве только для сокращения кода при первом вызове.

@0x90h · 27.11.2015, 13:55

Сообщение от 0x90h

Объявите db в хедере MainWindow:

В "теле класса"? А все-таки, где вы увидели глобальную переменную? Насчет целесообразности копирования соединения это к ТС

@OneMoreGres · 27.11.2015, 14:08

Сообщение от 0x90h

В "теле класса"?

C++ (Qt)

1
2
3

class foo {
#include <QtSql>
}

В принципе, конечно, не запрещено, но работать в подавляющем большинстве случаев не будет.

Сообщение от 0x90h

А все-таки, где вы увидели глобальную переменную?

Сообщение от 0x90h

C++ (Qt)

1 2	#include <QtSql> QSqlDatabase db;

По указанной выше причине делаю вывод, что объявление не внутри класса. Сочетание "хедер MainWindow" в данном случае конкретное место не указывает.

З.Ы. На всякий случай продублирую:

если не искать глубокий смысл (а человек, спрашивающий, как сделать переменную доступной для двух функций этого делать не будет)

@0x90h · 27.11.2015, 14:19

Сообщение от OneMoreGres

объявление не внутри класса

OneMoreGres, в хедере класса подключили заголовочник QtSql и объявили db в private-разделе, никаких глобальных переменных тут в принципе нет.

Добавлено через 4 минуты

C++ (Qt)

1 2	#include <QtSql> QSqlDatabase db;

подразумевалось, где должны находиться #include <QtSql> и QSqlDatabase db;. То, что я не расписал, с точностью до строчки, содержимое хедера, еще не основание делать выводы, подобные вашим о "глобальных" переменных и "вредности".

Не по теме:

Поисками "глубокого смысла" здесь занимаетесь как раз вы.

@OneMoreGres · 27.11.2015, 14:20

То, что вы подразумевали не совпадает с тем, что вы написали (учитывая уровень вопроса). Дальше развивать не буду. Считаете, что я не прав - ок, не возражаю.

@OneMoreGres 72 / 72 / 33 Регистрация: 07.11.2015 Сообщений: 281
	27.11.2015, 09:32	3
	0x90h, не обязательно (я бы даже сказал вредно) делать db глобальной переменной: QSqlDatabase::database() достаточно, чтобы получить то же соединение. QSqlDatabase is a value class. Changes made to a database connection via one instance of QSqlDatabase will affect other instances of QSqlDatabase that represent the same connection. 0

@0x90h 676 / 452 / 159 Регистрация: 01.10.2015 Сообщений: 1,259
	27.11.2015, 10:15	4
	OneMoreGres, а вы разницу между глобальными переменными и данными-членами, видимо, не очень осознаете 0

@OneMoreGres 72 / 72 / 33 Регистрация: 07.11.2015 Сообщений: 281
	27.11.2015, 13:51	5
	Не по теме: 0x90h, если отвечать в таком же стиле, то подключать заголовочные файлы в теле класса нельзя :p А именно это изображено в вас в примере, если не искать глубокий смысл (а человек, спрашивающий, как сделать переменную доступной для двух функций этого делать не будет). В любом случае, копировать экземпляр соединения с БД не имеет смысла, разве только для сокращения кода при первом вызове. 0

@0x90h 676 / 452 / 159 Регистрация: 01.10.2015 Сообщений: 1,259
	27.11.2015, 13:55	6
	Сообщение от 0x90h Объявите db в хедере MainWindow: В "теле класса"? А все-таки, где вы увидели глобальную переменную? Насчет целесообразности копирования соединения это к ТС 0

@OneMoreGres 72 / 72 / 33 Регистрация: 07.11.2015 Сообщений: 281
	27.11.2015, 14:20	9
	То, что вы подразумевали не совпадает с тем, что вы написали (учитывая уровень вопроса). Дальше развивать не буду. Считаете, что я не прав - ок, не возражаю. 0

Опции темы