question on string iterators

Question

I have following code

#include <iostream>
#include <string>
#include <algorithm>

using namespace std;

int main(){
  string d="georgia is nice country":
  string::iterator my;
  for (my=d.begin();my!=d.end();my++) {
    cout<<*my<<endl;
  }
  return 0;
}

but it causes a compiler error saying "my is undefined". What is wrong?

`using namespace std;` This is saving you to type five characters every time you type an identifier from the standard library. See [this answer](http://stackoverflow.com/questions/2879555/c-stl-how-to-write-wrappers-for-cout-cerr-cin-and-endl/2880136#2880136) for why this is _not_ a good idea. — sbi, Jul 10 '10 at 10:09

score 1 · Answer 1 · answered Jul 10 '10 at 07:38

1

string d="georgia is nice country"; // <-- semicolon, not colon

answered Jul 10 '10 at 07:38

Praetorian

106,671
19
240
328

I thought for a moment there you were saying georgia is not a nice country. – Borealid Jul 10 '10 at 07:38
just here is a little poor live bit everything will be ok soon – Jul 10 '10 at 07:43

score 1 · Accepted Answer · answered Jul 10 '10 at 07:40

1

You need to put a semicolon instead of a colon after the string.

string d="georgia is nice country"; // <-- semicolon!

answered Jul 10 '10 at 07:40

In silico

51,091
10
150
143

question on string iterators

2 Answers2