3

我正在编写我的 String 类版本，但 Valgrind 抱怨我<<为我的字符串实现的运算符。错误出现在错误的行，如果我按字符打印 char 效果很好。

我哪里错了？

Valgrind 错误：

==2769== 条件跳转或移动取决于未初始化的值

==2769== 在 0x4C2AC28：strlen（在 /usr/lib/valgrind/vgpreload_memcheck-amd64-linux.so 中）

==2769== by 0x4ECAD60: std::basic_ostream >& std::operator<< >(std::basic_ostream >&, char const*) （在 /usr/lib/x86_64-linux-gnu/libstdc++.so 中。 6.0.17)

==2769== by 0x400BD5: operator<<(std::ostream&, String&) (string.cpp:22)

==2769== by 0x400AAC: main (main.cpp:12)

我<<的字符串运算符：

ostream & operator << (ostream & o, String & inS) {
    o << inS._pData << " "; // the wrong line
    return o;
}

我的String班级：

class String {
    public:
         unsigned _size;
         char *   _pData;
         String();
         String(const char* inCString);
};

构造函数（对于char*）：

String::String(const char* inCString) {
    _size = strlen(inCString);
    _pData = new char[_size + 1];
    strncpy(_pData, inCString, _size);
}

主要.cpp：

int main(int, char**) {
    String s1("hello");
    cout << s1;
    return 0;
}
4

3 回答 3

10

我不建议使用这样的原始字符串。

然而，罪魁祸首在这里：

strncpy(_pData, inCString, _size+1);

或者，手动存储 NUL 终止字符：

_pData[_size] = 0;

由于缺少 NUL 终止字符，输出操作将继续运行超过字符串的末尾。（该行为可能看起来不错，因为该字符可能偶然为 nul，具体取决于编译器、选项等。）

暗示：

  • 考虑使用 C++ 风格而不是 C API
  • 如果你必须使用 C-style char*，至少使用stdrupandfree

  • 如果您坚持使用以 NUL 结尾的字符串，请考虑也使用 C++ 方式编写：

    #include <iostream>
#include <vector>

class String
{
public:
    std::vector<char> _data;
    String();
    String(const char* inCString);
};

std::ostream & operator << (std::ostream & o, String const& inS)
{
    o.write(inS._data.data(), inS._data.size());
    return o << ' ';
}

String::String(const char* inCString)
{
    for (const char* it=inCString; it && *it; ++it)
        _data.push_back(*it);
}

int main(int, char**)
{
    String s1("hello");
    std::cout << s1;
    return 0;
}
于 2012-10-05T14:53:40.440 回答
1

Because you fail to write the zero byte at the end. You need:

strncpy(_pData, inCString, _size + 1);
//                              ^^^^

You should always read the manual very carefully with the n-versions of C string functions, since they all have subtly different semantics.

于 2012-10-05T14:56:04.660 回答
-1

请注意，您没有显式初始化数据成员，而是为它们分配了一个值：

...in order to initialize, you should change to:

String::String(const char* inCString) :
    _size(strlen(inCString)),
    _pData(new char[_size + 1])
{
    strncpy(_pData, inCString, _size);
}
于 2012-10-05T14:54:26.203 回答