[C#]semplice utilizzo del db...

[the incredible]

Sto facendo un progetto per l'uni su c#, mi servono fare della query, update e delete.

Ho quindi pensato di fare una classe DB per fare la connessione al db, la chiusura e due metodi uno per insert/update e delete l'altro per le query.

Allora nel dettaglio quello che non riesco a capire è quando chiudere il db perchè il metodo che fa le query mi da problemi...

Eccolo nel dettaglio:

public SqlDataReader ExecuteQuery(string sql) {
this.GetConnection(); //metodo per connesione db
SqlCommand cmd = new SqlCommand(sqlQuery, this.connection);
SqlDataReader myReader = null;
try {
myReader = cmd.ExecuteReader();
}
catch (SqlException e) { }
this.CloseDB();//metodo per chiusura db
return myReader;
}

Secondo voi è giusto per ogni query aprire e chiudere il db?
Ed è sensato passare il myreader come ritorno?

[Jeremy75]

A mio avviso è bene non lasciare mai la connessione al Db aperta...quindi ritengo corretto aprire e chiudere ad ogni esecuzione delle query se......non sono contestuali, nel senso che, se avessi una stringa di Update ed una di Delete o anche dieci già tutte pronte ....creerei una transazione che esegue entrambe le istruzioni dopo di che chiuderei la connessione.
IMHO