Thread: How can i make a "letter could not be found" break?

I've made a small vector, that holds single characters, and i've made a small function that iterates through the vector to find a letter the user inputs. It works fine if the letter is in the vector, but how could i make it say "letter could not be found" or something similar? here's my function if it's any use:

Code:

char search(vector<char> line, char letter)
{
     char temp;
     vector<char>::const_iterator iter;
     iter = line.begin();

     while (*iter != letter)
     {
          if (*iter == letter)
          {
               temp = *iter;
          }
          else
          {
               iter++;
          }
     }

return temp;
}

Code:

  if ( iter == line.end() )
    cerr<<"The letter you specified could not be found"<<endl;
return temp;
}

Of course there are other problems (like what is the value of temp if the letter isn't found?

Initialize temp to some value that you know cannot be in the vector (like '\0'). If after the loop is done, the temp character is still that initial value, then you know the letter wasn't found.

However, since you are just returning the letter that the user input, you don't need the temp variable at all. You can make temp a bool that is false and set it to true when you find the letter. At that point you can even break out of the while loop. Then, at the end of your function, check the bool variable and output the error if it wasn't found.

Finally, good program design would probably actually have you just return true or false if the letter was found (or the index it was found at) and let the calling code output the error.

Code:

while (*iter != letter)

This is asking for a crash....

lol, well if it's asking for a crash then how else could i do it? with my limited experience it was the only way i could do it. Maybe using something like

Code:

for(iter = line.begin(); iter != end(); iter++)

??

And instead of using vector<char> you can use std::string

alright thanks guys

never used std::string before, so i think ill stick to what i know for the time being

Welshy, I think if you just take a few minutes to look at the capabilities of the string class, you'll immediately recognize its usefullness over C-strings.

but he's not using c-strings, he using a vector of chars. Now maybe he's storing sentences, which would be better in a string but maybe not.

I'm just learning how to use vectors and iterators, so i chose char for the simplicity factor. Line just stores a, b, c, d.

for(iter = line.begin(); iter != line.end(); ++iter)

That would be a good way to go. Make sure you determine whether you found the character or not by checking to see if the iter == line.end() after the loop or using a variable to remember that a match was found.

I've managed to get somewhere, in that it now recognises when a letter is or not found, but i get the positions as 97, 98, 99, 100, when the positions should be 0,1,2,3.

Code:

int search(vector<char> line, char letter)
{
     int temp = 0;
     vector<char>::const_iterator iter;

     for(iter = line.begin(); iter != line.end(); iter++)
     {
              if (*iter == letter)
              {
                       temp = *iter;
              }
              else if (iter == line.end())
                   break;
     }
return temp;
}

Code:

int main()
....
                      cout << "What letter would you like to search for? ";
                      cin >> letter;
    
                      position = search(line, letter);
                      
                      if (position == 0)
                      {
                               cout << "Sorry but that letter could not be found";
                      }
                      else
                      {
                               cout << "\n\n" << letter << " is in position " << position;
                      }
....

Ah your a star Darryl, dont know how i didnt think of that before, i was nearly there. Have some rep points