Question

我运行了以下代码，它在永远运行的while循环中崩溃了。当我调试此代码时，我在*(pointer+cnt)='\0'找到了问题; null字符永远不存在。我不知道如何在这里附加空终止符，以便程序不会崩溃。

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

char* decimal_binary(int);

int main()
{
   int n;
   char *ptr=NULL;

   printf("Enter the number\n");
   scanf("%d",&n);

   ptr=decimal_binary(n);
   //printing out the characters
   while(ptr!='\0')
   {
     printf("%c",*ptr);
     ptr++;
   }
   free(ptr);
   return 0;
}

char* decimal_binary(int n)
{
  int c,d,cnt=0;
  char *pointer=(char*)malloc(8+1);
  if(pointer==NULL)
     exit(EXIT_FAILURE);

  for(c=7;c>=0;c--)
  {
    d=n>>c;
    if(d&1)
        *(pointer+cnt)=1+'0';
    else
        *(pointer+cnt)=0+'0';
    cnt++;
   }
//Null not getting added at the end of this sequence.Hence while loop in    main runs forever.
*(pointer+cnt)='\0';
return pointer;
}

Answer 1

你选择写：

while(ptr!='\0')

这是一种有趣的写作方式：

while (ptr != 0)

或：

while (ptr != NULL)

你打算写的地方：

while (*ptr != '\0')

传统的*(pointer+cnt)撰写方式是pointer[cnt]。

你无法释放增加的指针;您必须释放malloc() - 或calloc()或realloc()或...

返回的内容

保留binary_decimal()返回的值的副本并释放副本（或增加副本并释放ptr中的值）。

您可以使用以下代码中的两个binary_decimal()函数之一：

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

char *decimal_binary(int);

int main(void)
{
    int n;
    char *ptr = NULL;

    printf("Enter the number\n");
    scanf("%d", &n);

    ptr = decimal_binary(n);
    char *cpy = ptr;
    //printing out the characters
    while (*ptr != '\0')
    {
        printf("%c", *ptr);
        ptr++;
    }
    putchar('\n');
    free(cpy);
    return 0;
}

char *decimal_binary(int n)
{
    int cnt = 0;
    char *pointer = (char *)malloc(8 + 1);
    if (pointer == NULL)
        exit(EXIT_FAILURE);

    for (int c = 7; c >= 0; c--)
    {
        int d = n >> c;
        if (d & 1)
            pointer[cnt] = 1 + '0';
        else
            pointer[cnt] = 0 + '0';
        cnt++;
    }
    pointer[cnt] = '\0';
    return pointer;
}

或者：

char *decimal_binary(int n)
{
    int cnt = 0;
    char *pointer = (char *)malloc(8 + 1);
    if (pointer == NULL)
        exit(EXIT_FAILURE);

    for (int c = 7; c >= 0; c--)
        pointer[cnt++] = ((n >> c) & 1) + '0';
    pointer[cnt] = '\0';
    return pointer;
}

这可能会被压缩得更多（甚至更不可读）：

char *decimal_binary(int n)
{
    char *pointer = (char *)malloc(8 + 1);
    if (pointer == NULL)
        exit(EXIT_FAILURE);

    for (int c = 7; c >= 0; c--)
        pointer[7 - c] = ((n >> c) & 1) + '0';
    pointer[8] = '\0';
    return pointer;
}

对于9字节的缓冲区，您可以很好地在main()中分配一个局部变量，并将地址传递给decimal_binary()，这样就不需要使用malloc()和{ {1}}不需要使用免费：

main()

如何将null终止符附加到索引的char指针的末尾

1 个答案: