Is there a better/cleaner way to do this?
int stockvalue = 0;
if (!Convert.IsDBNull(reader[\"StockValue\"]))
stockvalue = (int)reader[\"StockValue\"];
>
The way I handle this is
int? stockvalue = reader["StockValue"] as int?;
Very simple, clean and one line. If for some reason I absolutely can't have a null value (which I find poor reasoning for usually since I'd rather know if a value has meaning or if it was unitialized for a primitive type) I would do:
int stockvalue = (reader["StockValue"] as int?).GetValueOrDefault(-1);