Non e' bello dichiarare e soprattutto istanziare un oggetto connection globale per l'intera pagina. Se proprio vuoi lasciare un oggetto connection visibile all'intera pagina, limitati a dichiararlo e non anche a istanziarlo. Nel page_load lo istanzi e nel page_prerender lo chiudi. Altrimenti, dichiara istanzia usa e chiudi una connection ogni qualvolta ti serve e nel punto (procedura) in cui ti serve.
Per il resto, mi sembra che il codice cosi com'e' scritto sia anche vulnerabile ad attacchi di sqlinjection. Fossi in te, non prenderei un valore dalla querystring per metterlo in una query sql senza neanche controllarlo. Se proprio non riesci a fare a meno della querystring piuttosto utilizza un parametro che almeno limita la possibilita' di sqlinjection.