C# strange problem with constructor

Question

I have IMHO very strange problem with three-argument constructor , When I try to run the program visual studio shows me only one bug: " 'Sort.HeapSort' does not contain a constructor that takes 3 arguments 112 35" .

namespace Sort
{
    class HeapSort
    {
        private int[] A;
        private int heapSize;
        private int min; 
        private int max; 
        Random myRandom = new Random(); 

        HeapSort(int size, int min1, int max1) //this is the three argument constructor.
        {
            heapSize = size - 1;
            min = min1;
            max = max1;
            A = new int[size];
        }    
    }

    class Program
    {
        static void Main(string[] args)
        {
            int size = 30;
            int min = 0;
            int max = 100;

            HeapSort myHeapSort = new HeapSort(size,min,max); //In this line is the bug
        }
    }
}

perhaps you should decorate your classes with the proper access modifiers. Public, Private, protected, etc. — StingyJack
– StingyJack, Commented Nov 17, 2010 at 16:36
Could you fix the odd formating? (Moglbys naprawiac dziwny format?) — Chris Pfohl
– Chris Pfohl, Commented Nov 17, 2010 at 16:37
8 Answers in a blink of a sloth eye :pLooks like everyone is in the starting block — hoang
– hoang, Commented Nov 17, 2010 at 16:40

cdhowie · Accepted Answer · 2010-11-17 16:35:39Z

9

Your constructor is declared private, since you omitted an access specifier. Add the public keyword prior to the constructor definition.

answered Nov 17, 2010 at 16:35

cdhowie

172k25 gold badges303 silver badges324 bronze badges

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

1 Comment

Redwood Over a year ago

You should check the box beneath the update/down vote buttons to accept his answer then instead of editing the title to mark it solved.

Paul Sasik · Accepted Answer · 2010-11-17 16:35:53Z

3

You need to specify your constructor as public, thusly:

public HeapSort(int size, int min1, int max1)

The compiler allows you to omit the accessibility specifier and defaults to private. Bit of a syntactic sugar gotcha IMO that I wish would be done away with.

So, since you have a private constructor, your client code does not "see" it and the compiler tries to call the public constructor which naturally results in the error you're seeing.

answered Nov 17, 2010 at 16:35

Paul Sasik

81.9k23 gold badges152 silver badges194 bronze badges

Comments

Justin Niessner · Accepted Answer · 2010-11-17 16:35:53Z

2

Your constructor isn't public, it's private (you didn't include any modifiers so it defaults to private) therefore the calling code can't "see" it.

Change:

HeapSort(int size, int min1, int max1)

To:

public HeapSort(int size, int min1, int max1)

answered Nov 17, 2010 at 16:35

Justin Niessner

246k40 gold badges416 silver badges546 bronze badges

Comments

Igal Tabachnik · Accepted Answer · 2010-11-17 16:36:01Z

1

You need to add public to your constructor, otherwise it's considered private, and therefore inaccessible from within your Main().

answered Nov 17, 2010 at 16:36

Igal Tabachnik

31.6k15 gold badges96 silver badges159 bronze badges

Comments

LukeH · Accepted Answer · 2010-11-17 16:36:07Z

1

The constructor with three params has no accessibility modifier on it and so defaults to private.

Change the declaration to public HeapSort(int size, int min1, int max1) and you'll be good-to-go.

answered Nov 17, 2010 at 16:36

LukeH

271k59 gold badges373 silver badges411 bronze badges

Comments

Nickolay Olshevsky · Accepted Answer · 2010-11-17 16:36:45Z

0

It looks like you are missing 'public' keyword before constructor.

answered Nov 17, 2010 at 16:36

Nickolay Olshevsky

14.2k2 gold badges38 silver badges49 bronze badges

Comments

hoang · Accepted Answer · 2010-11-17 16:37:21Z

0

Make your constructor public !

answered Nov 17, 2010 at 16:37

hoang

1,9171 gold badge24 silver badges34 bronze badges

Comments

Jason · Accepted Answer · 2010-11-17 16:38:09Z

0

Your constructor is private. It needs to be public.

answered Nov 17, 2010 at 16:38

Jason

1,9791 gold badge16 silver badges19 bronze badges

Collectives™ on Stack Overflow

C# strange problem with constructor

8 Answers 8

1 Comment

Comments

Comments

Comments

Comments

Comments

Comments

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

8 Answers 8

1 Comment

Comments

Comments

Comments

Comments

Comments

Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Related