Question

在我的任务中，我仍然有点卡在另一部分上。

以下是提示的要求：

现在您可以修改LoadMovies函数以创建MovieList   对象并将每个Movie对象添加到它。功能   LoadMovies应该返回一个指向MovieList对象的指针。这意味着   你需要动态地在堆上创建MovieList对象。

更改main函数并将返回的MovieList指针存储在a中   变量。要测试一切是否按预期工作，您可以使用   MovieList对象的PrintAll函数。

到目前为止，这是我的代码：

class MovieList {

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

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

~MovieList() {
    delete [] movies;
}

int Length() {
    return movie_count;
}

bool IsFull() {
    return movie_count == movies_size;
}

void Add(Movie const& m)
{
    if (IsFull())
    {
        cout << "Cannot add movie, list is full" << endl;
        return;
    }

    ++last_movie_index;
    movies[last_movie_index] = m;
}

void PrintAll() {
    for (int i = 0; i < movie_count; i++) {
        movies[last_movie_index].PrintMovie();
    }
}

};

void ReadMovieFile(vector<string> &movies);
void LoadMovies();

enum MovieSortOrder
{
    BY_YEAR = 0,
    BY_NAME = 1,
    BY_VOTES = 2
};

int main()
{
    LoadMovies();

    // TODO:
    // You need to implement the Movie and MovieList classes and
    // the methods below so that the program will produce
    // the output described in the assignment.
    //
    // Once you have implemented everything, you should be able
    // to simply uncomment the code below and run the program.

    MovieList *movies = LoadMovies();

    //    // test methods for the Movie and MovieList classes
        //PrintAllMoviesMadeInYear(movies, 1984);
        //PrintAllMoviesWithStartLetter(movies, 'B');
        //PrintAllTopNMovies(movies, 5);

        //delete movies;
    return 0;
}

void LoadMovies()
{
    vector<string> movies;
    ReadMovieFile(movies);

    string name;
    int year;
    double rating;
    int votes;

    for (int i = 0; i < movies.size(); i++)
    {
        istringstream input_string(movies[i]);
        getline(input_string, name, '\t');
        input_string >> year >> rating >> votes;
        Movie movie (name, year, votes, rating);
        movie.PrintMovie();
    }
}

现在，我被困在哪里，教授要求我修改提示中的LoadMovies并将其转换为指针。我画的是空白。出于某种原因，如果我尝试编译它说：

C:\Users\Andy\Documents\C++ Homework\MovieStatisticsProgram\MovieStatsProgram.cpp:163: error: void value not ignored as it ought to be
     MovieList *movies = LoadMovies();
                                    ^

Answer 1

您应该在类定义中声明函数原型。

class MovieList {

public:
    MovieList();    // constructor
    ~MovieList();   // destructor

    void Add(Movie const& m);
    bool IsFull();
    ... etc

private:   // Your members should be private, in general
    Movie* movies;
    int last_movie_index;
    int movies_size;
    int movie_count = 0;    
}

然后，您的所有函数都应该是MovieList类的成员，以便他们可以访问成员变量，并使用this来引用。例如，这些将在类声明之外。

int MovieList::Length(){
    // work
}

bool MovieList::IsFull(){
    // work
}

同样适用于LoadMovies，它应该有签名

MovieList* MovieList::LoadMovies();   // Note it returns a MovieList pointer, not void

然后在main中你可以创建一个实例

int main()
{
    MovieList ml = MovieList();    // instantiate a MovieList

    Movie a = Movie();   // instantiate a couple of movies
    Movie b = Movie();
    ml.Add(a);           // call the Add method
    ml.Add(b);

    MovieList* movies = ml.LoadMovies();  // Calls method from this object
}

作为旁注，我注意到main中有一条评论//delete movies;。实际上，由于您的MovieList对象已分配数组，因此应负责自行清理。我的意思是析构函数应该是这样的，这样你就不会泄漏内存。

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

Answer 2

您没有创建MovieList（）类型的对象。

方法＆＃34; LoadMovies＆＃34;只读取并在本地组装一个向量。

函数LoadMovies（）不返回任何内容，您无法分配＆＃34;没有＆＃34;任何事情。

为什么不利用已有的方法。每个被读入的电影都需要进入一个MovieList对象。调用Add（）方法。

在LoadMovies（）中声明一个MovieList对象，并将返回类型更改为通过引用返回/指向MovieList的指针，如

MovieList* LoadMovies();

如果没有通过引用传入MovieList对象并将其填入。

void LoadMovies(MovieList& mList)

我认为你需要看一本书中的基础知识并从那里开始。这类问题可能会被严重贬低。

C ++向数组添加元素并将函数修改为指针

2 个答案: