'Contact' does not contain a constructor that takes 3 arguments

Question

0.00/5 (No votes)

See more:

Hello! I'm trying to make a phonebook program. I want to add a contact by this line:

C#

Contact person = new Contact("Bob", "Smith", "0500399");

The problem is that I get an error saying

C#

'Contact' does not contain a constructor that takes 3 arguments

My 'Contact' class looks like this:

C#

class Contact
    {
        public string FristName { get; private set; }
        public string LastName  { get; private set; }
        public string PhoneNr   { get; private set; }

        public void CreateContact (string firstname, string lastname, string phonenr)
        {
            this.FristName = firstname;
            this.LastName = lastname;
            this.PhoneNr = phonenr;
        }
    }

I know that I could set these values by doing

C#

person.CreateContact("Bob", "Smith", "0500399")

This is not what I want. I want to add it the same time I make the instance.

I'm new to programming, so please bear with me. Thank you in advance :-)

Posted 21-Nov-15 2:23am

Member 12039355

Updated 21-Nov-15 2:29am

v3

Add a Solution

2 solutions

Solution 2

To fix Bills code a bit (he didn't actually add a constructor!), your Contact class would look more like this:

public class Contact
{
    public string FirstName { get; private set; }
    public string LastName  { get; private set; }
    public string PhoneNr   { get; private set; }

    // A public constructor
    public Contact(string firstName, string lastName, string phoneNr)
    {
        this.FirstName = firstName;
        this.LastName = lastName;
        this.PhoneNr = phoneNr;
    }

    // CreateContact should return a new Contact object.
    // That's what Create... implies.
    // You'd use it like this:
    //     Contact myContact = Contact.CreateContact(f, l, p);
    public static Contact CreateContact(string firstName, string lastName, string phoneNr)
    {
        Contact newContact = new Contact(firstname, lastname, phonenr);
        return newContact;
    }
}

Posted 21-Nov-15 4:06am

Dave Kreskowiak

Updated 21-Nov-15 4:46am

v2

Comments

BillWoodruff 21-Nov-15 10:34am

+5 for covering my back :) (my answer edited)

Dave Kreskowiak 21-Nov-15 10:46am

No problem! ;)

Member 12039355 21-Nov-15 11:39am

Thanks! +5 :)

Krunal Rohit 21-Nov-15 11:20am

5!
-KR

Add a Solution

Add your solution here

Treat my content as plain text, not as HTML

Preview 0

…

Existing Members

Sign in to your account

...or Join us

Download, Vote, Comment, Publish.

Your Email
Password
Forgot your password?

Your Email
This email is in use. Do you need your password?
Optional Password

I have read and agree to the Terms of Service and Privacy Policy
Please subscribe me to the CodeProject newsletters

When answering a question please:

Read the question carefully.
Understand that English isn't everyone's first language so be lenient of bad spelling and grammar.
If a question is poorly phrased then either ask for clarification, ignore it, or edit the question and fix the problem. Insults are not welcome.
Don't tell someone to read the manual. Chances are they have and don't get it. Provide an answer or move on to the next question.

Let's work to help developers, not make them feel stupid.

This content, along with any associated source code and files, is licensed under The Code Project Open License (CPOL)

BillWoodruff · Accepted Answer · 2015-11-21T03:52:00

Why don't you just make the Constructor of the Class take parameters:

C#

public Contact(string firstname, string lastname, string phonenr)
{
    this.FristName = firstname;
    this.LastName = lastname;
    this.PhoneNr = phonenr;
}

Of course, if you do that, you really don't need a separate 'CreateContact method.

If you expect that some contacts will not have a phone number, you can make that parameter optional:

C#

public Contact(string firstname, string lastname, string phonenr = "")
{
    this.FristName = firstname;
    this.LastName = lastname;
    this.PhoneNr = phonenr;
}