alu0101028163 / alu0101028163/Maximum-Bandwidth
Anotaciones cara a la limpieza y estandarización del código
- Lingua principale
- C++
- Stelle
- 0
- Fork
- 0
- Metriche di merge delle PR
- Nessuna PR unita negli ultimi 30g
Descrizione
Entre que voy haciendo commits y terminando VNS, he ido mirando el proyecto en general, y creo que sería lo suyo limpiar el código cara a la entrega. Les dejo por aquí lo que creo que viene bien tener en cuenta:
- DeterministAlgoritm es una buena herramienta para hacer tests de funcionamiento con instancias pequeñas, pero no puedo usarlo fácilmente porque recibe el etiquetado en formato erróneo (debería ser un vector de enteros) y me obliga a transformarlo.
- La definición de Antibandwidth::solutionT tiene el propósito de hacer el código más versátil y legible, además de abstraer el tipo de dato subyaciente (en nuestro caso, un labeling es un vector de enteros). Sería un detalle, a la hora de limpiar el código,
- Ya de paso, podría ser una buena idea cambiarle el nombre a solutionT por "labelingT" o algo similar, para que sea más claro. También sería buena idea darle los mismos nombres a los parámetros cuyo propósito o forma coincida en distintos métodos (adjMatrix/graph, etc..)
- Importante: Cuidar la duplicación del código y primar la reutilización sobre reinventar la rueda (std::swap). También, podríamos evitar duplicar la librería catch.hpp en todas las carpetas que lo necesiten y dejarlo en un lugar accesible para todos los subproyectos. Propongo la carpeta Antibandwidth, que es donde van los genéricos.
Pensaba usar este issue como tablón para ir anotando este tipo de cosas según nos surjan.
Guida per i contributori
Nessuna guida per i contributori indicizzata per questo repository
Valutazione
Questa issue non è ancora stata valutata.