Getting strange error while trying to set up an iterator

Question

0.00/5 (No votes)

See more:

This is probably a silly error but i cant seem to find what i have done wrong.

The error i am getting is 'no operator "=" matches these opperands'.

Here is my code...

C++

void print_words(const map < string, int >& m1) {
    map<string, int>::iterator it;
    cout << "Number of non-empty words: " << m1.size() << '\n';
    
    int count = 0;
    for (it = m1.begin(); it != m1.end(); it++) {
    
    }
}

I get the error in the for loop in the it = m1.begin() statement and i cannot go on to print out the map if i cant iterate through it so any help would be much appreciated.

What I have tried:

I tried finding any simple typos or errors but cannot find any. I set up an iterator in another method and it worked completely fine so i am really stumped on why i am getting this error.

Posted 30-Sep-17 18:01pm

Member 13439041

Updated 30-Sep-17 21:19pm

Richard MacCutchan

v2

Add a Solution

Comments

Richard MacCutchan 1-Oct-17 3:17am

What error?

1 solution

Add a Solution

Add your solution here

Treat my content as plain text, not as HTML

Preview 0

…

Existing Members

Sign in to your account

...or Join us

Download, Vote, Comment, Publish.

Your Email
Password
Forgot your password?

Your Email
This email is in use. Do you need your password?
Optional Password

I have read and agree to the Terms of Service and Privacy Policy
Please subscribe me to the CodeProject newsletters

When answering a question please:

Read the question carefully.
Understand that English isn't everyone's first language so be lenient of bad spelling and grammar.
If a question is poorly phrased then either ask for clarification, ignore it, or edit the question and fix the problem. Insults are not welcome.
Don't tell someone to read the manual. Chances are they have and don't get it. Provide an answer or move on to the next question.

Let's work to help developers, not make them feel stupid.

This content, along with any associated source code and files, is licensed under The Code Project Open License (CPOL)

Shao Voon Wong · Answer 1 · 2017-09-30T20:52:00

Just use auto keyword to define your variable type. C++ compiler will infer the correct type for you.

C++

void print_words(const std::map < std::string, int >& m1) {
	std::cout << "Number of non-empty words: " << m1.size() << '\n';

	int count = 0;
	for (auto it = m1.begin(); it != m1.end(); ++it) {

	}
}