4

我在大学里学习 C，也通过“Learn C the Hard Way”。这个问题特别与Learn C the Hard Way - 练习17有关

在我编写的代码中，使用了一个结构数据库：

#define MAX_ROWS 100
...
...
struct Address {
    int id;
    int set;
    char name[MAX_DATA];
    char email[MAX_DATA];
};
...
...
struct Database {
    struct Address rows[MAX_ROWS];
}

本练习的“额外功劳”部分要求“更改代码以接受 MAX_DATA 和 MAX_ROWS 的参数，将它们存储在 Database 结构中，并将其写入文件，从而创建一个可以任意大小的数据库。”

现在忽略 MAX_DATA，因为我想一次只关注一个。

注意：这是我的完整代码

#include <stdio.h>
#include <assert.h>
#include <stdlib.h>
#include <errno.h>
#include <string.h>

#define MAX_DATA 512
#define MAX_ROWS 100

struct Address {
    int id;
    int set;
    char name[MAX_DATA];
    char email[MAX_DATA];
};

struct Database {
    struct Address rows[MAX_ROWS];
};

struct Connection {
    FILE *file;
    struct Database *db;
};

void die(const char *message, struct Connection *conn)
{
    if(errno) {
        perror(message);
    } else {
        printf("ERROR: %s\n", message);
    }

    free(conn->file);
    free(conn->db);
    free(conn);

    exit(1);
}

void Address_print(struct Address *addr)
{
    printf("%d %s %s\n",
            addr->id, addr->name, addr->email);
}

void Database_load(struct Connection *conn)
{
    int rc = fread(conn->db, sizeof(struct Database), 1, conn->file);
    if(rc != 1) die("Failed to load database.", conn);
}

struct Connection *Database_open(const char *filename, char mode)
{
    struct Connection *conn = malloc(sizeof(struct Connection));
    if(!conn) die("Memory error", conn);

    conn->db = malloc(sizeof(struct Database));
    if(!conn->db) die("Memory error", conn);

    if(mode == 'c') {
        conn->file = fopen(filename, "w");
    } else {
        conn->file = fopen(filename, "r+");

        if(conn->file) {
            Database_load(conn);
        }
    }

    if(!conn->file) die("Failed to open the file", conn);

    return conn;
}

void Database_close(struct Connection *conn)
{
    if(conn) {
        if(conn->file) fclose(conn->file);
        if(conn->db) free(conn->db);
        free(conn);
    }
}

void Database_write(struct Connection *conn)
{
    rewind(conn->file);

    int rc = fwrite(conn->db, sizeof(struct Database), 1, conn->file);
    if(rc != 1) die("Failed to write database.", conn);

    rc = fflush(conn->file);
    if(rc == -1) die("Cannot flush database.", conn);
}

void Database_create(struct Connection *conn)
{
    int i = 0;

    for(i = 0; i < sizeof(conn->db->rows) / sizeof(*conn->db->rows); i++) {
        // make a prototype to initialize it
        struct Address addr = {.id = i, .set = 0};
        // then just assign it
        conn->db->rows[i] = addr;
    }
}

void Database_set(struct Connection *conn, int id, const char *name, const char *email)
{
    struct Address *addr = &conn->db->rows[id];
    if(addr->set) die("Already set, delete it first", conn);

    addr->set = 1;

    char *res = strncpy(addr->name, name, MAX_DATA);
    addr->name[MAX_DATA - 1] = '\0';
    if(!res) die("Name copy failed", conn);

    res = strncpy(addr->email, email, MAX_DATA);
    addr->email[MAX_DATA - 1] = '\0';
    if(!res) die("Email copy failed", conn);
}

void Database_get(struct Connection *conn, int id)
{
    struct Address *addr = &conn->db->rows[id];

    if(addr->set) {
        Address_print(addr);
    } else {
        die("ID is not set", conn);
    }
}

void Database_delete(struct Connection *conn, int id)
{
    struct Address addr = {.id = id, .set = 0};
    conn->db->rows[id] = addr;
}

void Database_list(struct Connection *conn)
{
    int i = 0;
    struct Database *db = conn->db;

    for(i = 0; i < sizeof(conn->db->rows) / sizeof(*conn->db->rows); i++) {
        struct Address *cur = &db->rows[i];

        if(cur->set) {
            Address_print(cur);
        }
    }
}

int main(int argc, char *argv[])
{
    char *filename = argv[1];
    char action = argv[2][0];
    int id = 0;
    struct Connection *conn = Database_open(filename, action);

    if(argc < 3) die("USAGE: ex17 <dbfile> <action> [action params]", conn);


    if(argc > 3) id = atoi(argv[3]);
    if(id >= sizeof(conn->db->rows) / sizeof(*conn->db->rows)) die("There's not that many records.", conn);

    switch(action) {
        case 'c':
            Database_create(conn);
            Database_write(conn);
            break;

        case 'g':
            if(argc != 4) die("Need an id to get", conn);

            Database_get(conn, id);
            break;

        case 's':
            if(argc != 6) die("Need id, name, email to set", conn);

            Database_set(conn, id, argv[4], argv[5]);
            Database_write(conn);
            break;

        case 'd':
            if(argc != 4) die("Need id to delete", conn);

            Database_delete(conn, id);
            Database_write(conn);
            break;

        case 'l':
            Database_list(conn);
            break;
        default:
            die("Invalid action, only: c=create, g=get, s=set, d=del, l=list", conn);
    }

    Database_close(conn);

    return 0;
}

我的第一个想法是不要让“数据库”成为一个全局结构，而是将它放在我的主函数中，以便它可以流过程序的其余部分，但这似乎很尴尬，不得不在整个程序中传递这个变量。然而，这让我意识到我将不得不通过每个需要它的函数调用传递一个指向它的指针，这不是额外功劳所暗示的。

我觉得作为一个 C 新手我错过了一些东西。我的问题可能含糊不清，因为我不完全了解我所谈论的术语。

如果我的问题不清楚 - 我如何实际添加此功能？？？我真的很茫然。

4

1 回答 1

2

该练习意味着将您的更改Database如下：

struct Database {
    int max_rows;
    struct Address *rows;
};

你可以让你的数据库保持全局，但现在它的rows内部必须使用malloc. 当然，为了做到这一点，您需要知道要分配多少行。这就是“接受参数MAX_DATA和MAX_ROWS”的全部内容：添加代码main以解释max_rows参数，将其存储在max_rows您的中，并根据该数字进行struct Database分配。rows

您Database_load的Database_write, 和Database_close需要更改以读​​取和写入max_rows数据库文件。读取大小后负责Database_load分配；将负责调用.rowsmallocDatabase_closefree()rows

于 2013-03-14T05:52:23.647 回答