What is wrong with this class definition?

Question

I have a class like this:

class connect:IDisposable { public void OpenChannel(SqlConnection ch) { ch.ConnectionString="....."; ch.Open(); } public void Dispose() { } }

And another class like this:

public Cust { SqlConnection channel=new SqlConnection(); SqlCommand command=new SqlCommand(); public void Method() { using(connect con=new connect()) { con.OpenChannel(channel); command.connection=channel; ..... .... .... command.ExecuteNonQuery(); } }

But when I run ExcuteNotQuery() there is an error: "no open connection" So what is wrong?

Not technically wrong, but connect should start with a capital C. — BoltClock
– BoltClock, Commented Aug 6, 2011 at 15:59
it would help if you posted a working example that shows the problem extracted from the real code — BrokenGlass
– BrokenGlass, Commented Aug 6, 2011 at 16:03
This code won't even compile - so it's possible you've mis-transcribed the bug anyway... can you paste extracts from the actual code? — James Gaunt
– James Gaunt, Commented Aug 6, 2011 at 16:03
yeah the 'class' keyword is missing but it shouldn't compile without it. — IbrarMumtaz
– IbrarMumtaz, Commented Aug 6, 2011 at 16:04
On a design level - presumably Cust is a customer - not sure that a customer style entity usually has it's own connection/command fields. Unrelated to the issue - I'm just saying... — Marc Gravell
– Marc Gravell, Commented Aug 6, 2011 at 16:06

Akash Kava · Accepted Answer · 2011-08-06 16:04:21Z

public Cust { SqlConnection channel=new SqlConnection(); SqlCommand command;//=new SqlCommand(); public void Method() { using(connect con=new connect()) { con.OpenChannel(channel); //command.connection=channel; // create command from open connection command = channel.CreateCommand(); ..... .... .... command.ExecuteNonQuery(); } }

Collectives™ on Stack Overflow

What is wrong with this class definition?

1 Answer 1

Comments

Hot Network Questions

Collectives™ on Stack Overflow

1 Answer 1

Comments

Related