Сбой программы C ++ из функции печати

0

В настоящий момент моя программа сбой после отображения одной строки из 250, которые она должна отображать. Вот мой код:

string MovieList::PrintAll() {
   for (int i = 0; i < last_movie_index; i++) {
        movies[last_movie_index].Movie::PrintMovie();
   }
}

string Movie::PrintMovie() {
    cout << fixed << setprecision(1) << rating << "\t\t" << votes << "\t\t" << "(" <<
            year_made << ")" << "\t" << name << endl;
}

Полный фильм и класс MovieList:

class Movie {
public:
    Movie();
    Movie(string n, int y, int v, double r);
    string get_name();
    void set_name(string n);
    int get_year();
    void set_year(int y);
    int get_votes();
    void set_votes(int v);
    double get_rating();
    void set_rating(double r);
    string PrintMovie();

private:
    string name;
    int year_made;
    int votes;
    double rating;

};

Movie::Movie() {
    name = "null";
    year_made = 0;
    votes = 0;
    rating = 0.0;
}

Movie::Movie(string n, int y, int v, double r) {
    name = n;
    year_made = y;
    votes = v;
    rating = r;
}

string Movie::get_name() {
    return name;
}

void Movie::set_name(string n) {
    name = n;
}

int Movie::get_year() {
    return year_made;
}

void Movie::set_year(int y) {
    year_made = y;
}

int Movie::get_votes() {
    return votes;
}

void Movie::set_votes(int v) {
    votes = v;
}

double Movie::get_rating() {
    return rating;
}

void Movie::set_rating(double r) {
    rating = r;
}

string Movie::PrintMovie() {
    cout << fixed << setprecision(1) << rating << "\t\t" << votes << "\t\t" << "(" <<
            year_made << ")" << "\t" << name << endl;
}

class MovieList {
public:
    MovieList(int size);
    ~MovieList();
    int Length();
    bool IsFull();
    void Add(Movie const& m);
    string PrintAll();

private:
    Movie* movies;
    int last_movie_index;
    int movies_size;
    int movie_count = 0;

};

MovieList::MovieList(int size) {
    movies_size = size;
    movies = new Movie[movies_size];
    last_movie_index = -1;
}

MovieList::~MovieList() {
    delete [] movies;
}

int MovieList::Length() {
    return last_movie_index;
}

bool MovieList::IsFull() {
    return last_movie_index == movies_size;
}

void MovieList::Add(Movie const& m)
{
    if (IsFull()) {
        cout << "Cannot add movie, list is full" << endl;
        return;
    }
    last_movie_index++;
    movies[last_movie_index] = m;
}

string MovieList::PrintAll() {
   for (int i = 0; i < last_movie_index; i++) {
        movies[i].Movie::PrintMovie();
   }
}

Мои видеофильмы массива динамически выделяются (например, movies = new Movie[movies_size];). Я заметил, что использование функции cout << movies[1] << endl не будет работать в функции PrintAll. Неужели это возможно? И что я могу сделать, чтобы исправить это?

  • 2
    Разве это не должны быть движения [i] вместо ходов [last_movie_index] в цикле?
  • 0
    @alexm Аааа, да, я чувствую себя идиотом для этого. К сожалению, он все еще падает сразу после первой печати. Теперь, что произойдет, он напечатает: 9.2 453500 (1994) The Shawshank Redemption 0.0 13483293892 (-5963777736), а затем он все равно рухнет на меня
Показать ещё 6 комментариев
Теги:

1 ответ

0

Это не решит вашу проблему, но если вы хотите сделать cout << movies[i] << endl work, тогда вам необходимо определить следующую функцию в классе Movie:

friend ostream& operator<<(ostream& ostr, const Movie& movie){
    ostr << fixed << setprecision(1) << rating << "\t\t" << votes << "\t\t" << "(" <<
            year_made << ")" << "\t" << name ;
}
  • 1
    Я не верю, что объявления функций могут быть вложенными. Это опечатка?
  • 0
    Спасибо, это была ошибка копирования-вставки

Ещё вопросы

Сообщество Overcoder
Наверх
Меню