@Quaixy: Das ist uneinheitlich eingerückt. Konvention ist vier Leerzeichen pro Ebene.
Der erste Teil scheint auf Modulebene zu stehen. Auf Modulebene sollte nur Code stehen der Konstanten, Funktionen, und Klassen definiert. Das Hauptprogramm steht üblicherweise in einer Funktion die `main()` heisst.
Daten und Code sollte man nicht wiederholen. Das macht unnötig Arbeit bei Änderungen und ist fehleranfällig. Der Dateiname sollte nur einmal im Code stehen, als Konstante definiert.
Namen sollten zum Wert passen. `log_file` ist sicher aus irgendeinem Beispiel das nichts mit einer Datei mit Spielernamen zu tun hat.
Textdateien sollte man mit einer expliziten Kodierung öffnen. Sonst ist das Programm/die Datei nicht portabel.
Einmal wird ``with`` verwendet, sonst nicht. Das sollte man immer verwenden wenn man es kann.
Das öffnen und sofort wieder schliessen und das öffnen ohne das mit dem Rückgabewert von `open()` etwas gemacht wird, ist sinnfrei.
Namen sollten keine kryptischen Abkürzungen enthalten. Wenn man `players_file` meint, sollte man nicht `fplayers` schreiben.
Grunddatentypen haben in Namen nichts zu suchen. `players_list` wird auch überhaupt gar nicht an eine Liste gebunden sondern an `None`, denn die `writelines()`-Methode hat keinen expliziten Rückgabewert. An die Liste willst Du den Namen anhängen.
`writelines()` macht auch gar keinen Sinn wenn man da eine Zeichenkette übergibt. Da wird dann jedes Zeichen einzeln geschrieben.
Funktionen und Methoden bekommen alles was sie brauchen als Argument(e) übergeben. `add_name()` braucht also `players` als Argument.
`flush()` direkt vor `close()` ist sinnlos, eventuell gepufferte Daten werden durch das schliessen der Datei rausgeschrieben.
Wenn man das Leerzeichen zum trennen von Einträgen verwendet, sollte man sicherstellen, dass keine Leerzeichen *in* den Einträgen sind.
Da die Namen nicht durch Leerzeichen getrennt sind, sondern durch Leerzeichen abgeschlossen werden, muss die Datei wenn sie nicht komplett leer ist, mit einem Leerzeichen enden. Und in jedem Fall führt das ``.split(" ")`` zu einem zusätzlichen Element am Ende der Liste das einfach nur eine leere Zeichenkette ist und entfernt werden muss.
Zwischenstand (ungetestet):
Code: Alles auswählen
#!/usr/bin/env python3
PLAYERNAMES_FILENAME = "players.txt"
NAME_TERMINATOR = " "
def add_player(players, name):
if NAME_TERMINATOR in name:
raise ValueError(
f"name {name!a} contains a terminator ({NAME_TERMINATOR!a})"
)
with open(PLAYERNAMES_FILENAME, "a", encoding="utf-8") as file:
file.write(name + NAME_TERMINATOR)
players.append(name)
def main():
try:
with open(PLAYERNAMES_FILENAME, "r", encoding="utf-8") as file:
content = file.read()
if content and content[-1] != NAME_TERMINATOR:
raise ValueError(
f"expected terminator ({NAME_TERMINATOR!a}) at end,"
f" found {content[-1]!a} instead"
)
players = content.split(NAME_TERMINATOR)
assert players[-1] == ""
players.pop()
except FileNotFoundError:
players = []
...
if __name__ == "__main__":
main()
