Question

虽然我知道这是一个愚蠢的想法，但我想看看我是否可以为容器和非容器类型使用单个类。首先，我从此question.

复制粘贴的代码

然后我有两个辅助函数：一个用于确定成员函数变量的类型（T是否具有成员value_type），另一个用于确定operator *的返回值。

template <typename T>
typename std::enable_if<HasValueType<T>::value, typename T::value_type>::type
proxy_func_op() {

}

template <typename T>
typename std::enable_if<!HasValueType<T>::value, T>::type
proxy_func_op() {

}

template <typename T>
typename std::enable_if<HasValueType<T>::value, typename T::const_iterator>::type
proxy_func_mem() {
}

template <typename T>
typename std::enable_if<!HasValueType<T>::value, T*>::type
proxy_func_mem() {
}

我的班级看起来像这样：

template<typename T>
class MyIterator {

如果cur没有T成员，则

const_iterator应指向T而非value_type。如果是这种情况，则开始和结束都未使用。

    decltype(proxy_func_mem<T>()) begin;
    decltype(proxy_func_mem<T>()) end;
    decltype(proxy_func_mem<T>()) cur;
public:

这是我的init函数的逻辑。

    template <typename U = T>
    typename std::enable_if<HasValueType<U>::value, void>::type 
    init(U t) {
        static_assert(std::is_same<typename T::const_iterator, 
            decltype(proxy_func_mem<U>())>::value, 
            "Make sure correct function is called.");
        begin = t.begin();
        end = t.end();
        cur = begin;
    }

    template <typename U = T>
    typename std::enable_if<!HasValueType<U>::value, void>::type 
    init(U t) {
        static_assert(!std::is_same<typename T::const_iterator, 
        decltype(proxy_func_mem<U>())>::value, 
        "Make sure correct function is called.");
        cur = &t;
    }

我已将问题缩小到这一行。如果我删除init<T>(t)并直接复制粘贴第一个重载的内容，我会得到正确的结果。否则，我得到的结果不正确。

    explicit MyIterator(const T& t) {
        init<T>(t);
    }

    MyIterator& operator++() {
        static_assert(HasValueType<T>::value, "You cannot use this operator for non-containers.");
        if (cur + 1 != end)
            cur++;
        return *this;
    }

    decltype(proxy_func_op<T>()) operator *() {
        return *cur;
    }
};

例如，错误的输出是：

~~似乎正在调用正确的函数。有什么问题？~~

修改

出于某种原因，将函数签名更改为init(const U& t) {可以解决问题。任何人都可以解释原因吗？

The Code

Valgrind错误：

==4117== Invalid read of size 4
==4117==    at 0x401270: MyIterator<std::vector<int, std::allocator<int> > >::operator*() (main.cpp:78)
==4117==    by 0x400E8A: main (main.cpp:87)
==4117==  Address 0x514d0a0 is 0 bytes inside a block of size 20 free'd
==4117==    at 0x4A05FD6: operator delete(void*) (vg_replace_malloc.c:480)
==4117==    by 0x401CC5: __gnu_cxx::new_allocator<int>::deallocate(int*, unsigned long) (new_allocator.h:110)
==4117==    by 0x401999: std::_Vector_base<int, std::allocator<int> >::_M_deallocate(int*, unsigned long) (stl_vector.h:174)
==4117==    by 0x4014A4: std::_Vector_base<int, std::allocator<int> >::~_Vector_base() (stl_vector.h:160)
==4117==    by 0x4011A0: std::vector<int, std::allocator<int> >::~vector() (stl_vector.h:416)
==4117==    by 0x401209: MyIterator<std::vector<int, std::allocator<int> > >::MyIterator(std::vector<int, std::allocator<int> > const&) (main.cpp:67)
==4117==    by 0x400E75: main (main.cpp:85)

当我不致电init<T>(t)时，Valgrind检测不到任何错误。

Answer 1

init按值接受其参数意味着它是原始对象的副本。您正在存储该副本的迭代器，该副本在init返回时被销毁。销毁容器会使其迭代器失效，因此解除引用这些迭代器的行为具有未定义的行为。

构造函数内的重载函数导致引用错误

1 个答案: