Help with ISO errors

Trying to get program to compile I keep recieving the following error:" ISO C++ forbids comparision between pointer and integer"

Any help is appreciated
Alpha22

Code:

int enterAmts (float Ca[])

{
    int i = 0;
    int idx = 0;
    int count = 0;
    char cont;
    string namefile;
    string catName;
    float catAmts;
    float numcat = 0;
    
    infile.open(namefile.c_str());    
    infile >> catName[idx];  
   
   
   while(!infile.eof()){
       infile >> catName[idx];
       numcat ++;
       }   
       
    while(i < 10 && catName [idx]!= "zzz"){ //problem area
    
       cout << "Enter amt for " << catName[i] << "$" << flush;
       runResults << "Enter amt for " << catName[i] << "$" << endl;
       cin >> catAmts[i];
       runResults << catAmts[i] << endl;
       count += i;
       i++;
     }//endwhile
     infile.close();
    //  outfile.close();
  return 0;
}//endmain

Lose the space between the variable name and index. Also that index represents a single character and therefor will never equal "zzz"

Lose the space between the variable name and index.

Why? That will only make it look better. There's nothing wrong with the space.

Use the array without an index reference.

Code:

string catName;

If I then do this:

Code:

catName = "Sam";

what is catName[0]? What type is it? How do you indicate a char literal? Then, can this be correct:

Code:

catName [idx] != "zzz"

Should I change the inx to just i?

No. You want to change it to just plain "catName". catName[anything] is a character, and catName is a string. You're comparing in to a string, so you want catName, not catName[something].

CatName is a string of names for budget category's.

No, it isn't.

Code:

string catName;

Originally Posted by dwks

Why? That will only make it look better. There's nothing wrong with the space.

Use the array without an index reference.

Ick, I hate the way it looks. Personally, I think the index should appear as an extension of the variable, not a seperate thing. Anyway, the idea was really him not comparing against the single character, but I also think he was trying to compare against a substring, not the whole string.

If you are looking for a specific substring and you know where it would start in the string, then you can use the substr() function.

Code:

catName.substr(6,3); // substr([Starting Index], [Size of substring])

Thanks dwks and 7stud damm thing will compile now. Another set of eyes always helps.