это мой код для задачи hackerRank написать функцию
def is_leap(year):
x=str(year)
y=int(x[-2:])
return y%4==0 or y%400==0 and y%100!=0
year = int(input())
print(is_leap(year))
Все 5 тестовых случаев работали, кроме одного, когда year=2100, и я хотел бы знать, почему? что не так с моим кодом? редактировать: после запуска кода я получил следующее:
Сообщение компилятора: неправильный ответ
Ввод (стандартный): 2100
Ожидаемый результат: ложь
Проблема в том, что вы тестируете только две последние цифры, как видно из y=int(x[-2:]), для чего на самом деле нет причин. Ограничения программы уже говорят вам, что ввод будет целым числом от 1900 до 10000, поэтому вы можете просто работать с годом. Кроме того, ваш оператор return будет оценивать or перед and (см. здесь), поэтому последнее, что он проверяет, будет !=100, тем самым разбивая его на случай 400.
(y% 4 == 0 and y % 100 != 0) or y % 400 == 0
- person FlorianDe; 18.03.2019
Я не совсем уверен, почему ваш код не работает, но у меня есть решение:
def is_leap(year):
if year%4 == 0:
if year%100 == 0:
if year%400 == 0:
return True #divisible by 4, divisible by 100, and divisible by 400
else:
return False #divisible by 4 and divisible by 100
else:
return True #divisible by 4 and not divisible by 100
else:
return False #not divisible by 4
И чтобы проверить это:
for i in range(1000, 2501, 4):
if not is_leap(i):
print(str(i)) # this will print all the "irregular" leap years between 1000 and 2500
Кроме того, я не уверен, где вы запускаете этот код, но в вопросе действительно запрашиваются логические значения (True или False), и вместо этого вы возвращаете годы, поэтому, возможно, поэтому вы получаете Ошибка.
for i in range?
- person Lord Elrond; 18.03.2019
Мне нравится ваш лаконичный код оценки и возврата результата в одной строке кода! Однако есть две проблемы с вашим кодом: 1) вы смотрите на последние две цифры вместо проверки целого числа 2) неверный порядок логической оценки. Попробуйте этот простой код
def is_leap(year):
y=int(year)
return y%400==0 or (y%100 != 0 and y%4==0)
year = input()
print(is_leap(year))
