В вашем коде нет ничего плохого, но кажется, что что-то не так в вашем Request.Params
. Используете ли вы строку запроса, если это так, я бы предложил вам использовать Request.QueryString
вместо. Кроме того, что вы хотите, только если обновить счетчик, то вы можете избежать выбора запроса и уменьшить свой код до:
da db = new da();
String str = "UPDATE newstxt SET views = views + 1 WHERE id=" + Request.QueryString["id"].Trim();
db.docom(str);