Мне среди прочих попалась задача, которую вы тут уже разбирали.
Решение panama555 с использованием модуля datetime очень хорошее и короткое, но на собеседованиях часто просят решать без импорта модулей, а самостоятельно писать, поэтому снова поднимаю этот топик.
Моя программа работает и работает верно, но выглядит громоздкой и много ветвлений.
Подскажите, можно ли как-то упростить и улучшить ее читаемость? Можно ли как-то избегать такого количества if / else в коде?
Я еще не умею определять вычислительную сложность того или иного решения. Правильно ли, что я выделила проверку високосности года в отдельную функцию или это избыточно и надо было все делать в рамках основной функции?
Вроде так читаемость кода немного повышается, а так же в реальных условиях будет проще повторно использовать код с проверкой високосности года при необходимости, но я не знаю , сделает ли это программу плохой с точки зрения производительности.
Буду признательна за любые указания на мои ошибки и плохой codestyle
# Написать функцию date , принимающую 3 аргумента — день, месяц и год.
# Вернуть True, если такая дата есть в нашем календаре, и False иначе.
def is_year_leap(year):
"""
Принимает аргумент: порядковый номер года
Возвращает значение: если високосный год - True, иначе - False
"""
if year % 4 != 0:
return False
elif year % 100 == 0:
if year % 400 == 0:
return True
else:
return False
else:
return True
def date(day, month, year):
"""
Принимает аргументы: день, месяц, год
Возвращает значение: если дата правильная - True, иначе - False
"""
# Задаем количество дней в месяцах невисокосного года
set_months = {1: 31,
2: 28,
3: 31,
4: 30,
5: 31,
6: 30,
7: 31,
8: 31,
9: 30,
10: 31,
11: 30,
12: 31}
# Проверяем заданы ли верно год и месяц
if year > 0 and (month >= 1 and month <= 12):
# Изменяем количество дней для февраля в високосных годах
if month == 2 and is_year_leap(year) == True:
set_months[2] = 29
# Проверяем задан ли верно день
if day in range(1, set_months[month]+1):
return True
else:
return False
else:
return False
# Тест кейсы
print(date(31, 12, 2020)) # Правильная дата
print(date(1, 0, 2000)) # Месяц вне диапазона
print(date(1, 1, 0)) # Год вне диапазона
print(date(29, 2, 2000)) # 29 февраля високосный год
print(date(29, 2, 1900)) # 29 февраля невисокосный год