質問をすることでしか得られない、回答やアドバイスがある。

15分調べてもわからないことは、質問しよう!

新規登録して質問してみよう
ただいま回答率
85.50%
C++11

C++11は2011年に容認されたC++のISO標準です。以前のC++03に代わるもので、中枢の言語の変更・修正、標準ライブラリの拡張・改善を加えたものです。

Qt

QtはGUIプログラムの開発で広く使われているクロスプラットフォーム開発のフレームワークです。

Linux

Linuxは、Unixをベースにして開発されたオペレーティングシステムです。日本では「リナックス」と呼ばれています。 主にWebサーバやDNSサーバ、イントラネットなどのサーバ用OSとして利用されています。 上位500のスーパーコンピュータの90%以上はLinuxを使用しています。 携帯端末用のプラットフォームAndroidは、Linuxカーネル上に構築されています。

C++

C++はC言語をもとにしてつくられた最もよく使われるマルチパラダイムプログラミング言語の1つです。オブジェクト指向、ジェネリック、命令型など広く対応しており、多目的に使用されています。

Qt Creator

Qt Creatorは、Qtアプリケーションを開発するための開発統合環境(IDE)です。

Q&A

解決済

2回答

4932閲覧

Qt 自前クラスをsignal/slotでやり取りする場合

tuyudaku

総合スコア75

C++11

C++11は2011年に容認されたC++のISO標準です。以前のC++03に代わるもので、中枢の言語の変更・修正、標準ライブラリの拡張・改善を加えたものです。

Qt

QtはGUIプログラムの開発で広く使われているクロスプラットフォーム開発のフレームワークです。

Linux

Linuxは、Unixをベースにして開発されたオペレーティングシステムです。日本では「リナックス」と呼ばれています。 主にWebサーバやDNSサーバ、イントラネットなどのサーバ用OSとして利用されています。 上位500のスーパーコンピュータの90%以上はLinuxを使用しています。 携帯端末用のプラットフォームAndroidは、Linuxカーネル上に構築されています。

C++

C++はC言語をもとにしてつくられた最もよく使われるマルチパラダイムプログラミング言語の1つです。オブジェクト指向、ジェネリック、命令型など広く対応しており、多目的に使用されています。

Qt Creator

Qt Creatorは、Qtアプリケーションを開発するための開発統合環境(IDE)です。

0グッド

1クリップ

投稿2019/06/19 01:34

編集2019/06/19 03:54

スレッド間通信にsignal/slotを使って実装しようと考えています。

その際に通信するデータが自前クラスの予定なのですが
Qtのsignal/slotとthread(2)
上記のサイトを信じる限り、自前クラスをそのままやり取りするのはあまりオススメしないみたいです
やるとしても参照渡し?

サイトを見る限りQSharedPointerを使用したやり方が一番安全?なのかな
という感じですが
そもそも、自前クラスのやり取りはやめとけ!!
というものなのか、それとももっと良いやり方があるのかが分かりませんでした

状況によっては別の実装方法を考える必要もあります...

何かオススメのやり方を知っている方
クラスのやり取りはやめとけ!という考えの方
是非とも回答のほどよろしくお願いいたします。


######追記

質問の回答を待ちながら並行して実装を進めていたのですが
自前クラスをsignal/slotでやり取りするのが上手く行きませんでした...

Q_DECLARE_METATYPEqRegisterMetatype<>()の使い方がいまいち分かりません...
ご教授お願いいたします

c++

1//event.h 2 3class EventData 4{ 5public: 6 EventData() 7 { 8 code = 0; 9 string = ""; 10 } 11 12public: 13 int code; 14 QString string; 15};

これが実際にやり取りに使いたい自前クラスだとします。これを

c++

1// test.h 2#include <event.h> 3#include <QObject> 4 5class test: public QObject 6{ 7 Q_OBJECT 8 Q_DECLARE_METATYPE(EventData) 9 10signals: 11 void sDataTransmission(const EventData data); 12 13private slots: 14 void receiveData(const EventData data); 15 16public: 17 MainProcessor(); 18};

