我是否正确使用strcpy_s?

时间:2013-10-30 19:17:51

标签: c++ constructor strcpy

我有一个名为“Person”的Person类的构造函数,它看起来像这样:

    Person(const char * their_name, const char * email, int day, int month, int year)
        : name(0), email_address(0), birthday(day, month, year) {

        name = new char [strlen(their_name)+1];
        strcpy_s(name, strlen(their_name) +1, their_name);

        email_address = new char[strlen(email) + 1]; 
        strcpy_s(email_address, strlen(email) + 1, email); 

        cout << "\nPerson(...) FIRST CONSTRUCTOR CREATING: "<<name<<"\n";
        printOn(cout);
    }

我在这个类中有私有变量:

private: 
    char * name; 
    char * email_address; 
    Date birthday; 

我认为这里有一个错误或者错误,因为当我在main中测试我的构造函数时这样:

Person *p1 = new Person("Sarah", "iam@awesome.com", 2,2,1000);

打印到我的控制台:

Person(...) FIRST CONSTRUCTOR CREATING: Sarah
ààà

我不明白为什么在构造函数运行后打印a ...可以有人看到这个问题吗?

编辑:我的printOn方法

/*print person on output stream o*/
    virtual void printOn(ostream & o) const { 
        //print person
         o << "………";
    } 

并覆盖&lt;&lt;操作

ostream & operator<<(ostream & ostr, const Person & p) { 
    p.printOn(ostr); 
    return ostr; 
} 

2 个答案:

答案 0 :(得分:0)

目前尚不清楚printOn(cout)的功能是什么;在你的构造函数中。这可能是它的输出吗?

对于函数strcpy_s,则无需使用它。使用旧函数strcpy会好得多。在您的代码中,您调用strlen两次:一次分配内存,第二次使用strcpy_s。

答案 1 :(得分:0)

您的输出使用的文本编码与编辑器不同。打印出来时,代码编辑器中显示为的字符变为à

一般情况下,除非您知道如何选择正确的编码,否则不要使用高位ASCII字符,否则事情将无法按照您的想法运作。