应《网络安全法》要求，自2017年10月1日起，未进行实名认证将不得使用互联网跟帖服务。为保障您的帐号能够正常使用，请尽快对帐号进行手机号验证，感谢您的理解与支持！

24小时热门版块排行榜    

小木虫论坛-学术科研互动平台 » 计算模拟区 » 程序语言 » C/C++ » 我自己写的程序，求高手改错
51/1返回列表
查看: 1101  |  回复: 8
只看楼主 @他人 存档 新回复提醒 (忽略) 收藏    在APP中查看
当前只显示满足指定条件的回帖，点击这里查看本话题的所有回帖

earth洛溪

铜虫 (小有名气)

[求助] 我自己写的程序，求高手改错

我自己写的程序，求高手改错
题目：计算并输出n个学生某学期m们课程的平均成绩
CODE:
#include <stdio.h>
#include <stdlib.h>
void main()
{ int n,c,i,sum,average;
printrf("\nenter n number students");
scanf("%d",&n);
c=(int*)malloc(n*sizeof(int));
printf("\nenter %d number students",n);
c=0;
for(i=0;i<n;i++)

scanf("%d",c+i);
for(i=0;i<n;i++)

sum=*c+*(c+i);


average=sum/n;
printf("%d",average);



   
}

[ Last edited by jjdg on 2013-6-7 at 00:42 ]
回复此楼

» 猜你喜欢

» 本主题相关价值贴推荐，对您同样有帮助:
大鹏一日随风起，扶摇直上就九万里。
1楼 2013-05-24 11:37:25
已阅   回复此楼   关注TA 给TA发消息 送TA红花 TA的回帖

limmettle

金虫 (正式写手)

【答案】应助回帖

★
感谢参与，应助指数 +1
xzhdty: 金币+1, 谢谢参与 2013-06-11 23:24:03
根据楼主的这一句
c=(int*)malloc(n*sizeof(int));
就可以知道c是一个整型指针，因此必须定义为
int *c；

当
c=(int*)malloc(n*sizeof(int));
这一句执行过后，c的值是这个连续的n*sizeof(int)这一段内存的首地址，这时c指向该连续空间的第一个整数；c的值每加1，将会指向下一个整数。

因此，楼主的
c=0;
这一句实际上已经给c重新赋值，c已经不指向刚才开辟的内存空间，而指向内存中地址为0的空间，该空间是加载系统的空间，受系统保护，不允许入侵；

*(c+i)取出楼主开辟的连续空间的第i+1个整数的值，因此求和应该写成
sum=0;
for(i=0;i<n;i++) sum+=*(c+i);

开辟的空间使用完后应该及时释放，在本例中使用
free(c);
语句。既然使用了内存管理的函数，就应该管理好内存（有开辟就应该有释放），否则还不如直接使用数组呢，是吧？

另外，再给楼主一点建议吧
1.main函数的标准形式是不存在void main()这种的，规范的写法是
int main(){
    ...;
    return 0;
}
谭浩强老师的书看看就行，当不得真，不利于培养良好的编程习惯；我猜楼主c编程可能还不很熟练，建议空了翻翻《C Primer Plus》（中英文版都有的），很不错；
2.不建议二楼的朋友所说的使用scanf_s()函数，该函数不是ANSI C标准里的，在今后实际应用中可能存在移植性的问题。
赞一下(3人) 回复此楼
3楼2013-05-25 11:41:30
已阅   回复此楼   关注TA 给TA发消息 送TA红花 TA的回帖
查看全部 9 个回答

ausir

木虫 (正式写手)

【答案】应助回帖

★
感谢参与，应助指数 +1
xzhdty: 金币+1, 谢谢参与 2013-06-11 23:24:18
现在的编译器对数据类型的定义要求比较严格。把 int *c 定义为 int c 编译无法通过。
另外 c=0; 会导致原来分配的数组空间丢失，不知道指到那里，因此不能用。
在后面求和是，正确的加法是 sum=sum+*(c+i) 而不是*c+*(c+i)，计算从i=1起，到i<n,并且 sum 的初值要是 *c的值。
最后没有做内存释放，每次分配内存，会造成内存泄露。
做了以上修改后，就可以正确运算了。

在VS2012中，需要添加<stdlib.h>头文件，并且把scanf函数用scanf_s这个安全函数替代才可以正确运行。
赞一下(3人) 回复此楼
2楼2013-05-24 13:07:14
已阅   回复此楼   关注TA 给TA发消息 送TA红花 TA的回帖

earth洛溪

铜虫 (小有名气)
引用回帖:
3楼: Originally posted by limmettle at 2013-05-25 11:41:30
根据楼主的这一句
c=(int*)malloc(n*sizeof(int));
就可以知道c是一个整型指针，因此必须定义为
int *c；

当
c=(int*)malloc(n*sizeof(int));
这一句执行过后，c的值是这个连续的n*sizeof(int)这一段内存的 ...

谢谢啦，我是大一的，刚开始学c++，不怎么会用，感谢你哦
赞一下(1人) 回复此楼
大鹏一日随风起，扶摇直上就九万里。
4楼2013-05-28 19:29:11
已阅   回复此楼   关注TA 给TA发消息 送TA红花 TA的回帖

黑色风信子

荣誉版主 (文坛精英)

皇家神龍教KOKO或KOKO猪

【答案】应助回帖

★ ★ ★ ★
earth洛溪: 金币+3, ★有帮助 2013-06-11 16:10:43
xzhdty: 金币+1, 谢谢参与 2013-06-11 23:23:42
要不要这么复杂啊，不喜欢改你的，给你写一个吧
题目：计算并输出n个学生某学期某们课程的平均成绩
#include <stdio.h>
#include <stdlib.h>
void main()
{
  int n,i;
  float sum=0,temp,average;//总和和平均分可能带小数，定义为浮点型比较妥
  printf(“Please input the number of students：”);
  scanf(“%d”,&n);
  printf("\n";
  for(i=0;i<n;i++)
{
   printf("Please input the %d student's score",i);
   scanf("%f",&temp);
   sum=sum+temp;
   printf("\n";
}
  average=sum/n;
printf("The average of this course is %2.f\n",average);//精确到小数点后2位
}
赞一下(1人) 回复此楼
如果你哭，你的对手就会笑。如果你笑，你的对手就会哭。人生就像愤怒的小鸟，每次你失败的时候，总有几只猪在笑。你要做的就是无视嘲笑的声音，给自己打气。自.
6楼2013-06-06 19:08:57
已阅   回复此楼   关注TA 给TA发消息 送TA红花 TA的回帖
查看全部 9 个回答
普通表情 龙 兔 虎 猫
信息提示
关闭
请填处理意见
关闭