Can't Find Error

[Suzanne]

This is a program which takes info from a movie website. The program complies but for some reason when executing, the search function goes into an infinite loop.Can't identify why. Also my wget command doesn't work.

#include <iostream>
#include <iomanip>
#include <fstream>
#include <sstream>
#include <ctime>
#include <vector>
#include <algorithm>
#include <cstdlib>
#include <iterator>
#include <functional>

using namespace std;

class Movie
{
    friend istream& operator>>(istream&, const Movie&);
    friend ostream& operator<<(ostream&, const Movie&);
    
    public:
        void setVector(string&, string&, string&, string&, string&, string&, int&);
        void setInfo(string, string, string, string, string, string, int);
        string getTitle(const Movie&);
        string getDirector(const Movie&);
        string getRelDate(const Movie&);
        string getGenre(const Movie&);
        string getCast(const Movie&);
        string getSyn(const Movie&);
        int getRuntime(const Movie&);
    
    //private:
        string title;
        string director;
        string reldate;
        string genre;
        string cast;
        string synopsis;
        int runtime;
};

ostream& operator<<(ostream& out, const Movie& mv)
{
    out << mv.title << "|" << mv.director << "|" << mv.reldate << "|"
        << mv.genre << "|" << mv.cast << "|" << mv.synopsis << "|" << mv.runtime;
    
    return out;    
}

void Movie::setVector(string& t, string& d, string& rdate, string& g, string& c, string& s, int& run)
{
    title = t;
    director = d;
    reldate = rdate;
    genre = g;
    cast = c;
    synopsis = s;
    runtime = run;
}

void Movie::setInfo(string t, string d, string rdate, string g, string c, string s, int run)
{
    title = t;
    director = d;
    reldate = rdate;
    genre = g;
    cast = c;
    synopsis = s;
    runtime = run;
}

string Movie::getTitle(const Movie& rhs)
{
    return title;
}
string Movie::getDirector(const Movie& rhs)
{
    return director;
}
string Movie::getRelDate(const Movie& rhs)
{
    return reldate;
}
string Movie::getGenre(const Movie& rhs)
{
    return genre;
}
string Movie::getCast(const Movie& rhs)
{
    return cast;
}
string Movie::getSyn(const Movie& rhs)
{
    return synopsis;
}
int Movie::getRuntime(const Movie& rhs)
{
    return rhs.runtime;
}
 

[Suzanne]

class Dbase
{
    public:
        void add();
        void remove();
        void search();
        void summary();
        int infile();
        
    private:
        bool compare(const Movie&);
        void outfile();
        Movie mov;
        vector<Movie> m;
        vector<Movie>::iterator it;
};

bool Dbase::compare(const Movie& rhs)
{
    it = m.begin();
    cout << "Checking movie in database.." << endl;
    
    for (int i = 0; i < m.size(); i++)
    {
        
        if ((m[i].title == rhs.title) && (m[i].reldate == rhs.reldate))
        {
            cout << "Movie data found!!" << endl;
            return true;
        }
        it++;
    }
    return false;
}

int Dbase::infile()
{
    ifstream fin;
    string s, z, name;
    string temp[7];
    int i, rt;
    fin.open("movdb.txt");
    if (!fin.good())
    {
       /* cout << "Please enter the website name you want to grab" << endl;
        cin >> name;
        command="wget -q -O filename URL";
        system(command.c_str());
        return get;*/
       return 1; 
    }    
    else
    {
        while (getline(fin, s))
        {
            stringstream ss(s);
            i = 0;
            while (getline(ss, z, '|'))
            {
                temp[i] = z;
                ++i;
            }
            rt = atoi( temp[6].c_str() );
            mov.setVector(temp[0],temp[1],temp[2],temp[3],temp[4],temp[5],rt);
            m.push_back(mov);
        }
        return 0;
    }    
}

/*void Dbase::wCheck(int num)
{
   if(num == 1)
     {
       if(strcmp(("http://www.imdb.com",name.c_str())== 0) 
         cout << "Here: " << endl;
     }
}*/         
void Dbase::outfile()
{
    ofstream fout;
    fout.open("movdb.txt");
    
    if (!fout.good())
        cout << "Output file corrupted!!" << endl;
    else
        for (int i = 0; i < m.size(); i++)
            fout << m[i] << endl;
    
    fout.close();
}
    
void Dbase::add()
{
    
    if (compare(mov) == true)
    {
       cout << "Duplicate data was not added.." << endl; 
    }
    else
    {
        m.push_back(mov);
        cout << "A movie data has been added successfully!!" << endl;
    }
    outfile();
}

