Juan Juan - 4 months ago 21
C# Question

Is this a good way to execute queries in a SQL Server database from c#?

I have the next code to run queries in the database in my system. :

public static DataSet execute_query(string query)
DataSet ds = new DataSet();
SqlConnection con = new SqlConnection();
con.ConnectionString = DataAccessLayer.Properties.Settings.Default.cn;
SqlDataAdapter da = new SqlDataAdapter(query, con);
catch (Exception ex)
ds = null;
if (con.State == ConnectionState.Open) con.Close();

return ds;

  1. Is this a secure or good way to run queries in the database from c#?

  2. I have read about queries with parameters. How I can do this using parameters?


A few things I see that raise flags.

  1. Your unit of work and repository code are living in the same method. This limits transaction handling among other things. Meh - lots of concessions have been made with using DA code not implementing UOW / Repo. So it's not a requirement.
  2. You have a public method accept text to use as query. just make sure to separate that from any sort of user or api or good ole bobby drop tables may come to visit. (this answers your secure question - it is secure if it's seperated from the interfaces and your query is 'scrubbed' before coming in)
  3. DataSets - im not a fan, IMO they are very bloaty. I generally use a DataReader, populate IList<> and call it a day. But by all means, use the DataSets (or DataTables) if you like.
  4. No use of a using statement. You should dispose of objects implementing IDisposable, the using statement is an easy way to to this. Had you separated the UnitOfWork from the Repository then I wouldn't mention this - as the UnitOfWork would be disposed of elsewhere.

To use parameters make a SqlCommand object from your connection and query. In that object you can add parameters to the Parameters collection.

cmd.Parameters.AddWithValue("paramname", paramvalue);

Feed that to the DataAdapter Good luck!