Open
Conversation
Adicionalmente creo que ya encontre el problema de: "marcos.indice:8979589794: Se esperaba digito 128b" Parece ser debido a que en lee128b se lee el stream de entrada, pero este no se descomprime, haciendo que se encuentren caracteres inesperados.
Ahora es mas evidente como agregar varias tendencias a un arbol huffman. Se pueden agregar de un archivo o un mapa. Se podria agregar que tambien se agregue de una cadena, en este momento solo se tiene por constructor, o convertir una cadena a un mapa y usar agregar mapa. Se puede agregar la cantidad de fuentes que sea, mientras no se llame compresion o descompresion. Una vez se llame una de estas funciones, se "bloquea" el arbol y ya no se pueden hacer cambios adicionales.
Se realiza la misma operacion de lectura en varias partes del codigo, haciendo que, debido a los cambios que se realizaron para compresion, que la misma operacion dejara de funcionar. De momento puse el mismo codigo que esta en leeCad para solucionar, pero a futuro valdria la pena buscar el codigo que pueda estar repetido y abstraerlo a varias funciones. Hay mucho codigo repetido.
dhasane
commented
Aug 17, 2021
|
|
||
| # exit | ||
| comparar mm.indice md.indice | ||
| exit |
Contributor
Author
There was a problem hiding this comment.
sigue existiendo error en esta comparacion
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Se soluciono uno de los problemas en la mezcla de indices. No se estaba teniendo cuenta el escape de caracteres que fue agregado en leeCad. De momento se agrego la misma solución de leeCad y parece estar funcionando correctamente.
Algo que seria recomendable a futuro, es buscar todas las partes del código que hagan el mismo trabajo y unificarlas en una función, de forma que no haya que arreglarlo varias veces.
Adicionalmente se modifico la forma en la que se generaban los arboles de huffman, haciendo mas conveniente el agregar varias fuentes. De forma que se mantenga la consistencia del árbol, una vez este comience a ser utilizado (operaciones de lectura o escritura), se bloquea la posibilidad de modificarlo nuevamente. Esto porque se estaría utilizando un árbol distinto en caso de modificarlo entre lecturas/escrituras.