@RIN67630 Die Dokumentation hat das hier zu ``assert``: https://docs.python.org/3.10/reference/simple_stmts.html#the-assert-statement
Du hast da nicht die Python-Dokumentation verlinkt, sondern etwas externes das als „Entwurf“ gekennzeichnet ist, und das seit Mitte 2015 diese Notiz vom Autor ...

assert dient dazu, Invarianten zu testen. Benutzereingaben sind das Paradebeispiel für Nicht-Invarianten. Deswegen sollte hier ein if -Statement und nicht assert stehen.
Interessante Information, die die Mantra dieses Forums: "Alles steht in der Doku" widerspricht. Das steht nämlich nicht drin ...

Wenn Du eh schon im Binärcode herumpatchst, warum nicht gleich per:
struct.pack("<i", timezone_offset)
als Zahl einfügen?

...weil der User auch noch die Werte überschreiben können muss.
Ich mache ihm das leicht, in dem ich die Geo-Werte von der IP ableite, wenn er sein Gerät dort nutzen will ...

Namen werden in Python klein_mit_unterstrich geschrieben.

TZ_OFF ist der Macro-Name des C++ programs. Dort wird es als Konstante Kapitalgeschrieben

Dein Code ist genauso kaputt wie vorher. Du vergleichst noch immer Zeichenketten mit Zahlen.

War ein Kopierfehler, ist im Code korrigiert ...

Danke an allen.
Jetzt habe ich es so geändert:

gmtoff = "3600" # Offset to GMT in secs
Placeholder_TZ_OFF = b"TZ_OFF " # Offset to GMT in secs (exactly 8 chars incl spaces)
gmtoff = input("How many seconds ahead (or behind -) Greenwitch is normally the device ?") or gmtoff
assert -43200 <= gmtoff ...

Hallo,
ich bin etwas verduzt

die gleiche Variable gmtoff wird einmal moniert, dass sie String ist, und woanders moniert, dass sie int ist:


gmtoff = 3600 # Offset to GMT in secs
gmtoff = input("How much seconds ahead (or behind -) Greenwitch is normally the device ?") or gmtoff
assert( gmtoff ...

__blackjack__ hat geschrieben: Samstag 13. Juli 2024, 11:31 Einzelschrittdebugger sind in Python für mich eher nutzlos, weil man dann den Code so schreiben muss wie man das früher in Pascal & Co gemacht hat, um da überhaupt Schritte zu haben die man sinnvoll durchsteppen kann.

Passt mir aber wunderbar, genauso schreibe ich.

Ich denke nur, dass dir eine andere IDE (PyCharm ist hier wohl der Standard) hier evtl. auch nicht weiterhilft...

Danke, PyCharm macht es schon besser.
Es wirft zwar ein Warning aus, dass möglicherweise das Passwort angezeigt werden kann, aber macht brav weiter.

Dass es auch mit dem Terminal ...

Hallo,
als gelegentlichen Anwender von Python, auf Raspberry Pi, Windows, macOS nutzte ich Thonny als Editor/Debugger. Thonny ist auch gepriesen als "anfängerfreundlich".
Mehmals musste ich jedoch feststellen, dass das Ausführen über Thonny unerklärlichen Fehlern hervorbrachten , die bei direkter ...

DeaD_EyE hat geschrieben: Donnerstag 11. Juli 2024, 08:22 Spätesten wenn du anfängst Pfade aus anderen Pfaden zusammenzusetzen und Dateien löscht/kopierst, kann das Problem auftreten.

Nö, nö, Danke, keine keine Gefahr, wir bleiben brav im selben Verzeichnis und löschen nix.

Sirius3 hat geschrieben: Mittwoch 10. Juli 2024, 14:39 Dann mit Sets, denn jeder kennt Sets und mit Shell-Pattern kennt sich auch der Noob besser aus als mit komplizierten if-Abfragen:

Mein Saraksmusdetektor klingelt...

Auf https://github.com/rin67630/ESP_Binary_patcher/blob/main/PythonPatcher.py kommt os genau zweimal vor: Einmal in Zeile 3, wo es importiert wird, und einmal in Zeile 25, wo du os.listdir() verwendest.
Upps, richtig ! ursprünglich war zum Ausführen von ESPtools auch os genutzt. Ging dann doch ...

pillmuncher hat geschrieben: Mittwoch 10. Juli 2024, 13:38

Es werden nicht weniger libs geladen, wenn du das os Modul aus der Standardbibliothek verwendest, statt das pathlib Modul aus der Standardbibliothek.
[/quote]
Das OS Modul wird auch am Ende zum Flashen gebraucht. Also doch ein Modul weniger.

Hab ich auch zuerst gedacht, aber dann ist mir aufgefallen, dass Dateien gesucht werden, die die Endung '.bin' haben und in denen *nicht* '_patched' vorkommt. Das hatte ich zuerst übersehen, weil der OP not "_patched" in name verwendet hat, statt "_patched" not in name .
[/quote]
Dass musste ich ...

Sirius3 hat geschrieben: Mittwoch 10. Juli 2024, 13:17 @pillmuncher: das if spart man sich, indem man glob benutzt:

Genau das ist m.E. nicht noob freundlich.
Glob ist wieder Kauderwelsch...

Pathlib ist jetzt weg.
...
Das ist irgendwie nicht leichter zu verstehen. Viel einfacher als das hier geht es kaum:
for i, filepath in enumerate(Path().iterdir()):
if filepath.suffix == '.bin' and '_patched' not in filepath.stem:
print(f'[{i}] {filepath}')

Traceback (most recent call last ...

Warum benutzt Du os.listdir, wenn Du doch schon pathlib benutzt?

Pathlib ist jetzt weg.
Ich erwarte alles (PythonPatcher.py und Dateien) im gleichen Verzeichnis, egal welches.
Sie werden auch vom Anwender zusammen in einem Verzeichnis heruntergeladen.

Auch das würde man einfacher mit pathlib ...

Dann hofft mal, dass die Nutzer keine Verzeichnisse erstellen, die auf .bin enden. Im Beispielcode wird nicht geprüft, ob es sich um eine Datei handelt.

Psst !
Die –auch kritisierte– altmodische Art, die Datei mit open() zu öffnen, fängt das automatisch ab.
Traceback (most recent call last ...

Wenn du aber Code postest, musst du damit rechnen, dass der Code kritisiert wird. Es wird der Code kritisiert, nicht der Mensch.
Das war nicht das Problem. Dass mein Code kritisiert wird, ist berechtigt.
Ich habe aber eine andere Motivation. Ich will über alles lesbarer Code anbieten.
Für ...

Woher weisst Du wie die Python-Dokumentation in der Praxis aussieht wenn Du die gar nicht gelesen hast? Oder sind das jetzt Deine Ausreden das gar nicht erst anfangen zu müssen, weil das ja bestimmt alles ganz schlecht ist‽

Habe ich doch, damit das erste funktionierenden Skript erstellt, ganz ...