I'm getting a runtime error

Hi,

Could you please take a look at the following code? I'm getting a runtime error every time I enter the average score:

Code:

---


struct blah {
    char firstname[20+1];
    char lastname[25+1];
    float average;
};

typedef struct blah entry;

int main() {

    entry student;

    do {
        printf("First name: ");
        gets(student.firstname);
        if (strlen(student.firstname)==0) break;
        printf("Last name: ");
        gets(student.lastname);
        printf("Average score: ");
        scanf("%f", &student.average);
        // fflush(stdin);

    } while (1);



    return 0;

}

---

I simply can't see what I'm doing wrong.

How about starting with the FAQs which mention why gets() and fflush(stdin) are bad?

FAQ gets()

Additionally there are a couple of other problems:

1. You never initialize any of your variables from the struct. This is why you are getting your run time error.

2. After the first iteration this

Code:

---

if (strlen(student.firstname)==0) break;

---

Does not break you out of your loop.

I realize they're bad, thanks, but I'm still getting that runtime error on the input for average score.

1. You never initialize any of your variables from the struct. This is why you are getting your run time error.

Quote:

---

Originally Posted by andyhunter

1. You never initialize any of your variables from the struct. This is why you are getting your run time error.

---

Don't yell, I didn't notice your post (look at the time mine was posted).

Thanks though.

Ah, but CAPS is yelling. Bolding is 'look at me' :p