取消引用指针（并返回它）的问题

Question

这里我有一个函数创建一个字符串，将其指定给一个字符串指针并将其返回。我试着返回一个普通的字符串，它工作正常，但是当我集成指针并引用它们时，我的程序崩溃了。当我试图调试它，这是我得到的消息：在分配2.exe在0x00024cbf

未处理的异常：0000005：访问冲突读取位置0xcccccce4

这里是我的代码：

string* Recipe::getCookingTime() 
// @intput: none 
// @output: cooking time as a string 
{ 
    string temp; 
    string displayHrs; 
    string displayMins; 
    if(cookingTime_->numHours < 10) 
     displayHrs = intToString(0) + intToString(cookingTime_->numHours); 
    else 
     displayHrs = intToString(cookingTime_->numHours); 
    if(cookingTime_->numMinutes < 10) 
     displayMins = intToString(0) + intToString(cookingTime_->numMinutes); 
    else 
     displayMins = intToString(cookingTime_->numMinutes); 

    temp = "The time to cook the recipe is " + displayHrs + ":" + displayMins; 
    *cTime_ = temp; 
    return cTime_; 
} 

希望有人可以帮助我，谢谢！

Answer 1

问题在于，如果不先实际分配内存，则会取消引用cTime_变量。我不确定这是一个全局变量还是一个成员变量，但是您需要先使用“new”运算符来分配它的内存。因此，您将指向此变量（的地址）的指针返回给函数的调用方，但一旦此函数退出，它将删除“temp”变量，因此您返回的指针将指向无效内存。

的解决办法是使用“new”操作符：

string* Recipe::getCookingTime() 
// @intput: none 
// @output: cooking time as a string 
{ 
    string displayHrs; 
    string displayMins; 
    if(cookingTime_->numHours < 10) 
     displayHrs = intToString(0) + intToString(cookingTime_->numHours); 
    else 
     displayHrs = intToString(cookingTime_->numHours); 
    if(cookingTime_->numMinutes < 10) 
     displayMins = intToString(0) + intToString(cookingTime_->numMinutes); 
    else 
     displayMins = intToString(cookingTime_->numMinutes); 

    if(NULL == cTime_) 
    { 
     cTime_ = new string(); 
    } 

    *cTime_ = "The time to cook the recipe is " + displayHrs + ":" + displayMins; 
    return cTime_; 
} 

不过，我要提醒你，这是不好的设计，因为在这里你分配内存，并要求该呼叫知道他们有当它们完成时释放它。优选的方式做这将是具有来电分配变量，然后传递指针：

bool Recipe::getCookingTime(string* str) 
// @intput: none 
// @output: cooking time as a string 
{ 
    if(NULL == str) 
    { 
     // Received invalid pointer 
     return false; 
    } 
    string displayHrs; 
    string displayMins; 
    if(cookingTime_->numHours < 10) 
     displayHrs = intToString(0) + intToString(cookingTime_->numHours); 
    else 
     displayHrs = intToString(cookingTime_->numHours); 
    if(cookingTime_->numMinutes < 10) 
     displayMins = intToString(0) + intToString(cookingTime_->numMinutes); 
    else 
     displayMins = intToString(cookingTime_->numMinutes); 

    *str = "The time to cook the recipe is " + displayHrs + ":" + displayMins; 
    return true; 
} 

然后，当主叫用户想要使用他们能做到这一点的功能：

cTime_ = new string(); 
getCookingTime(cTime_); 

摘要 这里要记住的重要一点是，您必须在尝试分配指针之前分配指针正在引用的内存。另外，在函数中分配内存（使用新的操作符）通常是不好的设计，而不是明确地删除它。 分配内存的人应该几乎总是释放内存的人

Answer 2

*cTime_ = temp; 

看来你还没有分配内存cTime_。

我想知道你为什么要返回指向std::string的指针。为什么如下图所示不是你简单地返回std::string：

std::string Recipe::getCookingTime() 
{ 
    //your same code 
    return temp; //this is fine! 
} 

注意，返回类型的类型从std::string*改为std::string。

Answer 3

我只关注你的问题地址而不是代码中的其他任何地方。首先，我认为你没有发布你的完整代码，因为在你发布的内容中，我没有看到cTime_在该方法中的定义，所以你的代码甚至不会编译。其次，假设您将cTime_定义为指向字符串的指针，并将该指针指定给由字符串temp占用的内存。当该方法退出时，temp将超出范围，现在cTime_不再指向有效的内存位置，因此您将获得访问冲突。你可能会考虑这样的事情：

void Recipe::getCookingTime(string& str) 
{ 
    string displayHrs; 
    string displayMins; 
    if(cookingTime_->numHours < 10) 
     displayHrs = intToString(0) + intToString(cookingTime_->numHours); 
    else 
     displayHrs = intToString(cookingTime_->numHours); 
    if(cookingTime_->numMinutes < 10) 
     displayMins = intToString(0) + intToString(cookingTime_->numMinutes); 
    else 
     displayMins = intToString(cookingTime_->numMinutes); 

    str = "The time to cook the recipe is " + displayHrs + ":" + displayMins; 
} 

然后调用getCookingTime（）：

string s; 
getCookingTime(s); 

处理指针相反的，你现在会处理的参考。代码会更直接。