C++將元素添加到數組並將函數修改爲指針

這裏有什麼提示，詢問：

現在你可以修改LoadMovies函數創建一個MovieList 對象並添加每個電影的反對它。函數 LoadMovies應該返回一個指向MovieList對象的指針。這意味着您需要動態地在堆上創建MovieList對象。

變化的主要功能和所述返回MovieList指針存儲在一個變量。要測試一切是否按預期工作，您可以使用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(); 
            ^

來源

2014-12-05 andayn

你應該在類定義中聲明函數原型。

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() 
{ 
    delete[] movies; 
    movies = nullptr; 
}

來源

2014-12-05 20:19:55 CoryKramer

當我做函數原型時，我的編譯器總是會說函數不能被重載，並且函數被超標。 – andayn 2014-12-05 20:59:53

如果你定義了'class MovieList {...}'以外的函數，那麼他們需要這個範圍， 'MovieList :: IsFull（）'。如果您在類範圍內定義它們，則它們不會'class MovieList {bool IsFull（）; }'請注意，在我上面的例子中，在寫'Length'和'IsFull'的定義之前，我關閉了'class MovieList'的大括號。 – CoryKramer 2014-12-05 21:01:28

此外，在我的LoadMovies函數中，當我添加MovieList :: Add（movie）時，編譯器會說它不能在沒有對象的情況下調用，但是不是已經在參數中的對象？ – andayn 2014-12-05 21:05:13

您未創建MovieList（）類型的對象。

「LoadMovies」方法只進行讀取並在本地組裝一個向量。

函數LoadMovies（）不返回任何內容，您不能將「nothing」分配給任何東西。

爲什麼不利用已有的方法。每個讀入的電影都需要進入MovieList對象。調用Add（）方法。

聲明內部LoadMovies一個MovieList對象（）和改變返回類型以引用/指針MovieList返回，像

MovieList* LoadMovies();

如果在一個MovieList對象不通過引用傳遞並填充它。

void LoadMovies(MovieList& mList)

我認爲你需要看一本書的基礎知識，並從那裏去。這類問題可能會被嚴重低估。

來源

2014-12-05 20:23:06 KalyanS

C++將元素添加到數組並將函數修改爲指針

回答

相關問題