need help with a method

[laasunde]

This code handles text + const char* :

text& text::operator+ (const char* c)
{
   //code
}

but how can I handle a const char* + text expression ?

[Fordy]

Originally posted by laasunde
This code handles text + const char* :

text& text::operator+ (const char* c)
{
   //code
}

but how can I handle a const char* + text expression ?

You need to overload the operator for + for that situation...you cant really do it for char* as you cant overload operators on a built in data type

text operator+ (const char* c,const text& t)
{
   //code
}

[Stoned_Coder]

operator + must return an object an not a reference.

For a detailed explanation consult the bible part 1 (effective C++ by Scott Meyers) item 23 (Dont try to return a reference when you must return an object).

[ammar]

operator + must return an object an not a reference.

Correct me if I'm wrong...
I think so because if you creat the object in the function it should be destroyed aftert the function is executed.

[Fordy]

Originally posted by ammar
Correct me if I'm wrong...
I think so because if you creat the object in the function it should be destroyed aftert the function is executed.

Yes....if you return a reference, the object you are providing the reference to will be out of scope, and therefore effectively out of existance by the time you use it.

[laasunde]

Originally posted by Fordy
Yes....if you return a reference, the object you are providing the reference to will be out of scope, and therefore effectively out of existance by the time you use it.

Are you saying this wont work ?

text& text::operator+ (const char* c)
{
  text *obj = new text()

  //code  

  return obj;
}

[The Dog]

Originally posted by laasunde
Are you saying this wont work ?

text& text::operator+ (const char* c)
{
  text *obj = new text()

  //code  

  return obj;
}

That will work because the memory is dynamically allocated, but this is what you musn't do :

text& text::operator+ (const char* c)
{
  text obj;

  //code  

  return obj;
}

[Fordy]

Originally posted by laasunde
Are you saying this wont work ?

text& text::operator+ (const char* c)
{
  text *obj = new text()

  //code  

  return obj;
}

Firstly, no it wont as you are trying to return a pointer, when the func user will expect a reference....

Secondly, its an stupid way of doing things as the user will have to call delete to free that memory for a simply operator+ addition, and even more worryingly they wont have to the option to even do this as they wont be able to even access the pointer to actually call delete on it!

[The Dog]

Originally posted by Fordy
Firstly, no it wont as you are trying to return a pointer, when the func user will expect a reference....

I didn't even see that, it should've been :

text& text::operator+ (const char* c)
{
  text *obj = new text()

  //code  

  return *obj;
}

[laasunde]

Originally posted by The Dog
I didn't even see that, it should've been :

text& text::operator+ (const char* c)
{
  text *obj = new text()

  //code  

  return *obj;
}

Sorry I forgot about the dereference operator.

[laasunde]

Originally posted by Fordy
Firstly, no it wont as you are trying to return a pointer, when the func user will expect a reference....

Secondly, its an stupid way of doing things as the user will have to call delete to free that memory for a simply operator+ addition, and even more worryingly they wont have to the option to even do this as they wont be able to even access the pointer to actually call delete on it!

Your first point was a typo, the dereference operator should have been there.

Your second point, I see your point but how would solve the issue ?

Appreciate any response.

Cheers

[Fordy]

Originally posted by laasunde
Your first point was a typo, the dereference operator should have been there.

It doesnt matter...you are still creating an opject on the heap that you have no real way of freeing with a call to delete..therefore you can expect some nasty memory leaks

Originally posted by laasunde
Your second point, I see your point but how would solve the issue ?

Always return an object by value in this situation....its not the most efficient way, but its the safe way

[laasunde]

Originally posted by Fordy
It doesnt matter...you are still creating an opject on the heap that you have no real way of freeing with a call to delete..therefore you can expect some nasty memory leaks

Always return an object by value in this situation....its not the most efficient way, but its the safe way

So my overloading method should look something like this then :

text text::operator+ (const char* c)
{
  text obj;

  //  

  return obj;
}

[laasunde]

Originally posted by Fordy
It doesnt matter...you are still creating an opject on the heap that you have no real way of freeing with a call to delete..therefore you can expect some nasty memory leaks

Any good utillitys that can help me find possible memory leaks ?

[Zahl]

[QUOTE Stoned_Coder ]
operator + must return an object an not a reference.

For a detailed explanation consult the bible part 1 (effective C++ by Scott Meyers) item 23 (Dont try to return a reference when you must return an object). [/QUOTE]

Why can't you just return a reference if that's effecient?