0

我试图为作业创建一个地图，我在标题中声明了它，我试图访问它，但我不断收到错误。我不确定代码有什么问题。我在加载函数中实现了代码，但如果你能帮忙，我似乎无法让它在 get 函数中工作谢谢

这是头文件

class Movies {
    // data is private by default
    Movie *movies;
    int movieCnt;
    map<string,**string> Mymap;

public:
    Movies(string);
    int getMovieCount() const;
    const Movie * getMovie(string) const;
    ~Movies();

 private:
    void loadMovies(string);
    int getMovieHash(string) const;
};

这是代码

const Movie * Movies::getMovie(string mc) const {
    if(mc.length()==0)
        return NULL; // not found
    else
        return &(Mymap.find(mc));
}

Movies::~Movies() {delete[] movies; movies = NULL;}

void Movies::loadMovies(string fn) {
    ifstream iS(fn);  // technically should be c_str
    string s;
    getline(iS, s); // skip heading
    getline(iS, s);
    movieCnt=0;
    while(!iS.eof()) {
        Movie* m = new Movie(s);
        Mymap[(m->getTitle())] = *m;
        movieCnt++;
        getline(iS, s);
    }
    iS.close();
}
4

3 回答 3

1

您将指针星号放在错误的位置。它应该是这样的：

 map<string,string**> Mymap;

甚至更像 C++

 map<std::string, std::vector<std::vector<std::string>>> Mymap;

您的getMovie(string f)功能可以通过以下方式改进：

const Movie Movies::getMovie(string mc) const 
{
   if(mc.length() > 0)
   {
      auto it = Mymap.find(mc);
      if (it != Mymap.end())
         return *it; //by value
   }
   else
       throw std::runtime_error;
}
于 2013-03-12T08:29:47.260 回答
0

你的地图的价值应该是一个Movie对象。

map<string, Movie> Mymap;
于 2013-03-12T08:31:11.503 回答
0

请更具体地说明您的错误。怎么了？同时看看这个：

const Movie * Movies::getMovie(string mc) const {
if(mc.length()==0)
    return NULL; // not found
else
    return &(Mymap.find(mc));
}

在return &(Mymap.find(mc));您返回指向临时对象的指针时，这不太可能起作用。尝试返回对象本身，

const Movie Movies::getMovie(string mc) const {
if(mc.length()==0)
    return NULL; // not found
else
     std::map<string, string**>::const_iterator it = Mymap.find(mc);
     Movie m = it*;
     return m;
}

或条目的索引并使用索引从地图中获取它。

于 2013-03-12T08:31:12.127 回答