This is not working! I want to add new book to my library record

Question

0.00/5 (No votes)

See more:

C++

#include<stdio.h>
#include<conio.h>
#include<string.h>
#include<stdlib.h>
int main()
{	FILE *librecord;
	char another='Y';
	struct book
	{	
		char bname[40];
		int bid;
	};
	struct book b[100];
	librecord=fopen("librecord.txt","w");
	if(librecord==NULL)
	{	
		puts("Cannot open file!");
		exit(1);
	}
	while(another=='Y')
	{
		printf("\nEnter name and id of Book\n");
		scanf("%s %d\n",book.bname,&book.bid);
		fprintf(librecord,"%s %d\n","book.bname,book.bid");
		printf("Add Another Record(Y/N)");
		another=getche();
	}
	fclose(librecord);
	return 0;
}

What I have tried:

I have tried it many times. Actually i want to create a function of adding books to my library,with their unique ids entered by the user.

Posted 19-Jun-16 10:01am

Member 12592442

Updated 19-Jun-16 17:09pm

Maciej Los

v2

Add a Solution

Comments

Andreas Gieriet 19-Jun-16 21:40pm

What exactly is the symptom?
- you have no variable book
- the fprintf call does have hard coded non-sense string arguments
- the b[100] is not used
- you do not check for errors from scanf

Cheers
Andi

1 solution

Add a Solution

Add your solution here

Treat my content as plain text, not as HTML

Preview 0

…

Existing Members

Sign in to your account

...or Join us

Download, Vote, Comment, Publish.

Your Email
Password
Forgot your password?

Your Email
This email is in use. Do you need your password?
Optional Password

I have read and agree to the Terms of Service and Privacy Policy
Please subscribe me to the CodeProject newsletters

When answering a question please:

Read the question carefully.
Understand that English isn't everyone's first language so be lenient of bad spelling and grammar.
If a question is poorly phrased then either ask for clarification, ignore it, or edit the question and fix the problem. Insults are not welcome.
Don't tell someone to read the manual. Chances are they have and don't get it. Provide an answer or move on to the next question.

Let's work to help developers, not make them feel stupid.

This content, along with any associated source code and files, is licensed under The Code Project Open License (CPOL)

leon de boer · Answer 1 · 2016-06-19T17:09:00

What Andreas Gierit said but I will expand a little as you are missing a variable
You need a number of the book you are on which starts at zero and increments each time you press "Y" so it starts like

int onBook = 0;

You created an array of books called b now you use it (why create it if you dont use it) .. this line is wrong

scanf("%s %d\n",book.bname,&book.bid);

should be

scanf("%s %d\n",b[onBook].bname,&b[onBook].bid);

Get it onBook is the book index in array b your are loading the data into.

I will leave it to you to increment onBook when you press "Y" and you probably need to make sure it doesn't increment pas the length of the array b.