C вопрос, связанный с указателями и массивами - PullRequest
Новая
       3

C вопрос, связанный с указателями и массивами

2 голосов
/ 24 июня 2010

По сути, в приведенном ниже коде мой окончательный массив не содержит содержимого функции function1 ().Есть идеи, почему я не могу заставить это работать?Спасибо. 

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

  unsigned char *function1() 
 {
  unsigned char array2[] = { 0x4a,0xb2 };
  return (array2 );

  }

   main()

     {
unsigned char temp[] = { 0xaa, 0x0b, 0x03,0x04,0x05,0x06,0x07,0x08,0x09 };
unsigned char x[2];
unsigned char *packet;
int pkt_len;
pkt_len = sizeof(temp) + sizeof(x);
packet = (unsigned char*) malloc ( pkt_len +1);

memset( packet, 0x00, pkt_len +1);
unsigned char *pointer1 = malloc ( sizeof(temp) + 1);

memset( pointer1, 0x00, sizeof(temp) +1);
memcpy (pointer1, temp, sizeof(temp) );

memcpy (packet, pointer1, sizeof(temp) );
printf("\nPacket before copy is 0x%x\n", packet[8]);

    unsigned char *array2 = malloc ( sizeof (x) + 1)  ;
    array2 = (char *)function1();
printf("\nArray2 is 0x%x\n", array2[0]);
    memcpy (packet + sizeof(temp), array2, sizeof(x) );
printf("After copy, Packet contents are 0x%x\n", packet[9]);
  }

Ответы [ 3 ]

2 голосов
/ 24 июня 2010

Ниже приведены ошибки, которые я заметил в вашем коде.Вы написали 

  char temp [] = { a,s,d,f,g,h};
  char * pointer1, *array1;
  pointer1 = &temp;  
  memcpy (array1, pointer1, sizeof( temp) );

Теперь нет необходимости делать это pointer1 = &temp, имя любого массива само по себе является указателем.Следовательно, вы можете просто сделать 

<code>
  char temp [] = { a,s,d,f,g,h};
  char *pointer1;
  memcpy (pointer1, temp , sizeof( temp) );

Но подождите!

Достаточно ли указателя 1 для хранения содержимого temp []?В вашем коде вы не назначили пространство для pointer1, что может привести к сбою вашей программы.

Правильный способ сделать это - 

<code>
  char temp [] = { 'a','s','d','f','g','h'};
  char *pointer1 = malloc( sizeof(char) * (sizeof( temp) + 1) );
  memset( pointer1, 0x00, sizeof( temp) + 1 );
  memcpy (pointer1, temp , sizeof( temp) );

здесь, прежде чем копировать любое значение в pointer1, мы убедились, что на нем достаточно места.

Нет необходимостидля приведения значения возврата malloc ().В sizeof( temp) + 1 1 добавлен нулевой символ.Затем мы сделали memset (), который заполнил память, на которую указывает указатель 1, нулевым.Просто хорошая и здоровая практика.

Тогда вы 

memcpy ( pointer1 + sizeof(temp), pointer3, sizeof ( the temp array)

Опять же, у pointer1 достаточно места для содержимого pointer3?У вас есть область памяти, обозначенная pointer1 + sizeof(temp)?Это тоже приведет к краху вашей программы.Теперь вы либо используете realloc(), либо назначаете больший пробел указателю1 с помощью malloc() на более ранней стадии.Почему sizeof ( the temp array) здесь?Вам не кажется, что в указателе 3 должно быть количество байтов?

Наконец, в определении function1()

char *pointer2, *array2;
 // Now i need to have pointer2 point to contents of array2.
 pointer2 = &temp2;
 return pointer2

Что делает array2?Ничего такого!Тогда это должно быть удалено.Для возврата просто используйте 

return temp2;

, что означает, что pointer2 также бесполезно.

Надеюсь, это поможет.

0 голосов
/ 24 июня 2010

Не могу понять ваш код. Проблема проста, но код не работает. Далее этот кусок кода memcpy ( pointer1 + sizeof(temp), pointer3, sizeof ( the temp array)

Полагаю, вы хотите, чтобы конечный массив содержал {a, b, c, d, e, 1,2,3,4,5}. Ну, я бы посоветовал вам проверить указатели и массивы из K & R, а затем попытаться снова решить эту проблему.

0 голосов
/ 24 июня 2010

Не совсем понятно, что именно вы пытаетесь сделать, но я думаю это то, что вы ищете.Я прокомментировал это, чтобы показать, что он делает. 

void function1(char *dest, size_t len);

int main() 
{
  /* Allocate an array 'temp' */
  char temp[] = { 'a', 's', 'd', 'f', 'g', 'h' };

  /* Allocate an array 'array1', the same size as 'temp' */
  char array1[sizeof temp];

  /* Copy the contents of 'temp' into 'array1' */
  memcpy(array1, temp, sizeof temp);

  /* Call a function to copy new contents into 'array1' */
  function1(array1, sizeof array1);

  return 0;
}

void function1(char *dest, size_t len)
{
  char temp2[] = { 1, 2, 3, 4, 5 };

  /* Determine how much to copy - the _minimum_ of 'len' and 'sizeof temp2' */
  if (len > sizeof temp2)
  {
      len = sizeof temp2;
  }

  /* Copy contents of 'temp2' into 'dest' */
  memcpy(dest, temp2, len);
}
...