¿Es una buena idea mantener la solicitud de consulta como una propiedad? -- # campo con asp.net camp codereview Relacionados El problema

Is it a good idea to keep Request querystring as a property?


2
vote

problema

Español

Mi código SNIPPET es así:

  geometry2  

y voy a llamar a esta propiedad geometry3 siempre que lo necesite. ¿Es una buena práctica?

Original en ingles

My code snippet is like this:

public static string EventId {     get { return HttpContext.Current.Request["eventId"]; } } 

And I will call this property EventId whenever I need it. Is it a good practice?

     

Lista de respuestas

2
 
vote
vote
La mejor respuesta
 

¿Puede garantizar que class SnakeBody: def __init__(self, x, y): self.x, self.y = x, y def __repr__(self): return "SnakeBody({self.x}, {self.y})".format(self=self) 5 nunca cambiará? Si es así, podría no ser una mala idea, pero esto se reduce a algunas cosas

  • class SnakeBody: def __init__(self, x, y): self.x, self.y = x, y def __repr__(self): return "SnakeBody({self.x}, {self.y})".format(self=self) 6 nunca cambia;
  • cómo y dónde lo estás usando. Puede ser algo difícil para un nuevo programador en el proyecto que localice y realice lo que realmente hace (devuelve el 99887766555443327 para la solicitud HTTP actual), únicamente basada en el nombre de la propiedad; y
  • ¿Está seguro de que tanto 99887766555443328 99887766555443329 se inicializan antes de recuperar __repr__0 . Hay un cambio que estos lanzarán una excepción de referencia nula. Puede ser una muy buena idea hacer un simple 998877766555443331 antes de invocar la propiedad de solicitud.

Si su código puede acomodar para estos puntos, no veo ningún error inherente con hacer esto.

 

Can you guarantee that eventId will never change? If so it might not be a bad idea, but this comes down to a few things

  • eventId never changes;
  • how and where you're using it. It might be somewhat difficult for a new programmer in the project to locate and realize what it actually does (it returns the eventId for the current Http Request), solely based on the property name; and
  • Are you sure that both HttpContext.Current and Current.Request are initialized before retrieving eventId. There is a change these will throw a null reference exception. It might be a very good idea to do a simple if != null check before invoking the Request property.

If your code can accommodate for these points I don't see any inherently wrong with doing this.

 
 

Relacionados problema

3  Master Detalle Insertar en Entity Framework 6 Base de datos Primer MVC 5  ( Master detail insert in entity framework 6 database first mvc 5 ) 
Tengo un DB con una tabla maestra llamada "Facturas" y otra tabla de detalles "Facturas_Detalle". Me gustaría insertarlos, por lo que esta es una "base de dat...

2  Diferente enfoque a la siguiente acción ASP.NET WEB API  ( Different approach to the following asp net web api action ) 
Tal vez alguien aquí puede ayudarme a resolver esta implementación y hacerlo mejor. Primero las siguientes restricciones: 1 controlador & amp; Acción que...

4  Envoltura de sesión  ( Session wrapper ) 
Tengo dos tipos de envoltorios de sesión: tipo 1 public class SessionHandler { public static SessionHandler CurrentSession { get ...

3  Aplicación externa para la implementación de la aplicación web  ( External application for the web application deployment ) 
Necesito una aplicación externa que ayude a la implementación de la aplicación web. La aplicación debe crear la base de datos 99887766665544330 en el SQL Se...

3  Diseño de 3 capas y conjunto de datos múltiples  ( 3 layer design and multiple dataset ) 
Soy nuevo en la programación C # y OO. Tengo una página ASPX con 3 listas de casillas de verificación y me gustaría generarlas desde el DB. ¿Cómo debo estruct...

3  MARCA DE TEXTBOX JavaScript de doble modo  ( Dual mode javascript textbox watermark ) 
Solo estaba tratando de hacer una marca de agua AJAX alternativa desde "Fecha de inserción" (primer modo) a "MM / DD / YYAY" (segundo modo). Después de algún ...

1  Inserte el elemento seleccionado en ListBox usando ADO.NET y STOR PROC  ( Insert selected item in listbox using ado net and stor proc ) 
He creo este método de clase para ingresar el formulario de elementos seleccionados, lista de verificación para el recurso asociado, utilizo ADO.NET & AMP; La...

4  NHIBERNATE (UOW) + patrón de ccommand  ( Nhibernate uow ccommand pattern ) 
Estoy construyendo una aplicación ASP.NET MVC y me gustaría algunos comentarios para mi manera de consultar / ejecutar declaraciones al ORM. Estoy usando nh...

3  Paginación personalizada en ASP.NET Aplicación web  ( Custom paging in asp net web application ) 
Tengo un código siguiente para hacer una paginación personalizada desde una aplicación web ASP.NET. puntos de interés usa los botones de enlace como se...

6  API web y procedimientos almacenados  ( Web api and stored procedures ) 
Estoy creando un servicio web que obtiene datos a través de procedimientos almacenados de un DB y proporciona el resultado como JSON. La solución se construye...




© 2022 respuesta.top Reservados todos los derechos. Centro de preguntas y respuestas reservados todos los derechos