新手网站故障问题C++

Question

我正在开发一个项目，我可以在其中创建银行帐户并能够存款和取款。我要创建两个银行帐户和两个人 - 一个在堆栈上，另一个在堆上。我应该存款和取款各两次，并获得余额打印姓名、身份证件和帐号。目前，我认为是站点故障，读取或写入受保护的内存。我已经对我认为错误所在的地方发表了评论。我将不胜感激任何帮助。谢谢。

#include <iostream>
#include <string>
using namespace std;

class BankAccount {
private:
    double *balance;
    int *accountNumber;
public:
    BankAccount(){//default constructor
        *balance = 0.0;/***This is where is says the Access violation lies*/
        *accountNumber = 0;
    }

    BankAccount(double bal, int acctNum){//constructor
        balance = new double(bal);
        accountNumber = new int(acctNum);
    }
    ~BankAccount() {delete balance; delete accountNumber;}
    void Deposit(double amt) {
        *balance = *balance + amt;
    }

    virtual double GetBalance() {
        return *balance;
    }

    virtual double GetAccountNumber() {
        return *accountNumber;
    }

    virtual double Withdraw(double amt) {
        *balance = *balance - amt;
        return *balance;
    }
};

class Person {
    string *name;
    int *ID;

public:
    Person(){//default constructor
        *name = "name not yet defined";
        *ID = 0;
    }

    Person(string nameIn, int idIn){//constructor
        name = new string(nameIn);
        ID = new int(idIn);
    }

    virtual int GetID() {
    return *ID;
    }

    virtual string GetName() {
    return *name;
    }
};

class NamedBankAccount: public BankAccount {
private:
    Person *owner;
public: 
    NamedBankAccount(){
    }
    NamedBankAccount(Person *p): owner(p){/***This is where is says the Access violation lies*/
        p = new Person();
    }
    ~NamedBankAccount(){delete owner;}

    Person getPerson() {
        return *owner;
    }
};

int main() {

    Person *q = new Person("Joe", 54321);
    cout << q->GetName() << endl;
    cout << q->GetID() << endl;


    NamedBankAccount nba1;/***This is where is says the Access violation lies*/
    NamedBankAccount *nba2 = new NamedBankAccount(q);

    nba1.Deposit(50);
    nba1.Deposit(50);
    nba1.Withdraw(25);
    cout << nba1.GetBalance() <<endl;//should print 75

    nba2->Deposit(60);
    nba2->Deposit(60);
    nba2->Withdraw(20);
    cout << nba2->GetBalance() << endl;//should print 100

    getchar();
    return 0;
}

Answer 1

不要在这里使用指针。只需将这些字符串和整数作为成员变量即可。对于特定问题 - 在默认构造函数中分配之前您没有分配任何内存。

做类似的事情：

class BankAccount {
private:
    double balance;
    int accountNumber;
public:
    BankAccount() :
        balance( 0.0 ),
        accountNumber( 0 ) {}

    // ...

编辑：

关于代码的更多要点：

• 在构造函数中使用初始化列表而不是分配给成员变量 - 这避免了首先默认初始化成员然后分配给它们
• 基多态类 的两步过程应该有虚拟析构函数，因此派生类的实例可以通过指向基本
• 多态类型的指针正确销毁，通常需要遵循三法则以避免切片
• 不会使基类的所有成员函数虚拟，仅使那些您希望派生类重写的成员函数
• 在使类型多态之前思考 - 您真的拥有没有所有者的银行帐户吗？也许这只是一个值类型？
• 使访问器方法const，这样您就可以从const实例获取信息
• 检查错误（您肯定不希望允许从零或负余额帐户提款）

Answer 2

“不要使用指针”有点强，但 Nikolai 的意思是成员变量不应该是指向基本类型的指针，而只是那些类型

，即在 BankAccount 中，balance 应该对于其他人来说，只是一个 double 而不是 double*

，或者让 BankAccount() 调用 BankAccount(0.0, 0) 因为这将像 Person() 一样正确分配字段，但出乎意料的是，这并没有达到我在 C++ 中的预期，正如 Karl Knechtel 所说

Answer 3

您正在取消引用未初始化的指针，如果您更改它们的位置，它仍然会执行相同的操作。

你看，c++（和c）使用指针作为内存地址，如果你不初始化，那么它们将指向内存中的任何位置，因此取消引用可能会导致访问冲突（可能是因为你不知道你的指针是否指向）。

正确的方法是：

BankAccount(){//default constructor
    balance = new double; // Initialize pointer (make it points to a valid memory address)
    *balance = 0.0; // Give a value to the variable where balance is pointing
    accountNumber = new int; // Initialize pointer (make it points to a valid memory address)
    *accountNumber = 0; // Give a value to the variable where balance is pointing
}

或者，如果您想稍后分配内存：

BankAccount(){//default constructor
    balance = 0; // Make it point to address 0 (conventional value meaning it is uninitialized)
    accountNumber = 0; // Make it point to address 0 (conventional value meaning it is uninitialized)
}

当然，如上所述，在您的情况下，最好使用普通变量而不是指针。在使用指针之前，您应该阅读更多有关指针的内容，它们可能会很痛苦（我想我在这里代表 99.999% 的 C 和 C++ 程序员发言，我们都经历过）。