Question

我尝试将负十进制数转换为二进制数，此代码完全适用于我的计算机，但代码不能用于另一台计算机。

我没有得到它的可能性。我的代码有什么问题？

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

void decTobin(int dec, int s)
{
    int b[s], i = 0;

     while (dec >= 0 && i != s - 1) {
         b[i] = dec % 2;
         i++;
         dec /= 2;
     }

     int j = i;

     printf("%d", dec);

     for (j = i - 1; j >= 0; j--) {
         if (b[j] == NULL)
             b[j] = 0;

         printf("%d",b[j]);
     }
}

void ndecTobin(int dec, int s)
{
    int b[s], i = 0, a[s], decimal, decimalvalue = 0, g;

    while (dec >= 0 && i != s-1) {
        b[i] = dec % 2;
        i++;
        dec /= 2;
    }

    int j = i;

    printf("%d",dec);

    for (j = i - 1; j >= 0; j--) {
        if (b[j] == NULL)
            b[j] = 0;

        printf("%d",b[j]);
    }

    printf("\n");

    a[s - 1] = dec;

    for (j = s - 2; j >= 0; j--) {
        a[j] = b[j];
    }

    for (j = s - 1; j >= 0; j--) {
        if (a[j] == 0)
            a[j] = 1;
        else
            a[j] = 0;

        printf("%d",a[j]);
    }

    for (g = 0; g < s; g++) {
        decimalvalue = pow(2, g) * a[g];
        decimal += decimalvalue;
    }

    decimal = decimal + 1;
    printf("\n%d\n", decimal);
    decTobin(decimal, s);
}

int main()
{
    int a, b;

    printf("enter a number: ");
    scanf(" %d", &a);
    printf("enter the base: ");
    scanf("%d", &b);

    ndecTobin(a, b);
}

Answer 1

decimal和int b[s]未初始化。

通过不将decimal初始化为0，它可能在一天的机器上具有值0，否则会产生完全不同的结果。

void decTobin(int dec, int s) {
  //  while loop does not set all `b`,but following for loop uses all `b`
  // int b[s], i = 0;  
  int b[s] = { 0 }; // or int b[s]; memset(b, 0, sizeof b);
  int i = 0; 
}

void ndecTobin(int dec, int s) {
  int b[s], i = 0, a[s], decimal, decimalvalue = 0, g;
  decimal = 0;
  ...
  decimal += decimalvalue;
}

小点：

1）if (b[j] == NULL) b[j] = 0;很奇怪。 NULL最适合用作指针，但代码正在将b[j]，int与指针进行比较。此外，由于NULL通常具有0的算术值，因此代码看起来像if (b[j] == 0) b[j] = 0;。

2）decTobin()难以理解。它当然仅适用于非负dec和s。候选人简化：

void decTobin(unsigned number, unsigned width) {
  int digit[width];
  for (unsigned i = width; i-- > 0; ) {
    digit[i] = number % 2;
    number /= 2;
  }

  printf("%u ", number);  // assume this is for debug

  for (unsigned i = 0; i<width; i++) {
    printf("%u", digit[i]);
  }
}

Answer 2

看起来您只是将数字打印为二进制表示。如果是这样，这个版本就可以了。

void print_binary(size_t n) {
   /* buffer large enough to hold number to print */
   unsigned buf[CHAR_BIT * sizeof n] = {0};
   unsigned i = 0;

   /* handle special case user calls with n = 0 */
   if(n == 0) {
      puts("0");
      return;
   }
   while(n) {
      buf[i++] = n % 2;
      n/= 2;
   }

   /* print buffer backwards for binary representation */
   do {
     printf("%u", buf[--i]);
   } while(i != 0);
}

如果你不喜欢缓冲区，你也可以使用这样的递归来实现：

void using_recursion(size_t n)
{
   if (n > 1)
      using_recursion(n/2);

   printf("%u", n % 2);
}

另一种方法是首先打印评估最重要的位。然而，这引入了在下面的代码中跳过的前导零的问题。

void print_binary2(size_t n) {
   /* do not print leading zeros */
   int i = (sizeof(n) * 8)-1;
   while(i >= 0) {
      if((n >> i) & 1)
         break;
      --i;
   }

   for(; i >= 0; --i)
      printf("%u", (n >> i) & 1);
}

Answer 3

不同的OS /处理器组合可能导致C编译器以不同的字节数存储各种数字变量。例如，当我第一次学习C（80368，DOS 5上的Turbo C）时，int是两个字节，但现在，在64位Linux上使用gcc，我的int显然是四个字节。您需要包含一些方法来考虑变量类型的实际字节长度：一元运算符sizeof(foo)（其中foo是一种类型，在您的情况下，int）返回一个可以使用的无符号整数值确保你做正确的位移数。

将负十进制数转换为二进制数

3 个答案: