Tydzień w Pythonie

Question

Witam,

Jako że zakończyłem pierwszy tydzień nauki Pythona, to chciałbym abyście ocenili jeden z moich ostatnich projektów. Kod zamieszczam poniżej.

import os

while 1==1:
 print("--------------------------------------------------------------------")
 files = os.listdir(".")
 print("")
 print("")
 for item in files:
     if os.path.isfile(item):
        print("{}     JEST PLIKIEM".format(item))

     if os.path.isdir(item):
        print("{}     JEST FOLDEREM".format(item))
 print("")
 print("")
 print("....::::MENU::::....")
 print("1. Stwórz folder w lokalizacji")
 print("2. Usuń folder")
 print("3. Usuń plik")
 print("4. Zmień nazwę elementu")
 print("5. Napisz coś w pliku tekstowym")
 print("")
 choice = input("Wybór działania:    ")

 if choice=="1":
     print("")
     dir_name = input("Podaj nazwę: ")
     os.mkdir(dir_name)
     print("")
     print("Pomyślnie utworzono folder o nazwie", dir_name)


 if choice=="2":
     print("")
     deldir_name = input("Podaj nazwę: ")
     os.removedirs(deldir_name)
     print("")
     print("Pomyślnie skasowano folder o nazwie", deldir_name)

 if choice=="3":
     print("")
     delfile_name = input("Podaj nazwę: ")
     os.remove(delfile_name)
     print("")
     print("Pomyślnie skasowano plik o nazwie", delfile_name)

 if choice=="4":
     print("")
     old_name = input("Podaj nazwę elementu: ")
     new_name = input("Podaj nową nazwę elementu: ")
     os.renames(old_name,new_name)
     print("")
     print("Pomyślnie zmieniono nazwę", old_name,"na",new_name)

 if choice=="5":
     print("")
     open_name = input("Podaj nazwę pliku tekstowego: ")
     print("")
     file_text = input("Podaj tekst do wpisania: ")

     f = open(open_name, "a+")
     f.write(file_text)
     f.close()
     print("Pomyślnie wpisano tekst do pliku")

W miarę dobrze jak na tydzień czy coś poprawić? Z góry dziękuję

Answer 1

Ogólnie OK, ale stosuj się do zasady DRY. Za dużo powtarzającego kodu jest u Ciebie.

Zamiast if choice itd zastosuj konstrukcję if elif elif.....else. Chociaż tu użyłbym po prostu funkcji.

Zamiast while 1==1 napisz while True:    <- lepiej tak ;)

Zamiast open...... write..... close zapisz tak:

with open(file_name, "a+") as my_file:

      my_file.write(text)

Python sam zamknie plik.

Poza tym brakuje obsługi błędów (konstrukcje try....except...itd.) - a przy operacjach I/O to podstawa

Comments

Dzięki bardzo za odpowiedź. Wiesz, trzeba zaznaczyć, że 7 dni to mało by ogarnąć się w tak potężnym języku. Zastosuje się do twoich rad, aczkolwiek konstrukcji except nawet nie ruszyłem. Moduł os to taki szczyt moich aktualnych umiejętności. Jeszcze raz dziękuję i miłego wieczoru ;)

---

Nie ma za co dziękować, forum jest po to żeby sobie pomagać.

Tu jest dość dobrze wytłumaczona obsługa błędów: https://www.tutorialspoint.com/python3/python_exceptions.htm

Jeszcze zamieniłbym w "menu" wybór (choice) na typ int i porównywał liczby zamiast stringów.

Answer 2

To co kolega niżej zasada DRY się kłania oraz proponował bym rozbić program na funkcję.