Classes program

Hi, I am doing a assignment in which I have to create classes to store information about my books. However it is not working because I am getting crazy output and I don't know what's wrong. I keep on trying different things but I cannot get it to work. I'd appreciate it if someone could look at it. I think the problem may lie in the data file, but I'm not sure. Here is my program:

Code:

---

#include <iostream>
#include <fstream>
#include <string>
using namespace std;
const int SIZE = 15;
ifstream file_in;
ofstream file_out;
class cover {
      public:
            string title;
            string first;
            string last;
};

class inside {
      public:
           
            string genre;
            string publisher;
            int year;
            int pagenums;
};

class book {
      public:
            cover coverinfo;
            inside insideinfo;
};
void readdata (book [], int &);
void printdata (book [], int);
int main ()
{
    int n;
    book library [SIZE];
   
    file_in.open ("program8in.dat");
    file_out.open ("program8.out");
   
    readdata (library, n);
    printdata (library, n);
   
   
    return 0;
}
void readdata (book library [], int &n)
{
    file_in >> n;
   
   
    for (int count = 0; count < n; count++) {
        file_in >> library[count].coverinfo.title;
        file_in >> library[count].coverinfo.first;
        file_in >> library[count].coverinfo.last;
        file_in >> library[count].insideinfo.genre;
        file_in >> library[count].insideinfo.publisher;
        file_in >> library[count].insideinfo.year;
        file_in >> library[count].insideinfo.pagenums;
   
    }
    return;
}
void printdata (book library [], int n)
{
    for (int count= 0; count < n; count++) {
        file_out << "\t" << library[count].coverinfo.title;
        file_out << "\t" << library[count].coverinfo.first;
        file_out << "\t" << library[count].coverinfo.last;
        file_out << "\t" << library[count].insideinfo.genre;
        file_out << "\t" << library[count].insideinfo.publisher;
        file_out << "\t" <<library[count].insideinfo.year;
        file_out << "\t" << library[count].insideinfo.pagenums;
    }
    return;
}

---

This is my data file:

Code:

---

7
Harry Potter
JK
Rowling
SciFi
Scholastic
2003
870

---

And this is my output:

Code:

---

Harry        Potter        JK        Rowling        SciFi        205936        196984                                                196984        196984                                                27        512                                                152        -2141497075                                                -445879994        2358840                                                198376        2089878893                                                211408        211408

---

I'm sorry about the length, I know it's rather long but I'm stuck.

To be honest, all you are using are C-style structs, not OO classes.

What is happening is that you read in 7 and then loop 7 times to read in data. However, your data file only has one item which spans 7 lines. Not only that, but you are reading in using formatted input, i.e., you read in until the first whitespace. Rather, you want to read line by line, possibly by using std::getline(). Finally, year and pagenums are ints, so you need to convert the strings read to ints before storing them (or use formatted input to read directly to the ints, but you have to be careful to discard newlines when mixing formatted input with getline).

Thanks I fixed the first part by using getline() but how would I convert the strings read to ints for year and pagenums?

Read the FAQ on Convert a string to a int (C++).

Another solution is to simply use getline for the lines that may contain a space, then use operator>> for the lines that have integers. The only caveat is that when mixing operator>> and getline, you have to add a file_in.ignore() after the calls to operator>> to ignore trailing newlines. The benefit is that you don't have to take the extra step of converting strings to integers, it is done automatically by operator>>.