imobanco / icnab240

GNU General Public License v3.0
0 stars 0 forks source link

Utilizar uma lista de field é ineficiente #33

Open rodrigondec opened 4 years ago

rodrigondec commented 4 years ago

Resumo

todas as funções fazem um for field in fields com o intuito de substituir com base em algun field.identifier.

Sugestão

pode ser armazenado em um dicionário tal como:

fields = {
    'identifier': Field()
}

Dessa forma podemos manter a portabilidade de listagem de fields utilizando fields.values() e alterar nas funções 'pontuais' que precisam de um acesso O(1).

Dores

o miolo possuem identifier duplicados. Utilizar um simples dict não funciona.

Usar uma classe?

Uma estrutura mais completa?