[dbthegrandtour]

I am trying to build a very basic class in C++ I have a header file

#ifndef time
#define time

class time
{
	int month;		
				
};
#endif 

and i and just trying to cin month and cout month in my main

#include <iostream>
#include "time.h"
using namespace std;

int main()
{
	cin >> time.month;
	cout << time.month << endl;
}

I keep getting this error "error C2228: left of '.month' must have class/struct/union"
what is wrong with this code?

[cam888]

Try

class time
{
	   public:
			int month;
};

[sparkart]

You need to create an instance of the time class:

time my_time;

cin >> my_time.month;

And as cam pointed out.... month must be set to public

Not specifiying: public, private, protected defaults the member to private.

This post has been edited by e_barroga: 22 February 2009 - 03:21 PM

[programble]

Also, when using #ifndef time and #define time, you shouldn't be defining using the same name of your class. Use _TIME or something similar instead.

[dbthegrandtour]

e_barroga, on 22 Feb, 2009 - 02:20 PM, said:

You need to create an instance of the time class:

time my_time;

cin >> my_time.month;

And as cam pointed out.... month must be set to public

Not specifiying: public, private, protected defaults the member to private.

i have tried the my_time and it still gives me the same error "error C2228: left of '.month' must have class/struct/union"

[sparkart]

You might want to adjust three things with your source code.

For your defines in your header file, use something other than your class name:

#ifndef TIME_H
#define TIME_H

//Omitted Code

#end /*TIME_H*/

If you want to access class members outside of the class, you must make the member public

class my_class {
  public:
	int a_public_member;
};

Create an instance of your class:

int main () {
  my_class an_instance;
  cin >> an_instance.a_public_member;
  return 0;
}

[FrozenSnake]

I don't use names of already existing classes even if the header isn't included (time is in time.h) so I changed it to ttime.
Anyway I did like this:
ttime.h

#ifndef _TTIME_H_
#define _TTIME_H_

class ttime
{
	   public:
			int month;
};

#endif

main.cpp

#include <iostream>
#include "ttime.h"
using namespace std;

int main()
{
	ttime t;
	cin >> t.month;
	cout << t.month;
}

And it compiles and run for me.

This post has been edited by FrozenSnake: 22 February 2009 - 03:45 PM