Úkol č. 3 - Githubissues

helenafanfrlikova commented 2 years ago

https://github.com/helenafanfrlikova/python-autumn-2022/blob/main/ukol-03.py

Ahoj, odevzdávám úkol č. 3. Nebyla jsem si jistá, jestli to s tím True a False ještě nešlo nějak jednodušeji. Moc díky! Helena

pmoravkova commented 2 years ago

reseni mas spravne. Super! Akorat par poznamek: urcite s tim true a false to zlehcit jde. Vyhod vubec promennou 'navratova_hodnota' a namisto toho rovnou pis return

def spravne_cislo(cislo):

    cislo = str(cislo)

    if len(cislo) == 9:
        return True

    elif len(cislo) == 13:
        if cislo[0:4] == "+420":
            return True
        else:
            return False
    else:
       return False

Co se tyce druhe casti, tak k tomu jen tip a poznamka. Zkus se naucit importy davat vzdy nahoru, aby vzdy zezacatku uz bylo videt, co je treba a aby byly na jednom miste. A rozhodne neni uplne dobra praktika je importovat primo uvnitr funkce. Stejne tak, a neni to spatne, nebo ze by to nefungovalo ci tak, ale spis kvuli prehlednosti a jasnosti, funkci zkus spis definovat ne v ramci cyklu, podminky apod. Tu je to jedno, spis kosmeticka uprava, obcas kdyz se ti do toho pripletou frameworky, tridy apod, tak se muze stat, ze kdyz ji definujes v ramci cyklu, ze funkce nebude tak globalni jak chces, apod.
Chvalim tuhle cast, to jsi moc hezky vymyslela: math.ceil(len(zprava) / 180) * 3

helenafanfrlikova commented 2 years ago

Ahoj Pavlo,

ještě se zpožděním moc díky za tvůj feedback!

Měj se, ahoj

Helena

From: pmoravkova @.> Sent: Tuesday, October 11, 2022 8:41 PM To: helenafanfrlikova/python-autumn-2022 @.> Cc: helenafanfrlikova @.>; Author @.> Subject: Re: [helenafanfrlikova/python-autumn-2022] Úkol č. 3 (Issue #3)

reseni mas spravne. Super! Akorat par poznamek: urcite s tim true a false to zlehcit jde. Vyhod vubec promennou 'navratova_hodnota' a namisto toho rovnou pis return

def spravne_cislo(cislo):

cislo = str(cislo)

if len(cislo) == 9:
    return True

elif len(cislo) == 13:
    if cislo[0:4] == "+420":
        return True
    else:
        return False
else:
   return False

Co se tyce druhe casti, tak k tomu jen tip a poznamka. Zkus se naucit importy davat vzdy nahoru, aby vzdy zezacatku uz bylo videt, co je treba a aby byly na jednom miste. A rozhodne neni uplne dobra praktika je importovat primo uvnitr funkce. Stejne tak, a neni to spatne, nebo ze by to nefungovalo ci tak, ale spis kvuli prehlednosti a jasnosti, funkci zkus spis definovat ne v ramci cyklu, podminky apod. Tu je to jedno, spis kosmeticka uprava, obcas kdyz se ti do toho pripletou frameworky, tridy apod, tak se muze stat, ze kdyz ji definujes v ramci cyklu, ze funkce nebude tak globalni jak chces, apod. Chvalim tuhle cast, to jsi moc hezky vymyslela: math.ceil(len(zprava) / 180) * 3

— Reply to this email directly, view it on GitHub https://github.com/helenafanfrlikova/python-autumn-2022/issues/3#issuecomment-1275119843 , or unsubscribe https://github.com/notifications/unsubscribe-auth/A3GIVD3LW2E7OIEFXNID3ALWCWYDTANCNFSM6AAAAAARBMEVWY . You are receiving this because you authored the thread. https://github.com/notifications/beacon/A3GIVD2BH3HIKWV27BCQQH3WCWYDTA5CNFSM6AAAAAARBMEVW2WGG33NNVSW45C7OR4XAZNMJFZXG5LFINXW23LFNZ2KUY3PNVWWK3TUL5UWJTSMADEOG.gif Message ID: @. @.> >

helenafanfrlikova / python-autumn-2022

Úkol č. 3 #3