c - malloc函数返回的内存不保存数据

Question

在下面的代码中，我不明白为什么 win[] 没有获取值。我已经对下面代码中我感到困惑的行进行了评论。 请帮助我。 我是否犯了任何概念性错误？

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

void winner(int *speed,int *distance,int rats){
    float winner=(float)distance[0]/speed[0];
    float time;
    float *win=malloc(rats * sizeof(float));
    memset(win,0,rats*sizeof(float));
    for(int i=0;i<rats;i++) {
        time=(float)distance[i]/speed[i];
        if(time<=winner) {
            win[i]=time;   /*Problem is here*/ 
        }
    }
    for(int i=0;i<rats;i++) {
        if(win[i]!=0) {
            printf("%d\n",i+1);
        }
    }
    free(win);
}

int main() {
    int rats;
    int *speed,*distance;
    scanf("%d",&rats);
    speed=malloc(rats * sizeof(int));
    distance=malloc(rats * sizeof(int));
    for(int i=0;i<rats;i++) {
        scanf("%d",&speed[i]);
    }
    for(int i=0;i<rats;i++) {
        scanf("%d",&distance[i]);
    }
    winner(speed,distance,rats);
    free(speed);
    free(distance);
    return 0;

}

Answer 1

您的代码中至少存在 3 个问题:

• 如果其中一只老鼠的时间短于第一只老鼠，您不会更改获胜者
• 您没有为非获胜者初始化win[i]
• 您没有利用最佳时间展示获胜老鼠

所以代替:

    if(time<=winner){
       win[i]=time;   /*Problem is here*/
     }

你应该有:

    win[i]=time; // always initialize !
    if(time<=winner){
       winner = time; // keep best time
    }

然后代替

for(int i=0;i<rats;i++){
  if(win[i]!=0){
      printf("%d\n",i+1);
     }
 }

你应该有:

for(int i=0;i<rats;i++){
  if(win[i] == time){ // display all rats having best time
      printf("%d\n",i+1);
     }
 }