Non è sbagliato, ma io farei così:
- crei 3 classi, una per aprire il db, una per ottenere il datareader e chiuderlo e una per chiudere la connessione.
In questo modo se nel pageload hai diverse query fare puoi richiamare una volta la apri_connessione, richiamare la leggi_dati n volte a seconda delle tue esigenze e poi alla fine dell' evento usi la chiudi_db.