Adding shelves in a Library management software problem!

Question

3.00/5 (1 vote)

See more:

Here is my code for library management system. Now i have tried to add a shelves function in this code which is supposed to work in this manner: Each shelf can have 3 book entries and after that, books are entered into the next shelf.
The problem i'm having is that all the books are entered into the 1st shelf even after the 3rd entry. The code used for this function is in bold!

C++

//***************************************************************
//                   CLASS USED IN PROJECT
//***************************************************************



class book
{
	char bno[6];
	char bname[50];
	char aname[20];
	int count1;
	int shelf;
    
    
  public:
	
    
    
	book()
	{
		count1=0;
        	shelf=1;
        }
    
    
	void create_book()
	{
		cout<<"\nNEW BOOK ENTRY...\n";
		cout<<"\nEnter The book no.";
cin.ignore();
		cin.getline(bno,6);
		cout<<"\n\nEnter The Name of The Book ";
		cin.getline(bname,50);
		cout<<"\n\nEnter The Author's Name ";
		cin.getline(aname,20);
		cout<<"\n\n\nBook Created..";
		count1++;
		
		if (count1 == 3 )
		{
    			shelf++;
			count1=0;
    		}
	}

	void show_book()
	{
		cout<<"\nBook no. : "<<bno;
		cout<<"\nBook Name : "<<bname;
		cout<<"\nAuthor Name : "<<aname;
		cout<<"\nShelf number : "<<shelf;
	}

	char* retbno()
	{
		return bno;
	}

	void report()
	{cout<<bno<<setw(30)<<bname<<setw(30)<<shelf<<setw(30)<<aname<<endl;}


};         //class ends here

Posted 8-May-12 12:03pm

ShayanTanwir

Updated 8-May-12 12:16pm

TRK3

v2

Add a Solution

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)

TRK3 · Answer 1 · 2012-05-08T12:14:00

You need to make a static class member that holds the next count and shelf, and initialize them statically, rather than initializing them in the constructor...

class book
 {
 char bno[6];
 char bname[50];
 char aname[20];
 
 int my_count;
 int my_shelf;
 static int total_books_count;
 static int current_shelf;

 book () 
 {
    my_count = total_books_count;
    my_shelf = current_shelf;

    total_books_count++;
    if (total_books_count == 3) 
    {
        total_books_count = 0;
        current_shelf++;
    }
 }

 void create_book()
 {
    cout<<"\nNEW BOOK ENTRY...\n";
    cout<<"\nEnter The book no.";
    cin.ignore();
    cin.getline(bno,6);
    cout<<"\n\nEnter The Name of The Book ";
    cin.getline(bname,50);
    cout<<"\n\nEnter The Author's Name ";
    cin.getline(aname,20);
    cout<<"\n\n\nBook Created..";
 }

...

}

int book::total_books_count = 0;
int book::current_shelf= 0;

The way you have the code written, every book starts out with it's own count1 = 0 and shelf = 1 and each book thinks it's the only book...