为什么const char *不能用于stringstream和read_json的boost?

时间:2015-12-04 16:05:39

标签: c++ json boost

我正在尝试将一个字符指针传递给函数,该函数将在内部使用字符串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();
}

1 个答案:

答案 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,这样客户端代码就可以提供更大的缓冲区。