6

Is there any way to get the + operator to work for the Point object?

Take, for example, this tiny snippet:

this.cm1.Show((MouseEventArgs)e.Location+this.i_rendered.Location);

You see, I try to add two points to eachother. It just does not work (which was expected). I'd love to get this working.

Any ideas?

Cody Gray - on strike
  • 239,200
  • 50
  • 490
  • 574
Tgys
  • 612
  • 1
  • 11
  • 21

3 Answers3

10

I read the documentation for System.Drawing.Point (linked in Cody Gray's answer), and it has an instance method Offset. That method mutates the current Point (the designers chose to make Point a mutable struct!).

So here's an example:

var p1 = new Point(10, 20);
var p2 = new Point(6, 7);
p1.Offset(p2); // will change p1 into the sum!

In the same doc I also see an explicit conversion from Point to Size. Therefore, try this:

var p1 = new Point(10, 20);
var p2 = new Point(6, 7);
Point pTotal = p1 + (Size)p2; // your solution?
Jeppe Stig Nielsen
  • 60,409
  • 11
  • 110
  • 181
  • 1
    The second snippet gets somewhat close, yes, however, it looks terribly confusing. I think Cody Gray's answer is easiest and the clearest solution to apply in this case. – Tgys May 19 '12 at 11:51
7

It's not going to happen the way you expect. The only overload that the Point structure provides for the + (addition) operator is one that translates the coordinates of the Point by a Size.

There's no way to add two Point structures together, and I'm not even sure what that would mean.

Don't waste too much time figuring it out, either, considering that you cannot write extension methods that overload operators.

Fortunately, in a compiled language, there's no penalty for splitting up code into multiple lines. So you can re-write your code as follows:

Point newLocation = new Point(e.Location.X + this.i_rendered.Location.X,
                              e.Location.Y + this.i_rendered.Location.Y);
this.cm1.Show(newLocation);

Alternatively, you could use the Offset method, but I'm not convinced that enhances readability.

Community
  • 1
  • 1
Cody Gray - on strike
  • 239,200
  • 50
  • 490
  • 574
0

The simplest (and cleanest) solution is to just cast i_rendered.Location to a Size

Point newLocation = (e.Location + (Size)this.i_rendered.Location);

I don't see how or why anything needs to be split into convoluted X and Y components, when a simple cast does the job in the way the OP was asking for.