C# constructor overloading invalid arguments

Question

Trying to execute the code below prevents me from moving forward. In my "Theline" class I have a constructor that accepts two arguments (int) but the error message I get says "Cannot convert from 'ConsoleApplication1.Point' to 'int'" What am I missing?

public static void Main()
     {
     Point a0 = new Point(0, 0);
     Point a1 = new Point(-1, -1);
     Theline line = new Theline(a0, a1);
     }

You're missing the fact that there is no way to convert from a Point to an int. Why do you think this is legal code? What do you expect to happen? — Lasse V. Karlsen
– Lasse V. Karlsen, Commented Sep 12, 2015 at 20:44
You should change your TheLine class constructor to accept Points because to define the line you need two points. — Hamlet Hakobyan
– Hamlet Hakobyan, Commented Sep 12, 2015 at 20:45
in 2d scene point have two ints (x and y). if you want to create line from two points you have to pass in Theline two points. and there are some drawing algorithms such as Bresenham's Line Algorithm you can search for that in google. — M.kazem Akhgary
– M.kazem Akhgary, Commented Sep 12, 2015 at 20:52

Slashy · Accepted Answer · 2015-09-12 20:42:09Z

3

Your Theline connstuctor accepts int and you send it a Point object...

Simply change it to

Theline line = new Theline(a0.X, a1.Y);

You can either change the Theline constructor parameters to type of Point.

Or anything else you need..

Goodluck.

answered Sep 12, 2015 at 20:42

Slashy

1,8913 gold badges28 silver badges47 bronze badges

Sign up to request clarification or add additional context in comments.

Comments

user1176420 · Accepted Answer · 2015-09-12 20:44:58Z

0

Indeed, what Slashy says.

Or since you want a line to be created, make your TheLine constructor accept 2 points instead of ints.

answered Sep 12, 2015 at 20:44

user1176420

1232 silver badges9 bronze badges

2 Comments

pellep Over a year ago

How do I do that exactly? public TheLine(int a0.X, a0.Y, a1.X, a1.Y)?

user1176420 Over a year ago

It depends on your implementation of the Theline class, can you show me the code? Your suggested answer would work if you change your constructor so it accepts 4 ints. But letting it accept 2 Points seems more logical to me.

code4life · Accepted Answer · 2015-09-12 22:34:35Z

0

If you really, really want to use the Point, then I guess you could try:

public Theline(Point pointA, Point pointB) : this (pointA.X, pointB.Y)
{
}

answered Sep 12, 2015 at 22:34

code4life

15.8k8 gold badges56 silver badges82 bronze badges

Collectives™ on Stack Overflow

C# constructor overloading invalid arguments

3 Answers 3

Comments

2 Comments

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

3 Answers 3

Comments

2 Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Related