이 성병 :: shared_ptr의

Question

내가 가지고 내 프로젝트에서 이러한 두 가지 기능 : 나는 그들을 변환

char* V8StringToChar(v8::Handle<v8::String> str); 
char* V8StringToChar(v8::Local<v8::Value> val); 

:

std::shared_ptr<char> V8StringToChar(Handle<String> str) { 
    int len = str->Utf8Length(); 
    char* buf = new char[len + 1]; 
    str->WriteUtf8(buf, len + 1); 
    return std::shared_ptr<char>(buf, ArrayDeleter<char>()); 
} 
std::shared_ptr<char> V8StringToChar(Local<Value> val) { 
    return V8StringToChar(val->ToString()); 
} 

로 몸

template <class T> 
class ArrayDeleter { 
public: 
    void operator() (T* d) const 
    { delete [] d; } 
}; 
std::shared_ptr<char> V8StringToChar(v8::Handle<v8::String> str); 
std::shared_ptr<char> V8StringToChar(v8::Local<v8::Value> val); 

그리고 모든 사용은 (&*V8StringToChar(whatever))입니다.

그리고 완벽하게 구축됩니다.

그리고 런타임 오류가 발생합니다.

실패 할 수있는 경우가 있습니까? 그리고 좋은 해결책을 제공해주십시오. 대신

(&*V8StringToChar(whatever)) 

의

Answer 1

은 당신이 쓴 수 :

V8StringToChar(whatever).get() 

그러나 모두는 아마 잘못하고 어떤 상황에서 실패 보장한다.

이렇게하면 새 버퍼가 만들어지고 shared_ptr으로 반환되고 버퍼의 주소가 반환 된 다음 shared_ptr이 범위를 벗어나 버퍼가 삭제되어 매달린 포인터가 남습니다. 붐, 해당 주소에서 메모리에 액세스하려는 시도는 정의되지 않은 동작입니다. 감옥에 가거나, 감옥에 직접 가고,지나 가지 않거나, £ 200을 모으지 마십시오.

배열에 대한 기본 제공 지원 기능이 있기 때문에 함수가 std::unique_ptr<char[]>을 대신 반환합니다.

std::unique_ptr<char[]> V8StringToChar(Handle<String> str) { 
    int len = str->Utf8Length(); 
    std::unique_ptr<char[]> buf(new char[len + 1]); 
    str->WriteUtf8(buf.get(), len + 1); 
    return buf; 
} 
std::unique_ptr<char[]> V8StringToChar(Local<Value> val) { 
    return V8StringToChar(val->ToString()); 
} 

런타임 오류를 수정하려면 버퍼가 필요한만큼 스마트 포인터를 유지해야합니다.

std::unique_ptr<char[]> smartptr = V8StringToChar(whatever); 
char* ptr = smartptr.get()); 
doSomethingWithPtr(ptr); 
// now it's OK if `smartptr` goes out of scope