your-dear/labs

Курс Програмирование на Python

Labs

labs's Issues

Замечания лаб 1 @Your-Dear

@Your-Dear
Замечания ниже нужно исправить.
Замечания:

1. Оформление не по PEP8. Примеры ошибок:

# Неверно
num_lines= len(books)

#Верно
num_lines = len(books)

# Неверно
reader(csv_books,delimiter=";")

#Верно
reader(csv_books, delimiter=";")

# Неверно
dateTimeObj = datetime.strptime(date, "%d.%m.%Y %H:%M")

#Верно (должно быть low_case)
date_time_obj = datetime.strptime(date, "%d.%m.%Y %H:%M")

2. Это можно написать короче

year == 2014 or year == 2016 or year == 2017:

# Короче
year in [2014, 2016, 2017]:

3. Не используйте магические числа (типа 20, 30) в программе, суть которых понятна только вам. Обозначайте их как константы. Константы обычно указываются вначале программы после блока импортов. Пример:

import csv

MAX_NAME_LENGTH = 30

....

Замечания лаб2 @Your-Dear

@Your-Dear

• Оформление не по PEP8. Неверное обрамление пробелами в выражениях , неверное обрамление пробелами в параметрах функции, неверное название одной из функций, не соблюдена максимальная длина строки в 79 символов. Исправить. Читайте PEP8 https://peps.python.org/pep-0008/
• Вместо odd*-1 лучше воспользоваться функцией abs(odd)

Замечания лаб3 @Your-Dear

@Your-Dear

Нужно исправить:

• Использование в функциях глобальных переменных плохая практика. Лучше стараться сделать функцию чистой. Передавайте объекты, которые вы хотите модифицировать, явно через параметры функции.
• Сделайте размер окна и пути до медиа константами.
• Используйте относительный путь для файла картинки и музыки, а не абсолютный. Это ухудшает переносимость программы на разные компьютеры.
• Оформление не по PEP8 (максимальная длина строки 79 символов)