我正在尝试将一个字符指针传递给函数,该函数将在内部使用字符串read_json。但是在退出函数后,指针的内容会发生变化。以下是代码。
string tempstr;
void someFunc()
{
const char *ptr = getDiagnosticGroupValue("Device.ManagementServer");
printf("string : %s\n", getParameterValue(ptr, "URL"));
printf("string : %s\n", getParameterValue(ptr, "Username"));
}
const char* getParameterValue(const char *jsonstring, const char* param)
{
return getString(jsonstring, param);
}
const char* getString(const char *jsonstring, const char* param)
{
stringstream jsonstringstream;
jsonstringstream << jsonstring;
string paramstr(param);
try
{
boost::property_tree::ptree pt;
boost::property_tree::read_json(jsonstringstream, pt);
tempstr = pt.get<string>(paramstr);
return tempstr.c_str();
}
catch(...)
{
jsonstringstream.clear();
jsonstringstream.str("");
cout << "Node Doesn't exist" << endl;
return "N/A";
}
}
对于第一个节点(URL),它返回正确的值,我可以在someFunc()中打印字符串。但是ptr内容正在被修改(并且它不再是json格式字符串)。所以第二次,当调用getParameterValue()时,read_json抛出异常而不是json格式。我想知道为什么ptr内容会被改变,即使我声明为const char * ptr。任何人都可以告诉我如何解决这个问题。
我确认问题不在于临时对象。我通过创建全局字符串并从全局字符串返回char *来修改代码。但我面临同样的问题。还有一件事是当第一次调用getParameterValue()时,我在read_json()之后放置print语句来检查ptr的值,并且ptr在那时已经被破坏了。
getDiagnosticGrouapValue()的实现如下。
const char* getDiagnosticGroupValue(const char* group)
{
VZUAL& vzual = VZUAL::getInstance();
string groupStr(group, group + strlen(group));
return vzual.Diagnostics.getGroupValue(groupStr).c_str();
}
const string PlatformDiagnostics::getGroupValue(const string& _groupName_, const string& DUMMY)
{
boost::property_tree::ptree ptRead;
boost::property_tree::ptree ptWrite;
string path = _groupName_ + ".";
stringstream ss;
try {
boost::property_tree::read_json(Resource::diagnosticsFile.c_str(), ptRead);
}
catch (boost::property_tree::json_parser::json_parser_error &je) {
return Resource::defaultValue;
}
try {
BOOST_FOREACH(boost::property_tree::ptree::value_type &v, ptRead.get_child(path))
ptWrite.put(v.first.data(), v.second.data());
}
catch (...) {
return Resource::defaultValue;
}
boost::property_tree::write_json(ss, ptWrite);
return ss.str();
}
答案 0 :(得分:2)
您正在返回指向在getString函数内的堆栈上创建的字符串的指针。这是未定义的行为。
从您的代码中,所有变量:
string paramstr(param);
boost::property_tree::ptree pt;
boost::property_tree::read_json(jsonstringstream, pt);
return (pt.get<string>(paramstr)).c_str();
表示此字符串不是静态创建的,也不是全局创建的。
如果你要返回一个指向jsonstring某些部分的指针,那么你应该在jsonstring中找到这个字符串并返回指向它的指针。问题是它不会被空终止(所以你需要也返回lenght)。
其他解决方案是向getString提供一个将从getString返回的char数组。但是你必须确保它的大量,并且它太小然后返回NULL,这样客户端代码就可以提供更大的缓冲区。