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"

Quote:

---

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;

---

Quote:

---

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.