UTNianos

Versión completa: Animaladas de código
Actualmente estas viendo una versión simplificada de nuestro contenido. Ver la versión completa con el formato correcto.
Páginas: 1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 23 24 25 26 27 28 29 30 31 32 33 34 35 36 37
(28-01-2012 19:25)Nicosunshine escribió: [ -> ]Intentando optimizar la plataforma, encontramos un store en producción que tiene escondido, entre muchas lineas de código:



select
*,
(select co.UserName from Contact co where co.Id = c.Id) as UserName
from Contact c
join ContactInformation ci where c.Id = ci.ContactId
where c.Status = 'A'



que hijo de puta jajajajajj
Hace una semana me mande un javascript mas o menos así segun recuerdo.



RemoveErrorMessages()
{
var codeError = $("#codeError");
if( codeError.hasChild) {removeChild(codeError.FirstChild)}
var incompleteFieldsError = $("#incompleteFieldsError ");
if( incompleteFieldsError.hasChild) {removeChild(incompleteFieldsError .FirstChild)}
var fileError = $("#fileError ");
if( fileError .hasChild) {removeChild(fileError.FirstChild)}
}



Una basura.Algo que se podía resolver tranquilamente agregando una clase error a todos los divs y usando la linea


$('.error').removeChild()

no es animalada de codigo, sino mas bien de paradigma.
ahora van a venir los evangelistas de objetos a contarme porque esto esta mal.



class ClaseA
{
(...)
public function loader($objeto)
{
foreach ($objeto as $nombre => $valor)
{
$this->$nombre = $valor;
};
}
(...)
}

(...)
$generar = unserialize($objetoSerializadoClaseB);
$tablaDatos = new ClaseA();
$tablaDatos->loader($generar);


(14-02-2012 15:02)Jarry escribió: [ -> ]no es animalada de codigo, sino mas bien de paradigma.
ahora van a venir los evangelistas de objetos a contarme porque esto esta mal.



class ClaseA
{
(...)
public function loader($objeto)
{
foreach ($objeto as $nombre => $valor)
{
$this->$nombre = $valor;
};
}
(...)
}

(...)
$generar = unserialize($objetoSerializadoClaseB);
$tablaDatos = new ClaseA();
$tablaDatos->loader($generar);



este PHP. ..... . .. JAJAJAJAJ xD

como deciamos ayer
Cita:I Heart PHP

public static void ActualizacionSistemaActualizar(ActualizacionSistemaDs actualizacionDs, bool updateOnly)
{
Agregar(actualizacionDs);
}



amo cuando pasan parametros al pedo (?)
es no es nada, vi un metodo que tenia como 8 parametros que no usaba.
que feo es el codigo castellanizado
Peor es como lo hago yo. Mitad y Mitad....

Off-topic:
yo hago lo mismo jajajajaj

siempre meto en ingles, hasta que llega una palabra que n ola se en ingles y de paja buscar, la pongo en castellano xD
a veces cuando heredas de una clase, hay subclases que necesitan mas parametros que otras para determinado metodo, y te pasan esas cosas.
igual lo de mitad y mitad esta piola, osea, reservar el castellano para las cosas especificas del negocio, ponele

algo asi:

GetCuentasContables()
RemoveChequeFromChequera()
(17-02-2012 11:56)ebric escribió: [ -> ]igual lo de mitad y mitad esta piola, osea, reservar el castellano para las cosas especificas del negocio, ponele

algo asi:

GetCuentasContables()
RemoveChequeFromChequera()

en ese caso creo que seria

TraerCuentasContables()

u
ObtenerCuentasContables()


Yo ya me mareo bastante, hay veces que tengo variables en ingles, metodos en castellano...
Pasa mas cuando son varios los que tocan en el mismo proyecto y cada uno implementa su forma.
(17-02-2012 11:56)ebric escribió: [ -> ]igual lo de mitad y mitad esta piola, osea, reservar el castellano para las cosas especificas del negocio, ponele

algo asi:

GetCuentasContables()
RemoveChequeFromChequera()

CIerra
(17-02-2012 12:12)shadow_mx escribió: [ -> ]
(17-02-2012 11:56)ebric escribió: [ -> ]igual lo de mitad y mitad esta piola, osea, reservar el castellano para las cosas especificas del negocio, ponele

algo asi:

GetCuentasContables()
RemoveChequeFromChequera()

en ese caso creo que seria

TraerCuentasContables()

u
ObtenerCuentasContables()


Yo ya me mareo bastante, hay veces que tengo variables en ingles, metodos en castellano...
Pasa mas cuando son varios los que tocan en el mismo proyecto y cada uno implementa su forma.

no, justamente, en este caso lo primordial del negocio serian las Cuentas Contables, Cuentas, Cheques, etc.

una regla masomenos valida seria:

entidades = castellano
acciones, verbos, etc = ingles

igual, lo fundamental es que todo los devs manejan la misma convencion, sea cual fuere.
[Imagen: 2j4f1fm.png]
Páginas: 1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 23 24 25 26 27 28 29 30 31 32 33 34 35 36 37
URLs de referencia