Thread: To Call an Object Outside of Class

Hi...I'm new here. I'm also a beginning Computer Science student, C++ OOP. I have mostly successfully put together a program which contains two objects, firstDate and secondDate. This program accepts two dates, increments the dates by one week, and correctly identifies whether the incremented date is a leap year, what day of the week it is, and what month it is. I am stuck on one little step, however, and that is in trying to get the program to identify which of the two objects converted values are largest in a function definition. If anyone is willing to help me with how to call these values in the function listed below, it would be much appreciated.

First, I have defined the bool function in my header file, outside of the class:

Code:

bool isLarger(int, int);

Second, I have in my definition file this function definition(which doesn't function), also outside of the class and is the last one in the file:

Code:

bool isLarger(int, int)
{
	if (firstDate.convert() > secondDate.convert())
	{
		cout << "The first date entered is the most recent. " << endl << endl;
		return true;
	}
	else
	{
		cout << "The second date entered is the most recent. " << endl << endl;
		return false;
	}
}

Third, I have no idea how to call it in my application file, my main() function. Would it be something like:

Code:

CDate.isLarger(firstDate.convert, secondDate.convert);

I'm sure I'm just missing something totally obvious, but I appreciate the help. Let me know if I need to attach the entire file(s).

Many thanks

Thanks for replying so quickly!

I changed my call to this, and it compiled correctly:

Code:

bool isLarger(CDate firstDate, CDate secondDate)

How do I reference it from the main() function? One of my guesses (and the one with the least errors) was the following, but I received the error message "'CDate' : illegal use of this type as an expression":

Code:

isLarger(CDate firstDate, CDate secondDate)

Thanks again.

When passing arguments to functions, you pass then names of variables you want to pass, not including their type.
Then there's a question between by-value and by-const (or by pointer). By value makes copies of every variable you pass (which is what you do in your example). laserlight's example passed by reference - hence the objects aren't duplicated (which also means that function can modify them and the function that called it will also see the changes). There's also const, which implies that the function that takes the arguments shouldn't modify them.

Thank you!!

I changed the main call to

Code:

isLarger(firstDate, secondDate);

yet continued to receive an error message for unresolved externals. So I placed the call directly in my header file as well as the definition file and it does compile and run correctly now. I'll show - in my header file:

Code:

bool isLarger(CDate firstDate, CDate secondDate);

and in my definition file:

Code:

bool isLarger(CDate firstDate, CDate secondDate)
{
	if (firstDate.convert() > secondDate.convert())
	{
		cout << "The first date entered is the most recent. " << endl << endl;
		return true;
	}
	else
	{
		cout << "The second date entered is the most recent. " << endl << endl;
		return false;
	}
}

and in my main() function

Code:

isLarger(firstDate, secondDate);

Earlier in this conversation, Laserlight mentioned passing by constant reference. What did you mean by that? How do I make this more efficient?

Read above answer.
Also note that you should always have an exact copy of your functions in a header. They're called declarations. If they don't match, you can get unresolved external errors.

Wow Elysia, you replied just as I was sending off my question! Thank you...I'm going to go see if I understand what you were saying, then get back to the thread.

Thanks again.

So, would this line be passing by reference rather than by value?

Code:

bool isLarger(CDate& firstDate, CDate& secondDate)

I tried

Code:

 bool isLarger(const CDate& firstDate, const CDate& secondDate)

but received errors "cannot convert 'this' pointer from constant CDate to CDate&".

Yes, the first is correct. You're passing by reference when you add &.
The second should be correct too. Maybe you should post the code and what line the errors occours on?

Originally Posted by marQade

but received errors "cannot convert 'this' pointer from constant CDate to CDate&".

If you call isLarger() in CDate class, you might want to dereference 'this' when passing to function, as it is pointer.

Code:

isLarger(*this, secondDate)

instead of

Code:

isLarger(this, secondDate)

...not sure if this is your problem though.

Code:

bool isLarger(const CDate& firstDate, const CDate& secondDate)
{
	if (firstDate.convert() > secondDate.convert())
	{
		cout << "The first date entered is the most recent. " << endl << endl;
		return true;
	}
	else
	{
		cout << "The second date entered is the most recent. " << endl << endl;
		return false;
	}
}

The function declaration is identical.
I receive error message on the header line.

What is CDate, then? There's no such existing class to my knowledge. Is it your own defined class somewhere?
What's the declaration of this class? This error seems mysterious.