2015-04-08 37 views
9

Chciałbym napisać funkcję tę find:szablon funkcji z odpowiednimi parametrami do podzbioru typów krotka

multi_set<int, string, double, myType> m; //vector of tuples 
m.insert(/*some data*/); 
m.find<1,2>("something",2.123); 

Albo

m.find<0,3>(1,instanceOfMyType); 
m.find<1>("somethingelse"); 

Gdzie find mogą być parametryzowane odpowiadające każdej podgrupie parametrów krotka .

Mój kod do tej pory:

template <typename ... T> 
class multi_set{ 
    typedef tuple <T...> Tuple; 
    vector<tuple<T...>> data = vector<tuple<T...>>(); 

public: 
    void insert(T... t){ 
     data.push_back(tuple<T...>(t...)); 
    } 


    template<size_t ... Pos> 
    void find(???){ 
    // then I would like to use those params to search through data and 
    // return first matching item 
    } 
} 

Odpowiedz

5
// test whether a particular tuple is a match 
template<size_t... Pos> 
static bool is_match(const Tuple& tuple, const typename std::tuple_element<Pos, Tuple>::type &... args) { 
    std::initializer_list<bool> results = { (std::get<Pos>(tuple) == args)... }; 
    return std::all_of(results.begin(), results.end(), [](bool p) { return p; }); 
} 

// Find the first one that is a match. 
template<size_t... Pos> 
typename vector<Tuple>::const_iterator find(const typename std::tuple_element<Pos, Tuple>::type &... args) const { 
    return std::find_if(data.begin(), data.end(), [&](const Tuple & tup) { return is_match<Pos...>(tup, args...); }); 
} 

Możliwe jest również, aby mieć find wziąć paczkę parametru typ i doskonale się do przodu, zamiast podejmowania stałych typów z tuple_element. Zaletą jest to, że można uniknąć niepotrzebnej konwersji, jeśli == jest przezroczysta. Koszt jest taki, że nie można pobrać niczego, czego nie można już idealnie przekierować (np. Wzmocnione listy inicjalizacyjne, 0 jako stała wskaźnika zerowego). Korzyścią z boku wydaje się, że MSVC 2013 nie dusić w tej wersji:

// test whether a particular tuple is a match 
template<size_t... Pos, class... Args> 
static bool is_match(const Tuple& tuple, Args&&... args) { 
    std::initializer_list<bool> results = { (std::get<Pos>(tuple) == std::forward<Args>(args))... }; 
    return std::all_of(results.begin(), results.end(), [](bool p) { return p; }); 
} 

// Find the first one that is a match. 
template<size_t... Pos, class... Args> 
typename vector<Tuple>::const_iterator find(Args&&... args) const { 
    return std::find_if(data.begin(), data.end(), [&](const Tuple & tup) { return is_match<Pos...>(tup, std::forward<Args>(args)...); }); 
} 
+0

Daje to twoją ustawioną wydajność liniową podczas wyszukiwania ... dość słabe z perspektywy złożoności algorytmicznej. – StilesCrisis

+0

@StilesCzy dane, które wstawia OP, nie są zamówione? Na marginesie, w C++ 1z jest to znacznie mniej rozwlekłe. – Yakk

+0

Jeśli twoje '==' jest przezroczyste, powyższe jest nieefektywne. Jako przykład: 'std :: string' i' "hello world" '. – Yakk

-2

podpis swojej funkcji find byłoby

template<size_t ... Pos, typename ... Types> void find(Types... &t) 

realizacja poszukiwań jest do ciebie, można użyć cyklicznych szablonów lub zapętlenie nad parametrem Pakiety:

+0

Dlaczego zmuszasz 'find' do pobierania wartości pośredniej? – Yakk

+0

@Yakk ah dzięki za połówki – Valerij

+0

Powinieneś używać perfekcyjnego przekazywania wszędzie tam, gdzie to możliwe, przy pisaniu szablonów – Fiktik

1

Jest to funkcja, która przyjmuje wartość nasion, oraz zestaw lambda. Żywi tę wartość nasion przez każdy z lambdas kolejno:

template<class... Fs, class R> 
R chain(R r, Fs&&... fs) { 
    using in_order = int[]; 
    (void)(in_order{0, 
    (
     (r = std::forward<Fs>(fs)(r)) 
     , void(), 0 
    )... 
    }); 
    return r; 
} 

Wewnątrz klasy, używamy powyższego:

template<size_t... Pos, class...Us> 
typename std::vector<Tuple>::const_iterator 
find(Us const&... us) const { 
    return std::find_if(
    data.begin(), data.end(), 
    [&](const Tuple & tup) { 
     return chain(
     true, 
     [&](bool old){ 
      return old && (std::get<Pos>(tup) == us); 
     }... 
    ); 
    } 
); 
} 

