`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; 
}