Verschiedene Sprachen hinzugefügt #347
Conversation
Schweizerdeutsch, Deutsch (Schwäbisch), Französisch und Rumänisch als Uhrenlayout hinzugefügt.
|
Hey @clingner Willkommen im Projekt und vielen Dank für Ihren Beitrag zur Wortuhr. Ich habe jetzt einige kleinere Fehler behoben, leider musste ich sie im Online-Editor beheben und es wurden deshalb viele E-Mails herausgeschickt, aber jetzt sollte es funktionieren. Für den Moment sieht der Code sehr gut aus und so ich gehe davon aus, dass alles funktionieren sollte. Testen kann ich das aber erst die kommenden Tage. Für heute lass ich das Coden mal sein und werde das neue Jahr begrüßen :-) Mit freundlichen Grüßen |
| @@ -526,11 +534,21 @@ void ClockWork::setMinute(uint8_t min, uint8_t &offsetHour, bool &fullHour) { | |||
| case 3: | |||
| case 4: | |||
| case 5: | |||
| usedUhrType->show(FrontWord::m_fuenf); | |||
| usedUhrType->show(FrontWord::nach); | |||
| break; | |||
There was a problem hiding this comment.
Ich denke, das hier ist redundant mit der nächsten Anweisung und kann entfernt werden.
| case 6: | ||
| case 7: | ||
| case 8: | ||
| case 9: | ||
| case 10: | ||
| if (G.UhrtypeDef == Fr10x11) { | ||
| usedUhrType->show(FrontWord::m_zehn); |
There was a problem hiding this comment.
Hier würde ich usedUhrType->show(getFrontWordForNum(min)); einfügen
| if (G.UhrtypeDef == Fr10x11) { | ||
| usedUhrType->show(FrontWord::m_zehn); | ||
| } else { | ||
| usedUhrType->show(FrontWord::m_zehn); |
There was a problem hiding this comment.
und hier auch usedUhrType->show(getFrontWordForNum(min));
| @@ -712,9 +740,13 @@ void ClockWork::setMinute(uint8_t min, uint8_t &offsetHour, bool &fullHour) { | |||
| //------------------------------------------------------------------------------ | |||
|
|
|||
| void ClockWork::setHour(const uint8_t hour, const bool fullHour) { | |||
| switch (hour % 12) { | |||
| switch (hour % 24) { | |||
There was a problem hiding this comment.
ich würde gerne das hour % 12 stehen lassen.
Vorher würde ich noch eine kleine Funktion erstellen, welche überprüft ob Midnight vorhanden ist und gleichzeitig ob auch Mitternacht ist.
bool isMidnight (const uint8_t hour) {
return hasMitternacht() && hour == 0;
}
Dann könnte so etwas da stehen, wie:
bool midnight = isMidnight(hour);
switch (hour % 12)
case 0:
if (midnight) {
usedUhrType->show(FrontWord::h_mitternacht);
} else {
usedUhrType->show(FrontWord::h_zwoelf);
}
das würde den code noch schlanker machen :-), da man sich dann die Varianten 12 -24 spart, da diese ja gleich bleiben.
|
Zur anderen Frage, ja in früheren Versionen war es möglich die Uhr auch ohne WLAN komplett einzustellen. Das versuche ich demnächst wieder zu ermöglichen. Diese Funktion ist damals, mit der Umstellung auf die WiFiManager Library, herausgenommen worden. |
Schweizerdeutsch, Deutsch (Schwäbisch), Französisch und Rumänisch als Uhrenlayout hinzugefügt. Da da man bei der französischen Uhr einen Unterschied zwischen Mittag und Mitternacht macht, musste ich den entsprechenden Abschnitt "switch (hour)" entsprechend ergänzen. Ich besitze nur sehr mässige Programmierkenntnisse und habe es u.U. nicht gerade nobel gelöst. Ich bin ich froh für eine genaue Prüfung aller meiner Ergänzungen.
An dieser Stelle auch ein herzliches Dankeschön für die Arbeit die in diesem Projekt geleistet wurde und wird.
Ich habe eines noch nicht herausgefunden. Wenn ich kein W-Lan zur Verfügung habe, wie kann ich die Uhr einstellen und sie ohne W-Lan autonom laufen lassen? Gibt es diese Funktion nicht mehr? In der früheren Version war das noch möglich.