[enocean plugin]: Introduce Protocol Module and Load CRC and CONSTANTS from Module

[hasenradball]

Introduce Protocol Module

• add submodule constants
• add submodule crc8

remove CRC table from init.py and load it from module

• clean up init.py
• use static_method from module

remove Constants definition from __init__.py

• remove constants from main file
• add loading constants as required from constants module

Reason for Change

• set a basis for future code and structure improvements
• increase maintainability
• increase readability of __init__.py

Status

• tested
• no functional change actually
• is a basis for future improvements

[hasenradball]

@Morg42
Hi,
hier mal ein kleines Beispiel wir ich CONSTANTEN auslagern würde.
Erst mal nur am Beispiel der CRC Table.

PS: und natürlich vorher gestestet, läuft.

[Morg42]

Wieso braucht es für CRC überhaupt eine Tabelle? Und schau dir mal @staticmethod an...

[hasenradball]

@Morg42
Ich verstehe Deine Frage nicht ganz.
Was wolltest Du sagen?

@staticmethod als decorator kenne ich, ja kann man auch so nutzen.

Aber was passt Dir an der Tabelle nicht?
sieh mal hier Seite 111

[Morg42]

Aus prinzipiellen und ästhetischen Gründen "passt" mir nicht, riesige Tabellen mitzuschleppen, die man "mal eben" auch automatisch befüllen lassen kann ;)

Und ja, ich kenne den Standard :)

[hasenradball]

Was wäre dein alternativ Vorschlag?
Wie würdest Du die Tabelle generieren?

Klar gebe Dir recht schön ist sie nicht, aber man kann so immer nachvollziehen wie man zu den Werten kommt.
Wenn Du die nicht mitschleppst musst halt dazuschreiben wie diese gerneriert wird.

Aber das mit der Tabelle sehe ich erstmal als kleine Baustelle an. :-)

[hasenradball]

:-)

[hasenradball]

Constant optimisation is working and PR is ready to merge!

[Morg42]

Mach doch erstmal weiter, bis ein wirklicher Fortschritt erreicht ist.

[hasenradball]

@Morg42 werde ich auch tun, allerdings kann ich nicht so regelmäßig dran arbeiten, daher step1.
Ein Schritt nach dem anderen…
😊