2

私は以下のコードを持っています：

std::string myName = "BLABLABLA";

//check if there are illegal characters
for (unsigned int i = 0; i < myName.length(); i++)
{
    const char& c = myName[i];
    if (!(isalnum(c) || (c == '_') || (c == '-')))
    {
        return 0;
    }      

}

これは、行「const char＆c =myName[i];」でのvalgrindの出力です。

==17249== 51 bytes in 1 blocks are possibly lost in loss record 116 of 224
==17249==    at 0x4C2714E: operator new(unsigned long) (vg_replace_malloc.c:261)
==17249==    by 0x602A498: std::string::_Rep::_S_create(unsigned long, unsigned long,       
std::allocator<char> const&) (in /usr/lib64/libstdc++.so.6.0.16)
==17249==    by 0x602A689: std::string::_M_mutate(unsigned long, unsigned long,   
unsigned long) (in /usr/lib64/libstdc++.so.6.0.16)
==17249==    by 0x602AFB5: std::string::_M_leak_hard() (in 
/usr/lib64/libstdc++.so.6.0.16)
==17249==    by 0x602B0A4: std::string::operator[](unsigned long) (in /
/usr/lib64/libstdc++.so.6.0.16)

これには何の問題もありません...

4

1 に答える 1

2

はい、それは恐ろしい COW 実装です! 次のように、const (したがって、非変更) オーバーロードの使用を強制することもできます。

std::string const myName = "BLABLABLA";

//check if there are illegal characters
for (unsigned int i = 0; i < myName.length(); i++)
{
    const char& c = myName[i];
    if (!(isalnum(c) || (c == '_') || (c == '-')))
    {
        return 0;
    }      
}

または (元の文字列型を変更したくない場合):

std::string myName = "BLABLABLA";
std::string const &cref = myName;
//check if there are illegal characters
for (unsigned int i = 0; i < myName.length(); i++)
{
    const char& c = cref[i];
    if (!(isalnum(c) || (c == '_') || (c == '-')))
    {
        return 0;
    }      
}

等

COW 参照、それについてどこかに書いたことを知っていたからです。

于 2012-09-26T15:10:39.637 に答える