Buenas Practicas de Programacion

Buenas Prácticas de Programación

Todas las plantillas deben ser revisadas en los Navegadores Mozilla e Iexplorer antes de dar por terminada una tarea. Deben funcionar correctamente en los dos navegadores.


Grabar los archivos en formato ISO-8859-15 para el manejo correcto de tildes.


El nombre de plantillas, librerías, archivos, querys o funciones no deben llevar palabras reservadas de G. Ejm: query, fetch, gsqlcommand, value, ect.INCORRECTONombre Plantilla : ac-negquery.htmNombre Librería : fetch.libNombre Función : sub query { }Nombre Variable en javascript : value=”SI”;CORRECTONombre Plantilla : ac-negconsulta.htmNombre Librería : ciclo.libNombre Función : sub consulta { }Nombre Variable en javascript : valor=”SI”;


En todas las plantillas y librerías siempre se deben declarar y comentariar las variables y las funciones.INCORRECTOJAVASCRIPTnumero=””;PERL$numero=””;

CORRECTOJAVASCRIPTvar numero=””; //Almacena el numero del AcuerdoPERLmy $numero=””; #Almacena el numero del Acuerdo


Todos los campos input en el formulario deben tener las propiedades id y name (SIEMPRE deben tener las dos propiedades y preferiblemente tener el mismo valor, ejemplo id=secuencial name=secuencial).

Si presenta problemas el uso del Botón Nuevo del fmanager (o cualquier otra utilidad del fmanager u otra función) bajo ninguna circunstancia se deben eliminar estas dos propiedades. Se debe preguntar inmediatamente cual es la forma correcta de solucionarlo.<input type="text" name="numcuenta" id="numcuenta" />


Cuando el id es diferente al name deben revisar que si hacen referencia a ese input con document.getElementById, estén usando el nombre del id y no el del name. Aunque como dice el punto anterior es mejor que tanto la propiedad id como name tengan el mismo valor.


Se debe verificar que la numeración en el tabindex sea correcta y que funcione tanto en Mozilla como en Iexplorer.


También se debe revisar que después que se hagan peticiones AJAX (q2XML o serverload) el cursor se posicione en el campo correcto.


Cuando se envía un D.k cómo parámetro para abrir un popup u otra ventana se debe enviar como &D.k=xxxx para que en la plantilla que se abre se puedan recuperar directamente los campos de ese registro.

(En todas las plantillas de acuerdos antes se enviaba el deka del acuerdo como &acuerdodk=xxxx, con el tiempo lo hemos migrado y ahora usamos &D.k=xxxx).


INCORRECTOSi el parámetro se envía &D.k=84859<g:eval ## { $dekaAcuerdo="var.D.k"; %consulta=(); $consulta="search qname=consulta D.k=$dekaAcuerdo"; $rta=&gsqlcommand($consulta,resulset); $numAcuerdo="$$rta[0]{'numero'}";} ## /g:eval>

A pesar que que se está enviando el parámetro como D.k, están haciendo una consulta innecesaria.


INCORRECTOSi el parámetro se envía &acuerdodk=84859<g:eval ## { $dkAcuerdo="var.acuerdodk"; %consulta=(); $consulta="search qname=consulta D.k=$dkAcuerdo"; $rta=&gsqlcommand($consulta,resulset); $numAcuerdo="$$rta[0]{'numero'}";} ## /g:eval>

Se está enviando incorrectamente el parámetro lo que conlleva a realizar un query adicional que no sería necesario si se envía el parámetro como D.k.


CORRECTOSi el parámetro se envía &D.k=84859<g:eval ## {

$numAcuerdo="$D.numero";

} ## /g:eval>

Se está enviando correctamente el parámetro como D.k y se está recuperando correctamente el campo número.


En las plantillas que aún tienen 3 g:eval para llenar los campos actb deben modificarlo tal y como lo genera actualmente developer:


INCORRECTO (Parte 1 de 3)

<g:eval ## { $salida_LineaNombre = "var linea=new Array("; $salida_LineaClave = "var linea_clave=new Array(";

$salida_LineaDK = "var lineaDK=new Array(";

} ## /g:eval>


INCORRECTO (Parte 2 de 3)<g:query qname=qlinea D.y=grupocat nivel=1 order by clave limit=null></g:query>

<fetch fname=qlinea>

<g:eval ## {

$salida_LineaNombre .= "'$D.nombre',";

$salida_LineaDK .= "'$D.D.k',";

$salida_LineaClave .= "'$D.clave',";

} ## /g:eval>

</fetch>


INCORRECTO (Parte 3 de 3)

<g:eval ## { chop($salida_LineaNombre); $salida_LineaNombre .= ");"; chop($salida_LineaClave); $salida_LineaClave .= ");";

chop($salida_LineaDK); $salida_LineaDK .= ");";

chop($salida_LineaGrilla);

} ## /g:eval>


CORRECTO<g:query qname=qlinea D.y=grupocat nivel=1 order by clave limit=null></g:query><script>

