0

特定のキーが見つかった場合、その値をインクリメントしようとしています。何らかの理由keys:valuesで、ハッシュ テーブルからすべてをダンプすると、(ポインター) アドレスを取得し続けます。

Output
      a: 153654132 // should be 5
      b: 1 
      c: 153654276 // should be 3
      d: 1 
      e: 1 
      f: 153654420 // should be 3


int proc()
{
    struct st stu;
    gpointer ok, ov;

    //... some non-related code here

    if(!g_hash_table_lookup_extended(table, key, &ok, &ov)){
        stu.my_int = g_malloc(sizeof(guint));
        *(stu.my_int) = 0;
        g_hash_table_insert(table, g_strdup(key), GINT_TO_POINTER(1));
    }else{
        stu.my_int = g_malloc(sizeof(guint));
        *(stu.my_int)++;
        g_hash_table_insert(table, g_strdup(key), stu.my_int);
    }   
}

どんなアイデアでも大歓迎です。

4

2 に答える 2

1

++ は * よりも優先されます。したがって、ポインターが指すものではなく、ポインター stu.my_int 自体をインクリメントしています。おそらく (*stu.my_int)++ が必要です。

于 2010-03-08T03:49:39.137 に答える
1

8行で7つのエラーを数えました：

/* 1) ov, the actual value read from the hash table, is never used */
if(!g_hash_table_lookup_extended(table, key, &ok, &ov)){
    stu.my_int = g_malloc(sizeof(guint));
    *(stu.my_int) = 0;
    /* 2) stu.my_int not freed: memory leak */
    g_hash_table_insert(table, g_strdup(key), GINT_TO_POINTER(1));
    /* 3) g_strdup(key) not freed: memory leak */
}else{
    stu.my_int = g_malloc(sizeof(guint));
    /* 4) stu.my_int not freed: memory leak */
    /* 5) stu.my_int not initialized: stu.my_int contains junk */
    *(stu.my_int)++;
    /* 6) incrementing a pointer, not the value pointed by */
    g_hash_table_insert(table, g_strdup(key), stu.my_int);
    /* 7) g_strdup(key) not freed: memory leak */
}

これが私がそれを書く方法です（テストされていません）：

gpointer ov;
gint value;

/* ... */

if(g_hash_table_lookup_extended(table, key, NULL, &ov)) {
    value = GPOINTER_TO_INT(ov);
} else {
    value = 1;
}

g_hash_table_insert(table, key, GINT_TO_POINTER(value));
于 2010-03-09T14:43:27.903 に答える