Refactor for more readable code and more C++

[boerge1]

Ich habe ein paar Stunden verbracht und die DEV Version umgeschrieben. Der Code ist jetzt lesbarer und C++ hat Einzug gehalten. Ich habe mich bemüht, das alte Verhalten nicht zu ändern, außer dass nach dem Startup und nach dem Karte auflegen als Feedback ein kurzer Ton kommt.

Folgende Issues oder neue Features wurden implementiert:

• komplettes Refaktoring

• 95

• 90

• 76

• 75

• 43

• 37

• Neues Feature: neuer Mode: Hörbuch einzeln (nur ein Titel wird gespielt und Fortschritt gespeichert
• Neues Feature: Pause, wenn Karte entfernt wird (lässt sich per Einstellungen steuern)

[xfjx]

Hallo boerge,

das sieht tatsächlich sehr gut aus ;)

Könntest du mir wohl deine Kontaktdaten per Mail zukommen lassen? Ich arbeite aktuell auch an einem kompletten Rewrite, eventuell magst du da auch mal drüber schauen und ggf. unterstützen? Ziel wäre dabei, dass ganze noch deutlich modularer und erweiterbar zu machen.

Am besten einfach eine Mail an Thorsten@voss.earth

Danke

[phschoen]

is there any update if this pr can be integrated soon? just asking since there the last pr/commit is 2 years ago.

[boerge1]

@phschoen I think this PR will be never merged as no other PR is merged. But of course you can use my DEV branch. I made many improvements, the main is using a state machine instead of "spaghetti code" and waits.

[boerge1]

Ich habe noch weiter entwickelt und diesen PR erweitert:

• völlig neues Logging
• es lassen sich viele Sachen in der Datei const.hpp konfigurieren
• Implementierung einer State Machine
• alle Tracks werden beim Auflegen der Karte in eine Queue geschrieben und nicht erst beim nextTrack ermittelt
• neuer Mode für Short Cut und Karte: wiederhole die letzte Karte
• ein Short Cut lässt sich jetzt löschen (nach Auswahl des Short Cuts einfach das Admin Menü abbrechen)
• Konfigurationsdateien für VS Code + plattformio

Achtung: es ist jetzt c++17 erforderlich. Wer nicht plattformio verwendet, muss die Datei platform.txt editieren compiler.cpp.flags=-c -g -Os {compiler.warning_flags} -std=gnu++17 -fpermissive -fno-exceptions -ffunction-sections -fdata-sections -fno-threadsafe-statics -Wno-error=narrowing -MMD -flto

[phschoen]

@boerge1 danke fürs update. ich bin mittlerweile auch auf deinem dev branch. :+1: @xfjx schade das es keine Updates mehr gibt vielleicht überlegst du dir eine community github Organisation/branch mit mehreren Review Leuten zu eröffnen, sodass die Arbeit nicht an nur einem hängt.

[boerge1]

Wird fortgesetzt im Projekt https://github.com/tonuino/TonUINO-TNG