genexus icon indicating copy to clipboard operation
genexus copied to clipboard

parametros en objetos

Open ramunozp opened this issue 3 years ago • 3 comments

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.

ramunozp avatar Jan 09 '22 23:01 ramunozp

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: @.***>

gxsoft avatar Jan 09 '22 23:01 gxsoft

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

dmonza avatar Feb 18 '22 17:02 dmonza

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

ramunozp avatar Feb 18 '22 19:02 ramunozp