Russian Qt Forum

Добрый день

Очень старый код который однако поддерживать обязан. Базовый класс (псевдокод)

C++ (Qt)
class BaseControl {
  BaseControl( void );
  void Init( Window * win, const std::string & txt ... );
  ...
  virtual const std::string & SerialName( void ) const;
  ...
  virtual void Read ( Stream & stream );
  virtual void Write( Stream & stream );
};
 

Ну и плюс фабрика (с темплейтпми) и макруха регистрации в ней. Работает так:
читается имя из потока, по нему фабрика делает new нужного типа и зовется виртуальный Read, который зовет конкретный Init

Однако дефаулт конструктор оказался очень неудобным (при переводе на Qt) , и я решил вместо Init иметь нормальные конструкторы во всех классах наследниках (десятка 2). Конечно развалилась хвабрика чтения. Нужно ли ее "домучивать"? Или лучше сбацать в духе "современного С++", там function, bind и все такое? Тогда как?

Спасибо

Почему фабрика развалилась?
В фабрике нельзя просто заменить make() на make( QWidget * parent, Qt::WindowFlags ) или что там используется?

Можно вообще сделать отдельно Qt Widgets и оставить эту сериализацию.

C++ (Qt)
 
class MyWidget : public QWidget
{
public:
    MyWidget ( QWidget * parent, Qt::WindowFlags );
};
 
 

C++ (Qt)
class BaseControlSerializer
{
  BaseControlSerializer( void );
  virtual void Init( QWidget * win, const std::string & txt ... ) = 0;
  ...
  virtual const std::string & SerialName( void ) const;
  ...
  virtual void Read ( Stream & stream ) = 0;
  virtual void Write( Stream & stream ) = 0;
 
  virtual QWidget * widget () = 0;
};
 
template < typename _Widget >
class GenericControlSerializer : public BaseControlSerializer
{
  _Widget * m_widget;
 
private:
  GenericControlSerializer ( void );
  virtual void Init( QWidget * win, const std::string & txt ... ) override;
  ...
  virtual void Read ( Stream & stream ) override;
  virtual void Write( Stream & stream ) override;
 
  virtual QWidget * widget () override { return m_widget; };
};
 
 

Можно, просто интересно "попробывать" новое  :)

Конкретный контрол выглядит так

C++ (Qt)
class MyControl : public QWidget, public BaseControl {
..
};

Или так

C++ (Qt)
class MyStaticText : public QLabel, public BaseControl {
..
};

Пока наметил так

C++ (Qt)
class BaseControl {
 ..
 static BaseControl * Read( Stream & stream );
 .. 
};

И покидать эти статики в мапу. Но может как-то по другому (в свете новых фич языка и.т.п.) ?

Так не катит :) Собсно десериализация - это "тривиально" (здесь это слово подходит). Сначала создаем объект конструктором и потом читаем из потока, объект всегда валиден. Но в данном случае я не могу безболезненно создать объект (см примеры классов выше). Сделал так

C++ (Qt)
static MyControl * CreateForRead( Window * win )
{
  return new MyControl(win, QRect(0, 0, 256, 100), "Dummy");
}

Для всех 25 контролов. Правда здесь окно имеется. Интересно а что если бы его не было? Тогда не вижу хорошего решения

Поэтому я и предложил отделить сериализацию от объекта.
Фабрика может создавать не сам Control, а ControlSerializator, который в свою очередь вычитывает из stream необходимые параметры и, если все ОК, создает Control.