c++ - 结构数组在调用时打印零，但在未调用时显示正确的输入(短代码)

Question

该程序的目的是提供一个银行帐户界面，用户可以在其中创建 4 个帐户并可以在它们之间转移资金。我选择了一组结构来处理“银行账户”信息，并选择了一个开关来处理用户选项。

问题:

帐户创建函数，在案例“a”中调用，似乎按预期创建帐户，并正确显示它们。但是，其他情况下的结果与设计不同。

Case 'b' 将显示输入的数组信息，只有没有被调用(包括显示数组信息的函数)，否则当 display() 被调用时它打印零(什么它被初始化为在创建帐户之前)。

case 'c' 会显示输入的数组信息，如果没有调用，以及如果只调用显示日期/时间的函数。否则它打印零。

问题:

为什么结构数组在我调用显示函数时显示为零，但在未调用时会显示用户输入，只要没有其他内容(除了 'c' 中的时间函数)，还有什么可以我要修复它吗？

注意事项:

我将帐户创建函数转换为返回一个指向结构的指针，希望能有所帮助，它似乎没有改变任何东西。我将 switch 语句转换为一系列 if 以查看是否会改变任何东西，但它没有(反转回 switch)。 Case 'd' 和 case default 按预期工作。程序不完整(我的 transfer() 没有写)。我知道“使用命名空间标准；”是诺诺。

非常感谢您花时间和提供的任何建议。

#include <iostream>
#include <ctime>
const int SIZE = 4;
using namespace std;

struct bank  //Bank Account Structure to hold account numbers and balances
{
     int num; // number of account
     int checkdigit; // ending account number checkdigit appended to account number
     float bal; // balance of account
};

void menu(); //MenuPrompt

bank* input(bank[]); // Struct Account Input
void display(bank[]); // Struct Account Display
void initialize(bank[]); // Struct Account Initialization

char transferprompt(bank s[]); // Transfer Menu

int createaccounts(); //Create Account Numbers and Balances
void time();    //Displays the Current date, month, year, and time.
void clearscreen(); //Prints 40lines to clear the screen

int main()
{
    bank accounts[SIZE]; // Array of bank-account structs
    initialize(accounts); // Removes garbage values
    char select; //User input variable
    bank *ptr;
    ptr = &accounts[0];
    menu(); //Runs MenuPrompt
    cin >> select; //Grabs user input
    select = static_cast<char>( tolower( select ) ); //Ensures user input is lowercase (for switch statement)
    clearscreen();

    switch ( select )
    {
        case 'a': //Creates Accounts
        input(accounts);
        clearscreen();
        main();
        display(accounts);
        break;

        case 'b': //Transfers between accounts
        //display(accounts);
        transferprompt(accounts);
        //main();
        break;

        case 'c': //Displays day, month, year, time, + Account Information.
        time();
        break;

        case 'd': //Exits Program
        break;

        default: //Invalid Input failsafe; Restarts menu prompt.
        cout<<"Invalid selection. Restarting...\n";
        main();
        break;
    }
}
char transferprompt(bank s[])
{
}
bank* input(bank s[]) // Creates Bank Accounts
{
        bank accounts[SIZE]; // Array of bank-account structs
        bank *ptr;           // Pointer to struct
        ptr = &accounts[0];  // Points pointer to array of bank-account structs
    cout << "Account Creation Selected; Create " << SIZE << " accounts..." << endl;
    for (int i = 0; i < SIZE ; i++)
    {
        cout << "Enter Account Number: ";
        cin >> s[i].num;
        s[i].checkdigit = s[i].num % 5;
        cout << "Enter Account Balance: ";
        cin >> s[i].bal;
    }
    return ptr;
}
void display(bank s[]) //Displays Bank Accounts
{
    for (int i = 0; i < 4; i++)
    {
        cout << "Account: " << s[i].num << s[i].checkdigit;
        cout << " has balance: $" << s[i].bal << endl;
    }

}
void initialize(bank s[]) // Removes Garbage values
{
for (int i = 0; i < 4; i++)
    {
        s[i].num = 0;
        s[i].checkdigit = 0;
        s[i].bal = 0;
    }
}
void clearscreen() // Clears screen
{
    cout << string(40, '\n');
}
void time() // Display Time function
{
    time_t t = time(NULL);
    tm* ptr = localtime(&t); // ptr = Pointer to compute time
    cout << endl;
    cout << endl;
    cout << "Date:  " << (ptr->tm_mon)+1 <<"/"<< (ptr->tm_mday)<< "/" << (ptr->tm_year)+1900 << endl;
    cout << "Time:  " << (ptr->tm_hour) <<":"<< (ptr->tm_min)<< ":" << (ptr->tm_sec) << endl;

}
void menu() // Menu Prompt
{
    cout << "Parkville Bank Client Program; Edit your accounts:\n";
    cout << "a. Create " <<SIZE<<" accounts\n";
    cout << "b. Transfer money from 1 account to another\n";
    cout << "c. Display account balances\n";
    cout << "d. Quit the program\n\n";
}

Answer 1

这个函数不正确:

bank* input(bank s[]) // Creates Bank Accounts
{
   bank accounts[SIZE]; // Array of bank-account structs
   bank *ptr;           // Pointer to struct
   ptr = &accounts[0];  // Points pointer to array of bank-account structs
   // ... lines removed ...
   return ptr;  // <-- Undefined behavior
}

您正在返回一个指向局部变量/数组的指针。返回局部变量的指针或引用是未定义的行为。原因是因为数组是本地的，一旦函数返回，它在逻辑上就不再存在了。所以你最终指向一个不存在的实体。

你可能想做的是这样的:

bank* input(bank* s) // Creates Bank Accounts
{
    cout << "Account Creation Selected; Create " << SIZE << " accounts..." << endl;
    for (int i = 0; i < SIZE ; i++)
    {
        cout << "Enter Account Number: ";
        cin >> s[i].num;
        s[i].checkdigit = s[i].num % 5;
        cout << "Enter Account Balance: ";
        cin >> s[i].bal;
    }
    return &s[0];
}

由于传入了 s，您可能打算改为传递该实体的第一个元素的地址。另请注意:

bank* input(bank s[]) // Creates Bank Accounts

与此没有什么不同:

bank* input(bank* s) // Creates Bank Accounts

随着数组衰减为指针。

我真的建议您使用诸如 std::vector 之类的容器来代替原始数组和全局常量来跟踪实体的数量(例如，SIZE ).然后

---

您的程序的另一个问题是您正在递归调用 main。不要这样做，因为它不是合法的 C++。相反，请使用适当的循环结构，例如 while() 和 do-while()。