c语言问题为啥没错误却运行结果与预计不相符

Question

/********程序功能*********/
/********输入某十进制整数，将其转换为二进制数并输出在屏幕上*********/
#include "stdio.h"
main()
{
int k=0;
int x, r,b[99],d[99],p,q;
int m,n,j;
int c[99],i,s;
printf("please input a single number:\n");
scanf("%d",&x);
do
{
r=x%2;
b[k++]=r;
x/=2;
}while(x);
p=0;
q=k-1;
for(;p<k;p++,q--)
d[q]=b[p];
for(p=0;p<k;p++)
b[p]=d[p];

m=k%4;
n=4-m;
j=k+n;
for(i=0;i<n;i++)
c[i]=0;
for(s=0;i<j;i++,s++)
c[i]=b[s];
for(i=0;i<j;i++)
{if((i+1)%4==0)
printf("\b");
printf("%c",c[i]);}
printf("\n");
}

Answer 1

　　其实，十进制整数转二进制最简单的办法我觉得是利用移位操作，和按位与，因为数据在计算机里是二进制的形式。当然，如果是大数的问题就不好这样做，我觉得还是写个通用的函数方便点，然后递归，迭代，都可以。看你的输入应该不是大数。
　　然后，看你复杂的for循环处理我看了半天感觉是想要处理输出格式。话说，你的代码我已经无力吐槽了。
　　例如：
　　for( ; p < k; p++, q--)
　　d[q]=b[p];
　　for( p = 0; p < k; p++)
　　b[p]=d[p];
　　这一段我想，你是要把顺序倒回来。
　　这里你可以用
　　for（i= 0； i < k / 2; i++ ）
　　{
　　temp = b[i];
　　b[i] = b[ k - i - 1];
　　b[k - i - 1] = temp;
　　}
　　这样子看的懂点。
　　然后
　　m=k%4;
　　n=4-m;
　　j=k+n;
　　这一段你应该是想要取4的整倍数这样写也是没问题的
　　不过，要是我的话，会这样写 ceil(n / 4.0) * 4，用数学函数库里的取上底函数。
　　还有就是这一段了：
　　for(i=0;i<j;i++) // 这里我觉得既然要利用数组c输出，那么还是从0到j置零的好。
　　c[i]=0;

　　i = 0; // 这里你 i没有重新赋值，倒置下面for循环出问题了。
　　for(s=0;i<j;i++,s++)
　　c[i]=b[s];
　　for(i=0;i<j;i++)
　　{
　　if( i && i % 4 == 0 ) // 四个一组。
　　printf("\t"); //你确定你这里要的不是\t而是 \b？
　　printf("%d",c[i]);// 最后这里看你的题目应该是%d输出，而不是%c
　　}
　　以下是完整代码：
　　#include "stdio.h"
　　#include <math.h>
　　int main()
　　{
　　int k=0;
　　int x, r,b[99],d[99],p,q;
　　int m,n,j;
　　int c[99],i,s;
　　printf("please input a single number:\n");
　　scanf("%d",&x);
　　do
　　{
　　r=x%2;
　　b[k++]=r;
　　x/=2;
　　}while(x);
　　p=0;
　　q=k-1;
　　int temp;
　　for(i= 0; i < k / 2; i++ )
　　{
　　temp = b[i];
　　b[i] = b[ k - i - 1];
　　b[k - i - 1] = temp;
　　}
　　j = ceil( k / 4.0) * 4;

　　for(i=0;i<j;i++)
　　c[i]=0;

　　i = 0;
　　for(s=0;i<j;i++,s++)
　　c[i]=b[s];
　　for(i=0;i<j;i++)
　　{
　　if( i && i % 4 == 0 )
　　printf("\t");
　　printf("%d",c[i]);
　　}
　　printf("\n");
}
附上一段代码
#include "stdio.h"
#include <math.h>
int main()
{
int k;
const int bit = 31; // 假设四个字节。
printf("please input a single number:\n");
while( scanf("%d",&k) )
{
int i = 0;
do
{
printf("%d",(k & 0x80000000) >> bit); // 从高位开始输出。
i++;
if( i % 8 == 0 )
{
printf(" "); //格式控制。
}
k <<= 1;
}while( i != 32 );
putchar('\n');
}
}

Answer 2

何必如此麻烦，十几行代码就能解决的事情
# include <stdio.h>
main()
{
int k=0, x, b[99]={0};
printf("Please enter a number:\n");
scanf("%d",&x);
if (x == 0) //如果x==0,后面将不会有输出,所以先输出一个0
printf("0");
for(; x; x/=2,k++) //b[0]是二进制右起第一位,b[1]是第二位,以此类推
if(x%2) b[k]=1; //如果x%2==1，二进制的右起第k+1位变成1
for(k--; k>=0; k--) //在第一个k--之前,k是二进制数的位数，先减1才能正确输出
printf("%d", b[k]); //将二进制数的每一位输出
printf("\n");
}
在VC++6.0中运行，结果完全正确。

希望对你有帮助！

追问

对是对了，但我想要的输出形式是每4个二进制位中间用空格隔开高位不足补零。总之还是谢谢你

追答

你那样也简单，在第二个for循环之前算出 k%4，然后相应地补0。中间空格隔开的问题，在第二个for循环内加上检测k%4是否等于0，如果等于则输出一个空格即可。代码很简单，自己写吧。

参考资料：自己写的

Answer 3

for(;p<k;p++,q--)
d[q]=b[p];
for(p=0;p<k;p++)
b[p]=d[p];
//这样的代码，真的能把人弄晕 ，你不适当加点注释，都不知道有些语句的目的是什么

Answer 4

楼上说的是