Signal из Dialog

Рекомендуємо хостинг TIMEWEB

Стабільний хостинг, на якому розміщується соціальна мережа EVILEG. Для проектів на Django радимо VDS хостинг.

В общем,с теми вопросами которые сверху я вроде разобрался, но появился другой , как вытащить какую-то информацию из Dialog? Я так понимаю,данные еще хранятся как мы выходим из Dialog,но как вытащить? У меня есть форма из нее я вызываю диалог, в диалоге выбираю после в tableview и мне надо вернуть обратно выбранное значение,как это сделать?

Все зависит от того, как был создан диалог. Если он является локальной переменной, как в следующем варианте:

void Widget::on_pushButton_clicked()
{
    Dialog dialog;
    dialog.exec();
}

То при закрытии диалог будет разрушен, и вся информация, которая хранится в нём, будет потеряна. Если же диалог объявлен более глобально, например в объявлении класса, в котором он вызывается, то можно добавить в диалог специальные методы, которые будут возвращать информацию.

А вообще, можно переопределить некоторые методы, например метод accept() , который вызывается при нажатии кнопки ОК . А в этом методе вызывать сигнал, к которому можно подключить слот приёма информации.

void Widget::on_pushButton_clicked()
{
    Dialog dialog;
    connect(&dialog, &Dialog::sendInfo, this, &Widget::receiveInfo);
    dialog.exec();
}

void Widget::receiveInfo(bool info)
{
    qDebug() << "Send Info" << info;
}

В самом диалоге это будет выглядеть так.

Заголовочный файл

class Dialog : public QDialog
{
    Q_OBJECT

public:
    explicit Dialog(QWidget *parent = 0);
    ~Dialog();

    virtual void accept() override;

signals:
    void sendInfo(bool info);

private:
    Ui::Dialog *ui;
};

Файл реализации

void Dialog::accept()
{
    emit sendInfo(true);
    QDialog::accept();
}

Сделал вот так, правильно ли это реализовано? просто как бы работает,но может стоит лучше? Запуск диалогового окна

void decorticket::ViewClients()
{
    viewclients *dlg = new viewclients(this);
    if(dlg->exec() != QDialog::Rejected){
         ui->fio->setText(dlg->selectclient());
    }
    delete dlg;
}

connect(ui->selectbutton, SIGNAL(clicked()), this, SLOT(selectclient()));

QString viewclients::selectclient()
{
    accept();
    return ui->clientview->model()->index(ui->clientview->selectionModel()->currentIndex().row(),0).data().toString();
}

public slots:
    QString selectclient();

Неправильно реализовано.

1. Коннект сигнала к слоту абсолютно бессмысленен. Да. он срабатывает, Но данные вы в итоге забираете не через сигнал-слотовое соединение, а обычным вызовом публичного метода. В результате метод selectclient() вызывается дважды. То есть до кучи ещё и метод accept() дважды вызывается - это совсем уж не хорошо. Он должен один раз вызваться.
2. Не надо сохранять объект диалога в куче, то есть выделять память под него через new, если вы его в этом же методе удаляете. Посмотрите как у меня реализовано . А то так забудете где-нибудь delete и будет утечка памяти.
3. Если хотите через сигналы и слоты, то коннект должен быть внутри основного окна к сигналу из диалога, а внутри основного окна нужно делать слот.
4. Ну и если учесть специфику сигнал/слотовых соединений, то return в слотах в некоторой мере менее необходим, чем в обычных методах типа get.
5. И используйте новый синтаксис сигналов и слотов на указателях. В этой статье он поясняется . Ошибочные макросы не ловятся на этапе компиляции, а новый синтаксис ловит. К тому же с новым синатксисом можно лямбды использовать.

Спасибо за эти пункты выше) Скажите пожалуйста,сейчас я правильно реализовал? Запуск диалогового окна и slot приема

void decorticket::ViewClients()
{
    viewclients dlg;
    connect(&dlg, &viewclients::send, this, &decorticket::receive);
    dlg.exec();
}

void decorticket::receive(QString number)
{
    ui->fio->setText(number);
}

virtual void accept() override;

void viewclients::on_selectbutton_clicked()
{
    emit send(ui->clientview->model()->index(ui->clientview->selectionModel()->currentIndex().row(),0).data().toString());
    accept();
}

Да. Теперь лучше выглядит. С методом тоже правильно поняли. Вместо переопределения метода accept() можете взять слот, прикрученный к кнопке и вызывать сигнал и метод accept() в этом самом слоте.

А теперь по код стайлу. Название классов никто и ни в каких командах так не пишет. Или пишите в стиле CamelCase , как написали метод ViewClients() , то есть

void DecorTicket::ViewClients()

Или в стиле с нижним подчёркиванием.

void decor_ticket::ViewClients()

Но если будете писать на Qt, то используйте код стайл Qt, то есть CamelCase. И название методов в данном случае должны начинаться с маленькой буквы, то есть:

void DecorTicket::viewClients()

Спасибо большое за разъяснения. Буду исправлять ошибка