(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 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.