Opening And Closing The DB Connection

Hi most of my code follows the following format in a lot of my pages, my question is; Am i approaching it the right way in terms of performance, sql injection attacks and anything someone more knowledgeable than myself can think off, please feel free to criticise the code. Thank youprotected void Page_Load(object sender, EventArgs e)

{string strID = Request.QueryString["id"];

SqlConnection conn = new SqlConnection(ConfigurationManager.ConnectionStrings["streamConnectionString"].ConnectionString);SqlCommand command = new SqlCommand("stream_EventsByID", conn);

command.CommandType = CommandType.StoredProcedure;command.Parameters.Add("@eventID", SqlDbType.Int).Value = Request.QueryString["id"];

conn.Open();

SqlDataReader reader = command.ExecuteReader(CommandBehavior.CloseConnection);

eventList.DataSource = reader;

eventList.DataBind();

conn.Close();

}

}