콜백 함수의 개체 삭제

Question

서버의 데이터를 저장/업데이트하기 위해 HTTP 요청을 보내고 있습니다. 요청은 비동기 적으로 이루어지고 완료되면 콜백 함수가 호출됩니다. 모든 경우는 예외적 인 경우를 제외하고는 가끔씩 응용 프로그램이 콜백에서 충돌합니다.

이

내가 뭐하는 거지입니다 :

이

void User::saveOnServer(){ 
    Request *request = new Request(); 

    // Send request to the server and register the callback. 
    request ->setCallback(&userCallback, (void*)this); 
} 

콜백 :

배에서

void userCallback(void *data){ 
    User *user = (User*)data; 

    // Do something here. 
    // Delete user if it's a zombie. 
    if(user->zombie) 
     delete user; 
} 

, 내가 필요 User에서

user = new User(); 
user->saveOnServer(); 
user->zombie = true; // Mark the user that it needs to be deleted in the callback. 

, 나는 saveOnServer() 방법을 서버에 요청을 보낸 후 새 사용자를 만드십시오.

user = new User(); 
user->saveOnServer(); 
user->zombie = true; 
// Some code comes here. 
if(user) 
    delete user; 
user = new User(); 

그런 경우에는 이미 삭제 된 콜백으로 사용자를 삭제할 때 응용 프로그램이 충돌합니다. 또 다른 문제는 콜백이 사용자를 삭제하지만 main에있는 user 포인터가 여전히 일부 주소 (매달린 포인터)를 가리키고 있으므로 다시 삭제하려고한다는 것입니다.

이 경우 메모리를 관리하는 가장 좋은 방법은 무엇인지 모르겠습니다. 나는 zombie을 가지고 있습니다. 왜냐하면 콜백이 사용자를 삭제하기를 원치 않을 때가 있기 때문입니다.

Answer 1

좀비 사용자의 saveOnServer을 호출하면 요청은 해당 사용자 개체의 유효한 "소유자"입니다. 그것을 사용하고 나중에 삭제하려는 다른 것이 있기 때문에 직접 해방하지 마십시오.

실제로 서버 작업이 비동기 적으로 반환 될 수있는 경우 사용자 개체는 언제든지 파괴 될 수 있습니다. 다른 코드에서 완전히 사용을 중지해야합니다. 당신은 요청에 해당 개체의 제어를 허용 한, 당신은 다른 곳에서 사용을 중지해야합니다

user = new User(); 
user->zombie = true; // set *before* transferring ownership to server 
user->saveOnServer(); 
user = NULL; 
//some code comes here 
user = new User(); 

을 더 이상 객체를 사용하는 요청을하지 않으려면, 당신은 몇 가지 기능을 제공해야 서 v 저장 (save-on-server) 조치가 오브젝트를 사용하지 않도록 "취소"합니다.

---

또 다른 옵션은 스마트 포인터를 사용하는 것입니다. 주 코드에서 shared_ptr에 개체를 저장하십시오. 요청 개체에서 weak_ptr에 저장합니다. 그렇게하면 주 코드가 사용자 객체를 파괴하려고 할 때 간단히 user.reset()을 호출 할 수 있습니다. 그런 다음 콜백이 weak_ptr을 사용하려고 시도하면 지적 된 객체를 더 이상 사용할 수 없다는 것을 알게됩니다. 스마트 포인터를 사용하는 경우 함수는 delete을 사용해야합니다. 포인터 개체는 사용자의 수명을 관리합니다.saveOnServer 기능에

shared_ptr<User> user = make_shared<User>() 
user->saveOnServer(); 
//some code comes here 
user.reset(new User()); 

, 객체에 weak_ptr 만들 shared_from_this를 사용

void userCallback(weak_ptr<User> data){ 
    shared_ptr<User) user = data.lock(); 
    if (!user) 
    return; 

    //do something here 
}