TEMA 10: Lo que NO hay que hacer

Tamaño: px

Comenzar la demostración a partir de la página:

Download "TEMA 10: Lo que NO hay que hacer"

Rosario Río Ojeda
hace 6 años
Vistas:

1 TEMA 10: Lo que NO hay que hacer Introducción...1 Mezclar módulos...1 No crear una estructura de clases...3 Insertar la logica en los aspx...3 Gestionar los eventos en Page_Load...3 Utilizar Response.Write...4 Generar HTML...4 Crear campos públicos y no utilizar propiedades en las Clases...4 Utilizar subrayado en los nombres de las variables...5 Especificar el tipo de dato en el nombre de la variable...5 Poner más de una clase en el mismo fichero...6 Introducción Al crear aplicaciones con otro tipo de entornos y lenguajes de programación vamos adquiriendo esquemas y usos de programación que no son correctos dentro la plataforma orientada a objeto de.net. Es tan importante conocer aquello que hay que evitar como las buenas prácticas A continuación se muestra una serie de cosas que hay que evitar a toda costa. Mezclar módulos La programación en.net se caracteriza por separar las tecnologías. Por un lado el aspecto, por otro el código, por otro los recursos dependientes de la cultura y así con todos los elementos de nuestro sitio web. Es una muy mala práctica que nos lleva a crear software de muy mala calidad el mezclar los módulos que deben ir separados. Por ejemplo insertar dentro de un aspx: presentación, código de servidor, código de cliente y estilos. Cuando habría que tener 4 ficheros diferentes:.aspx,.cs,.js,.css. Por ejemplo el siguiente aspx (Default.aspx) está fatal: <%@ Page Language="C#" AutoEventWireup="true" %> <html> <head runat="server"> <title>pagina mal</title> <script type="text/javascript"> function hola() alert("hola"); </script> Tema 1 / Página 1

2 <style type="text/css">.bonito text-decoration:none; </style> </head> <body> <form id="form1" runat="server"> <div> <% string saludo; saludo = "Hola"; // bla bla bla %> </div> </form> </body> </html> En su lugar habría que hacer: Default.aspx <%@ Page Language="C#" AutoEventWireup="true" CodeFile="Default.aspx.cs" Inherits="_Default" %> <html> <head runat="server"> <title>pagina mal</title> <link href="misestilos.css" rel="stylesheet" type="text/css" /> <script src="codigocliente.js" type="text/javascript"></script> </head> <body> <form id="form1" runat="server"> <div> </div> </form> </body> </html> Default.aspx.cs using System; public partial class _Default : System.Web.UI.Page protected void Page_Load(object sender, EventArgs e) string saludo; saludo = "Hola"; // bla bla bla Tema 1 / Página 2

3 codigocliente.js function hola() alert("hola"); misestilos.css.bonito text-decoration:none; No crear una estructura de clases.net es un sistema de programación orientado a objetos por lo tanto la calidad de las aplicaciones dependerán de un buen diseño orientado a objetos que deriva en un buen diagrama de clases. Nunca hay que utilizar programación secuencial sin jerarquía de clases. Para eso mejor no utilizar.net. Insertar la logica en los aspx Los aspx sólo se tienen que encargar de gestionar los eventos de los controles de la página. Toda la lógica de la aplicación estará en las clases, contenidas en la carpeta App_Code Además a la hora de generar las pruebas unitarias estas sólo se pueden realizar en las clases, Visual Studio no permite pruebas unitarias en los aspx Gestionar los eventos en Page_Load Cada evento lo tiene que gestionar el gestor de eventos correspondiente. No comprobar el estado de los eventos en el Page_Load Por ejemplo en este Page_Load gestionamos el evento de que el usuario pulse un botón protected void Page_Load(object sender, EventArgs e) if (Request.Form["BotonPulsa"] == null) HazCosicasDelBoton(); Tema 1 / Página 3

4 La forma correcta sería: protected void Page_Load(object sender, EventArgs e) protected void BotonPulsa_Click(object sender, EventArgs e) HazCosicasDelBoton(); y en el aspx asociar el evento con el botón <asp:button ID="BotonPulsa" runat="server" onclick="botonpulsa_click" Text="Pulsa ya" /> Utilizar Response.Write Nunca hay que utilizar Response.Write para generar contenido en la página. Generar HTML Hay que evitar generar etiquetas html escribiéndolas directamente en la página desde programación. Response.Write("<div> Contenido div </div>"); Crear campos públicos y no utilizar propiedades en las Clases Los campos de la clase tienen que ser privados y su nombre tiene que estar precedido por _. Para acceder a los valores usaremos las propiedades Mal: public class ClaseMal public int numero; public string nombre; Tema 1 / Página 4

5 Bien: public class ClaseMal private int _numero; private string _nombre; public int Numero get return _numero; set _numero = value; public string Nombre get return _nombre; set _nombre = value; Utilizar subrayado en los nombres de las variables Nunca utilizar subrayado en los nombres de variables. Seguir la guía de estilo en cuanto a formato de nombres de variables. Mal string variable_mal_declarada; Bien string variablebiendeclarada; Especificar el tipo de dato en el nombre de la variable Nunca especificar el tipo de una variable en su nombre. Es innecesario en un lenguaje fuertemente tipado como.net Mal int icodigo; string snombre; Bien int codigo; string nombre; Tema 1 / Página 5

6 Poner más de una clase en el mismo fichero Cada clase tiene que almacenarse en un fichero con su mismo nombre. No utilizar un mismo fichero para insertar más de una clase. que los ficheros son gratis!! Tema 1 / Página 6

Documentos relacionados

TEMA 4: FLUJO DE UNA APLICACIÓN

TEMA 4: FLUJO DE UNA APLICACIÓN 1. EVENTOS EN ASP.NET Los eventos en ASP.NET se clasifican en tres grupos: (1) Eventos a nivel de aplicación. Son eventos globales de nuestra web, como pueden ser el inicio