[РЕШЕНО] правильная реализация паттерна Pimpl

[kambala]

Здравствуйте. Первый раз в жизни пришлось использовать паттерн Pimpl (pointer to implementation) для того, чтобы код имел нормальный вид (перед этим хедер был написан через #ifdef Q_WS_MACX #else #endif и просто ужасен, могу показать ). Покритикуйте пожалуйста - какие ошибки допущены, как сделать лучше.

Класс реализует простенький месседжбокс с чекбоксом и кнопками да/нет. В мак ос это просто обертка над NSAlert, а на других платформах - полноценный QDialog.

Вот тут прикреплён окончательный вариант реализации: http://www.prog.org.ru/index.php?topic=22129.msg154409#msg154409

[Kurles]

Здравствуйте. Первый раз в жизни пришлось использовать паттерн Pimpl (pointer to implementation) для того, чтобы код имел нормальный вид (перед этим хедер был написан через #ifdef Q_WS_MACX #else #endif и просто ужасен, могу показать ). Покритикуйте пожалуйста - какие ошибки допущены, как сделать лучше.

Класс реализует простенький месседжбокс с чекбоксом и кнопками да/нет. В мак ос это просто обертка над NSAlert, а на других платформах - полноценный QDialog.

Вроде все хорошо, но в самом Qt немного другая реализация. Вот здесь об этом подробно описано.

[kambala]

да, эту статью я читал, но для простенького случая там слишком много ненужного кода

[Авварон]

Неправильно всё. Нахрена там вообще виртуальность?

[kambala]

не придумал как по-другому сделать один из классов наследником QDialog при одном и том же интерфейсе

[Авварон]

Интерфейсом является сам класс. Наследования там не надо. Вы просто реализуете его ф-ии под конкретную платформу в отдельном файле. Если есть платформонезависимые ф-ии, реализуете их в общем .cpp. Линковщик сам подцепит нужную имплементацию.

[Igors]

Лучше смотрится так

Код

C++ (Qt)
class MessageCheckBoxImpl;
 
class MessageCheckBox
{
public:
    MessageCheckBox(const QString &text, const QString &checkboxText, QWidget *parent = 0);
    virtual ~MessageCheckBox();
 
    void setChecked(bool checked);
    bool isChecked();
 
    int exec();
 
private:
    MessageCheckBoxImpl * mImpl;
};
 

А потом 2 файла cpp для каждой из платформ. Да, придется в каждом написать напр MessageCheckBox::setChecked, но это лучше чем размазывать в хедере. Др словами более "идейно" не светить никаких деталей MessageCheckBoxImpl

[kambala]

спасибо, переделал (пркрепил к посту). но напрягает необходимость написания одной и той же реализации конструктора в обоих файлах реализаций:

Код

C++ (Qt)
MessageCheckBox::MessageCheckBox(const QString &text, const QString &checkboxText, QWidget *parent /*= 0*/) : _impl(new MessageCheckBoxImpl(text, checkboxText, parent)) {}

[Igors]

спасибо, переделал (пркрепил к посту). но напрягает необходимость написания одной и той же реализации конструктора в обоих файлах реализаций:

Код

C++ (Qt)
MessageCheckBox::MessageCheckBox(const QString &text, const QString &checkboxText, QWidget *parent /*= 0*/) : _impl(new MessageCheckBoxImpl(text, checkboxText, parent)) {}

Не вижу напряга, т.к. конструктор на платформу завязан. Вот если бы образовались методы которым не нужен impl (т.е. работают на уровне setChecked), то можно отнаследоваться (наследник уже платформо-независим)

[kambala]

в данном случае конструктор одинаков и там и там, потому получается дублирование кода может в таком случае лучше сделать MessageCheckBoxImpl структурой с нужными полями без всяких методов и конструкторов, а текущий конструктор MessageCheckBoxImpl перенести в конструктор MessageCheckBox?

[Igors]

в данном случае конструктор одинаков и там и там, потому получается дублирование кода может в таком случае лучше сделать MessageCheckBoxImpl структурой с нужными полями без всяких методов и конструкторов, а текущий конструктор MessageCheckBoxImpl перенести в конструктор MessageCheckBox?

Одинаков "формально", по написанию (и это не надолго). А на деле тело конструктора знает платформу, поэтому должно быть 2 тела, каждое в своем cpp. Думайте проще: отот impl должен быть в хедере "совсем бестелесным", с него извне ничего не взять даже если захотеть. Не надо бояться разумного дублирования, в конце-концов делегирование - тоже дубляж