`std :: string :: substr`返回無效指針？

Question

我正在寫一個函數，根據特定字符的位置將C++ string分成兩個獨立的string。但是，當我使用它返回的指針時，它們似乎是無效的。這裏有一個例子：

#include <iostream> 
#include <string> 

bool SplitString(std::string &input, char splitChar, std::string **left, std::string **right) 
{ 
    std::string::size_type offset = input.find(splitChar); 
    if (offset == std::string::npos) return false; 
    *left = &input.substr(0, offset); 
    *right = &input.substr(offset + 1); 
    return true; 
} 

int main(int argc, char *argv[]) 
{ 
    std::string input = "Left side:Right side"; 
    std::string *left; 
    std::string *right; 

    if (SplitString(input, ':', &left, &right)) { 
     std::cout << "left = \"" << *left << "\"" << std::endl; 
     std::cout << "right = \"" << *right << "\"" << std::endl; 
    } else { 
     std::cout << "Didn't find ':' - this shouldn't happen!" << std::endl; 
    } 

    return 0; 
} 

這應該輸出如下：

left = "Left side" 
right = "Right side" 

爲什麼不工作？

Answer 1

你並不需要在這裏指針。您可以參考left和right。可以說，你不需要參考input（你不修改它，如果你這樣做，如果你必須對同一個輸入執行其他操作，會怎麼樣？）所以要麼通過值來傳遞它（如果你打算在副本上工作）或const&（以表示不修改它的意圖）。

bool SplitString(std::string input, char splitChar, std::string& left, std::string& right) 
{ 
    std::string::size_type offset = input.find(splitChar); 
    if (offset == std::string::npos) return false; 
    left = input.substr(0, offset); 
    right = input.substr(offset + 1); 
    return true; 
} 

// ... 

    std::string left; 
    std::string right; 

    if (SplitString(input, ':', left, right)) { 

// ... 

Answer 2

原來，當我編譯我的測試代碼g++時，它給了我一個Visual C++編譯器不給我的警告。問題是由std::string::substr返回的對象被認爲是「臨時的」，並且在函數之外使用時無效。解決方案是使用*left = new std::string(input.substr(0, offset))而不是*left = &input.substr(0, offset)。以類似的方式顯然修改*right行。

不要忘了delete當你完成他們的新對象！

這裏的工作代碼：

#include <iostream> 
#include <string> 

bool SplitString(std::string &input, char splitChar, std::string **left, std::string **right) 
{ 
    std::string::size_type offset = input.find(splitChar); 
    if (offset == std::string::npos) return false; 
    *left = new std::string(input.substr(0, offset)); 
    *right = new std::string(input.substr(offset + 1)); 
    return true; 
} 

int main(int argc, char *argv[]) 
{ 
    std::string input = "Left side:Right side"; 
    std::string *left; 
    std::string *right; 

    if (SplitString(input, ':', &left, &right)) { 
     std::cout << "left = \"" << *left << "\"" << std::endl; 
     std::cout << "right = \"" << *right << "\"" << std::endl; 
    } else { 
     std::cout << "Didn't find ':' - this shouldn't happen!" << std::endl; 
    } 

    delete left; 
    delete right; 
    return 0; 
}