Почему функция возвращает None?

Question

[mirindanda]

from datetime import date, timedelta
import os 
os.system('cls')

def date():
    s = input("Write 1 for today, 2 for yesterday and 3 for random date: ")
     
    if s == 1:
        last_date = (date.today())
        link_today = "https://api.nasa.gov/planetary/apod?api_key=DEMO_KEY&date=" + str(last_date)
        return link_today
    elif s == 2:
        yesterday = print(date.today() + timedelta(days=-1)
        return yesterday
date()

Answer 1

[Сергей Горностаев]

Во-первых, input возвращает строку, которая очевидно не может быть равна числу. Во-вторых, нет альтернативных условий.

Comments

[mirindanda]

Не понимаю, тогда как можно задать их?

from datetime import date, timedelta
import os 

os.system('cls')




def date():
    s = input("Write 1 for today, 2 for yesterday and 3 for random date: ")
     
    if s == 1:
        last_date = (date.today())
        link_today = "https://api.nasa.gov/planetary/apod?api_key=DEMO_KEY&date=" + str(last_date)
        return link_today
    elif s == 2:
        yesterday = print(date.today() + timedelta(days=-1))
        return yesterday
date()

[Сергей Горностаев]

mirindanda, повторяю: строка '1' не равна числу 1. Используйте приведение типов.

[Андрей]

У вас не проходит ни одно условие, из-за того что как описали выше input дает str тип, а вы сравниваете с int,

if int(s) == 1 # так будет работать, но только в том случае если пользователь введет число, если ввести букву, все поломается

Так же если пользователь введет 3, у вас с таким кодом тоже ничего не пройдет, добавьте else или просто в теле функции допишите что будет происходить, если ни одно условие не пройдет

Answer 2

[kurrbanov]

1. Оберните input() в int():

s = int(input("Write 1 for today, 2 for yesterday and 3 for random date: "))

2. И print здесь зачем? Он же ничего не возвращает, отсюда и переменная yesterday будет None.

yesterday = print(date.today() + timedelta(days=-1)