0

これは C を苦労して学ぶことからの問題です。これは C のデータベース管理システムです。私は 3 つの構造を持っています:-

 struct Address {
         int id;
         int set;
         char *name;
         char *email;
 };

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

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

データベース構造を初期化しようとしています。ただし、セグメンテーション違反が発生しています

    void Database_create(struct Connection *conn, int no_of_rows)
    {
      int i = 0;
  conn->db->row_num = no_of_rows; 

      for(i = 0; i < conn->db->row_num; i++) {

      // make a prototype to initialize it
      struct Address addr;
      addr.id = i;
      addr.set = 0;

      // then just assign it
      conn->db->rows[i] = addr;
      }
  }

これらの構造体にメモリを割り当てる別の関数を作成しました。

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

    int number = conn->db->rows;

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

        conn->db->row = malloc(sizeof(*conn->db->row) * number);
        if(!conn->db->row) die("Memory error");

        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");

      return conn;
   }

valgrind は、Database_open() で「サイズ 4 の初期化されていない値の使用」と言っています

ここで私が間違っていることを誰かが提案できますか?

4

1 に答える 1

1

dbinConnectionおよびrowinDatabaseは、初期化されていないポインターです。それらを初期化し、それらが指す構造体のストレージを提供する必要があります。

ポインターではなくメンバーとしてConnection持つように変更することで、おそらくいくつかの動的割り当てを節約できますDatabase

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

データベース行にメモリを割り当てる必要があります

conn->db.row = malloc(no_of_rows * sizeof(*conn->db.row));

Database_create次に、次のようになります

int Database_create(struct Connection *conn, int no_of_rows)
{
    int i = 0;
    conn->db.rows = no_of_rows;
    conn->db.row = malloc(no_of_rows * sizeof(*conn->db.row));
    if (conn->db.row == NULL) {
        /* out of memory */
        return 1;  /* indicate failure to caller */
    }
    for(i = 0; i < conn->db->rows; i++) {
        conn->db.row[i].id = i;
        conn->db.row[i].set = 0;
    }
    return 0; /* indicate success to caller */
}

これは、メモリがすでに割り当てられていることを前提としていることに注意してください。Connection

于 2013-07-17T16:38:09.657 に答える