C语言中关于指针和数组的问题

Question

基本上在下面的代码中，我的最终数组似乎没有 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]);
  }

Answer 1

以下是我在您的代码中观察到的错误。
您写道

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

现在无需执行此操作 pointer1 = &temp，任何数组的名称本身就是一个指针。
因此你可以简单地做

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

但是等等！

pointer1 有足够的空间来存储 temp[] 的内容吗？在您的代码中，您没有为pointer1分配任何空间，这可能会使您的程序崩溃。

正确的方法是

  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()，将pointer1指向的内存填充为null。只是良好而健康的做法。

那么你

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

再说一遍，pointer1有足够的空间容纳pointer3的内容吗？您是否拥有pointer1 + sizeof(temp)指向的内存区域？它也会使你的程序崩溃。
现在您可以使用 realloc() 或在早期使用 malloc() 为指针1 分配更大的空间。

为什么在这里使用sizeof（临时数组）？你不认为它应该有pointer3中的字节数吗？

最后在function1()的定义中

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

array2做了什么？没有什么！那么应该将其删除。
要返回，只需使用

return temp2;

，这意味着指针2也是无用的。

希望有帮助。

Answer 2

目前还不清楚您想要做什么，但我认为这就是您想要的。我对其进行了评论以显示它在做什么。

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

Answer 3

我似乎无法理解你的代码。问题很简单，但是代码无法运行。
接下来是这段代码
memcpy (pointer1 + sizeof(temp),pointer3,sizeof(temp array)

我猜你希望最终的数组包含 { a,b,c,d,e,1,2,3, 4,5}。
好吧，我建议您检查 K&R 中的指针和数组，然后再次尝试此问题。