Si è valida, sicuramente meglio che mettere le query nella pagina aspx.
Bisogna però valutare i costi di usare questo approccio.
Metti che hai una tabella di prodotti. Puoi fare una classe

public class Prodotto
{
public int idProdotto;
....
}

Per rappresentare il singolo prodotto e fare una classe GestioneProdotti o la classe Prodotto stessa, con metodi statici per fare Update Insert e delete, es:

public static Prodotto GetProdotto(int idProdotto)
{
// chiamata alla sp
Prodotto o = new Prodotto()
p.idProdotto = readersp["idProdotto"]..
}

Una cosa del genere. Che con whidbey si farà ancora più in fretta

L'approccio che ti consiglio io è, niente query sql nella classe, solo richiamo alle sp che fanno tutto. Quello che fai nelle classi è avere i vari metodi che fanno da ponte tra la classe che rappresenta il prodotto e il db sql. Il GetProdotto che t'ho proposto, serve per fare la select del singolo record. Farai un altro metodo che fa l'update, un altro per add, list e delete.

Whidbey è la prossima release di asp.net
http://www.aspitalia.com/articoli/whidbey/alpha1.aspx

Non mi sembra un gran vantaggio fare delle classi alla quale passare la query, risparmi 3 righe per aprire la connessione ed eseguirla.
La mia proposta era di fare le classi che hanno già i metodi che eseguono le query e ti danno un dataset se è una lista, o l'id del record (se ti serve, se fai un insert) o l'oggeto Prodotto se fai una get.

Allora:
hai una bella di prodotti:
id
Nome

Fai una classe che la rappresenta

public class Prodotto
{
public int Id;
public string Nome;
}

Ora in questa classe metti i metodi per fare, list, get, insert, update e delete:

public class Prodotto
{
...
public static DataSet ListProdotti()
{
SqlConnection con = new SqlConnection(...);
// meglio ancora se è una stored procedure
SqlDataAdapter cmd = new SqlDataAdapter ("select * from prodotti", con);
DataSet ds = new DataSet();
cmd.Fill(ds);
return ds;
}

public static Prodotto GetProdotto (int id)
{
SqlConnection con = new SqlConnection(...);
SqlCommand cmd = new SqlCommand("select * from prodotti WHERE id = " + this.Id "', con);
SqlDataReader reader = cmd.ExecuteReader();
if (reader.Read())
{
Prodotto p = new Prodotto();
p.Id = id;
p.Nome = (string)reader["nome"];
}
else
return null;
}

public void Update()
{
SqlConnection con = new SqlConnection(...);
SqlCommand cmd = new SqlCommand("update prodotti set nome = '" + this.Nome + "' WHERE id = " + this.Id "', con);
cmd.ExecuteNonQuery();
}

ecc

Con lo stesso principio fai anche il delete ecc. Il codice è uno schizzo, non copiarlo così eh
Così dalla tua pagina aspx, farai
datagrid1.DataSource = Prodotto.ListProdotti();

Se vuoi avere un singolo prodotto farai (es id = 1)
Prodotto p = Prodotto.GetProdotto(1);

lo potrai modificare
p.Nome = "ciao";
e aggiornare
p.Update();

Più chiaro di così