Thread: Trouble with eof()

I am writing a program that reads in a file and converts the data to a different format. I am having issues with using the end of file function. The program compiles with out any error but when I run it I think it gets stuck in an infinite loop. Can anyone tell me what I am doing wrong? I am just learning C++ so any and all advice is appreciated!

Code:

/*
 *  RasMolMaker
 *  
 *
 *  Created by  on 6/22/10.
 *  Copyright 2010 __MyCompanyName__. All rights reserved.
 *
 */
#include <fstream>
#include <iostream>
#include <string>

int main(const int argc, const char** argv)
{
	char ch;
	const char* c_str();
	std::string str,suffix,RasFile;
	std::string filename;
	suffix = ".pdb";
	
	// Get filename and open stream
	std::cout << "Enter filename:\n";
	std::cin >> filename;
	std::cout << "Opening " << filename << "...\n";	
	std::ifstream infile(filename.c_str());
	
	// Create RasMol file and open stream
	RasFile = filename + suffix;
	std::cout << "Output will be to file " << RasFile << "\n";
	std::ofstream outfile(RasFile.c_str());
	
	int index = 1;
	double x,y,z;
	while (!infile.eof())
	{
		//infile.get(ch, 1);
		if (ch == 'm') 
		{
			//infile.getline(str, '\n');
			continue;
		}
		else {
			infile >> x >> y >> z;
			outfile << "ATOM  " << index;
			index ++;
		}

	}
	return 0;
}

Edit: Also, when I uncomment the infile.get and infile.getline I get a ton of error messages. The first one is:

Code:

RasMolMaker.cpp:35: error: no matching function for call to ‘std::basic_ifstream<char, std::char_traits<char> >::get(char&, int)’
/usr/include/c++/4.2.1/istream:292: note: candidates are: typename std::basic_istream<_CharT, _Traits>::int_type std::basic_istream<_CharT, _Traits>::get() 
[with _CharT = char, _Traits = std::char_traits<char>]

First, the get member function come in a variety of flavors:

int_type get( );
basic_istream& get(
char_type& _Ch
);
basic_istream& get(char_type *_Str,
streamsize _Count);
basic_istream& get(
char_type *_Str,
streamsize _Count,
char_type _Delim
);
basic_istream& get(
basic_streambuf<Elem, Tr> *_Strbuf
);
basic_istream& get(
basic_streambuf<Elem, Tr> *_Strbuf,
char_type _Delim
);

None of these match what you've got. When I compile I get an error because of this:

1>------ Build started: Project: CProg4, Configuration: Debug Win32 ------
1>Compiling...
1>CProg4.cpp
1>c:\documents and settings\******\my documents\visual studio 2005\projects\cprog4\cprog4.cpp(28) : error C2664: 'std::basic_istream<_Elem,_Traits> &std::basic_istream<_Elem,_Traits>::get(_Elem *,std::streamsize)' : cannot convert parameter 1 from 'char' to 'char *'
1> with
1> [
1> _Elem=char,
1> _Traits=std::char_traits<char>
1> ]
1> Conversion from integral type to pointer type requires reinterpret_cast, C-style cast or function-style cast
1>Build log was saved at "file://c:\Documents and Settings\******\My Documents\Visual Studio 2005\Projects\CProg4\Debug\BuildLog.htm"
1>CProg4 - 1 error(s), 0 warning(s)
========== Build: 0 succeeded, 1 failed, 0 up-to-date, 0 skipped ==========

Second, you should avoid using eof tests to control your loops. Unless you know what you're doing it usually ends up badly. You should instead test the result of the input operation, for example:

Code:

while(infile.get(ch))
{
    // Do something
}

Edit: Also, when I uncomment the infile.get and infile.getline I get a ton of error messages. The first one is:

See, now you've just changed things, you're original post said it compiled without errors but that older version had the infile.get() line uncommented which could not have been. Now you're getting the error I get and that's because you're using the wrong version of get.