Conversation
| print("Zadej unikátní čísla v rozsahu 1 - 49") | ||
|
|
||
| try: | ||
| numbers = [] |
There was a problem hiding this comment.
lepsi je set, protoze delas membership test pres in operator
|
|
||
| except ValueError: | ||
| print("Toto není číslo!") | ||
| return None |
There was a problem hiding this comment.
Takhle ne. Tahle logika je chybna. Ty obalujes v try cely ten loop a to neni ok, protoze kdyz kupr. hned v prvni iteraci zada cislo 99 tak to vyskoci s None a ty v tom listu nic nemas. Toto neni spravne reseni. Tim try nesmis obalit cely ten loop, ale musis ten try/except mit uvnitr toho loopu
|
|
||
| def lotto_hit(x, y): | ||
| """Funkce pro porovnání listů""" | ||
| intersection_set = set(x) & set(y) |
There was a problem hiding this comment.
to je presne ono - ty vytvoris listy a pak je tady predelavas na sety. To je lepsi aby ty funkce uz hned vytvorily ty sety a nemueslo se to dit tady
| """Hlavní funkce Lotta, která volá ostatní""" | ||
| x = lotto_user() | ||
| if x is None: | ||
| return |
There was a problem hiding this comment.
toto tady nema byt, tobe nemuze takova podminka nasat pokud mas tu logiku spravne, user musi vybrat 6 cisel
| #zacyklení hádání | ||
| x = 0 | ||
|
|
||
| while x < 10: |
There was a problem hiding this comment.
Tohle je pomerne komplikovane reseni. Ty tady vubec nemas pracovat s 10 pokusy jako s kontrolni promenou v tom loopu. To, ze ten pocitace to uhadne maximalne do 10 pokusu je dano tim, ze kdyz je ten algoritmus spravne napsany, tak bezi v logaritmicke komplexite a log(1000) je 10. S tim ale ty vubec nemas pracovat, to se deje proste automaticky.
ta podminka na while ma vypadat trochu jinak, zkus se jeste zamyslet.
| @app.route('/index', methods=['GET', 'POST']) | ||
| def game(): | ||
|
|
||
| global min, max, guess |
There was a problem hiding this comment.
globalni promenne se moc pouzivat neuc, to je bad practise a pouzije se to jen kdyz musis neco hodne ohybat a hacknout. To tady neni potreba
| import re | ||
| from random import randint | ||
|
|
||
| def roll(kostky): |
There was a problem hiding this comment.
ufff.. no jako.. celkem komplikovane zase. V pythonu jde casto o citelnost a nejakou primocarost toho kodu, tohle da pomerne znacnou namahu to precist a vyznat se co je co...
Rozhodne to jde zjednodusit, kdyztak se podivej jak, ja jsem nejak posilal svoji vetev.
Ale ze je videt zes to asi delala sama a nejak o tom premyslela.
Good Job
No description provided.