to kompiluje w brzękiem, ale nie g ++ 4.9.2 - g ++ nie lubi pakietów parametrów w lambdach.

Pamiętaj, że przyjmujemy Us const&... - pozwala to na przejrzyste ==, co jest ważne w niektórych przypadkach. std::string == char const* to klasyczny przykład, w którym jeśli wymusisz, aby find przyjął taką samą wartość jak w krotce, wymusisz niepotrzebne przydzielanie w wywołaniu find.


w C++ 1z, wywołanie chain można zastąpić:

(... && (std::get<Pos>(tup) == us)) 

który jest koncepcyjnie identyczne, ale znacznie łatwiejsze do odczytania. Jest to znane jako "wyrażenie krotnie".


Problem z powyższym polega na tym, że korzysta z referencji przekazywanych, co powoduje problemy z perfekcyjnym przekazywaniem.

Najbardziej irytującą z nich jest niemożność użycia {} do konstruowania argumentów.

Jeśli używamy typów dopasowania my zamiast zmuszać nieprzejrzystego porównania, które mogą być drogie (w porównaniu do zbadania std::string"hello this is a c string". - powoduje alokację ewentualnie jeśli zmusimy ciąg c do std::string)

sposób obejścia tego jest do type erase down to the concept of equality with a given type.

template<class...>struct voider{using type=void;}; 
template<class...Ts>using void_t=typename voider<Ts...>::type; 
template<class T>struct tag{using type=T;}; 

template<class...>struct types{using type=types;}; 

template<class T> 
using block_deduction = typename tag<T>::type; 

template<class F, class Sig, class T=void> 
struct erase_view_op; 

template<class F, class R, class...Ts, class T> 
struct erase_view_op<F, R(Ts...), T> 
{ 
    using fptr = R(*)(void const*, Ts&&...); 

    fptr f; 
    void const* ptr; 

private: 
    template<class U> 
    erase_view_op(U&& u, int): 
    f([](void const* p, Ts&&...ts)->R{ 
     U& u = reinterpret_cast<U&>(*static_cast<std::decay_t<U>*>(const_cast<void*>(p))); 
     return F{}(u, std::forward<Ts>(ts)...); 
    }), 
    ptr(static_cast<void const*>(std::addressof(u))) 
    {} 
public: 
    template<class U, class=std::enable_if_t< !std::is_same<std::decay_t<U>,erase_view_op>{} && std::is_convertible< std::result_of_t<F(U,Ts...)>, R >{} >> 
    erase_view_op(U&& u):erase_view_op(std::forward<U>(u), 0){} 

    template<class U=T, class=std::enable_if_t< !std::is_same<U, void>{} >> 
    erase_view_op(block_deduction<U>&& u):erase_view_op(std::move(u), 0){} 

    erase_view_op(erase_view_op const&) = default; 
    erase_view_op(erase_view_op&&) = default; 

    R operator()(Ts... ts) const { 
    return f(ptr, std::forward<Ts>(ts)...); 
    } 
}; 

struct equality { 
    template<class lhs, class rhs> 
    bool operator()(lhs const& l, rhs const& r)const { 
    return l==r; 
    } 
}; 
template<class T> 
using erase_equal_to = erase_view_op< equality, bool(T const&), T >; 
using string_equal_to = erase_equal_to<std::string>; 

int main() { 
    static_assert(std::is_same< bool, std::result_of_t< std::equal_to<>(decltype("hello"), std::string const&) > >{}, "hmm"); 
    string_equal_to s = "hello"; 
    string_equal_to s2 = {{"hello"}}; 
    (void)s2; 
    std::string x = "hello"; 
    std::string y = "jello"; 
    std::cout << s(x) << s(y) << '\n'; 
} 

potem przepisać find:

template<size_t... Pos> 
typename std::vector<Tuple>::const_iterator 
find(erase_equal_to< std::remove_reference_t<std::tuple_element_t<Pos, Tuple>> >... us) const { 
    return std::find_if(
    data.begin(), data.end(), 
    [&](const Tuple & tup) { 
     return chain(
     true, 
     [&](bool old){ 
      return old && us(std::get<Pos>(tup)); 
     }... 
    ); 
    } 
); 
} 

który ma zarówno przejrzyste równości i pozwala {} konstrukcja oparta (dobrze, że nie wymaga {{}} konstrukcja oparta - zewnętrzna powiedzieć budujemy gumki, wewnętrzna, aby zbudować T).

+0

@ T.C. po co usuwać ziarno "prawda"? Czy nie jest potrzebna pusta paczka argumentów? – Yakk

+0

Nie, składanie pustego opakowania z && daje 'true'. –

+0

@ T.C. bah, a co jeśli mój 'operator &&' jest przeciążony lub wartościuje się razem? "Szczególny przypadek najczęstszym przypadkiem" "przyjazny dla użytkownika" pfah. – Yakk