C++ error C2011

[lionofgod]

Hi, Im working with making my first class and I have everything set up but when i try to build, i get the apparently "famous" c2011 error. I've tried different methods of solving this problem after searching online, but nothing worked.
Here is my code
This code creates the gradebook object

// 

#include "stdafx.h"
#include <iostream> 
#include "gradebook.h" //this is the header file for my class

using namespace std;



int main()
{
	
	Gradebook mygradebook ("CS 101") ; // 

}

This is my header file.

#include "stdafx.h"
#include <iostream>

#include <string>

using namespace std;

class Gradebook {
public:
	Gradebook (string name);
	void Setname(string name);
	string Getname();
	void DisplayMessage( );
private :
	string Coursename ; };

This is my .cpp file, for the header,

#include "stdafx.h"
#include "gradebook.h"
#include <string>





	using namespace std;
class Gradebook {
public:
	Gradebook::Gradebook (string name){  //constructor statement
	Setname(name);
	}
	void Gradebook::Setname(string name) {   // Mutator method
		Coursename = name;
	}
	string Gradebook::Getname() {
	return Coursename ;
	}
	
	void Gradebook::DisplayMessage( ){ //display message statement
	cout << "Welcome To The Gradebook For " << Getname() << endl ; 
	}

private: 
	string Coursename ;
};

The error i get is

gradebook.cpp(11) error C2011: 'Gradebook' : 'class' type redefinition
          gradebook.h(8) : see declaration of 'Gradebook'

I've tried using the #indef, endif, method but i wasnt exactly sure how to use that and it didnt work out Can someone help me out here???

[manasij7479]

[EDIT: BOGUS POST; SEE 4th ONE INSTEAD]

I've tried using the #indef, endif, method but i wasnt exactly sure how to use that and it didnt work out. Can someone help me out here???

It seems you've correctly identified the problem.

So, after that, this will definitely help.

[Elysia]

Also some advice:
- Improve your indentation. Visual Studio's default indentation is good. If you don't know how to indent properly, then let it indent for you.
- Don't use "using namespace x" in headers. Use fully qualified types (ie std::string instead of string). You should easily find a why with a simple google search.

[Ushakal]

This isn't a problem with inclusion guards, though it's a good idea to have them anyway. The problem is that you're "redefining" your class in the .cpp file. You should remove the public/private identifiers, the private members, and the "class Gradebook" bit from the .cpp file and you should stop getting that error.

For example, this is how it should look:

// Example.h File
class Example
{
public:
     int Function();
private:
     int m_data;
};

 // Example.cpp file

#include "Example.h"

int Example::Function()
{
     // ...
     return m_data;
}

[manasij7479]

The problem is that you're "redefining" your class in the .cpp file.

OMG. I can't believe I could not see it.
Thank You very much for spotting it !

[lionofgod]

This isn't a problem with inclusion guards, though it's a good idea to have them anyway. The problem is that you're "redefining" your class in the .cpp file. You should remove the public/private identifiers, the private members, and the "class Gradebook" bit from the .cpp file and you should stop getting that error.

For example, this is how it should look:

// Example.h File
class Example
{
public:
     int Function();
private:
     int m_data;
};

 // Example.cpp file

#include "Example.h"

int Example::Function()
{
     // ...
     return m_data;
}

Im sorry, but I lost you, I removed the public/ private identifiers
Now my code looks like this

#include "stdafx.h"
#include "gradebook.h"
#include <string>

	using namespace std;
 class Gradebook{ //if i omit class Gradebook, then nothing works 

	Gradebook::Gradebook (string name){  //constructor statement
	Setname(name);
	}
	void Gradebook::Setname(string name) {   // Mutator method
		Coursename = name;
	}
	string Gradebook::Getname() {
	return Coursename ;
	}
	
	void Gradebook::DisplayMessage( ){ //display message statement
	cout << "Welcome To The Gradebook For " << Getname() << endl ; 
	}


	string Coursename ;
};

What else should I take out? If I take out class Gradebook, nothing will work?
BTW thanks for the replies

[Elysia]

#include "stdafx.h"
#include "gradebook.h"
#include <string>

	using namespace std;
 class Gradebook{ //if i omit class Gradebook, then nothing works 

	Gradebook::Gradebook (string name){  //constructor statement
	Setname(name);
	}
	void Gradebook::Setname(string name) {   // Mutator method
		Coursename = name;
	}
	string Gradebook::Getname() {
	return Coursename ;
	}
	
	void Gradebook::DisplayMessage( ){ //display message statement
	cout << "Welcome To The Gradebook For " << Getname() << endl ; 
	}


	string Coursename ;
};

Remove red parts.
If it doesn't work, specify the error. "Doesn't work" doesn't tell us much!

[lionofgod]

0.o Tyvm
I thought that if I only tried removing

class Gradebook

forgot to remove the

{ };

Everything works fine now, Thanks