Sending SqlDataReader to constructor as parameter and constructor argument error

Question

I created a public class and wrote a public constructor with parameters like:

public Patient(SqlDataReader reader, string p) {
        if (p == "L") {
            Name = reader[0].ToString();
        }
        else { }
    }

then I used this constructor like

SqlDataReader reader = command.ExecuteReader();
                while (reader.Read())
                {
                    Patient myP(reader, "L");                    
                }

Then during debug, I got the error: Error 49 Expected ; or = (cannot specify constructor arguments in declaration)

Where is the problem and how can I fix it?

I should say: you might find it more effective to use tools like ORMs or micro-ORMs to populate the objects; the "use a data-reader to fill an object" is pretty-much a solved problem — Marc Gravell
– Marc Gravell, Commented May 29, 2013 at 6:50

Marc Gravell · Accepted Answer · 2013-05-29 06:47:58Z

5

That syntax simply isn't valid; try:

Patient myP = new Patient(reader, "L");

or (identical result):

var myP = new Patient(reader, "L");

answered May 29, 2013 at 6:47

Marc Gravell

1.1m273 gold badges2.6k silver badges3k bronze badges

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

Comments

Soner Gönül · Accepted Answer · 2013-05-29 06:49:42Z

1

You should use constructor with new operator like;

Patient p = new Patient(reader, "L");

Your syntax is invalid for C#.

From new Operator (C# Reference)

Used to create objects and invoke constructors.

answered May 29, 2013 at 6:49

Soner Gönül

99.1k103 gold badges224 silver badges375 bronze badges

Collectives™ on Stack Overflow

Sending SqlDataReader to constructor as parameter and constructor argument error

2 Answers 2

Comments

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Related