刚学完c。求教，自写一小程序不知道错哪里。一直遵循努力自己解决问题的原则。但。。

。风中劲草。

#include<stdio.h>
                                                                  //我的目的是输入i个学生成绩（每个学生4门成绩）；然后一个一个的输出各学生的平均
void average(float *p,int x,int y);        // 成绩，然后输出所有学生的所有成绩。
void search(float (*p)[4],int x,int y);   //(×p)是指向一维数组的指针变量                       

int main()
{       
        int j=0;
        int i=0;
        int student=0;
        float score[i][4];
        float (*p)[4]=NULL;

        printf("请输入学生人数：\n");
        scanf("%d",&student);

        printf("你输入的学生人数是：%d\n",student);
       
        printf("请输入学生成绩：\n");
       
        p=score;
        for(i=0;i<student;i++)
                for(j=0;j<4;j++)
                           scanf("%f",*(p+i)+j);                   //&(*(*(p+i)+j)))
       
        average(*p,student,4);
        search(p,student,4);

        return 0;
       
}
       
void average(float *p,int x,int y)
{
        float aver=0;
        int i=0;
        int j=0;
        float sum=0;

        for(i=0;i<x;i++)
          {     
                sum=0;
                 for(j=0;j<y;j++)
                  {
                        sum=sum+*p;
                  }
                aver=sum/y;
                printf("\n第%d个同学的平均成绩为：%5.2f\n",i,aver);
          }
}


void search(float (*p)[4],int x,int y)
{
        int i=0;
        int j=0;
        float fenshu=0;
       
        for(i=0;i<x;i++)
          {       
                printf("\n第%d个同学各科成绩：\n",i);
                for(j=0;j<y;j++)
                  {
                           fenshu=*(*(p+i)+j);
                        printf("%5.2f",fenshu);       
                  }
          }       
       
}

Kasiotao

回复 1# 。风中劲草。
没仔细看你的代码，不知道你具体有什么问题，但是很明显有一点错误，我看你申请数组的时候score[4]，我想你的意图应该是根据学生的总数来动态申请数组大小吧，4应该是一个学生有4门成绩？
那你这样做是不对的，因为你的i初始化为0，那么你可以理解为实际上申请的数组大小就是一个1行4列的矩阵，假设你的学生总数>1,那么你从*p地址开始填写内存的时候，就会溢出，也就时你填写了你并没有申请的地址空间，这是严重错误的。一般动态申请数组使用malloc函数，你可以去查一下。

   

cuusrname

float score[i][4];  // 声明数组长度不能用变量

void search(float (*p)[4],int x,int y);   //(×p)是指向一维数组的指针变量
通常用2级指针 float **p

看编译器的提示改

dxyf1524

        float score[4];
        float (*p)[4]=NULL;
你换个方法做个吧，不知道你的编译怎么通过的，
用指针数组比较复杂，考虑简单的方法吧
实在不行，用结构体

zhaohongjian000

To 3楼、4楼：怎么通过的？当然是用gcc啊，不然这么神奇的语法。。。
To 2楼：float score[0][4];  声明的是0行4列的数组，也就是没分配任何空间。

To 楼主：
过去，C语言中是没法声明一个变量长度的数组的。后来在C99标准中添加了这个特性，而gcc默认情况下（不是用-std=c99）就支持这个特性。当然，长度为0的数组也是一个gcc的扩展。下面两个链接作为参考：

http://gcc.gnu.org/onlinedocs/gcc/Variable-Length.html
http://gcc.gnu.org/onlinedocs/gcc/Zero-Length.html


需要注意的是，虽然你可以声明一个变量长度的数组，但是这个数组的长度不会随着这个变量的变化而变化，而是在声明之后就固定下来了。所以，下面的语句：

1.          int j=0;
   
2.         int i=0;
   
3.         int student=0;
   
4.         float score[i][4];

复制代码

score是个长度为0的数组，它实际上不占任何空间，所以向里面存东西是错误的，会溢出。

正确的做法：
1. 取得学生人数后再声明score，不是一个好方法。
2. 学习使用malloc/free函数，自己分配内存。

ntwarren

还有：

你定义P为指针：float (*p)[4]=NULL;
然后你这里用：scanf("%f",*(p+i)+j);
你想想p为地址，*(p+i）之后还是地址吗？不是地址你还可以用来放在scanf中做参数吗？

Kasiotao

回复 5# zhaohongjian000
额，大早起来脑袋有点晕，索引开始和数组长度居然弄晕了，你说的是对的，其实没有分配任何空间。
这个地址确实是可以用的，但是写到哪里就不得而知了，会损坏什么关键数据也不得而知了。
to楼上，*(p+i)当然是地址，因为这里p本身是一个二级指针，但是是一个没有被内核分配给该进程的地址，换句话说就是在操作一个非法空间，gcc编译也是可以通过的，甚至得到结果都有可能正确，但是做法显然是错误的

   

ntwarren

回复 7# Kasiotao


    不听我的话，你还是自己写个程序试一下吧，你这里定义p就是一级指针。不想解释了。

zhaohongjian000

回复 8# ntwarren


    我觉得你才需要写个程序试一下。

ntwarren

回复 9# zhaohongjian000

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

int main(){
    int a[8][10] = {4};
    int (*p)[4];
    p = a;
   

    printf("%d\n",*((int*)p));
    printf("%d\n",a[0][0]);

}