Please support our C++ advertiser: Programming Forums
Views: 3166 | Replies: 15
 |
•
•
Join Date: Mar 2005
Posts: 16
Reputation: 
Rep Power: 4
Solved Threads: 0
Newbie Poster
Help with Classes !!! (Homework) I am doing an assignment that has to use a class dayType.
The program has to set the day, print the day and return the next day.
I am receiving the error:
--------------------Configuration: Program9 - Win32 Debug--------------------
Compiling...
Program9.cpp
C:\Documents and Settings\Faculdade\ISM3232\Assig9\Program9\Program9.cpp(45) : error C2447: missing function header (old-style formal list?)
Error executing cl.exe.
Program9.exe - 1 error(s), 0 warning(s)
Can somebody help me please ?
I am reading the book 100 times but can figure out what is happening.
Thanks a lot
The program has to set the day, print the day and return the next day.
I am receiving the error:
--------------------Configuration: Program9 - Win32 Debug--------------------
Compiling...
Program9.cpp
C:\Documents and Settings\Faculdade\ISM3232\Assig9\Program9\Program9.cpp(45) : error C2447: missing function header (old-style formal list?)
Error executing cl.exe.
Program9.exe - 1 error(s), 0 warning(s)
Can somebody help me please ?
I am reading the book 100 times but can figure out what is happening.
Thanks a lot
#include <iostream>
#include <string>
using namespace std;
class dayType
{
public:
void setDay(char day);
void nextDay();
void printDay() const;
private:
string dayofweek[7];
int current;
char userEntry;
};
int main();
{
dayType myDay;
char day;
cout << "Please enter M for Monday, T for Tuesday, W for Wednesday" << endl;
cout << "R for Thursday, F for Friday, S for Saturday or U for Sunday" << endl;
cout << endl;
cin >> day;
myDay.setDay(day); // set the day
cout << endl;
myDay.printDay() const; // print the day
cout << endl;
myDay.nextDay(); // return the next day
cout << endl;
cout << myDay.printDay() << endl;
return 0;
} // end of main
void dayType::setDay(char day)
{
if (day = "m" || "M")
{
(dayofweek [0] == "MONDAY");
current = 1;
userEntry = day;
}
if (day = "t" || "T")
{
(dayofweek [1] == "TUESDAY");
current = 2;
userEntry = day;
}
if (day = "w" || "W")
{
(dayofweek [2] == "WEDNESDAY");
current = 3;
userEntry = day;
}
if (day = "r" || "R")
{
(dayofweek [3] == "THURSDAY");
current = 4;
userEntry = day;
}
if (day = "f" || "F")
{
(dayofweek [4] == "FRIDAY");
current = 5;
userEntry = day;
}
if (day = "s" || "S")
{
(dayofweek [5] == "SATURDAY");
current = 6;
userEntry = day;
}
if (day = "u" || "U")
{
(dayofweek [6] == "SUNDAY");
current = 7;
userEntry = day;
}
}
void dayType::nextDay()
{
{
current++;
if (current = 7)
current = 0;
}
{
current++;
if (current = 6)
current = 7;
}
{
current++;
if (current = 5)
current = 6;
}
{
current++;
if (current = 4)
current = 5;
}
{
current++;
if (current = 3)
current = 4;
}
{
current++;
if (current = 2)
current = 3;
}
{
current++;
if (current = 1)
current = 2;
}
}
void dayType::printDay() const
{
cout << "The day is: " << dayofweek << endl;
printDay();
}
•
•
Join Date: Feb 2005
Posts: 464
Reputation:
Rep Power: 4
Solved Threads: 18
Posting Pro in Training
Couple of problems:
if (day = "m" || "M")
must be
if ( day == 'm' || day == 'M' )
because day is a single character. (Even if it were a string you couldn't do a comparison like that, you'd need to use strcmp)
You must specify day once for each comparison. Plus, you want to do double = if you are checking for equality. A single = sets a variable...
if (day = "m" || "M")
must be
if ( day == 'm' || day == 'M' )
because day is a single character. (Even if it were a string you couldn't do a comparison like that, you'd need to use strcmp)
You must specify day once for each comparison. Plus, you want to do double = if you are checking for equality. A single = sets a variable...
•
•
Join Date: Jul 2005
Posts: 26
Reputation:
Rep Power: 4
Solved Threads: 0
Light Poster
The setday and nextday functions may be candidates for a switch. Which would be good because you can have it do something then in the default case if the user inputs something besides a char that corresponds to a day of the week.
•
•
Join Date: Mar 2005
Posts: 16
Reputation:
Rep Power: 4
Solved Threads: 0
Newbie Poster
OK. Thanks for the explanation. I fixed.
Do you see anything else wrong ?
What am I missing ?
Coulf you please give me a hint ?
Do you see anything else wrong ?
What am I missing ?
Coulf you please give me a hint ?
•
•
Join Date: Mar 2005
Posts: 16
Reputation:
Rep Power: 4
Solved Threads: 0
Newbie Poster
A switch inside a function on a class ? Is that OK ?
•
•
Join Date: Jun 2005
Location: Cambridge, MA
Posts: 1,330
Reputation:
Rep Power: 7
Solved Threads: 44
int main(); •
•
Join Date: Mar 2005
Posts: 16
Reputation:
Rep Power: 4
Solved Threads: 0
Newbie Poster
How do I get rid of this error:
Deleting intermediate files and output files for project 'Program9 - Win32 Debug'.
--------------------Configuration: Program9 - Win32 Debug--------------------
Compiling...
Program9.cpp
c:\documents and settings\faculdade\ism3232\assig9\program9\program9.cpp(45) : error C2447: missing function header (old-style formal list?)
Error executing cl.exe.
Program9.exe - 1 error(s), 0 warning(s)
Deleting intermediate files and output files for project 'Program9 - Win32 Debug'.
--------------------Configuration: Program9 - Win32 Debug--------------------
Compiling...
Program9.cpp
c:\documents and settings\faculdade\ism3232\assig9\program9\program9.cpp(45) : error C2447: missing function header (old-style formal list?)
Error executing cl.exe.
Program9.exe - 1 error(s), 0 warning(s)
•
•
Join Date: Feb 2005
Posts: 464
Reputation:
Rep Power: 4
Solved Threads: 18
Posting Pro in Training
Post the offending line or full code..
•
•
Join Date: Mar 2005
Posts: 16
Reputation:
Rep Power: 4
Solved Threads: 0
Newbie Poster
ahahahahahah
#include <iostream> // library to use cin and cout
#include <string> // library to strings
using namespace std;
class dayType
{
public:
void setDay(char day);
void nextDay();
void printDay() const;
private:
string dayofweek[7];
int current;
char userEntry;
};
int main();
{
dayType myDay; // class named dayType - object named myDay
char day;
cout << "Please enter M for Monday, T for Tuesday, W for Wednesday" << endl;
cout << "R for Thursday, F for Friday, S for Saturday or U for Sunday" << endl;
cout << endl;
cin >> day;
myDay.setDay(day); // set the day
cout << endl;
myDay.printDay() const; // print the day
cout << endl;
myDay.nextDay(); // return the next day
cout << endl;
cout << myDay.printDay() << endl;
return 0;
} // end of main
void dayType::setDay(char day)
{
if ( day == 'm' || day == 'M' )
{
(dayofweek [0] == "MONDAY");
current = 1;
userEntry = day;
}
if ( day == 't' || day == 'T' )
{
(dayofweek [1] == "TUESDAY");
current = 2;
userEntry = day;
}
if ( day == 'w' || day == 'W' )
{
(dayofweek [2] == "WEDNESDAY");
current = 3;
userEntry = day;
}
if ( day == 'r' || day == 'R' )
{
(dayofweek [3] == "THURSDAY");
current = 4;
userEntry = day;
}
if ( day == 'f' || day == 'F' )
{
(dayofweek [4] == "FRIDAY");
current = 5;
userEntry = day;
}
if ( day == 's' || day == 'S' )
{
(dayofweek [5] == "SATURDAY");
current = 6;
userEntry = day;
}
if ( day == 'u' || day == 'U' )
{
(dayofweek [6] == "SUNDAY");
current = 7;
userEntry = day;
}
}
void dayType::nextDay()
{
{
current++;
if (current = 7)
current = 0;
}
{
current++;
if (current = 6)
current = 7;
}
{
current++;
if (current = 5)
current = 6;
}
{
current++;
if (current = 4)
current = 5;
}
{
current++;
if (current = 3)
current = 4;
}
{
current++;
if (current = 2)
current = 3;
}
{
current++;
if (current = 1)
current = 2;
}
}
void dayType::printDay() const
{
cout << "The day is: " << dayofweek << endl;
printDay();
} •
•
Join Date: Feb 2005
Posts: 464
Reputation:
Rep Power: 4
Solved Threads: 18
Posting Pro in Training
A couple of issues:
int main(); should be int main() (no ;)
myDay.printDay() const; should be myDay.printDay();
You can't do this:
cout << myDay.printDay() << endl;
because printDay() does not return anything (it returns void)
Also,
You'll end up with some kind of endless loop/recursion here as you are calling your self inside:
void dayType::printDay() const
{
cout << "The day is: " << dayofweek << endl;
printDay();
}
int main(); should be int main() (no ;)
myDay.printDay() const; should be myDay.printDay();
You can't do this:
cout << myDay.printDay() << endl;
because printDay() does not return anything (it returns void)
Also,
You'll end up with some kind of endless loop/recursion here as you are calling your self inside:
void dayType::printDay() const
{
cout << "The day is: " << dayofweek << endl;
printDay();
}
|
•
•
•
•
Currently Active Users Viewing This Thread: 1 (0 members and 1 guests)
Linear Mode
