MainWindowクラスから他のクラスの呼出方法

Question

Qt5.5.1、QtCreator3.5.1でウィジェットAppを作成しています。

MainWindowクラスからPrinterクラスのprintルーチンを呼び出す場合に
ポインタを介しての呼び出しの場合は、突然プログラムを終了します。
尚、添付コードのＭａｉｎＷｉｎｄｏｗクラスの該当箇所を参照ください

Q1)この突然プログラムを終了する件について、コメント頂け
ますと大変ありがたいです。

添付コード
printer.h================
#ifndef PRINTER_H
#define PRINTER_H
#include <QObject> //for QString str;
class Printer
{
public:
Printer();
void print();
QString str;
};
#endif // PRINTER_H

printer.cpp================
#include "printer.h"
#include <QObject> //for qPrintable
Printer *obj;
Printer::Printer()
{
str="OK";
obj=this;
qDebug("constructor");
}
void Printer::print(){
qDebug("%s", qPrintable(str));
}

mainwindow.h================
#ifndef MAINWINDOW_H
#define MAINWINDOW_H

#include <QMainWindow>
namespace Ui {
class MainWindow;
}
class MainWindow : public QMainWindow
{
Q_OBJECT

public:
explicit MainWindow(QWidget *parent = 0);
~MainWindow();
private:
Ui::MainWindow *ui;
};
#endif // MAINWINDOW_H

mainwindow.cpp================
#include "mainwindow.h"
#include "ui_mainwindow.h"
#include "printer.h"

extern Printer *obj;

MainWindow::MainWindow(QWidget *parent) :
QMainWindow(parent),
ui(new Ui::MainWindow)
{
ui->setupUi(this);
Printer p; //正常に動作する
p.print(); //正常に動作する

//Printer *pp; //Form画面の表示がなく、突然終了
//pp->print();

//obj->print(); //Form画面の表示がなく、突然終了
}

MainWindow::~MainWindow()
{
delete ui;
}

以上

Answer 1

こんにちは。

Printer p; //正常に動作する
obj->print(); //Form画面の表示がなく、突然終了

この２つを有効にしておけば、前者でPrinterのコンストラクタが呼び出されて、グローバル変数objにPrinterオブジェクトが設定されるので動作はすると思います。

しかし、Printerオブジェクトが複数生成されるような場合、objに設定されているものは最後にコンストラクトされたものですし、その最後のものがデストラクトされるとobjが指すオブジェクトは不正（既に破棄）になります。
たいへんバギーな設計です。

例えば、Printerオブジェクトを複数生成する必要がないのでしたら、シングルトン化すると対策できると思います。
複数必要でしたら、Printerオブジェクトの管理方法をきちんと設計（どうやって管理し、いつ誰が生成／破棄するのかなどなど）しましょう。

Comments

[nqf27650]

//=================
Printer p;
obj->print();
前者でPrinterのコンストラクタが呼び出されて、グローバル変数objにPrinterオブジェクトが設定されるので動作はすると思います。
<---了解しました。感謝

下記の場合は、pにはポインターが入りますか？
また、ｐの後処理も必要ですか？
ご説明頂けますと、大変助かります。

Printer p; //正常に動作する
p.print();

[Chironian]

> 下記の場合は、pにはポインターが入りますか？

この場合、pはオブジェクトそのものです。ポインタではありません。
{から}の間で定義された変数はその}までが有効範囲です。
その有効範囲から出る時に自動的に解放されます。（デストラクタが呼ばれ、記憶領域が解放される）

Answer 2

ポインタ変数の宣言のみで実体がないからです。

pp = new Printer();

のように実体を生成する必要があります。

objのほうはChironianさんも指摘のとおり、動作はすると思います。
以下の類似ソースでは正常に動作します。

#include <iostream>
using namespace std;

class Printer;
Printer *obj = NULL;

class Printer{
public:
	Printer(){
		obj = this;
 	}
	void Print( string s){
		cout << s << endl;
	}
};

class Wnd{
public:
	Wnd(){
		Printer p;
		p.Print("p");
		obj->Print("obj");
	}
};

int main() {
	Wnd w;
        // "p"
        // "obj"
	return 0;
}

Comments

[nqf27650]

＞
class Printer;
Printer *obj = NULL;
＜ーーこの場合、print()を呼び出す場合の書き方は？
お手数ですが、宜しくお願いします。

[8524ba23]

提示したソースではWndクラスのコンストラクタ内で呼び出していますが、質問は別の場所でも呼び出したいという意味でしょうか？
この例ではobjの指す実体pのスコープ（寿命）はコンストラクタ内に限るので、ほかの場所では呼び出してはいけません。
（クラス）変数のスコープやポインタ、コンストラクタ、デストラクタなどについて基礎的な教科書やWebページで再確認してみてください。