Question

我正在编写我的String类的版本，但Valgrind抱怨我为我的字符串实现了<<运算符。错误发生在错误的行，如果我通过char打印char它工作得很好。

我哪里错了？

Valgrind错误：

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

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

== 2769 == by 0x4ECAD60：std :: basic_ostream＆gt;＆amp;的std ::运营商LT;＆LT; ＆gt;（std :: basic_ostream＆gt;＆amp;，char const *）（在/ usr / lib / x86_64-linux-gnu / libstdc ++。so.6.0.17）

== 2769 == by 0x400BD5：operator＆lt;＆lt;（std :: ostream＆amp;，String＆amp;）（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);
}

Main.cpp的：

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

Answer 1

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

然而，罪魁祸首在于：

strncpy(_pData, inCString, _size+1);

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

_pData[_size] = 0;

通过错过NUL终止字符，输出操作将继续运行超过字符串的结尾。（行为可能看起来没问题，因为角色可能是偶然的，具体取决于编译器，选项等。）

提示：

考虑使用C ++样式而不是C API
如果您必须使用C风格char*，至少使用stdrup和free

如果你坚持做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;
}

Answer 2

因为你没有在结尾写入零字节。你需要：

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

你应该始终read the manual非常谨慎地使用n - 版本的C字符串函数，因为它们都有微妙的不同语义。

Answer 3

请注意，您没有明确初始化数据成员，而是为其分配值：

...为了初始化，你应该改为：

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

带有char *和＆lt;＆lt;条件的跳转valgrind操作者

3 个答案: