帮我检查一下这段C语言.

Question

用指针法把4插入1,3,5,7,9数列中.我得到的结果是134790,不知错在那里?
#include <stdio.h>
void main(){
int *p,i,t,a[6]={1,3,5,7,9},b=4,l,h,j;
p=a;
for(i=1;i<6;i++,p++)
if (*p>b) break ;
t=b;*p=b;h=*p;
for(j=6;j<i;j--,p++)
l=*p,*p=h;h=l;
p=a;
for(i=1;i<7;i++,p++)
printf("%d",*p);
}

Answer 1

其实你这个程序错误的地方非常多(LZ不要生气啊，我初学的时候还写不出这样的呢)

先说你的程序错在哪里，主要错在3个地方：
错误1. 逻辑设计的错误（这个比较严重，稍后再说）
错误2. 在下列语句：
for(i=1;i<6;i++,p++)
if (*p>b) break ;
在这个语句执行后，程序中p已经指向了数组a的第3个元素：a[2] = 5
所以你随后的“*p=b;”这一句，其实就是将p指向的元素的内容赋值为b(也就是4)，
所以a[2]的内容就变成了4，所以结果就变成了134790
错误3. 数组的插入算法错误
下面这个语句是无法实现数组的插入的：
for(j=6;j<i;j--,p++)
l=*p,*p=h;h=l;

正确做法：
正确的逻辑应该是这样的：
本题的要求"用指针法把4插入1,3,5,7,9数列中"，其实他的实现分为以下步：
1. 定位(找到正确的插入位置，并记录) //这个过程，你已经做了

2. 数组正向遍历，找到数组尾 //由于本题的数组是固定的，所以直接可以用数组下标a[5]，就不需要遍历了：）

3. 数组插入
//这一步是关键，应该按照如下方法实现：
起始位置：定义两个指针(假设为*p,*q)，一个p指向数组尾元素(也就是a[5])，一个q指向p的前一个元素
数组逆向遍历：这里的遍历还包含赋值的意思，即每次将q指向地址内的值赋给*p，赋值完成后，p,q依次进行--操作，指向前一位置
上述逆向遍历直到找到先前记录的"定位"的位置为止，也就是a[2]
最后再将相关地址a[2]的值置位为4。

方法和思路远比贴给你一段代码重要，希望我讲明白了：）

Answer 2

#include <stdio.h>
void main(){
int *p,i,t,a[6]={1,3,5,7,9},b=4,l,h,j;
p=a;
l=6;
for(i=0;i<l;i++)
{if (*(p+i)>b)
for(j=5;j>=i;j--)
{*(p+j)=*(p+j-1);
l--;
}

}

*(p+i-1)=b;
for(i=0;i<6;i++)
printf("%d",*(p+i));
}

Answer 3

就是插入一个值弄了这么多变量，代码好乱，看的头晕！！

将你的代码该了一下：

#include <stdio.h>
void main(){
int *p,i,j,b=4,a[6]={1,3,5,7,9}; /*这里初始化数组是从0下标开始的*/
for(p=a,i=0;i<6;i++) /*你用的时候却是从下标为1开始*/
if (*(p+i)>b) break ; /*查找要插入的位置i*/
for(p=a,j=4;j>=i;j--) /*将i以后的数据都往后移动从最后一个开始移动*/
*(p+j+1)=*(p+j);
p=a;/*让指针指向数组头部*/
*(p+i)= b; /*将上面找到的下标为i的值替换为要替换的值b=4*/
for(p=a,i=0;i<6;i++)
printf("%d",*(p+i));
}

Answer 4

t=b;*p=b;h=*p; 这句有毛病，这样做就把5用4覆盖了
可如下修改
#include <stdio.h>
void main(){
int *p,i,t,a[6]={1,3,5,7,9},b=4,l,h,j;
int*q;
p=a;
for(i=1;i<6;i++,p++)
if (*p>b)
break ;
q=&a[4];
for(;q>=p;q--)
*(q+1)=*q;
*p=b;
p=a;
for(i=1;i<7;i++,p++)
printf("%d",*p);

}

Answer 5

写的太混乱了
我照你那代码修改了些：
/*在数组a[6]中按序插入一数*/
#include "stdio.h"
main()
{
int *p,a[6]={1,3,5,7,9},temp;
scanf("%d",&a[5]);
p=&a[5];
for(p=&a[5];p>=a;p--)
{
if(*p>*(p-1))
break;
else
{
temp=*p;
*p=*(p-1);
*(p-1)=temp;
}

}
p=a;
for(p=a;p<a+6;p++)
printf("%d ",*p);
}

运行如下：输入 4
输出：1 3 4 5 7 9

Answer 6

for(i=1;i<6;i++,p++)
if (*p>b) break ;
这里，你找到了应该把4放在的位置，现在你要做的是把这个位置上的5先保存起来，否则4放上去就把5覆盖了。

t=b;*p=b;h=*p;
而这里，你把4保存在了t上，这是毫无意义的，应该是这样：
t=*p;*p=b;

下面你要把4以后的元素都后移一个位置，并保持它们原来的顺序。但是你做错了太多。
1、for后面要是有多个语句，需要用大括号括起来，才能一起执行。
2、你的p现在指向的位置应该是最后一个元素，而不是不变就行的
for(j=6;j<i;j--,p++)
l=*p,*p=h;h=l;
应该是这样的：
p=a+5;
for(j=6;j>i;j--,p--)
{
t=*p;*p=t;
}