What Is Wrong With My Name Finder?

[Baumer8993]

For my C++ class we have a big list of movies, and we have to print out the names when we enter a certain director. Here is my code:

cout << "Enter Part Of A Director Name: ";
	getline(cin, str);

	for (int i=0; i<numMovies; i++)
	{
		found = films[i].title.find(blank);

		if (found != string::npos)
			printMovie(films[i]);
	}
	

str is a string. When I try to run this part it just jumps into the loop, and prints every movie out. I copy, and pasted this from my professor example, and it works there, but not in mine! What am i doing wrong? In case it is needed here is the code to my entire program.

#include <iostream>
#include <iomanip>
#include <string>
#include <fstream>
using namespace std;

// Structure Definitions
struct movie
{
	int rank;
	string title;
	int year;
	float boxOffice;
	string rating;
	string director;
};

// Function Prototypes
int loadMovies(movie[]);
void yearRange(movie[], int);
void ratingFinder(movie[], int);
void printMovie(movie);

int main()
{
	movie films[600];
	string str;
	int numMovies;
	unsigned found;
	int choice;

	numMovies = loadMovies(films);
	cout << "Loaded " << numMovies << " movies..." << endl;

	cout << "1. See Movies For A Year Range" << endl;
	cout << "2. See Movies For A Rating" << endl;
	cout << "3. See Movies For A Director" << endl;
	cout << "4. Exit" << endl;
	cout << "Enter The Number For Your Choice: ";
	cin >> choice;

	if (choice == 1)
	{
		yearRange(films, numMovies);
	}
	else if (choice == 2)
	{
		ratingFinder(films, numMovies);
	}
	else if (choice == 3)
	{
	
	cout << "Enter Part Of A Director Name: ";
	getline(cin, str);

	for (int i=0; i<numMovies; i++)
	{
		found = films[i].title.find(blank);

		if (found != string::npos)
			printMovie(films[i]);
	}
	

	}
	else if (choice == 4)
	{
	}


	return 0;
}

int loadMovies(movie m[])
{
	ifstream inFile;
	int count = 0;
	string junk;

	inFile.open("movies.txt");
	if(!inFile)
		cout << "Could Not Open File." << endl;
	else
	{
		cout << "File Opened" << endl;
		
		// Pick Out What We Want:
		inFile >> m[count].rank;
		while(!inFile.eof())
		{
			getline(inFile, junk, '|');
			// Read The Title
			getline(inFile, m[count].title, '|');
			// Read The Year
			inFile >> m[count].year; // Quits At The Decimal Point.
			getline(inFile, junk, '|');
			// Clear The $
			inFile.ignore(); // Ignores The Next Space
			// Read Box Office
			inFile >> m[count].boxOffice; // Read Up To |
			inFile.ignore(); // Ignores The Next |
			// Get The Rating
			getline(inFile, m[count].rating, '|');

			// Skip The Next Three Fields
			getline(inFile, junk, '|');
			getline(inFile, junk, '|');
			getline(inFile, junk, '|');

			// Read The Director
			getline(inFile, m[count].director, '|');

			// Get Rid Of The Last Part
			getline(inFile, junk);

			// Set Up For Next Read
			count++;
			inFile >> m[count].rank;   // Read The First Integer

		}

		inFile.close();
	} 

	return count;
}

void yearRange(movie m[], int numMovies)
{
	int low, high;

	cout << "Enter The Lower Year Number: ";
	cin >> low;
	cout << "Enter The Higher Year Number: ";
	cin >> high;

	for(int i = 0; i < numMovies; i++)
	{
		if(m[i].year >= low && m[i].year <= high)
		{
			printMovie(m[i]);
		}
	}

	return;
}

void ratingFinder(movie m[], int numMovies)
{
	int choice;
	string rating;


	cout << "1. G" << endl;
	cout << "2. PG" << endl;
	cout << "3. PG-13" << endl;
	cout << "4. R " << endl;
	cout << "Enter The Rating You Are Looking For: ";
	cin >> choice;

	if(choice == 1)
	{
		rating = "G";
	}

	else if(choice == 2)
	{
		rating = "PG";
	}
	
	else if(choice == 3)
	{
		rating = "PG-13";
	}

	else if(choice == 4)
	{
		rating = "R";
	}

	for(int i = 0; i < numMovies; i++)
	{
		if(rating == m[i].rating)
			printMovie(m[i]);
	}

	
	return;
}

void printMovie(movie m)
{
	cout << setw(4) << right << m.rank << "  ";
		if (m.title.length() < 30)
			cout << setw(30) << left << m.title;
		else
		{
			for(int i = 0; i < 30; i++)
				cout << m.title[i];
		}

	cout << setw(5) << right << m.year << "  "
		 << setw(8) << right << m.boxOffice << "  " 
		 << setw(6) << left << m.rating
		 << m.director << endl;
	return;
}

[cire]

When I try to run this part it just jumps into the loop, and prints every movie out. I copy, and pasted this from my professor example, and it works there, but not in mine!

If by "works" you mean "doesn't compile", you might be on to something. blank is not defined anywhere.

[Baumer8993]

It runs, and compiles. It just prints out the entire list of movies, and I did put blank in there somewhere. I changed it from something else which did not work in the first place.

[cire]

I suspect the reason blank was in there was because you were supposed to replace it with something more appropriate. Like maybe the input you just got from the user?

[Baumer8993]

Ok I got it to work without the getline. Thank you for the help!