MoveToThread (все-таки хочется понять ..)

[Igors]

А если делать это в одном классе, то нельзя будет создавать Scanner имеющим родителя.

Ну и чем это грозит? Scanner не удалится автоматычно при удалении родителя? Так это и нежелательно т.к. все равно нитку надо завершать. Поэтому лучше напр сделать указатель Scanner членом и в деструкторе разобраться

[andrew.k]

А если делать это в одном классе, то нельзя будет создавать Scanner имеющим родителя.

Ну и чем это грозит? Scanner не удалится автоматычно при удалении родителя? Так это и нежелательно т.к. все равно нитку надо завершать. Поэтому лучше напр сделать указатель Scanner членом и в деструкторе разобраться

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

[Igors]

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

Технически никаких, но это "архитектурно некрасиво" Чтобы завершить нитку в деструкторе Scaner, указатель на QThread должен быть членом Scanner, и зависеть от него. Тогда чего было притворяться и избегать наследования от QThread ?

[andrew.k]

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

Технически никаких, но это "архитектурно некрасиво" Чтобы завершить нитку в деструкторе Scaner, указатель на QThread должен быть членом Scanner, и зависеть от него.

Естественно QThread член класса. Я ж написал, хочу все в одном классе, чтобы все само работало.
А внешний юзер даже не знал ничего про потоки.

членом Scanner, и зависеть от него. Тогда чего было притворяться и избегать наследования от QThread ?

Чтобы получить "архитектурно красиво".
Это в этом топике еще как-то лояльно обсуждают подход. Оригинальная же статья не выделяет преимуществ и недостатков, а говорит о том, что наследоваться неправильно. QThread уже готовый класс и нечего от него наследовать.

[Igors]

Естественно QThread член класса. Я ж написал, хочу все в одном классе, чтобы все само работало.
А внешний юзер даже не знал ничего про потоки.

Как я понимаю, смысл того что пишут (и активно повторяют) в том что Scanner не должен ничего знать о том в какой QThread он выполняется. Завязывая его с QThread (через наследование или членом класса) мы теряем гибкость. Напр др экземпляр Scanner потребовался в главной нитке или хотим иметь что-то еще (кроме Scanner) в той же нитке, или двигать его туда-сюда. Это достигается легко, напр

Код

C++ (Qt)
void MyClass::RunScanner( void )
{
 if (!mThread) 
   mThread = new QThread(this);
 mScanner = new Scanner(0);
 mScanner->moveToThread(mThread);
 ..
}
 

Да, так 2 класса, но это не накладно. Можно все в одном, ну тогда наследоваться или делать один членом другого (особой разницы нет). 

[andrew.k]

Естественно QThread член класса. Я ж написал, хочу все в одном классе, чтобы все само работало.
А внешний юзер даже не знал ничего про потоки.

Как я понимаю, смысл того что пишут (и активно повторяют) в том что Scanner не должен ничего знать о том в какой QThread он выполняется. Завязывая его с QThread (через наследование или членом класса) мы теряем гибкость. Напр др экземпляр Scanner потребовался в главной нитке или хотим иметь что-то еще (кроме Scanner) в той же нитке, или двигать его туда-сюда. Это достигается легко, напр

Код

C++ (Qt)
void MyClass::RunScanner( void )
{
 if (!mThread) 
   mThread = new QThread(this);
 mScanner = new Scanner(0);
 mScanner->moveToThread(mThread);
 ..
}
 

Scanner всегда должен выполняться в отдельном потоке, в этом и смысл. Поэтому зачем пользователя этого класса заставлять каждый раз делать одно и тоже, если об этом может позаботиться сам класс?

Да, так 2 класса, но это не накладно. Можно все в одном, ну тогда наследоваться или делать один членом другого (особой разницы нет). 

в том и дело, что разница есть. Если делать членом, то экземпляр класса не должен иметь родителя.
Хотелось бы чтобы он был. Но в общем-то это практически не важно. Поэтому пока сделал без родителя.

Естественно QThread член класса. Я ж написал, хочу все в одном классе, чтобы все само работало.
А внешний юзер даже не знал ничего про потоки.

Как я понимаю, смысл того что пишут (и активно повторяют) в том что Scanner не должен ничего знать о том в какой QThread он выполняется.

Так он и так не знает, его реализация не завязана на потоке. Просто его код выполняется в потоке созданном самим классом.
При желании его так же можно подвинуть в любой другой поток.