Question

我不确定标题，因为我不确定问题是否来自＆＃34; copyablility＆＃34;我的容器我尝试了所有的东西，但我无法摆脱这个错误。

这是我的代码的简化版本（请不要挑战类设计，我真的想在BOOST_FOREACH中保留最终使用的语法）：

template <typename T>
class MyContainer
{
public:
    typedef typename std::vector<T>::iterator iterator;
    typedef typename std::vector<T>::const_iterator const_iterator;

    MyContainer(std::vector<T>& vec, boost::mutex& mutex) :
        m_vector(vec),
        m_lock(mutex)
    {
    }

    iterator begin() { return m_vector.begin(); }
    const_iterator begin() const { return m_vector.begin(); }
    iterator end() { return m_vector.end(); }
    const_iterator end() const { return m_vector.end(); }


private:
    std::vector<T>& m_vector;
    boost::lock_guard<boost::mutex> m_lock;
};

template <typename T>
struct GetContainer
{
    GetContainer(std::vector<T>& vec, boost::mutex& mutex) :
        m_vector(vec),
        m_mutex(mutex)
    {
    }

    MyContainer<T> Get()
    {
        return MyContainer<T>(m_vector, m_mutex);
    }

    std::vector<T>& m_vector;
    boost::mutex& m_mutex;
};



int main()
{
    std::vector<int> v;
    v.push_back(1);
    v.push_back(2);
    boost::mutex m;

    GetContainer<int> getter(v, m);

    BOOST_FOREACH(int i, getter.Get())
    {
        std::cout << i << std::endl;
    }

    return 0;
}

编译器抱怨没有MyContainer :: MyContainer的复制构造函数（const MyContainer＆amp;）。我也有：错误：没有匹配函数来调用'MyContainer :: MyContainer（boost :: foreach_detail _ :: rvalue_probe＆gt; :: value_type）'

我遵循可扩展性提示： http://www.boost.org/doc/libs/1_58_0/doc/html/foreach/extensibility.html#foreach.extensibility.making__literal_boost_foreach__literal__work_with_non_copyable_sequence_types

但是，制作

MyContainer<T> : private boost::noncopyable

没有解决问题。也没有定义功能

boost_foreach_is_noncopyable

或专门化模板struct

is_noncopyable

for MyContainer（事实上，我如何将此模板专门用于模板类型？）

最后＆＃34;提示＆＃34; ：如果我从任何地方删除互斥锁和锁（我只是将向量传递给GetContainer和MyContainer），它就可以工作了。但是，如果我做了

，它就不起作用

MyContainer<T> : private boost::noncopyable

（我希望它应该，所以我不确定我的问题是BOOST_FOREACH，但也许是因为我用我的getter返回了MyContainer的副本？）

如果你在此读到我的话，我感谢你，并提前感谢你的帮助。

Answer 1

似乎是具有仅移动类型的BOOST_FOREACH的限制。我没有找到解决方法¹（除了 - 丑陋 - 显而易见的方法将lock_guard放入shared_ptr）。

但是，您没有指定c ++ 03要求，因此您可以通过将lock_guard替换为unique_lock来使其无需BOOST_FOREACH。

这是我对c ++ 11中的事情的看法（注意它是如何通用的）：

<强> Live On Coliru

#include <boost/thread.hpp>
#include <boost/range.hpp>

namespace detail {
    template <typename R, typename M>
    struct RangeLock {
        RangeLock(R&r, M& m) : _r(r), _l(m) {}
        RangeLock(RangeLock&&) = default;

        using iterator = typename boost::range_iterator<R>::type;
        iterator begin() { using std::begin; return begin(_r); }
        iterator end  () { using std::end;   return end  (_r); }

        using const_iterator = typename boost::range_iterator<R const>::type;
        const_iterator begin() const { using std::begin; return begin(_r); }
        const_iterator end  () const { using std::end;   return end  (_r); }

     private:
        R& _r;
        boost::unique_lock<M> _l;
    };
}

template <typename R, typename M>
    detail::RangeLock<R,M> make_range_lock(R& r, M& mx) { return {r,mx}; }
template <typename R, typename M>
    detail::RangeLock<R const,M> make_range_lock(R const& r, M& mx) { return {r,mx}; }

#include <vector>
#include <map>

int main() {

    boost::mutex mx;

    std::vector<int> const vec { 1, 2 };
    std::map<int, std::string> const map { { 1, "one" }, { 2, "two" } };

    for(int i : make_range_lock(vec, mx))
        std::cout << i << std::endl;

    for(auto& p : make_range_lock(map, mx))
        std::cout << p.second << std::endl;

    for(auto& p : make_range_lock(boost::make_iterator_range(map.equal_range(1)), mx))
        std::cout << p.second << std::endl;

}

打印

1
2
one
two
one

¹甚至没有使用Using BOOST_FOREACH with a constant intrusive list

中的所有方法

Answer 2

如果可以帮助我发布我的答案......

使用C ++ 03 ，我最终提供了一个复制构造函数，以便能够将该类与BOOST_FOREACH一起使用。因此，问题转移到另一个主题：以逻辑和合适的方式复制类。

在我的情况下，我＆＃34;分享锁和向量＆＃34;，如果用户不想做错误，用户不应该使用此副本，但在BOOST_FOREACH中＆＃34; 39;没关系：

我将互斥锁更改为recursive_mutex

我将锁定更改为unique_lock 和

MyContainer(const MyContainer& other) :
                            m_vector(other.vec),
                            m_lock(*other.m_lock.mutex())
{
}

使用C ++ 11

感谢Chris Glover关于提升填充列表的一个C ++ 11解决方案：

你无法在C ++ 03中做你想做的事。为了实现它，你需要C ++ 11移动语义才能将MyContainer移出Get 功能。即使不使用BOOST_FOREACH，以下代码也会失败;
GetContainer<int> getter(v, m); 
MyContainer<int> c = getter.Get(); // <-- Error. 
这是一个必要的变化的例子;我将scoped_lock更改为 unique_lock并添加了一个移动构造函数。
template <typename T> 
class MyContainer 
{ 
public: 
[...]
    MyContainer(MyContainer&& other) 
        : m_vector(other.m_vector) 
    { 
        m_lock = std::move(other.m_lock); 
        other.m_vector = nullptr; 
    } 

无法迭代函数返回的不可复制容器

2 个答案: