cpp一些基本问题

Question

我的任务如下： 创建具有 char*name 和 int Age 的 Person 类。使用变量动态分配内存实现构造函数、析构函数、函数 init 和友元函数 show。然后将这个类转化为头文件和cpp文件并在其他程序中实现。好的，这是我的 Person 类：

#include <iostream>
using namespace std;

class Person {   
    char* name;
    int age;
public:   

    Person(){
        int size=0;
        cout << "Give length of char*" << endl;
        cin >> size;
        name = new char[size];      
        age = 0;
    }

    Person::~Person(){
        cout << "Destroying resources" << endl;
        delete [] name;
        delete take_age();
    }  

    friend void show(Person &p);

   int* take_age(){
       return &age;
   }

   char* take_name(){
         return name;      
   }

    void init(char* n, int a) {
        name = n;
        age = a;
    }
}; 

void show(Person *p){
    cout << "Name: " << p->take_name() << "," << "age: " << p->take_age() << endl; 
}

int main(void) {
    Person *p = new Person;  
    p->init("Mary", 25);

    show(p);

    system("PAUSE");
    return 0;
}

现在带有标头/实现部分：
- 我需要在头文件/实现文件中引入构造函数吗？如果是 - 怎么办？
- 我的 show() 函数是一个友好的函数。我应该以某种方式考虑它吗？

我已经未能在考试中返回此任务，但我仍然想知道如何实现它。

Answer 1

通过从 char * 切换到 std::string 可以解决您的许多问题。你会很高兴你这么做了。

std::string 类负责内存分配、释放以及复制。

如果这是家庭作业，请说服您的教授为初学者使用 std::string 并为指针部分保留 char *。还要提醒您的教授，C++ 语言与 C 语言不同。这是这些领域之一。

Answer 2

使用 delete 或 delete[] 时不需要 *。只需向其提供一个指针变量即可，例如。

delete[] name;

此外，您的 take_age 成员声称返回 int* 但您实际上返回 int 成员本身。如果您想这样做，您需要使用 & 获取会员的地址。正如 @Jerry 所评论的，这不是您想要在这里做的事情。

Answer 3

尽管该网站上的一些人显然认为这是完全可以接受的，但很好的做法（请参阅

恕我直言，这就是您应该采取的第一步。

Answer 4

在典型情况下，管理一个指针和动态分配的内存块（例如本例中的名称）对于一个类来说就足够了。因此，Thomas Matthews 是对的：在这种情况下你真的应该使用字符串。如果您要自己处理它，您仍然应该将该责任拆分到它自己的类中，并将该类的对象嵌入到您的 Person 对象中。如果有的话，std::string 已经尝试做太多事情了；你最好做一些少做的事，而不是多做的事。

您的删除应该与您的分配完全匹配。在这种情况下，唯一的分配是：

    name = new char[size];      

所以唯一的删除应该是：

    delete [] name;

就 friend 函数而言，您通常希望将友元声明放在类定义内，但将函数定义放在类定义之外：

class Person { 
// ...
    friend void show(Person const &);
// ...
};

void show(Person const &p) { 
     // ...
}

还有其他可能性，但这就是总体思路。特别是，朋友从来都不是成员函数。您拥有的是一个名为 show 的（全局）函数的声明和一个完全独立的成员函数的定义——它们碰巧具有相同的名称，但实际上根本不是同一个函数。

这表明了另一点：常量正确性。您将参数作为对 Person 的引用传递。除非它要修改 Person 对象（在这种情况下，show() 似乎是一个糟糕的名称选择），否则它可能应该引用 const 对象。同样的一般思想也适用于 take_age() ——因为它只检索一个值，所以它应该是一个 const 函数：

int take_age() const { return age; }

我可能已经尝试涵盖太多了，所以我暂时闭嘴...

Answer 5

我认为你应该调查你的代码的以下部分（例如，它们下面是什么，这里发生了什么，等等......）

int * take_age(); // You should return plain `int` here, I assume


~Person(){
    cout << "Destroying resources" << endl;
    delete *[] name; // Do you understand why did you put `*` here?
    delete * take_age(); // Do you understand why did you write this? What behaviour you were trying to achieve?

，实际上，等等。我认为，只有当您完成了基本内容后，您才能继续进行标题设计问题和友元函数。

Answer 6

首先，感谢您尝试找到正确的方法来实施您的课程，特别是在已经错过答案之后。

从您在顶部的描述来看，我认为您可能误解了这项作业的一些要求。首先，我的解释是设置名称和年龄的值应该在 init() 函数中而不是在构造函数中进行。正如其他几位海报所提到的，您的构造函数应该简单地将您的类初始化为已知的良好状态。例如，

Person() {
    name = NULL;
    age = 0;
}

然后在初始化函数中，您可以分配值。查看原始的 init() 函数，可能应该提到的是，简单地将指针值 (char *) 分配给另一个指针 (char *) 仅复制指针的值，而不是它表示的数据。因此，对于名称值的分配，您需要计算所需缓冲区的大小，分配缓冲区并自行复制数据。基本的 init() 函数可能看起来像

init(const char *n, int a) {
    // Calculate the required name length plus a NULL-terminator
    size_t nameLen = strlen(n) + 1;

    // Free any previous value.
    if (name != NULL) {
        delete[] name;
    }

    // Allocate the name buffer and copy the name into it.
    name = new char[nameLen];
    memcpy(name, n, nameLen);

    // Store the age.
    age = a;
}

最后，在析构函数中，您释放类分配的所有资源，在本例中是名称缓冲区。

~Person() {
    if (name != NULL) {
        delete[] name;
    }
}

如果您有一本书或与您的班级相关的东西，您可能需要查看有关指针的信息。它们可能有点棘手，但学习很重要。我怀疑这就是为什么问题指定使用 char * 作为字符串而不是 STL 字符串类。

对于有关将信息放入头文件和源文件中的问题，通常认为创建包含类声明和成员函数原型的头文件，然后在单独的源文件中提供方法的实现是一种很好的做法。对于一些简单的函数，您可以直接在头文件中提供实现。

在单独的源文件中提供类成员定义时，关键是提供类名以正确确定函数的范围（即 Person::）。因此，您的头文件可能包含类似的类定义

// Header file (e.g., person.h)

class Person {
private:
    char *name;
    int age;

public:
    Person() { name = NULL; age = 0 };
    ~Person() { if (name != NULL) delete[] name; }

    void init(const char *n, int a);

    // Other method declarations and/or definitions
};

，然后在源文件中

// Source file (e.g., person.cpp)

void Person::init(const char *n, int a) {
    // ...
}

// Rest of method definitions

使用您的 person 类的源文件只需要包含带有您的类定义的头文件。

Answer 7

我认为你的问题在于这一行：
朋友 void(Person &p);
需要它做什么呢。

我需要在头文件/实现文件中引入构造函数吗？
构造函数可以位于 .h 或 .cpp 文件中。没关系。一般来说，如果函数很短，可以将其包含在 .h 文件中。任何更长的内容都应该放在 .cpp 中。

我的 show() 函数是一个友好的函数。
不确定你的意思是什么。友元函数存在于类定义之外。您的 show 函数是在类内部定义的，因此不需要成为友元。

Answer 8

除了之前发布的答案之外，我还有两点建议给您：

• 不要使用“朋友”。这里有些人可能不同意我的观点，但“朋友”确实不应该再成为 C++ 的一部分，因为它违背了 OOP 的含义。
• 命名你的方法：避免将你的方法命名为“take_name”或“take_age”。按照惯例，由于这些是 getter，请考虑将它们命名为“getName”和“getAge”。通过这种方式，你最终会得到开发人员更多的尊重。