3

Learn C The Hard WayでCスキルを磨いています。現在、17回目の演習を行っています。

私は「エクストラクレジット」の部分をやっています。そのページに記載されているデータベースコードを作成して、

「MAX_DATAとMAX_ROWSのパラメーターを受け入れるようにコードを変更し、それらをDatabase構造体に格納し、それをファイルに書き込んで、任意のサイズのデータ​​ベースを作成します。」

そこで、#defineディレクティブをコメントアウトし、AddressとDatabaseの構造体を次のように変更しました。

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

struct Database {
    int MAX_DATA;
    int MAX_ROWS;
    struct Address *rows;
};

そのため、ユーザーからMAX_DATAパラメーターとMAX_ROWSパラメーターを取得して、ユーザーの好みに応じてデータベースを作成できます。私がコードで変更した他のものは-

Database_create関数:

void Database_create(struct Connection *conn, int MAX_DATA, int MAX_ROWS)
{
        int i = 0;
        conn->db->MAX_DATA = MAX_DATA;
        conn->db->MAX_ROWS = MAX_ROWS;
        conn->db->rows = malloc(sizeof(struct Address) * MAX_ROWS);
        for(i = 0; i < MAX_ROWS; i++) {
                struct Address addr = {.id = i, .set = 0};
                conn->db->rows[i] = addr;
        }
}

「4バイトの無効な読み取り」を取得している部分で、SegFaultは次のとおりです。

void Database_set(struct Connection *conn, int id, const char *name, const char *email)
{
    struct Address *addr = &conn->db->rows[id];
    int MAX_DATA = conn->db->MAX_DATA;
    if(addr->set) die("Already set, delete it first");
    /* This if statement gives error for addr->set */
    addr->set = 1;
    addr->name = malloc(sizeof(char) * MAX_DATA);
    addr->email = malloc(sizeof(char) * MAX_DATA);
    // WARNING: bug, read the "How To Break It" and fix this
    char *res = strncpy(addr->name, name, MAX_DATA);
    // demonstrate the strncpy bug
    if(!res) die("Name copy failed");

    res = strncpy(addr->email, email, MAX_DATA);
    if(!res) die("Email copy failed");
}

このコードスニペットがすべてではないことは知っていますが、ここにコード全体を貼り付けることはできません。だから、私はここにそれを投稿しました:http: //pastebin.com/EbKShT3r'c 'オプションを使用して最初の実行のためのデータベースを作成して書くことができます。ただし、「s」オプションを使用してエントリを追加すると、セグメンテーション違反が発生します。

編集:それで、最後に、@ WhizCraigによって与えられたソリューションでこのプログラムを動作させました。しかし、メモリを解放するために、これが私が試していることです:

void Database_close(struct Connection *conn)
{
    int i;
    if (conn) {
        int MAX_ROWS = conn->db->MAX_ROWS;
        for (i=0; i<MAX_ROWS; i++) {
            struct Address *row = conn->db->rows+i;
            if (row->set) {
                free(row->name);
                free(row->email);
            }
        }
        free(conn->db->rows);
        if(conn->file) fclose(conn->file);
        if(conn->db) free(conn->db);
        free(conn);
    }
}

そして、Valgrindでエラーが発生し、メモリリークしています。上記のコードのエラーがわかりません。ただし、主な問題は解決されたようです:)

4

2 に答える 2

1

Rohitがここに投稿したコードhttp://pastebin.com/MvLXkDCzを変更して、メモリリークを修正しました。エラーなしでValgrindを実行します。関連する変更は以下に掲載されています。

void Database_close(Connection *conn)                                                 
{                                                                                     
  size_t i;                                                                           
  if(conn) {                                                                          
    if(conn->db && conn->db->rows) {                                                  
      for(i = 0; i < conn->db->max_rows; i++) {                                       
        Address *cur = conn->db->rows[i];                                             
        free(cur->name);                                                              
        free(cur->email);                                                             
        free(cur);                                                                    
      }                                                                               
      free(conn->db->rows);                                                           
    }                                                                                 
    if(conn->file) fclose(conn->file);                                                
    if(conn->db) free(conn->db);                                                      
    free(conn);                                                                       
  }                                                                                   
}   

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

完全なプログラムはここにあります:https ://github.com/sookoor/Learn-C-the-Hard-Way/blob/master/ex17.c

于 2014-06-18T16:33:27.797 に答える
0
struct Address {
    int id;
    int set;
    char *name;
    char *email;
};

struct Database {
    int MAX_DATA;
    int MAX_ROWS;
    Address **rows; // USE ARRAY OF POINTERS
};

void Database_create(Connection *conn, int MAX_DATA, int MAX_ROWS)
{
        int i = 0;
        conn->db->MAX_DATA = MAX_DATA;
        conn->db->MAX_ROWS = MAX_ROWS;
        conn->db->rows = (Address**)malloc(sizeof(Address*) * MAX_ROWS);

        for(i = 0; i < MAX_ROWS; i++) {
                conn->db->rows[i] = (Address*)malloc(sizeof(Address));
                conn->db->rows[i]->id = i;
                conn->db->rows[i]->set = 0;
        }
}

void Database_set(Connection *conn, int id, const char *name, const char *email)
{
    if (!(conn && conn->db && conn->db->rows && conn->db->rows[id])) return;

    Address *addr = conn->db->rows[id];
    int MAX_DATA = conn->db->MAX_DATA;

    if(addr->set == 0) die("Already set, delete it first");

    addr->set = 1;
    addr->name = malloc(sizeof(char) * MAX_DATA);
    addr->email = malloc(sizeof(char) * MAX_DATA);

    char *res = strncpy(addr->name, name, MAX_DATA);
    if(!res) die("Name copy failed");

    res = strncpy(addr->email, email, MAX_DATA);
    if(!res) die("Email copy failed");
}

void Database_close(Connection *conn)
{
        size_t i;
        if(conn) {
                if (con->db && conn->db->rows) {
                        for (i = 0; i < conn->db->MAX_ROWS; i++) {

                                Address *cur = conn->db->rows[i];
                                free(cur);

                        }
                }

                if(conn->file) fclose(conn->file);
                if(conn->db) free(conn->db);
                free(conn);
        }
}
于 2013-03-29T11:57:30.787 に答える