копировать:
аюсь создать свой собственныйboost::adaptors::transformed
.
Вот связанныйповысить код.
Вот его использование (изменено сТАК ответ от LogicStuff): -
C funcPointer(B& b){
//"funcPointer" is function convert from "B" to "C"
return instance-of-C
}
MyArray<B> test; //<-- any type, must already have begin() & end()
for(C c : test | boost::adaptor::transformed(funcPointer)) {
//... something ....
}
Результат будет таким же, как: -
for(auto b : test) {
C c = funcPointer(b);
//... something ...
}
Моя попыткая создалCollectAdapter
которые стремятся работать какboost::adaptor::transformed
.
Это работает нормально в большинстве распространенных случаев.
Вот полныйдемонстрация а такжерезервное копирование. (такой же, как приведенный ниже код)
Проблемная частьCollectAdapter
- ядро моей библиотеки.
Я не знаю, должен ли я кешироватьcollection_
по-указатель или жепо значению.
CollectAdapter заключает в себе основуcollection_
(например, указатель наstd::vector<>
): -
template<class COLLECTION,class ADAPTER>class CollectAdapter{
using CollectAdapterT=CollectAdapter<COLLECTION,ADAPTER>;
COLLECTION* collection_; //<---- #1 problem? should cache by value?
ADAPTER adapter_; //<---- = func1 (or func2)
public: CollectAdapter(COLLECTION& collection,ADAPTER adapter){
collection_=&collection;
adapter_=adapter;
}
public: auto begin(){
return IteratorAdapter<
decltype(std::declval<COLLECTION>().begin()),
decltype(adapter_)>
(collection_->begin(),adapter_);
}
public: auto end(){ ..... }
};
IteratorAdapter
(используется выше) инкапсулирует базовый итератор, изменяет поведениеoperator*
: -
template<class ITERATORT,class ADAPTER>class IteratorAdapter : public ITERATORT {
ADAPTER adapter_;
public: IteratorAdapter(ITERATORT underlying,ADAPTER adapter) :
ITERATORT(underlying),
adapter_(adapter)
{ }
public: auto operator*(){
return adapter_(ITERATORT::operator*());
}
};
CollectAdapterWidget
(используется ниже) это просто вспомогательный класс для построенияCollectAdapter
-пример.
Может использоваться как:
int func1(int i){ return i+10; }
int main(){
std::vector<int> test; test.push_back(5);
for(auto b:CollectAdapterWidget::createAdapter(test,func1)){
//^ create "CollectAdapter<std::vector<int>,func1>" instance
//here, b=5+10=15
}
}
проблемаПриведенный выше код работает нормально в большинстве случаев, кроме случаев, когдаCOLLECTION
это временный объект.
В частности, когда я создаюадаптер адаптера адаптера ....
int func1(int i){ return i+10; }
int func2(int i){ return i+100; }
template<class T> auto utilityAdapter(const T& t){
auto adapter1=CollectAdapterWidget::createAdapter(t,func1);
auto adapter12=CollectAdapterWidget::createAdapter(adapter1,func2);
//"adapter12.collection_" point to "adapter1"
return adapter12;
//end of scope, "adapter1" is deleted
//"adapter12.collection_" will be dangling pointer
}
int main(){
std::vector<int> test;
test.push_back(5);
for(auto b:utilityAdapter(test)){
std::cout<< b<<std::endl; //should 5+10+100 = 115
}
}
Это приведет к ошибке во время выполнения. Вотдемо-свисающий указатель.
В реальном использовании, если интерфейс более удивительный, например, использование|
Оператор, ошибка будет еще сложнее обнаружить: -
//inside "utilityAdapter(t)"
return t|func1; //OK!
return t|func1|func2; //dangling pointer
ВопросКак улучшить мою библиотеку, чтобы исправить эту ошибку, сохраняяспектакль & прочность & maintainablilty на том же уровне?
Другими словами, как кэшировать данные или указательCOLLECTION
(это может бытьадаптер или жереальная структура данных) элегантно?
В качестве альтернативы, если легче ответить путем кодирования с нуля (чем модифицировать мой код), сделайте это. :)
Мои обходные путиТекущий код кешейпо указателю.
Основная идея обходных путей заключается в кешированиипо значению вместо.
Позволятьадаптер кэшироватьзначение изCOLLECTION
.
Вот главное изменение:
COLLECTION collection_; //<------ #1
//changed from .... COLLECTION* collection_;
Недостаток:-
Вся структура данных (например,std::vector
) будет скопировано значение - ненужный ресурс.(при использовании для
std::vector
напрямую)Обходной путь 2 (две версии библиотеки, лучше?)Я создам 2 версии библиотеки -AdapterValue
а такжеAdapterPointer
.
Я должен создать связанные классы (Widget
,AdapterIterator
и т. д.)
AdapterValue
- по значению, (предназначен дляutilityAdapter()
)AdapterPointer
- по указателю, (предназначен дляstd::vector
)Недостаток:-
Много повторяющегося кода = низкая ремонтопригодностьПользователи (кодеры) должны очень хорошо понимать, какой из них выбрать = низкая надежностьОбходной путь 3 (определить тип)Я могу использовать специализацию шаблона, которая делает это:
If( COLLECTION is an "CollectAdapter" ){ by value }
Else{ by pointer }
Недостаток:-
Не очень хорошо взаимодействует между многими классами адаптеров.Они должны узнать друг друга:признанное = должен кешироватьпо значению.
Извините за очень длинный пост.