1

为什么调用 c2.view() 会打印出客户 ID 的 ID 和名称？

我已经盯着这个看了一段时间，找不到原因。我要么错过了一些非常明显的东西，要么我不明白 cstrings 是如何工作的 :)

客户.h

#ifndef CUSTOMER_H
#define CUSTOMER_H
 class Customer
 {
 private:
     char accountID[6];
     char name[30];
 public:
     Customer();
     Customer(char[], char[]);
     void view();
     Customer operator=(const Customer&);

 };
#endif

客户.cpp

#include <string>
#include <iostream>
#include "Customer.h"
using namespace std;



Customer::Customer()
{
    strcpy(accountID, "");
    strcpy(name, "");
}

Customer::Customer(char acc[], char n[])
{
    strcpy(accountID, acc);
    strcpy(name, n);
}

void Customer::view()
{
    cout << "Customer name: " << name << endl;
    cout << "Customer ID: " << accountID <<endl;
}

Customer Customer::operator=(const Customer& right)
{
    strcpy(accountID, right.accountID);
    strcpy(name,  right.name);
    return* this;
}

驱动程序.cpp

#include <iostream>
#include "Customer.h"
using namespace std;

int main()
{
    char id[] = "123456";
    char n[] = "Bob";
    Customer c1;
    Customer c2(id, n);
    c1.view();
    c2.view();
    system("pause");
    return 0;
}

输出：

Customer name:
Customer ID:
Customer name: Bob
Customer ID: 123456Bob
Press any key to continue . . .
4

4 回答 4

4

您正在传递一个包含七个字符的字符串：

char id[] = "123456"; // one more character for null termination '\0'

但是您的数组大小为 6。因此，当您 print 时accountId，您会超出'6'字符并打印出它旁边的任何内容，在这种情况下恰好是name.

std::strings使用而不是字符数组可以为自己省去很多麻烦。

于 2012-11-27T22:22:18.990 回答
1

因为 accoutID 的长度为 6，并且当您在 n 中进行 strcpy 时，您会覆盖 accountID 的终止符，该终止符溢出到 name[0]

于 2012-11-27T22:24:18.573 回答
1

strcpy复制直到它到达一个空终止符；\0因为你没有定义并且你正在运行调试，所以 name 恰好占据了与 id 相邻的内存，并且也被复制到 id 的缓冲区中。

如果你要构建发布，你很可能会在那里胡说八道。无论哪种方式，如果您使用 C 字符串，则在所有字符串的末尾都需要空终止符。

strcpy的实现类似于；

while (*ptr2 != '\0')
{
    // copy string2 into string1's buffer until we reach it's null termintor
    *ptr1 = *ptr2
    ptr1++;
    ptr2++;
}
*(ptr1 + 1) = '\0'  // append null terminator

如您所见，它依赖于空终止符，如果不存在，您将获得缓冲区溢出。

于 2012-11-27T22:24:39.963 回答
1

使用 C++ std::string。您正在写超出accountID成员的范围。这char id[] = "123456";有七个要素。

在这种情况下发生的是终止空字符首先结束，name[0]然后被覆盖，strcpy (name, n)你得到一个连续的序列123456Bob\0

于 2012-11-27T22:22:26.493 回答