копировать пустой указатель на другой пустой указатель? - PullRequest
Новая
       64

копировать пустой указатель на другой пустой указатель?

0 голосов
/ 09 мая 2019

Я пытаюсь скопировать пустой указатель на другие индексы другого массива указателей, все работает нормально для символов, но возникает проблема для целых чисел и удваивается

Вот моя структура: 

typedef struct vector_struct {
 size_t e_sz;
 char e_type;

 #define V_INT 1 
 #define V_DOUBLE 2
 #define V_CHAR 3

 unsigned no_e;
 unsigned cur_cap;
 void* e_array;

}* vector_type;
 //for typecasting
  #define vector_int_at(v,i) (*((int*)v_at(v,i)))
  #define vector_double_at(v,i) (*((double*)v_at(v,i)))
  #define vector_char_at(v,i) (*((char*)v_at(v,i)))

  void* v_at(vector_type v, iterator i) {
    if ( v == NULL )
     fail("v_at() Error: no such vector");
    if ( i == iterator_end )
     i = v->no_e -1;
    if ( i < 0 || i >= v->no_e )
     fail("v_at() Error: no such element in this vector");
    return (v->e_array+i); //return element
   }

Я хочу скопировать значение в void* e_array, оно может быть целым, символьным или двойным. 

void v_push_back(vector_type v, void* new_val){
 if ( v->no_e >= v->cur_cap ) {
    /*** need to expand the array ***/
    v->cur_cap += (v->cur_cap)? v->cur_cap : 2;

    v->e_array = check_a(realloc(v->e_array, sizeof(void*) * v->cur_cap));
    //allocate memory

   }
/*** copy new_val into the array at end position (v->no_e) ***/
   printf("%d  %d %p\n",v->e_sz,v->no_e,v->e_array+(v->no_e *v->e_sz));
    memcpy(
          v->e_array +(v->no_e)*v->e_sz,
          new_val,
          v->e_sz
          );

 (v->no_e)++;
}

Я вызвал эту функцию так: 

c1 = 'o'; v_push_back(vc1,(void*)&c1); //for character
for ( i1 = 0 ; i1 < 10 ; i1++ ){ //for integer
    v_push_back(vi1,(void*)&i1);
}

//I call these like
  for ( i1 = 0 ; i1 < vector_size(vi1) ; i1++ )
   printf(" %ld",vector_int_at(vi1,i1));
  printf("\n");

Но кажется, что вещи не работают для целых чисел, но хорошо работают для символов, но для целых чисел значения мусора копируются, как показано: введите описание изображения здесь

Пожалуйста, помогите мне. спасибо

Вот полный проект: https://s3.amazonaws.com/assignment.pointers.vectorc.implementation/C-VectorImplementation1.zip

1 Ответ

1 голос
/ 09 мая 2019

Я вижу несколько проблем,

  1. Чтобы получить i-е местоположение текущего типа вектора, нужно умножить i на v->e_sz. 

    void* v_at(vector_type v, iterator i)
{
    .....
    return ((char*)v->e_array+i*v->e_sz); //return element    
}

  2. Вы выделяете sizeof(void *) то, что вы хотите: 

     v->e_array = check_a(realloc(v->e_array, v->e_sz * v->cur_cap));

  3. Также на данный момент вы сохраняете адрес как int, но вам необходимо сохранить егозначение. 

    void v_push_back(vector_type v, void* new_val){
     if(v->e_type==V_INT){
        //  (v->e_array+(v->no_e))=new_val;
        ((int*)v->e_array)[v->no_e]=*(int*)new_val;
                                    ^------storing its content
     }
}

    Без каких-либо проверок типов должно работать простое memcpy. 

    void v_push_back(vector_type v, void* new_val){
    ....
    memcpy(
          v->e_array +(v->no_e)*v->e_sz,
          new_val,
          v->e_sz
          );
       //   *(char*)(v->e_array+v->no_e)=*(char*)new_val;

     (v->no_e)++;
 }
...