The function itself is just returning as void. But, the point of the posted code is not about what the function is returning. It is the code related to using T-SQL and C# to return data from a SQL database that I would like reviewed. Especially the way the using statements are structured.
public void GetOffice(int syncID)
{
string strQry = @"
Select so.SyncID,
so.title
From Offices o
Left Outer Join SyncOffices so On so.id = o.SyncID
Where o.SyncID = @syncID
";
using (SqlConnection conn = new SqlConnection(Settings.ConnectionString))
{
using (SqlCommand objCommand = new SqlCommand(strQry, conn))
{
objCommand.CommandType = CommandType.Text;
objCommand.Parameters.AddWithValue("@syncID", syncID);
conn.Open();
SqlDataReader rdr = objCommand.ExecuteReader();
if (rdr.Read())
{
this.OfficeName= rdr.GetString(1);
}
rdr.Close();
}
}
}