Еще с темплейт

[Igors]

Добрый день

Первый класс

class MyClass1 : public MyBase {...

void MyClass1::DelRows( void )
{
 ...
 for (int i = 0; i < count; ++i)
  node->data1.erase(i);
 ...
}

Второй класс похож но нацелен на др контейнер

class MyClass2 : public MyBase {...

void MyClass2::DelRows( void )
{
 ...
 for (int i = 0; i < count; ++i)
  node->data2.erase(i);
 ...
}

Др словами отличие только в разных контейнерах (data1 и data2) - но я не могу подать их в аргументах. Как здесь "схлопнуть"?

Спасибо

[Tuxford]

А node какого типа?

[Igors]

А node какого типа?

Одинакового для обоих классов

[Tuxford]

Имхо с теплейтами не получится или что-то недопонял.

Вариант: в node добавить erase, который буде делать это уже непосредственно у data1 или data2?

[Авварон]

Код:

template<typename Container>
void coolErase(Container &c, int count)
{
    for (int i = 0; i < count; ++i)
        c.erase(i);
}

void MyClass1::DelRows( void ) 
{
    ...
    coolErase(node->data1, count);
    ...
}

[m_ax]

Код

C++ (Qt)
 
template <class R>
R & get(Node * node);
 
template<>
data1_type & get<data1_type>(Node * node)
{
    return node->data1;
}
 
template<>
data2_type & get<data2_type>(Node * node)
{
    return node->data2;
}
 
 
template <class T>
class MyClass : public MyBase {...
 
void MyClass::DelRows( void )
{
 ...
    for (int i = 0; i < count; ++i) {
        T & data = get(node);
        data.erase(i);   
    }
 ...
}
 
 

[Igors]

Код:

template<typename Container>
void coolErase(Container &c, int count)
{
    for (int i = 0; i < count; ++i)
        c.erase(i);
}

void MyClass1::DelRows( void ) 
{
    ...
    coolErase(node->data1, count);
    ...
}

Так я экономлю только на строчке for, а практически одинаковые методы DelRows остаются у обоих классов

Код

C++ (Qt)
 
template <class R>
R & get(Node * node);
 
template<>
data1_type & get<data1_type>(Node * node)
{
    return node->data1;
}
 

Понял, но не устраивает "размах", все-таки планировалось наследование. Но все равно спасибо, сам бы не додумался 
 

[m_ax]

Понял, но не устраивает "размах", все-таки планировалось наследование.

О каком размахе идёт речь? Спецмализация двух функций против реализации двух однотипных классов?

Но все равно спасибо, сам бы не додумался

Вообще это по сути паттерн type traits (класс характеристик), когда в зависимости от типа можно менять поведение:

Код

C++ (Qt)
template <class T>
struct MyTraits {};
 
template<>
struct MyTraits<data1_type>
{
    static data1_type some_action() { ...  }
    static other_type other_action() { ... }
    и т.д.. 
};
 
 
template <class T, class Traits = MyTraits<T>>
class MyClass : public MyBase {...
 

 

Т.е. у нас появляется возможность менять поведение MyClass не влезая в его внутренности.

[Igors]

О каком размахе идёт речь? Спецмализация двух функций против реализации двух однотипных классов?

Объявлять 2 класса темплейтными только для того чтобы схлопнуть один метод (строк 20)? Мне кажется лучше перетерпеть небольшой дубляж, но не усложнять код

А нет ли возможности как-то замкнуть эти самые traits внутри класса?
Спасибо

[m_ax]

Объявлять 2 класса темплейтными только для того чтобы схлопнуть один метод (строк 20)? Мне кажется лучше перетерпеть небольшой дубляж, но не усложнять код

Не понял.. Вначале вы говорили, что

Др словами отличие только в разных контейнерах (data1 и data2)

Всё.. Пишите один шаблонный класс вместо двух нешаблонных реализаций.

А нет ли возможности как-то замкнуть эти самые traits внутри класса?

Можно:

Код

C++ (Qt)
template <class T>
void MyClass::DelRows( void )
{
 ...
    for (int i = 0; i < count; ++i) {
        T & data = get(node);
        data.erase(i);   
    }
 ...
}
 

[Igors]

Всё.. Пишите один шаблонный класс вместо двух нешаблонных реализаций.

Так ведь объявив базовый с шаблоном я вынужден и оба дочерних объявлять с шаблонами. Оба класса - таблички, наследники QTreeWidget. Но содержимое айтемов совершенно разное, поэтому только удаление "совпало". Не смертельно, просто интересно какие есть ходы.

[m_ax]

Так ведь объявив базовый с шаблоном я вынужден и оба дочерних объявлять с шаблонами.

Базовый не шаблонный.

Код

C++ (Qt)
template <class T, class Traits = MyTraits<T>>
class MyClass : public MyBase {};
 
typedef MyClacc<data1_type> MyClass1;
typedef MyClacc<data2_type> MyClass2;
 
 

[Igors]

А можно так?

Код

C++ (Qt)
template <class T>
void DoErase( const T *, Node * node, size_t index )  { static_assert(false, "Invalid T"); } 
 
template <>
void DoErase( const MyClass1 *, Node * node, size_t index )  { node->data1.erase(index); } 
 
template <>
void DoErase( const MyClass2 *, Node * node, size_t index )  { node->data2.erase(index); } 
 

[m_ax]

Можно и так..
Я думал (во всяком случае такой вывод можно сделать из вашей формулировки) что как раз вместо того, чтоб писать две реализации MyClass1/2 различающихся только тем, что в одном используется data1, а в другом data2, написать один шаблонный класс..

 

[Igors]

Я думал (во всяком случае такой вывод можно сделать из вашей формулировки) что как раз вместо того, чтоб писать две реализации MyClass1/2 различающихся только тем, что в одном используется data1, а в другом data2, написать один шаблонный класс..

Ну почему Вы вечно все путаете   Я же только писал что нет

Но содержимое айтемов совершенно разное, поэтому только удаление "совпало".