4

我有这段代码,CBString 只是我用于某些处理的字符串类

  char * scrummyconfigure::dosub(strtype input)
  {
    CBString tstring;
    tstring = input;
    uint begin;
    uint end;

    begin = tstring.findchr('$');
    end = tstring.findchr('}',begin);        

    CBString k = tstring.midstr(begin+2,end-2); // this is BASE
    strtype vname = (strtype) ((const unsigned char*)k);
    strtype bvar = (strtype) "BASE";
    assert(strcmp(bvar,vname) == 0); // this never fails
    // theconf is just a struct with the map subvars
    // subvars is a map<const char *, const char *>
    out(theconf->subvars[bvar]); // always comes up with the value
    out(theconf->subvars[vname]); // always empty

    uint size = end - begin;
    tstring.remove(begin, size);

    return (const char *)tstring; // it's OKAY! it's got an overload that does things correctly
    //inline operator const char* () const { return (const char *)data; } <-- this is how it is declared in the header of the library
  }

为什么 strcmp 总是说字符串是相同的,但只有我声明为 bvar 的变量才返回任何内容?

4

3 回答 3

5

//子变量是一个map<const char *, const char *>

这个映射的关键不是一个字符串,而是一个内存地址。相应的检查将是

assert( bvar == vname);

这可能会失败。您需要将键类型更改为字符串类(或者std::string有意义CBString地使用 map.

于 2012-07-26T00:23:06.420 回答
5

我假设strtype是通过以下方式定义的:

typedef char * strtype

您的问题是您假设 vname 和 bvar 具有相同的值,而实际上,它们具有不同的值,每个值都指向包含相同数据的内存块。

std::map正在愚蠢地将它们与 == 进行比较,我敢打赌,如果您将它们与 == 进行比较,您会得到错误的结果,正如预期的那样。你为什么不使用这个std::string类?

编辑:我改写了你的方法不那么可怕:

// implied using namespace std;
string ScrummyConfigure::removeVariableOrSomething(string tstring)
{
    uint begin; // I'll assume uint is a typedef to unsigned int
    uint end;

    begin = tstring.find('$', 0);
    end = tstring.find('}', begin);        

    string vname = tstring.substr(begin + 2, end - 2); // this is supposedly BASE
    assert(vname == "BASE"); // this should be true if vname actually is BASE

    out(this->conf->subvars[bvar]);  // wherever theconf used to be, its now a member
    out(this->conf->subvars[vname]); // of ScrummyConfigure and its been renamed to conf

    uint size = end - begin;
    tstring.erase(begin, size);

    return tstring; // no casting necessary
}
于 2012-07-26T00:23:24.763 回答
5

仅仅因为字符串相同并不意味着 std::map 会将它们视为相同的键。这取决于 std::map 使用的 Compare 类,默认为less<KeyType>- 产生与应用小于运算符相同的结果。

您可以定义一个类,该类定义operator()为对您的 strtypes 进行适当的比较,并在定义 std::map 时将其作为第三个模板参数传递。或者,按照建议,使用 std::string 作为您的 strtype。

于 2012-07-26T00:29:20.543 回答