为什么被调用函数的指针不将值返回给调用函数？

Question

#include<stdio.h>
#include<stdlib.h>
unsigned int *bin(int);
int main(void)
{
  unsigned int n=0,*i=NULL;
  printf("Enter no:");
  scanf("%d",&n);
  i=bin(n);
  printf("Binary no: %d\n",*i);
  return 0;
}
unsigned int *bin(int n)
{
  unsigned int i=0,j=0;
  static unsigned int *result=NULL;
  result=(unsigned int*)malloc(1*sizeof(unsigned int));
  printf("Result=%p\n",result);
  j=(unsigned int)result;
  for(i=(1<<31);i>0;i=(i>>1))
  {
    if(n & i)
    {
      *result=1;
      result++;
    }
    else
    {
      *result=0;
      result++;
    }
}
  result=(unsigned int*)j;
  printf("Result=%p\n",result);
  return result;
}
Output : 
Enter no:6
Address of Result=0x2576010
Address of Result=0x2576010
Binary no: 0

本程序的目的是将十进制数转换为二进制数。主要功能是调用bin()函数将十进制数转换为二进制数。

代码逻辑：- 让我们取无符号整数（32 位），它由 0-31 位组成。要打印无符号整数的二进制表示，从第 31 位开始，检查第 31 位是 ON 还是 OFF，如果是 ON，则打印“1”，否则打印“0”。现在检查第 30 位是 ON 还是 OFF，如果是 ON 打印“1”，否则打印“0”，对从 31 到 0 的所有位执行此操作，最终得到数字的二进制表示。

我很困惑应该分配多少空间来存储 32 位整数。以及如何释放分配给结果的内存。请帮我解决这个代码。

Answer 1

考虑到 1 和 0 的数据类型是 char，您将需要分配 32 字节（至少）的内存来存储对应于 n 中每个位的 1 或 0。此外，我建议使用 int 作为数据类型并分配 32x4（字节）的内存。您的最终代码可能如下所示：

#include<stdio.h>
#include<stdlib.h>
unsigned int *bin(int);
int main(void)
{
  unsigned int n=0,*result =NULL;
  printf("Enter no:");
  scanf("%d",&n);
  result =bin(n);
  printf ("binary representation is: ");
  int i;
  for ( i=0;i<32;i++)
    printf("%d ",result[i]);
  free(result);
  return 0;
}

unsigned int *bin(int n)
{
  unsigned int i=0;
  static unsigned int *result=NULL;
  result=(unsigned int*)malloc(32*sizeof(unsigned int));
  printf("Result=%p\n",result);
  unsigned int* j=NULL;
  j=result;
  for(i=(1<<31);i>0;i=(i>>1))
  {
    if(n & i)
    {
      *j=1;
      j++;
    }
    else
    {
      *j=0;
      j++;
    }
}

  return result;
}

Answer 2

Shubham，我真的认为你走错了方向，因为首先没有十进制或二进制整数之类的东西。让我建议您使用 malloc 和不需要的 static 指针来解决您的问题：

/*
 * return a 32-bit long binary string from 'integer'
 *  this string is dynamically allocated (malloc)
 * return NULL in case of error (so DON'T free it in this case)
 */
char* bin(long integer)
{
 char* pChResult=malloc(33*sizeof(char));
 char* pCh;
 unsigned long i;
 if(pChResult!=NULL) {
   for(pCh=pChResult, i=1<<31; i>0; pCh++, i>>=1) {
    if(i&integer)
     *pCh='1';
    else
     *pCh='0';
    }
   *pCh='\0';
  }
 return(pChResult);
}

你的主菜也会很轻：

int main(void)
{
  unsigned int n;
  char* result;
  printf("Enter no:");
  scanf("%d",&n);
  result = bin(n);
  if(result==NULL)
   {
    fprintf(stderr, "internal error!\n");
    return(-1);
   }
  printf("binary representation is: %s\n", result);
  free(result);
  return(0);
}

Answer 3

32 位整数值强制使用signed long 或unsigned long

在函数bin 中，只要您需要存储结果，就一直使用*result 然后return(result);

主要使用*result 和free(result); 当你完成结果时

为什么不直接unsigned long bin(long);？

最后，你的函数什么都不做！应该类似于void bin(long, char[33]);（不需要malloc）或char* bin(long);（带有malloc）