salida_LineaDK=new Array(); salida_LineaClave=new Array();salida_LineaNombre=new Array();<fetch fname=qlinea> salida_LineaDK.push('$D.D.k'); salida_LineaClave.push('$D.clave'); salida_LineaNombre.push('$D.nombre');

</fetch>

</script>


Propiedades que deben tener los campos de los formularios. Si alguna de las propiedades con el llamado a estas funciones presenta incompatibilidad con algo específico que se esté programando en la plantilla, se debe consultar inmediatamente para encontrar la forma correcta de solucionarlo.<input type="text" name="numcuenta" id="numcuenta" value="" class="inputAUTO" size="10" maxlength="10" tabindex="1" onfocus="foco(this);" onblur="dejar(this);" />name --> nombre del campoid --> ide del campo

value --> valor que se le asigna

class --> ahi se usa input o inputAUTO (por ahora usamos solo input)

size --> tamaño de visualización de la caja de texto

maxlength --> cantidad de caracteres que se pueden escribir en una caja de texto

tabindex --> para la navegación con el tabulador

onfocus --> se debe llamar a la función foco(this) para el manejo de los colores

onblur --> se debe llamar a la función dejar(this) para el manejo de los colores


Siempre que se tenga que desarrollar una plantilla nueva, se debe hacer con Developer, que es nuestra herramienta de desarrollo. Si tienen inconvenientes, por medio de los jefes se debe hacer llegar esta información al grupo de Developer.

Al usar developer, tenemos código actualizado dentro de nuestras plantillas.


Siempre que se llame a la función KeytoUppercase en la propiedad onKeyPress (onKeypress="KeytoUppercase(event)") se debe enviar cómo parámetro el objeto (event) para que no genere errores en Mozilla.


Siempre que desde la plantilla se vaya a navegar a otra plantilla se debe hacer con el formulario gsecureform, para cuando se vaya a implementar la seguridad de las plantillas. En el body, al finalizar el formulario de la plantilla se debe crear el siguiente formulario:<form id='gsecureform' name='gsecureform' action='' method='post'></form>

Cuando se vaya a navegar a la plantilla necesaria se debe hacer de la siguiente manera:document.getElementById("gsecureform").action="$C.cgiurl/v.cgi?D.templ=$C.htmg/htmngf/acdomicilio.htm&email="+"var.email";

document.getElementById("gsecureform").submit();

La etiqueta <form ...> al igual que la etiqueta <input ...> debe llevar las propiedades id y name.



Siempre que se hagan peticiones AJAX se debe programar el sincronismo con la función controladora, para que la plantilla trabaje correctamente en Mozilla.


En las plantillas NO se deben usar las funciones setinterval y settimeout.


Cada plantilla debe lanzar solo las consultas necesarias para iniciar la funcionalidad de la misma, no se deben hacer mas consultas de las necesarias porque esto influye en los tiempos de carga.


Cuando se escriben consultas se debe verificar que tanto la clase como los campos realmente existen, para evitar errores en el futuro con consultas mal escritas.

Siempre se deben verificar las consultas.


En las plantillas intermedias que utilizamos para realizar validaciones, se debe implementar el uso de métodos, ya que al tener g:eval en la plantilla intermedia se parsea el código de la misma cada vez que se carga.


Con g:eval<g:evalrequire "$C.htmg/acuerdolib/gestion.lib";&alta_acuerdo($tvendible,$origen,$fechavalor,$fechactual);&alta_acpsimple($dkacuerdoalta,$fechavalor,$fechactual);..../g:eval>

Con Métodos$C.bancamethods/acuerdos/gestion::method::gestion('salida','salida1');eval.salidaeval.salida1


Uso correcto de la función gsqlcommand para inserción, modificación y eliminación.

&gsqlcommand("insert D.y=acuerdo numero=$numero timefecha=$fecha”);

&gsqlcommand("update set D.k=$deka to numero=$numstr"); Siempre que se va a eliminar un registro se debe comprobar que la variable que contiene el deka o la lista de dekas tenga valores, para no eliminar clases completas. if ($rta) { &gsqlcommand("delete D.y=acuerdo D.k=$rta"); }


Consulta sin parámetro adicionalour %qacuerdo=();$qacuerdo=&gsqlcommand("search qname=qacuerdo

D.y=acuerdo top=10");print "<br>qacuerdo=$qacuerdo";


Consulta con parámetro hashour %qacuerdo=();$qacuerdo=&gsqlcommand("search qname=qacuerdo

D.y=acuerdo top=10",'hash');print "<br>qacuerdo=$qacuerdo";foreach $deka(keys %sqlreg){print "<br>Numero = $sqlreg{$deka}{'numero'}";}


Consulta con parámetro resulsetour %qacuerdo=();$qacuerdo=&gsqlcommand("search qname=qacuerdo

D.y=acuerdo top=10",'resulset');my $longitud=@$qacuerdo;for($k=0;$k<$longitud;$k++){ print "<br>Numero = $$qacuerdo[$k]{'numero'}";}


Consulta con readk$hacuerdo=&gsqlcommand("readk D.k=$deka");print "<br>numero=$$hacuerdo{'numero'}";

Documents

Buenas Practicas de Programacion