Skip to content

Workshop_denisa#10

Open
denulka wants to merge 2 commits intoCodersLab-CZ:masterfrom
denulka:workshop
Open

Workshop_denisa#10
denulka wants to merge 2 commits intoCodersLab-CZ:masterfrom
denulka:workshop

Conversation

@denulka
Copy link

@denulka denulka commented Feb 4, 2024

No description provided.

print("Zadej unikátní čísla v rozsahu 1 - 49")

try:
numbers = []

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lepsi je set, protoze delas membership test pres in operator


except ValueError:
print("Toto není číslo!")
return None

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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)

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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:
Copy link

@ExperimentalHypothesis ExperimentalHypothesis Feb 6, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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):
Copy link

@ExperimentalHypothesis ExperimentalHypothesis Feb 6, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants