Am 27.08.24 um 00:01 schrieb Stefan Ram:
sources =[ source.strip() for source in '''
C:\otherdir\example.txt
C:\somedir\example.txt
C:\somedir\example1.txt
''' .strip().split( '\n' ) if source ]
a) splitlines() ist eleganter
b) Wenn in einer Zeile nur Leerzeichen stehen, landet sie trotzdem in
Deiner Liste.
# Generator, um alle Zeilen zu strippen, erspart mehrmaliges .strip()
sources = (source.strip() for source in '''
…
'''.splitlines()) # generator expression
# Liste mit nicht-leeren Zeilen
sources = [source for source in sources if source]
if not target_path.is_dir():
raise RuntimeError( f'Target must be a directory! Target is
"{target}".' )
else:
Nach einem "raise" brauchst Du keinen else-Zweig. Wenn Du ihn weg lässt,
hat der Code weniger Ebenen und ist leichter zu lesen. DIto unten.
if filecmp.cmp( source_path, target_file_path,
shallow=False ):
pass
Hier würde ich eine "okay" ausgeben.
HTH
--
Schönen Gruß
Hartmut Goebel
Dipl.-Informatiker (univ), CISSP, CSSLP, ISO 27001 Lead Implementer
Information Security Management, Security Governance, Secure Software
Development
Goebel Consult, Landshut
http://www.goebel-consult.de
Blog:
https://www.goebel-consult.de/blog/2020/alternative-android-betriebssysteme/
Kolumne:
https://www.goebel-consult.de/blog/cissp-gefluester/2011-11-in-troja-nichts-neues/
_______________________________________________
python-de Mailingliste -- python-de@python.org
Zur Abmeldung von dieser Mailingliste senden Sie eine Nachricht an
python-de-le...@python.org
https://mail.python.org/mailman3/lists/python-de.python.org/
Mitgliedsadresse: arch...@mail-archive.com