void Dbase::remove()
{
    char rmv;
    string a;
    
    cout << "Please enter movie data data that you want to remove.." << endl;
    cin.ignore(1000, '\n');
    cout << "Title: ";
    getline(cin, mov.title);
    cout << "Release Date: ";
    getline(cin, mov.reldate);
    
    if (compare(mov) == true)
    {
        while ((rmv != 'Y') && (rmv !='y') && (rmv != 'N') && (rmv != 'n'))
        {
            cout << "Confirm to remove movie data?? (Y/N): ";
            cin >> rmv;
        }
        if (rmv == 'Y' || rmv == 'y')
        {
            m.erase(it);
        }
    }
    else
        cout << "Movie not found!!" << endl;
}

void Dbase::search()
{
   int opt; string first_v, second_v;
   cout << "Please choose a search option 1.Title 2.Director 3.Release 4.Title & Director 5.Genre 6.Title & Release" << endl;
   cin >> opt;
   
  //Using switch case here to search for individual move details
  while(opt > 0 && opt < 7)
  {
   switch(opt)
    {
     case 1: cout << "Please provide movie title: " <<endl;
             cin >> first_v;
             it = m.begin();
             while(it != m.end())
             {   
               if(first_v == mov.title)
               cout << "The movie is: " << mov.title << endl;
               else
               it++;
             }
             break;
             
    case 2:  cout << "Please provide movie director's name: " <<endl;
             cin >> first_v;
             it = m.begin();
             while(it != m.end()){
             if(first_v == mov.director)
             cout << "The movies directed by this person are: " << mov.title << endl;
             it++;
             }
             break;
             
    case 3:  cout << "Please provide movie release date: " <<endl;
             cin >> first_v;
             it = m.begin();
             while(it != m.end()){
             if(first_v == mov.reldate)
             cout << "The movies with this release date are: " << mov.title << endl;
             it++;
             }
             break;
             
    case 4:  cout << "Please provide movie director's name & title: " <<endl;
             cin >> first_v;
             cin >> second_v;
             it =m.begin();
             while(it != m.end()){
             if(((first_v == mov.director) && (second_v == mov.title))||(first_v == mov.title) && (second_v == mov.director))
             cout << "The movie is: " << mov.title<< endl;
             it++;
             }
             break;
             
    case 5:  cout << "Please provide genre of movie: " <<endl;
             cin >> first_v;
             it = m.begin();
             while(it != m.end()){
             if(first_v == mov.genre)
             cout << "The movies of this genre are: " << mov.title<< endl;
             it++;
             }
             break;
             
    case 6:  cout << "Please provide title of the movie & its release date: " <<endl;
             cin >> first_v;
             cin >> second_v;
             it = m.begin();
             while(it != m.end()){
             if((first_v == mov.title)&&(second_v == mov.reldate)||(first_v == mov.reldate)&&(second_v == mov.title))
             cout << "The movies are: " << mov.title<< endl;
             it++;
             }
             break;
   
   }

 } 
}

void Dbase::summary()
{
    cout << setw(30) << left << "Title" << setw(25) << "Director" << setw(20) << "Release Date"
         << setw(30) << "Casts" << setw(15) << "Genre" << setw(10) << right << "Runtime" << endl;
    
    for (int i = 0; i < m.size(); ++i)
        cout << m[i] << endl;
}

class MovieManager
{
    public:
        MovieManager();
        void mainMenu();
        void optionCase(int);
        
    private:
        int choice;
        Dbase db;
};

MovieManager::MovieManager()
{
    while (choice != 5) 
        mainMenu();
}

void MovieManager::mainMenu()
{
    cout << "===========================" << endl;
    cout << "     Main Menu option:     " << endl;
    cout << "---------------------------" << endl;
    cout << "1. Add" << endl;
    cout << "2. Remove" << endl;
    cout << "3. Search" << endl;
    cout << "4. Summary" << endl;
    cout << "5. Quit" << endl << endl;  
    cout << "Enter your selection: ";
    
    db.infile();
    
    cin >> choice;
    optionCase(choice);

}

void MovieManager::optionCase(int choice)
{
    switch (choice)
    {
        case 1:
            db.add();
            break;
            
        case 2:
            db.remove();
            break;
            
        case 3:
            db.search();
            break;
            
        case 4:
            db.summary();
            break;
        case 5:
            break;
    }
}

int main()
{   
    MovieManager i;
    cout << "I'm done" << endl;
    system("pause");
    return 0;
}

[magnificence7]

It seems that the variable opt in your search function doesn't change, so when opt is between 0 and 7, it keeps between it, so the loop keeps running.

[Suzanne]

No that's not the problem. It was the while loop. But now I have modified the code & my search is too messy. Anyone can suggest a better way to do partial search & how to implement update in add function. Also need help with wget command.