2

我正在编写一个程序，该程序具有银行客户（BankAccount）的结构，该结构具有 3 个元素 - 帐号、客户名称和银行余额。我有一个已经包含记录的文件，并且我有一个修改余额的功能。但是，我无法这样做。余额保持不变。我认为我的 fseek() 是错误的，但我不确定如何/为什么。我在一些可能不需要的地方使用了 fflush(stdin) ，但我认为这与问题无关。我发表评论是为了让您了解我的逻辑，以防万一我的概念有一些误解。这是代码：

 void modify(){

    int account_number;
    FILE *ptr;
    BankAccount account;

    ptr = fopen("account.txt", "r+");
    printf("Enter account number: ");
    fflush(stdin);
    scanf("%d", &account_number);

    while (!feof(ptr)) // To search the whole "account.txt" file.
    {
        fread(&account, sizeof(BankAccount), 1, ptr); //brings record into memory
        if (account.account_number == account_number){ // if record's account number is same as account number entered by user above
            printf("***Account found***\n\nAccount number: %d\nAccount name: %s\nAccount balance: %.2f\n", account.account_number, account.name, account.balance);
            printf("\nEnter new balance: ");
            fflush(stdin);
            scanf("%f", &account.balance); // rewrites account's balance in memory
            fseek(ptr, -sizeof(BankAccount), SEEK_CUR); //pointer seeked to the beginning of the record to overwrite it with the one in memory
            fwrite(&account,sizeof(BankAccount), 1, ptr); // record overwritten
            return;

        }
    }


    printf("Account not found\n");
    fflush(stdin);
    getch();
}

这是我的项目的整个 .cpp 文件，如果您想运行它：源代码。我会很感激一些指导。提前致谢。

4

1 回答 1

1

问题可能是fseek()电话：

fseek(ptr, -sizeof(BankAccount), SEEK_CUR);

from的返回值sizeof()是无符号类型；对它的否定将是一个非常大的数字。从技术上讲，这是不正确的（fseek()应该得到 a long，而不是 a size_t）。但是，如果sizeof(size_t) == sizeof(long)，您可以侥幸逃脱（它对我有用）。

问题的另一个方面可能是您在返回之前没有关闭文件（无论您是否找到记录）。那肯定是内存泄漏。它还可能影响数据写入磁盘的方式。这可能是你麻烦的根本原因。您还有另一个功能，即打开文件以读取数据以查看更改，但由于文件未关闭，因此数据尚未写入磁盘。 注意：由于源现在可用，这是问题的原因。

由于您没有向我们展示数据结构，因此另一个问题可能是类型float与成员double的不匹配balance；同样，唯一的问题可能是float帐户余额的类型不合适（例如，它不能可靠地将大约 100,000.00 美元以上的余额表示为最接近的美分 - 例如，输入 199999.99 会显示为 199999.98）。

注意：在现代版本的Linux和Windows上，fflush(stdin)是已定义的操作（并且已定义的行为是明智且有用的）。根据 C 标准和 POSIX，它会产生未定义的行为。使用它时要小心——请注意，它不是便携式操作。

转换为 SSCCE（简短、独立、正确的示例），对您的代码进行非常轻微的修改（添加fclose()）对我有用：

#include <stdio.h>
#include <stdlib.h>

typedef struct BankAccount
{
    int account_number;
    char   name[20];
    float balance;
} BankAccount;

static void modify(void)
{
    int account_number;
    FILE *ptr;
    BankAccount account;

    ptr = fopen("account.txt", "r+");
    printf("Enter account number: ");
    fflush(stdin);
    scanf("%d", &account_number);

    while (!feof(ptr))
    {
        fread(&account, sizeof(BankAccount), 1, ptr);
        printf("***Account read***(%d: %s: %.2f)\n", 
               account.account_number, account.name, account.balance);
        if (account.account_number == account_number)
        {
            printf("***Account found***\n\nAccount number: %d\nAccount name: %s\nAccount balance: %.2f\n", account.account_number, account.name, account.balance);
            printf("\nEnter new balance: ");
            fflush(stdin);
            scanf("%f", &account.balance);
            fseek(ptr, -sizeof(BankAccount), SEEK_CUR);
            fwrite(&account, sizeof(BankAccount), 1, ptr);
            fclose(ptr);
            return;
        }
    }

    printf("Account not found\n");
    fflush(stdin);
    fclose(ptr);
}

static void write(void)
{
    FILE *fp = fopen("account.txt", "w");
    if (fp == 0)
    {
        fprintf(stderr, "Create file failed\n");
        exit(1);
    }
    static const BankAccount data[] =
    {
        { 1, "His", 20.00 },
        { 2, "Hers", 2000.00 },
        { 3, "Theirs", 1.00 },
    };
    if (fwrite(data, sizeof(data), 1, fp) != 1)
    {
        fprintf(stderr, "Write file failed\n");
        exit(1);
    }
    fclose(fp);
}

static void read(void)
{
    FILE *fp = fopen("account.txt", "r");
    if (fp == 0)
    {
        fprintf(stderr, "Open file failed\n");
        exit(1);
    }
    BankAccount ac;
    while (fread(&ac, sizeof(ac), 1, fp) == 1)
    {
        printf("A/C: %4d %-20s  %8.2f\n", ac.account_number, ac.name, ac.balance);
    }
    fclose(fp);
}

int main(void)
{
    write();
    read();
    modify();
    read();
    return 0;
}

编译器甚至不知道fseek()使用编译器选项进行的转换：

$ gcc -O3 -g -std=c11 -Wall -Wextra -Wmissing-prototypes -Wstrict-prototypes \
      -Wold-style-definition -Werror ba.c -o ba

运行时显示：

A/C:    1 His                      20.00
A/C:    2 Hers                   2000.00
A/C:    3 Theirs                    1.00
Enter account number: 2
***Account read***(1: His: 20.00)
***Account read***(2: Hers: 2000.00)
***Account found***

Account number: 2
Account name: Hers
Account balance: 2000.00

Enter new balance: 4000
A/C:    1 His                      20.00
A/C:    2 Hers                   4000.00
A/C:    3 Theirs                    1.00

请注意检查函数中是否已到达文件末尾的正确形式read()。如果你打电话给feof()，你在 99.9% 的时间里都做错了。

于 2013-12-22T14:14:28.207 回答