Thread: Class that defines a location??

I am tring to write a class that defines location based on latitude & longitude. I want to include members for East/West, North/South, and all latitude and longitude. Also a function must be included that will display the class's info and a function to set the info. Can anyone help?

I'm working with Visual C++ 2008 Express.

What s/he meant is, post your current code. What have you done so far?

This is what I have so far for the **.h file.

Code:

//location.h

class location
{
private:
	int degrees;
	int minutes;
	int	seconds;

public:
	location(int d, int m, int s);
	~location();

	void setdegrees(int d);
	void setminutes(int m);
	void setseconds(int s);

	int getdegrees();
	int getminutes();
	int getseconds();
	void properties();
};

These should be const functions:

Code:

int getdegrees() const;
int getminutes() const;
int getseconds() const;

since they don't (or at least shouldn't) modify the objects state.

It looks OK so far. So which parts are you having trouble with?

I'm not sure how where or how to get the East/West, North/South into the code. I would also like to initialize the class to a specific location. Is this something that can be done?

Here is what I have for my complete code, so far. If I make my funtions const I get errors.

Code:

//location.h

class location
{
private:
	int degrees;
	int minutes;
	int	seconds;

public:
	location(int d, int m, int s);
	~location();

	void setdegrees(int d);
	void setminutes(int m);
	void setseconds(int s);


	int getdegrees();
	int getminutes();
	int getseconds();
	void properties();
};



//location.cpp

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

using namespace std;

location::location(int d, int m, int s)
{
	degrees = d;
	minutes = m;
	seconds = s;
}

location::~location()
{

}

void location::setdegrees(int d)
{
	degrees = d;
}

void location::setminutes(int m)
{
	minutes = m;
}

void location::setseconds(int s)
{
	seconds = s;
}

int location::getdegrees()
{
	return degrees;
}

int location::getminutes()
{
	return minutes;
}

int location::getseconds()
{
	return seconds;
}

void location::properties()
{
	cout<<"  Degrees"<<getdegrees() <<endl;
	cout<<"  Minutes"<<getminutes() <<endl;
	cout<<"  Seconds"<<getseconds() <<endl;
}




//main

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

int main(int argc, char *argv[])
{
	location city(51, 30, 0);
	city.properties();
	
	system("PAUSE");
	return 0;
}

This is only giving lat or lon. Not both.

You can enhance your constructor in the following way:

Code:

location::location(int const& d, int const& m, int const& s) : degrees( d), minutes( m), seconds( s) { }

Initializing the attributes in the initializer list is faster.

Code:

void location::setdegrees(int const& d)
{
	degrees = d;
}

Passing by reference is faster, passing by const reference is faster and can prevent errors (if you erroneously modify the argument, the compiler will tell you). A good coding practice is to make input arguments 'const&' .

Just a link for my comment on the initializer list:

http://www.ubookcase.com/book/Addiso...8lev1sec2.html