int uploadsID;
int pageNumber;
int x;
int y;
int w;
int h;
bool isValidUploadID = int.TryParse(context.Request.QueryString["uploadID"], out uploadsID);
bool isValidPage = int.TryParse(context.Request.QueryString["page"], out pageNumber);
bool isValidX = int.TryParse(context.Request.QueryString["x"], out x);
bool isValidY = int.TryParse(context.Request.QueryString["y"], out y);
bool isValidW = int.TryParse(context.Request.QueryString["w"], out w);
bool isValidH = int.TryParse(context.Request.QueryString["h"], out h);
if (isValidUploadID && isValidPage && isValidX && isValidY & isValidW & isValidH)
{
Este es un controlador de ajax, verificando que todos los parámetros aprobados estén bien. ¿Esto se considera malo, y hay una mejor manera de escribir esto, o no es tan importante?C# ¿hay alguna manera más agradable de escribir esto?
Puede usar un objeto que contenga todas las propiedades y un constructor que requiera NameValueCollection (Request.QueryString o simplemente Request) y prepare object, este objeto expone un método: "IsValid" y verifica si todos los parámetros son correctos para la solicitud actual. –
Es posible que desee considerar publicar esto en [codereview] (http://codereview.stackexchange.com/) – Benjol