C程序结构

Question

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

struct s {
    char ch[20];
    float a;
};

int main()
{
    struct s p[10];
    int i;
    for(i=0;i<10;i++)
    {
        scanf("%s%f",p[i].ch,p[i].a);
    }
}

What is wrong with this code?

Its giving runtime error.

What's the problem?

Answer 1

Most of the errors come from this line.

scanf("%s%f",p[i].ch,p[i].a);

You should use the address of p[i].a , and also restrict the numbers of chars to write in p[i].ch .

scanf( "%19s%f", p[i].ch, &p[i].a );

Answer 2

I haven't touched C code for a while but shouldn't it be something like

scanf("%s%f",p[i].ch,&(p[i].a));

(You have to give the memory address of the variables to the scanf function.)

Answer 3

I think the problem is in the p[i].a parameter; use &p[i].a instead.

Answer 4

At the line:

scanf("%s%f", p[i].ch, p[i].a);

You are using p[i].a as a float* (pointer), while it's a float . You're invoking undefined behavior. You probably wanted to do it like this:

scanf("%s%f", p[i].ch, &p[i].a);

Answer 5

Change your code like this:

#include <stdio.h>
#include <string.h>
struct s {
    char ch[20];
    float a;
};

int main(){
    struct s p[10];
    int i;
    for(i=0;i<10;i++){
        scanf("%s%f",p[i].ch, &p[i].a);
    }
}

Note that variable a is a float type; you need to pass its memory address when using scanf .