Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Nueva estructura de directorios #78

Open
adrigm opened this issue Dec 22, 2013 · 2 comments
Open

Nueva estructura de directorios #78

adrigm opened this issue Dec 22, 2013 · 2 comments

Comments

@adrigm
Copy link
Contributor

adrigm commented Dec 22, 2013

No me acaba de convencer la nueva estructura de directorios. Algunas cuestiones:

  • ¿Por qué Core.hpp está dentro de su propia carpeta si se supone que es un archivo que incluye todos los archivos del Core?
  • El log debería ser parte del Core ya que este es el módulo base de el que van a depender todos los demás módulos ya que en él se encuentran todas las tareas básicas del motor.
  • Lo mismo con StringsUtils todo esto debe pertenecer al módulo Core que para eso es el Core, creo que se intentaba es dar la máxima modularidad posible y no creo que esto sea del todo conveniente en este proyecto.
  • Los archivos cpp se encuentren en src/GDE/source/ y los archivos hpp se encuentran en src/GDE/include/GDE/ creo que es un poco incosistente que no se encuentre en la misma estructura.

Siento reabrir este debate de nuevo, pero creo que hay que dejar el sistema lo mejor posible antes de que esto avance más y sea más complicado.

@danigomez
Copy link
Contributor

  • mmm No entiendo el problema de que el header esté ahí, podrías explicarlo mejor??
  • Creo que la idea de el módulo Core es que tenga lo que hace al funcionamiento del engine en si, Log y StringUtils son algo ajeno al engine en si, no hacen a su funcionamiento. Ese es el críterio que creo se uso para hacer la división de los módulos.
  • Lo del módulo de Utils separado también me parece bien, en la medida que vayamos agregando más funcionalidades en Util va a estar bueno que esté separado en su propio módulo.
  • Sobre lo que decís de los paths, estoy de acuerdo, deberiamos cambiar el path de include a src/GDE/include/ directamente.

@rickyah
Copy link
Contributor

rickyah commented Dec 23, 2013

  1. No entiendo qué problema hay. Core contiene los headers de core, y debería estar en su carpeta. No veo el problema, lo siento. Estaba pensando en más adelante crear un GDE.h que incluya el resto de headers de cada módulo, de cara a tener un único fichero para incluir todo (para facilitar la vida a los usuarios del engine)
  2. (ver siguiente línea)
  3. Ya lo había comentado aquí: Organización del código #57 (comment)
  4. Eso ha sido para poder indicarle al compilador que use la carpeta include como un directorio donde buscar headers, pero a la vez poder referenciar los headers dentro del código con la cadena 'GDE' delante (e.g. #include <GDE/Core>) También es más fácil copiar los headers así: con copiar la carpeta include ya están colocados correctamente. Se puede cambiar pero entonces hay que quitar la cadena GDE de los includes (e.g. #include <Core>)

Y sobre esto:

creo que se intentaba es dar la máxima modularidad posible y no creo que esto sea del todo conveniente en este proyecto

Dar la máxima modularidad posible es siempre conveniente, en este y cualquier proyecto, salvo casos muy excepcionales.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants