Я буквально вчера начал изучение Python по книге "A byte of python" и сегодня её закончил. Я выполнил заключительную задачу в книге (Адресный справочник), программа работает как нужно, однако, мой вопрос касается качества кода. По итогу я посмотрел подсказку в книге по этой задаче и понял, что делал совсем не так, как там советуется. Мне интересно знать, насколько эффективен мой код (для меня он походит на какое то чудовище), а так же насколько он хорошо оформлен. Буду рад советам и рекомендациям
Код я загрузил на гитхаб, посмотреть его можно здесь: https://github.com/GRUB666/Sundry/blob/master/ContactData.py
Вот чисто код (но отображается он криво):
import os
import pickle
#Name of save file
file_way = 'Contacts.dat'
List = []
class Date():
'''Save info about date
Atributes: __year, __month, __day
Functions: getters and setters, printDate() - format: [year.month.day]'''
def __init__(self, year = 1991, month = 1, day = 1):
self.__year = year
self.__month = month
self.__day = day
#Getters
def getYear(self): return self.__year
def getMonth(self): return self.__month
def getDay(self): return self.__day
#Setters
def setYear(self, year): self.__year = year
def setMonth(self, month): self.__month = month
def setDay(self, day): self.__day = day
def printDate(self):
print("{}.{}.{}".format(self.__year, self.__month, self.__day))
class ContactData():
'''Info about person in contacts
Atributes: __name, __birthday, __number, __adress
Functions: getters and setters, printInfo()'''
def __init__(self, name = '', birthday = Date(), number = '', adress = ''):
self.__name = name
self.__birthday = birthday
self.__number = number
self.__adress = adress
#Getters
def getName(self): return self.__name
def getBirthday(self): return self.__birthday
def getNumber(self): return self.__number
def getAdress(self): return self.__adress
#Setters
def setName(self, name): self.__name = name
def setBirthday(self, birthday): self.__birthday = birthday
def setNumber(self, number): self.__number = number
def setAdress(self, adress): self.__adress = adress
def printInfo(self):
print("Имя: ", self.__name)
print("День рождения: ", end = ''); self.__birthday.printDate()
print("Номер телефона: ", self.__number)
print("Адрес: ", self.__adress)
def initializateList():
'''Load list of contacts from memory
(Called only once during program execution)'''
global List
try:
f = open(file_way, 'rb')
List = pickle.load(f)
f.close()
except FileNotFoundError:
List = []
def saveContacts():
'''Save List of contacts in file'''
global List
global file_way
f = open(file_way, 'wb')
pickle.dump(List, f)
f.close()
def showAllContacts():
'''Outputs names of all contacts'''
global List
if len(List) > 0:
print("Всего контактов: ", len(List), "\n\n")
for i in range(len(List)):
print("{} - {}".format(i + 1, List[i].getName()))
else:
print("Контактов нет")
print("\n\n")
def createNewContact():
global List
'''Create object ContactData
return new List of contacts'''
os.system('cls' if os.name == 'nt' else 'clear')
NewContact = ContactData()
print("Заполните информацию о новом контакте (Неизвестную информацию можно не заполнять, вы сможете добавить её потом)")
NewContact.setName(input("Введите имя для нового контакта: "))
NewContact.setBirthday(Date(input("Введите год рождения: "), input("Введите номер месяца рождения: "), input("Введите день рождения: ")))
NewContact.setNumber(input("Введите номер телефона: "))
NewContact.setAdress(input("Введите адрес: "))
List.append(NewContact)
saveContacts()
def deleteContact():
'''Delete contact from List
return new List of contacts'''
global List
while True:
os.system('cls' if os.name == 'nt' else 'clear')
showAllContacts()
ch = int(input("Введите индекс элемента, который вы хотите удалить (для выхода введите -1): "))
if ch > 0 and ch < len(List) + 1:
del List[ch - 1]
break
elif ch >= len(List) + 1:
print("Этого индекса нет в списке!")
input("Введите любой символ для продолжения: ")
else:
break
saveContacts()
def showAdditionalInfo():
'''Show full info about contact'''
global List
while True:
os.system('cls' if os.name == 'nt' else 'clear')
showAllContacts()
ch = int(input("Введите индекс элемента, о котором вы хотите узнать информацию (для выхода введите -1): "))
if ch > 0 and ch < len(List) + 1:
print("\nДополнительная информация о: " + List[ch - 1].getName() + ": ")
List[ch - 1].printInfo()
break
elif ch >= len(List) + 1:
print("Этого индекса нет в списке!")
input("Введите любой символ для продолжения: ")
else:
break
def changeContact():
'''Change contact
return new contact List'''
global List
while True:
os.system('cls' if os.name == 'nt' else 'clear')
showAllContacts()
ch = int(input("Введите индекс элемента, информацию о котором вы хотите изменить (для выхода введите -1): "))
if ch > 0 and ch < len(List) + 1:
while True:
ind = input("Выберите столбец, который вы хотите изменить:\n1 - Имя\n2 - День рождения\n3 - Номер телефона\n4 - Адрес\n5 - Выход")
if ind == '1':
List[ch - 1].setName(input("Введите новое имя: "))
elif ind == '2':
List[ch - 1].setBirthday(Date(input("Введите новый год рождения: "), input("Введите новый номер месяца рождения: "), input("Введите новый день рождения: ")))
elif ind == '3':
List[ch - 1].setNumber(input("Введите новый номер телефона: "))
elif ind == '4':
List[ch - 1].setNumber(input("Введите новый адрес: "))
else:
break
break
elif ch >= len(List) + 1:
print("Этого индекса нет в списке!")
input("Введите любой символ для продолжения: ")
else:
break
saveContacts()
#Load Data
initializateList()
#MainLoop
while True:
os.system('cls' if os.name == 'nt' else 'clear')
showAllContacts()
ch = input("Что вы хотели бы сделать?\n\n1 - Создать новую запись\n2 - Удалить существующую\n3 - Узнать подробнее о какой либо записи\n4 - Изменить запись\n\n")
if ch == '1':
createNewContact()
elif ch == '2':
deleteContact()
elif ch == '3':
showAdditionalInfo()
input("\nВведите любой символ для продолжения: ")
elif ch == '4':
changeContact()
else:
print("Этого индекса нет в списке!")
input("Введите любой символ для продолжения: ")
P.S. Мне как С++ программисту сложно привыкнуть к некоторым различиям, и у меня неприятное ощущение, будто бы я невольно пытаюсь перенести С++ в Питон
global
вам не нужно). Вместо методов-геттеров/сеттеров (в стиле Java) можно использовать свойства (@property) с сеттерами. Плюс, если вы никак не ограничиваете что в поля пишется, можно использовать просто публичные поля.