Help with beginning programmer

[perimate]

Hi,
I'm trying to write a simple program, but I keep getting errors.
Is there anything glaringly wrong in the following code? Thanks!

// INSTALLATION:  MTSU                                         
// REMARKS:  The program's goal is to pick out and print all of the valid email
// addresses in the information given and nothing else.
// 
// 
#include <iostream>
#include <fstream>
#include <ostream>
#include <string>
using namespace std;

int main()
{ 
   
                             
   string word;                    
   int i=0;
   char ch['@'];
   size_t found;
   
   ifstream fileIn;
   ofstream fileOut;
   fileIn.open("mail.dat");
   fileOut.open("addresses.dat");
   if (!fileIn)
   {
      cout << "Can't open the input file." << endl; 
      return 1;
   }
   while (! fileIn.eof())
   {
      fileIn >> word;
      found=word.find_first_of(ch);
      if (found!=string::npos);
      {
         fileOut << word;
         cout << word << endl;
         i++;
      }
   }
   // close files
   fileIn.close();
   fileOut.close();
   return 0;
}

[Warnis]

char ch['@'];
This defines a char array with (what I assume is) the size of the ascii value for @. Just define it like you do the int above.

[ceruleus]

@Warnis that's actually pretty funny... that's probably exactly what that evaluates to (unless the compiler just disallows it)

[S G H]

Warnis is right, and no compiler can disallow it.
You should do something like
char ch = '@';
or
char ch[] = "@";
I personally do not know which one is the right one, i don't use iostream that much, eventually try them both.

[zb0S216C]

+1 Warnis

@Perimate: Did you mean to use functional notation? char ch( '@' )?

_Wazzak

[zvRX92yv]

File streams aren't booleans! WHY DOES EVERYONE KEEP DOING THAT!?!

Use leStream.good().

And instead of endl, please use '\n', endl does a bunch of extra stuff like flushing the buffer..