刪除回調函數中的對象

Question

我正在發送一個HTTP請求來保存/更新服務器上的數據。該請求是異步完成的，並在完成時調用回調函數。一切正常，除了有時候，應用程序在回調中崩潰。

這是我在做什麼：

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

在User，我有一個saveOnServer()方法：

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 = 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(); 

如果你不想請求使用該對象了，那麼你就需要提供一些設施用於「取消」保存服務器操作，以便它不使用該對象。

---

另一種選擇是使用智能指針。在您的主代碼中，將對象存儲在shared_ptr中。在請求對象中，將其存儲在weak_ptr中。這樣，如果您的主代碼想要銷燬用戶對象，則可以簡單地調用​​。然後，如果回調嘗試使用weak_ptr，則會發現指向的對象不再可用。當使用智能指針時，函數都不應該使用delete。指針對象將爲您管理用戶的生命週期。

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

在saveOnServer功能，使用shared_from_this創建weak_ptr對象：

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

    //send request on server and register the callback 
    weak_ptr<User> self(shared_from_this()); 
    request ->setCallback(&userCallback, self); 
} 

在回調，使用weak_ptr：

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

    //do something here 
}