search

oslugr / ugr-transparente

Portal de transaparencia de la UGR
GNU General Public License v3.0
2 stars 1 forks source link

Evitar constantes en el código #4

Closed JJ closed 6 years ago

JJ commented 10 years ago

Usar ficheros de configuración para prácticamente todo: cadenas, puertos, conexiones a base de datos...

torresj commented 10 years ago

He añadido un fichero de configuración y he sacado todas las costantes. También he reestructurado el codigo

JJ commented 10 years ago

¡Constante!

JJ commented 10 years ago

¡Código!

torresj commented 10 years ago

Jajaja, ya sospechaba que no te iba a gustar mucho. Lo hago leyendo un fichero de texto no?

JJ commented 10 years ago

No. Json o yaml. El 19/03/2014 15:18, "Jaime Torres" notifications@github.com escribió:

Jajaja, ya sospechaba que no te iba a gustar mucho. Lo hago leyendo un fichero de texto no?

Reply to this email directly or view it on GitHubhttps://github.com/oslugr/ugr-transparente/issues/4#issuecomment-38055211 .

torresj commented 10 years ago

Ok, me pongo a ello

torresj commented 10 years ago

Si leo desde un archivo json el valor de las variables globales seguiría siendo incorrecto? Tengo k consultar el archivo json cada vez que vaya a usar una de esas variables? O lo leo justo cuando lanzo el servidor y la exporto para poder usarlas en toda la app? No se si me he explicado muy bien...

JJ commented 10 years ago

No, basta con que lo leas al principio. Se trata de a) separar los aspectos particulares de la aplicación aparte y b) hacer fácil de editar la configuración sin meterse en el código, que no deben de tocarlo más que Los Verdaderos Programadores.

El 19 de marzo de 2014, 18:01, Jaime Torres notifications@github.comescribió:

Si leo desde un archivo json el valor de las variables globales seguiría siendo incorrecto? Tengo k consultar el archivo json cada vez que vaya a usar una de esas variables? O lo leo justo cuando lanzo el servidor y la exporto para poder usarlas en toda la app? No se si me he explicado muy bien...

Reply to this email directly or view it on GitHubhttps://github.com/oslugr/ugr-transparente/issues/4#issuecomment-38078228 .

JJ

torresj commented 10 years ago

Dadas las prisas que hay para el lunes, esto lo dejo para la semana que viene