třetí úkol - Githubissues

ana-ku commented 2 years ago

Ahoj, mám hotový třetí úkol. https://github.com/ana-ku/ukoly-python-Czechitas/blob/main/python3.py

AnetaPopelova commented 2 years ago

Ahoj,

program funguje 👍, napada me ale me par veci, ktere by slo zlepsit.

je (uzitecnym) pravidlem nejdriv na zacatku definovat funkce, pak je teprve poustet a napr. ptat se na input
zadanim bylo vytvorit dve samostatne funkce (jedna bude kontrolovat cislo, druha pocitat cenu), ktere pak postupne zavolas
pokud zadam spatne cislo, mohlo by mi to vyhodit chybu hned a ne az po zadani zpravy (jako uzivatel bych byla dost frustrovana, ze ma jiste peclive formulovana zprava prisla vnivec)
takze neco jako: pokud je vystup (return) prvni funkce informace, ze cislo je spravne, pust druhou funkci, jinak vyhod chybovou hlasku
neni treba pouzivat pass, zpravidla chceme, aby nam funkce neco vratila
je fajn udrzovat nazvy funci a promenych srozumitelne - phone_number misto nr atp. - pozdeji to ocenis, fakt ;-)

Podminky mas napsane dobre, chce to jen trochu ucesat :-) Necham to issue zatim otevrene, myslim ze stoji za to zkusit to rozepsat do dvou samostatnych funkci.

ana-ku commented 2 years ago

Ahoj Anet, díky za feedback Upravila jsem to, ale musím říct, že jsem chvíli nevěděla, jak dál v první funkci. Kontroluju číslo v první if podmínce. Pokud je správné, ideálně bych chtěla, aby funkce pokračovala dál a nic se nestalo, proto jsem tam dala pass. Teď jsem to teda upravila, aby return zprava. Je to takhle OK?

AnetaPopelova commented 2 years ago

Mela jsem na mysli neco takoveho:

def phone_number_check (phone_number):
    phone_number = phone_number.replace(' ','')
    if len(phone_number) == 9:
        valid = True
    elif len(phone_number) == 13 and phone_number[0:4] == '+420':
        valid = True
    else:
        valid = False
    return valid

Vyhoda je, ze pak mas k dispozici promenou "valid", se kterou muzes dal pracovat (napr. i v dalsich programech). Zaroven je lepe citelne, co vlastne funkce dela... Ale to uz je asi o me osobni preferenci...

ana-ku / ukoly-python-Czechitas

třetí úkol #3