These are chat archives for u3games/L2J_EventEngine

19th
Jul 2015
Lucas
@luksdlt92
Jul 19 2015 05:35
nice
por las variables escritos en mayusculas y es como yo entendia y al contrario de lo q tu dices
hoy me dio dudas sobre el tema de los nombres de las variables
solo las constantes se escriben en su totalidad en mayusculas
no existe diferencia en el nombre de las variables entre static o no
@luksdlt92 q opinas?
fissban
@fissban
Jul 19 2015 09:44
mejoro un poco mas ahora xD
blob
fissban
@fissban
Jul 19 2015 10:47
@luksdlt92 te deje varios PR, revisalos y mergealos por favor asi puedo continuar trabajando mas tranquilo
Lucas
@luksdlt92
Jul 19 2015 11:56
@fissban cuando vuelva de votar los reviso
respecto a la notación, yo tenía entendido que todo lo que es static va con mayúsculas y todo lo que es de instancia con guión bajo
al menos esa notación usan en l2j
Lucas
@luksdlt92
Jul 19 2015 16:41
@fissban me tomo el atrevimiento de fixear algo de tu primer pr así no perdemos más tiempo
Lucas
@luksdlt92
Jul 19 2015 17:59
listo, squashie todos los commits de ese pr (los fusioné), dejé una descripción general para todo y agregué un commit más con un par de fixes (hacele code review pls, ya lo probé y funciona bien pero quizás tenés alguna discrepancia de cómo lo hice)
fissban
@fissban
Jul 19 2015 20:10
@luksdlt92 ya revise los fix q hicistes....hoy comienzo a trabajar en revisar los eventos asique has lo q tengas q hacer y lleva todo a unstable...quiero evitar tener q enrededarme entre los fix en cada branch y perderme algo q luego pueda traerme problemas
el tema de las variables meditalo, pero l2jserver solo define las variables como static las constantes y yo defino variables static las constantes y las de clase.
Lucas
@luksdlt92
Jul 19 2015 20:40
@fissban ya implementé la sugerencia del map, lo pruebo y mergeo
acá te dejo un video sobre los singletons
si me decís que acis usa el singleton como lo venís aplicando, empiezo a dudar del lugar donde aprendió a programar ese muchacho
no presenta absolutamente ningún beneficio
Lucas
@luksdlt92
Jul 19 2015 20:45
respecto a las variables, mirando bien tenés razón (fue una gran sorpresa para mí), las cosas que no son static final van con _asdasd, sin importar si son de clase o no y las static final con todo mayúscula
después cambio las cosas que cambiamos sobre eso
ahora empiezo a preferir la notación de google jajaja
Lucas
@luksdlt92
Jul 19 2015 20:51
fijate por ejemplo L2World, esa es una aplicación, a mí entender, correcta del singleton
todos los métodos son de instancia excepto getInstance
por eso a L2World se lo llama siempre L2World.getInstance().algo
actualmente (y como aparentemente hace acis) hacés un singleton de algo que siempre va a tener una única instancia, que es la clase
salvo algún caso extraño que desconozco (que se ejecute en otra VM) la clase nunca va a estar duplicada, por lo que hacer un singleton de clase no sirve para nada
Lucas
@luksdlt92
Jul 19 2015 21:59
respecto al pr, el de minorfixes ya está mergeado, voy a ir agregando los JARs a los prs de la versión que se está mergeando, va a ser útil
Lucas
@luksdlt92
Jul 19 2015 23:15
ya armé el PR del rework de los buffs, hice un par de comentarios
al principio le di el OK sin mirar detalladamente, sori
fissban
@fissban
Jul 19 2015 23:25
sobre la nomenclatura y si son o no static dejamelo a mi q hoy lo armo y mañana lo subo
Lucas
@luksdlt92
Jul 19 2015 23:42
bancá que mergiemos todo
sino me voy a volver put* con el rebase
se te puede hablar en skype o preferís por acá?