c++

1// test.cpp 2#include <test.h> 3 4test::test() 5{ 6 qRegisterMetatype<EventData>(); 7 8 connect(this, SIGNAL(sDataTransmission(const EventData)), aite, SLOT(receiveData(const EventData))) 9 connect(aite, SIGNAL(sDataTransmission(const EventData)), this, SLOT(receiveData(const EventData))) 10} 11 12void test::receiveData(const EventData data) 13{ 14 qDebug() << "test: " << QThread::currentThread(); 15}

とこのように記述してみたのですが、Q_DECLARE_METATYPE(EventData)の部分でエラーが出てしまいました。
実際にエラーが出ているのはマクロ内部で、出ているエラーは
explicit specialization in non-namespace scope 'class test'
specialization of 'template<class T> struct QMetaTypeId' must appear at namespace scope
'struct QMetaTypeId<EventData>' redeclared with different access
等と数種類出ている状態です...

どのようにして使用すれば良いのでしょうか

気になる質問をクリップする

クリップした質問は、後からいつでもMYページで確認できます。

またクリップした質問に回答があった際、通知やメールを受け取ることができます。

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

guest

回答2

0

ベストアンサー

Q_DECLARE_METATYPE(EventData)event.h の、クラスの外 に記述します。

cpp

1class EventData 2{ 3}; 4 5Q_DECLARE_METATYPE(EventData)

詳細は Q_DECLARE_METATYPEのドキュメント をご覧ください。

メソッドの引数の const EventData dataconst EventData &data の方がベターです。

SIGNAL(), SLOT() の中の引数の方は、EventData のみ記載するのが普通です。

cpp

1connect(this, SIGNAL(sDataTransmission(EventData)), aite, SLOT(receiveData(EventData))); // サンプルはセミコロンが抜けていました。

それから、thisaite が別のスレッドで動いている場合は、connect の第5引数で明示的に Qt::QueuedConnection を指定しておくと、意図しないトラブルを回避できると思います。

投稿2019/06/19 04:10

tasuku.

総合スコア347

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

tuyudaku

2019/06/19 08:03

>クラスの外 に記述します。 なるほど 何とか上手く行きました! >const EventData &data の方がベターです。 >EventData のみ記載するのが普通です。 早速変更しました!w >Qt::QueuedConnection を指定しておくと、意図しないトラブルを回避できると思います。 なるほど、確かにそうですね 明示的にやっておくのは大事ですね! 回答ありがとうございました!
guest

0

自前クラスの中身次第ですけれど、いくつかのデータ(int, QByteArrayなど)を含む単純な構造体であれば
https://qiita.com/false-git@github/items/52303f42c518522f801d#myobject
の形式が安全でいいのではないでしょうか。

コピーのコストに関しては、int や bool 等であればほぼ無視できますし、QString や QByteArray なども、浅いコピー しかしませんので気にする必要はないかと思います。

投稿2019/06/19 02:14

tasuku.

総合スコア347

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

tuyudaku

2019/06/19 03:55 編集

>いくつかのデータ(int, QByteArrayなど)を含む単純な構造体であれば 大きなものでなければ余計なことはせずに普通に渡したほうがいいのですね 安心しました! 追加の質問になってしまうのですが よろしければ追記の方もよろしくお願いいたします...
guest

あなたの回答

tips

太字

斜体

打ち消し線

見出し

引用テキストの挿入

コードの挿入

リンクの挿入

リストの挿入

番号リストの挿入

表の挿入

水平線の挿入

プレビュー

15分調べてもわからないことは
teratailで質問しよう!

ただいまの回答率
85.50%

質問をまとめることで
思考を整理して素早く解決

テンプレート機能で
簡単に質問をまとめる

質問する

関連した質問