DRY up kod, który różni się tylko w stałej

Question

Czy istnieje sposób DRY this (bez makr)?

void clear_tp_vec(vector<const Tk*>& tps){ 
    for(auto it=tps.begin();it!=tps.end();++it){ 
     const ValT* vp=dynamic_cast<const ValT*>(*it); 
     if(vp!=NULL) 
      delete vp; 
    } 
    tps.clear(); 
}; 

void clear_tp_vec(vector<Tk*>& tps){ 
    for(auto it=tps.begin();it!=tps.end();++it){ 
     ValT* vp=dynamic_cast<ValT*>(*it); 
     if(vp!=NULL) 
      delete vp; 
    } 
    tps.clear(); 
}; 

Jedyne drugi różni się przeciążenia jest const kwalifikator (jej brak). Komentowanie którejkolwiek wersji łamie mój kod.

Answer 1

Komentarze (@ kubek w „sprawiają, że szablon” @ user1764961 w „pozbyć się” null testu) doprowadziły do następującej, bardzo oczywistej, zmodyfikowanej wersji:

template<typename T> 
void clear_tp_vec(vector<T>& tps) { 
     for(auto it=tps.begin(); it!=tps.end(); ++it) 
      delete dynamic_cast<const ValT*>(*it); 
     tps.clear(); 
} 

Answer 2

można użyć następujących funkcji szablonu:

template < typename T > 
void clear_tp_vec(vector< T* >& tps) 
{ 
    typedef typename conditional< is_const<T>::value, const ValT, ValT >::type OUT; 
    for (auto it = tps.begin(); it != tps.end(); ++it) 
    { 
     OUT* vp = dynamic_cast< OUT* >(*it); 
     if (vp != NULL) 
      delete vp; 
    } 
    tps.clear(); 
}; 

Answer 3

można zadzwonić usuwać na wartości null więc badanie nie jest potrzebne. Powinieneś także używać standardowych algorytmów na iteratorach, a nie iterować siebie.

Można użyć Boost.Lambda napisać coś bardzo ogólny tutaj używając zastępcze:

#include <boost/lambda/bind.hpp> 
#include <boost/lambda/lambda.hpp> 
... 
using namespace boost::lambda; 
std::for_each(tps.begin(), tps.end(), bind(std::ptr_fun(operator delete), *_1));