dmonza / genexus

Estandar de desarrollo en Genexus
http://www.sincrum.com
37 stars 23 forks source link

parametros en objetos #4

Closed ramunozp closed 4 months ago

ramunozp commented 2 years ago

original:: // mal parm(in:PaiId,out:&PaiNom);

// bien parm( in:PaiId, out:&PaiNom);

Propuesta: si bien eso es lo típico que se ve y se propagó en GX, para mi no es lo mejor ni lo mas legible, sobre todo cuando la cantidad de parámetros es grande. yo propongo y uso lo siguiente:

// MEJOR parm( in: PaiId, out: &PaiNom );

donde los Tab's son los que da por defecto GX y primero se deben dejar todos los IN y al final los OUT, como debería ser. De esta forma, si un procedimiento tiene parámetros es muy fácil ver como es y sobre todo si la lista de parámetros es extensa.

gxsoft commented 2 years ago

De Acuerdo con Mejor

-- Saludos, gab @gxsoft

On Sun, Jan 9, 2022 at 8:36 PM Rodo @.***> wrote:

original:: // mal parm(in:PaiId,out:&PaiNom);

// bien parm( in:PaiId, out:&PaiNom);

Propuesta: si bien eso es lo típico que se ve y se propagó en GX, para mi no es lo mejor ni lo mas legible, sobre todo cuando la cantidad de parámetros es grande. yo propongo y uso lo siguiente:

// MEJOR parm( in: PaiId, out: &PaiNom );

donde los Tab's son los que da por defecto GX y primero se deben dejar todos los IN y al final los OUT, como debería ser. De esta forma, si un procedimiento tiene parámetros es muy fácil ver como es y sobre todo si la lista de parámetros es extensa.

— Reply to this email directly, view it on GitHub https://github.com/dmonza/genexus/issues/4, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAA6LQSL46KIXJN45GFRTTDUVILYLANCNFSM5LSNK6KQ . Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Android https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

You are receiving this because you are subscribed to this thread.Message ID: @.***>

dmonza commented 2 years ago

@ramunozp está interesante la propuesta. Capaz se puede agregar como opcional en el caso que existan demasiados parámetros. Aunque en este último caso es preferible un SDT.

ramunozp commented 2 years ago

Si claro, si son muchos parametros, siempre es mejor un SDT, aun que me ha tocado tener SDT muy grandes que no son soportados o no funcionan muy bien por parametros y he tenido que pasarlos por websession.

De todas maneras lo que indico no es para competir con un sdt por parametros, si no mas bien, como quedan mejor escritos los parametros y, a mi juicio, como debería ser.

Se ve mejor en GX que aquí, como texto.

Captura de Pantalla 2022-02-18 a la(s) 16 13 01