Question

我的问题涉及以下功能：

/*Adds the transaction to the head of the linked list attached to the account.*/
void Bank::Worker::UpdateTransactionHistory(int account_number, string transaction, Bank *our_bank_ptr) {

    transaction_node new_trans;
    new_trans.transaction = transaction;

    if (our_bank_ptr->accounts[account_number].head == nullptr) {   //If this is the first transaction
        our_bank_ptr->accounts[account_number].head = &new_trans;

    } else {    //If this isn't the first transaction, disconnect the account from its current transaction list, connect the new transaction to the account and then connect the old list to the new transaction.
        transaction_node temp;
        temp = *(our_bank_ptr->accounts[account_number].head);

        our_bank_ptr->accounts[account_number].head = &new_trans;

        new_trans.next = &temp;
    }

if (our_bank_ptr->accounts[account_number].head) //here the correct string is printed
            cout << our_bank_ptr->accounts[account_number].head->transaction;
}

它意味着更新new_trans的事务字段，然后将其链接到给定帐户的其余事务列表。就在我从更新事务函数返回之前，我测试以确保正确添加了字符串。函数的最后一行是cout << our_bank_ptr->accounts[account_number].head->transaction;，它正确地输出事务字符串。

但是，当我从函数返回并立即调用完全相同的代码行时，编译器告诉我，更新的函数的事务字段仍然未初始化。尽管事实上它是作为指针传入的。

哎呀！？我想如果我通过指针将信息传递给函数，那么我在函数过程中对该信息所做的任何事情都是永久性的？我在这里缺少什么？

感谢您的帮助，

亚当

Answer 1

您将指针设置为指向局部变量（new_trans）。函数退出后，该变量将被销毁，指针悬空。因此，尝试取消引用它会导致未定义的行为。在您的情况下，这当前表现为单元化打印。但它可以做任何其他事情。

如果你需要指针，并且需要它们指向持久值，你必须动态分配值。但真正的问题是：你需要指针吗？

通过指针初始化字段的麻烦

1 个答案: