## C++: what did I do wrong?

Since I'm doing this for a problem of sorts on a site anyone can join, I won't release my whole code, but here's what I tried.

Code:
#include <iostream>

enum month {jan,feb,mar,apr,jun,jul,aug,sep,oct,nov,dec};
enum weekday {sun,mon,tue,wed,thur,fri,sat};

int year;
int numberofdays(month);
void plusone7(weekday&);

int main()
{
int number = 0;
int suns = 0;
//Some code
std::cout << number << " days total." << std::endl;
weekday dayofweek;
for (int day = 1,dayofweek = mon;day<=number;day++,plusone7(dayofweek)) //"No matching function for call to plusone7"
{
if (dayofweek == sun)
{
suns++;
}
}
std::cout << suns << " is the number of sundays" << std::endl;
}

//Defines a few functions

{
{
}
else
{
}
}
But the prototype and the bit at the end would make there a matching function, I thought?

 PhysOrg.com science news on PhysOrg.com >> Hong Kong launches first electric taxis>> Morocco to harness the wind in energy hunt>> Galaxy's Ring of Fire
 Mentor How do you know there's something wrong? Do you get a compiler error message? Does it compile, but not produce the output you expected?
 No, it doesn't compile, the error message is put in as a comment in the code.

Mentor

## C++: what did I do wrong?

Aha, I missed the error message before because it was out of view to the right. I had to scroll the text-box over, in order to see it.

The prototype for plusone7() looks like it matches OK.

Here's a possibly wild guess: it might be that you can't use a void function with the comma operator. The expression "day++,plusone7(dayofweek)" wants to return the value of plusone7(dayofweek), so the compiler may be looking for a non-void version of plusone7().

Try moving plusone7(dayofweek) to the beginning of the loop body and see if that makes a difference.

Do a Google search for "comma operator in C++" and you'll find some explanations of how the comma operator works.

 Huh. Code:  for (int day = 1,dayofweek = mon;day<=number;day++) { if (dayofweek == sun) { suns++; } plusone7(dayofweek); } doesn't help.
 Try defining the function plusone7 at the beginning of your script (immediately after calling int main{}, or at least before calling it during your loop). Some compilers are rather funny about these things.

 Quote by Number Nine Try defining the function plusone7 at the beginning of your script (immediately after calling int main{}, or at least before calling it during your loop). Some compilers are rather funny about these things.
Also tried that, unfortunately didn't help. :(

I did figure out what was wrong, though: http://www.cplusplus.com/forum/beginner/72969/

 Mentor Lesson to be learned here: Beware the comma operator my son. Especially beware thinking that a comma is the comma operator when it is in fact that comma is a separator. The comma operator can be oh-so-nice in collapsing multiple lines into one, particularly with for loops. My advice is to avoid that temptation toward that kind of cleverness. Don't do it unless the alternative is even harder to understand.
 Admin Wow, a nice one. I think it has quality of a good homework or test problem for the C++ class.
 @Borek: It's for a problem from a very well-known site.