VPF::обвал по таймеру - Форум программистов

hexzo

Дата 21.4.2011, 13:41 (ссылка)

(нет голосов)

Загрузка ...

Бывалый

Профиль
Группа: Участник
Сообщений: 178
Регистрация: 30.1.2011

Репутация: нет
Всего: нет

Почему, если я хочу провести какие-то манипуляции с интерфейсом по таймеру, который срабатывает после старта приложения, то прога обваливается.

main.cpp

Код


#include <QtGui>
#include "widget.h"

using namespace std;

int main(int argc, char **argv) {
    QApplication app(argc, argv);
    MyWidg *widg = new MyWidg;
    widg->show();

    QTimer *timer = new QTimer;
    timer->setSingleShot(1);
    timer->start(1000);
    QObject::connect(timer, SIGNAL(timeout()), widg, SLOT(doit()));
    return app.exec();
}

widget.h

Код


#include <QtGui>

class MyWidg : public QWidget {
    Q_OBJECT
public:
    MyWidg();
private:
    QHBoxLayout *hlayout;
    QTextEdit *label;
public slots:
    void doit();
};

widget.cpp

Код


#include "widget.h"

MyWidg::MyWidg() : MyWidg::QWidget(0) {
    QTextEdit *label = new QTextEdit;
    QHBoxLayout *hlayout = new QHBoxLayout(this);
    hlayout->addWidget(label);    
}

void MyWidg::doit() {
    label->setText("new text");  //вот оно, падение
}

P.S. Конструктор стопудово отрабатывает до начала работы слота doit(), я qDebugами проверял. И если в слоте doit() что-нибудь делать не с элементами интерфейса, то всё работает путём.

Modul

Дата 21.4.2011, 14:32 (ссылка)

(нет голосов)

Загрузка ...

Бывалый

Профиль
Группа: Участник
Сообщений: 238
Регистрация: 17.2.2006

Репутация: нет
Всего: нет

Цитата(hexzo @ 21.4.2011, 13:41

)

Почему, если я хочу провести какие-то манипуляции с интерфейсом по таймеру, который срабатывает после старта приложения, то прога обваливается.

Ты 2 раза объявил label.

Код


 QTextEdit *label = new QTextEdit;

Надо так:

Код


label = new QTextEdit;

А это что ?

Код


MyWidg::MyWidg() : MyWidg::QWidget(0) {...

Это сообщение отредактировал(а) Modul - 21.4.2011, 14:40

--------------------

I'll be back !

hexzo

Дата 21.4.2011, 14:43 (ссылка)

(нет голосов)

Загрузка ...

Бывалый

Профиль
Группа: Участник
Сообщений: 178
Регистрация: 30.1.2011

Репутация: нет
Всего: нет

Цитата(Modul @ 21.4.2011, 14:32

)

Ты 2 раза объявил label.

Ой, на такие грабли я ещё ни разу не наступал. smile

А почему в конструкторе всё как надо работает, то есть можно с label что угодно делать, а в слоте при обращении к label происходит обвал? Это разные ссылки что ли? smile

math64

Дата 21.4.2011, 22:04 (ссылка)

(нет голосов)

Загрузка ...

Эксперт

Профиль
Группа: Завсегдатай
Сообщений: 2505
Регистрация: 12.4.2007

Репутация: 23
Всего: 72

Кстати, точно также два раза объявлен hlayout - но он в слоте не используется, так что его можно не объявлять полем класса.
В конструкторе создаётся локальная переменная label, а поле класса label остаётся неинициализированным (даже не NULL!)
В doIt() обращается к этому мусору в label и происходит обвал.
Пиши в конструкторе

Код


QTextEdit *label = new QTextEdit;
this->label = label

или

Код


this->label = new QTextEdit;

или

Код


label = new QTextEdit;

borisbn

Дата 21.4.2011, 22:04 (ссылка)

(нет голосов)

Загрузка ...

Эксперт

Профиль
Группа: Завсегдатай
Сообщений: 4875
Регистрация: 6.2.2010
Где: Ростов-на-Дону

Репутация: 48
Всего: 135

Цитата(hexzo @ 21.4.2011, 14:43

)

А почему в конструкторе всё как надо работает, то есть можно с label что угодно делать, а в слоте при обращении к label происходит обвал? Это разные ссылки что ли?

Цитата(hexzo @ 21.4.2011, 13:41

)

QTextEdit *label = new QTextEdit;

создаёт "местную" переменную, с которой ты можешь делать всё, что хочешь до выхода из ф-ции

--------------------

Женщины отличаются от программистов тем, что у них чары состоят из стрингов

0 Пользователей читают эту тему (0 Гостей и 0 Скрытых Пользователей)
0 Пользователей:
« Предыдущая тема \| С/С++: Кроссплатформенное программирование, Qt/Gtk+/wxWidgets \| Следующая тема »