2010-05-20 23 views
3

我最近尝试构建自己的共享和弱指针。可以编译使用Visual Studio在GCC(4.5.0)不编译下面的错误代码:C++ - 使用GCC编译时'operator ='不匹配

main.cpp: In function 'int main()': 
main.cpp:18:27: error: no match for 'operator=' in 'wp1 = weak_ptr<int>(((const shared_ptr<int>&)((const shared_ptr<int>*)(& sp1))))' 
weak_ptr.h:59:9: note: candidate is: void weak_ptr<T>::operator=(weak_ptr<T>&) [with T = int, weak_ptr<T> = weak_ptr<int>] 

这里是我的代码的最重要的部分:

1)弱指针实现(注意的operator=声明)

#include "smart_ptr_wrapper.hpp" 
#include "shared_ptr.h" 

template <typename T> 
class weak_ptr { 
private: 
    // Weak wrapper implementation 
    typedef smart_ptr_wrapper<T> weak_ptr_wrapper; 
    weak_ptr_wrapper* wrapper; 

private: 
    // Shared wrapper additional routines 
    void increase_reference_count() { 
     ++(wrapper->weak_count); 
    } 
    void decrease_reference_count() { 
     --(wrapper->weak_count); 

     // Dispose the wrapper if there are no more 
     // references to this object 
     // @note This should actually lock the wrapper to 
     // preserve thread safety 
     if (wrapper->strong_count == 0 && wrapper->weak_count == 0) { 
     delete wrapper; 
     } 
    } 

public: 
    // Default constructor to grant syntax flexibility 
    weak_ptr() : wrapper(NULL) { } 

    weak_ptr(const shared_ptr<T>& pointer) : wrapper(pointer.wrapper) { 
     increase_reference_count(); 
    } 

    weak_ptr(const weak_ptr& p) : wrapper(p.wrapper) { 
     increase_reference_count(); 
    } 

    weak_ptr& operator= (weak_ptr& p) { 
     // Set new reference counts 
     // @note If this is 'just-a-pointer', which was created 
     // using default constructor then our wrapper would be 'NULL' 
     if (wrapper != NULL) { 
     decrease_reference_count(); 
     } 
     p.increase_reference_count(); 
     // Set new wrapper 
     wrapper = p.wrapper; 

     return *this; 
    } 

    ~weak_ptr() { 
     decrease_reference_count(); 
    } 

    T* get() const { return (wrapper->strong_count == 0) ? NULL: wrapper->raw_pointer; } 
    T* operator->() const { return get(); } 
    T& operator* () const { return *get(); } 

    // User comparison operation 
    operator void*() const { 
     return (get() == NULL); 
    } 
}; 

2)的main.cpp

int main() { 
    shared_ptr<int> sp1(new int(4)); 
    weak_ptr<int> wp1(sp1); 
    // Next line can't be compiled by gcc... Why? 
    wp1 = weak_ptr<int>(sp1); 
    return 0; 
} 

问:

为什么会发生这种情况?我可能很愚蠢,但我看不出这个代码有什么问题,并且无法解决GCC行为。我也很感激,如果有人能解释为什么这个代码会编译,为什么它能在MSVS下工作(我的意思是,为什么一个编译器可以很好地工作,为什么还会失败)。谢谢。

更新:完整的代码和编译器错误在这里可以看到 - http://codepad.org/MirlNayf

+0

在GCC中工作(http://codepad.org/GUdC3D0L) – 2010-05-20 16:58:22

+0

由于旧的编译器扩展,它适用于MSVC++。禁用扩展,并且它也会在MSVC中失败。 – AnT 2010-05-20 17:03:03

回答

5

你的赋值运算符需要一个参考,而不是一个const参考:

weak_ptr& operator= (weak_ptr& p) 

然而,表达weak_ptr<int>(sp1)导致暂时的,它只能被转换为const引用,因为它是一个右值。这样想一下:你不能修改表达式的结果,但你的赋值操作符要求它可以。

解决的办法是宣布你的赋值运算符是这样,而不是:

weak_ptr& operator= (const weak_ptr& p) 

为什么VC++接受这已经超出了我......也许你应该让一些标准符合性标志。

+0

MS调用VC绑定rvalues到non-'const'引用一个“扩展”的事实。我认为这是一个错误。 – sbi 2010-05-20 17:35:30

0

这是因为您首先从sp1构建了一个新对象,然后分配它,这不是我期望的预期行为。但是,根本的错误是因为赋值应该采用const引用。