small problem but....

Hello guys,

I want to know what my Visual .net compiler is skipping the process of the input of the book title after the first loop.

Quote:

---

#include <stdio.h>


struct books
{
char titel[50];
char auther[50];
float price;
int isbn;
} book_info[5];


int main()
{
int i;
for (i=0; i<5; i++)
{


printf("title: ");
gets(book_info[i].titel);
putchar('\n');

printf("auther: ");
gets(book_info[i].auther);
putchar('\n');

printf("price: ");
scanf("%f", &book_info[i].price);
putchar('\n');

printf("ISBN: ");
scanf("%d", &book_info[i].isbn);
putchar('\n');


}


for (i=0; i<5; i++)
{
puts(book_info[i].titel);
puts(book_info[i].auther);
printf(" Price is %.3f and ISBN is %d", book_info[i].price , book_info[i].isbn);
}


return 0;


}

---

the problem at the red line.

at the first loop everything gose well but the second loop the program dose not ask for the value of book_info[i].title but instead is asks
for the book_info[i].aurther



i'm not a begginer in programming but i'm in C and i can't find what is the problem


any help

scanf() leaves a newline on the input buffer so when it gets back around to gets() it thinks the user just hit ENTER.

Check the FAQ on this site for information on how to reliably get data from the user.

Another thing: Don't use gets(). Ever. For any reason. An explanation of why it's bad is also in the FAQ.

thanks man.

yes the problem is in scanf

after i removed scanf from the loop it worked well.

don't ask me why we are using gets() or whatever, it just our teacher is using it.

anyway....thanks.


:)

Quote:

---

don't ask me why we are using gets() or whatever, it just our teacher is using it

---

I see this comment so often, in reference to so many bad things. I know some teachers don't like to be shown up, but most will accept a different way of doing things quite happily (assuming it doesn't affect the substance of what they're teaching you), so long as you can explain why you did something a certain way.

Who knows, it may even be worth extra credit? Comment out the gets and rewrite it using fgets to show you know how to use both. Add a comment about why you did it, I'd be willing to bet $5.00 that your teacher will commend your work, not shoot you down for it. (Course you may have to come to Aussie to collect if I'm wrong).

you are completly correct. I agree with you.

But the point that this course is just basics of programming, he would not care much which function we use since we use a function that reads strings from user.

the main problem that the exam (after few hours) is going to be paper-based exam :p and imagine if he asks the same thing i did...about gets()...then i can't not answer the question saying

forgets gets() sir it is bad ...use fgets() :D

i will get zero.... :p

Sure you can. You can do it in a sentence or two:

Quote:

---

gets is a bad choice because it does no boundry checking. fgets specifies the size of the input buffer and is the better choice by far.

---

Quzah.