可以在main函数内完成排序，但是当我想把排序单独用一个函数sort处理时，程序老是出

技术小菜鸟

自学C到现在，这是我写的代码量最多的一段了，我可以在main函数内完成排序，但是当我想把排序单独用一个函数sort处理时，程序老是出错，向大神们求教啊！（我看了错误提示自己估计是红色区域出错了，不过自己不清楚原因）

# include <stdio.h>
# include <malloc.h>

void sort(struct student * p, int l)
{
        struct student t;
        int i, j;
    for(i = 0; i <= l-1; ++i)
        {
                for (j = 0; j <= l-1-i; ++j)
                     if (p[j].score < p[j+1].score)
                                 t = p[j];
                             p[j] = p[j+1];
                                 p[j+1] = t;
        }
}

struct student
{
        int num;
        char name[100];
        char sex;
        float score;
};

int main (void)
{
        int i;
        int len;
        struct student * pArr;

        printf("请输入学生个数：");
        scanf("%d", &len);
        pArr = (struct student *)malloc(len * sizeof(struct student));

    for (i = 1; i <= len; ++i)
        {
            printf("请输入第%d个学生信息:\n", i);
            scanf("%d %s %c %f", &pArr.num, pArr.name, &pArr.sex, &pArr.score);
            printf("\n");
        }

        sort(pArr, len);

    for (i = 1; i <= len; ++i)
        {
              printf("输出的是成绩第%d名学生信息:\n", i);
            printf("%d %s %c %f", pArr.num, pArr.name, pArr.sex, pArr.score);
            printf("\n");
        }

    return 0;
}

bruceteen

这个问题很简单，因为“代码不排版乃缺德行为”，所以你只要将代码排版一下就看出来了

1. void sort(struct student * p, int l)
   
2. {
   
3.     struct student t;
   
4.     int i, j;
   
5.     for(i = 0; i <= l-1; ++i)
   
6.     {
   
7.         for (j = 0; j <= l-1-i; ++j)
   
8.             if (p[j].score < p[j+1].score)
   
9.                 t = p[j];
   
10.         p[j] = p[j+1];
    
11.         p[j+1] = t;
    
12.     }
    
13. }
    

复制代码

技术小菜鸟

回复 2# bruceteen
我擦原来是这个呀，注意力没放在这方面，我说找了半天都没发现


   

技术小菜鸟

回复 2# bruceteen
不过好像试了下，代码还是不行啊


   

zonglin

1. for (i = 0; i < len; ++i)
   
2.         {
   
3.             printf("请输入第%d个学生信息:\n", i+1);
   
4.             scanf("%d %s %c %f", &pArr[i].num, pArr[i].name, &pArr[i].sex, &pArr[i].score);
   
5.             printf("\n");
   
6.         }
   
7. 
   
8.         sort(pArr, len);
   
9. 
   
10.     for (i = 0; i < len; ++i)
    
11.         {
    
12.             printf("输出的是成绩第%d名学生信息:\n", i+1);
    
13.             printf("%d %s %c %f", pArr[i].num, pArr[i].name, pArr[i].sex, pArr[i].score);
    
14.             printf("\n");
    
15.         }
    
16. 
    

复制代码

技术小菜鸟

回复 5# zonglin
不懂，什么意思啊？


   

bruceteen

技术小菜鸟 发表于 2013-07-30 19:05
回复 5# zonglin
不懂，什么意思啊？

对于一个数组 array[N]，其下标范围是[0,N)。看你的代码，你喜欢写成[0,N-1]也无所谓
但在main函数中，你写成了[1,N]，对比一下5楼的代码和你的代码，就看出来了

技术小菜鸟

回复 7# bruceteen
你的意思我懂了，我以后会注意的，但是这个代码不带中间的f（）函数运行是正常的，主要错误不是这个，求大神改一个可以运行的代码

   

seabiscuitxf

第一个错误是5楼指出的
第二个错误是sort函数：你应该是想用冒泡排序，其中两个for循环不是你那样写的，我觉得应该是这样写
for（int i= 0; i < l-1; ++i)
  for(int j=i+i; j< l; ++j)
     if (p[i].score < p[j].score)
        {
           t = p[i];
           p[i] = p[j];
           p[j] = t;
         }

技术小菜鸟

回复 9# seabiscuitxf
测试完毕，问题没有解决，你的for循环其实和我的都行，角度不一样
，求大神给个能行的代码啊