Workshop 03022024 by Diana-Prusova · Pull Request #5 · CodersLab-CZ/workshop

Diana-Prusova · 2024-02-02T14:48:03Z

No description provided.

ExperimentalHypothesis · 2024-02-03T07:36:51Z

1] guessing game/main.py

+                print("Too big number...")
+                continue
+            else: 
+                return "!!! YOU WIN !!!"


v returnu ten string nevracet - tim spis kdyz delas typing a rikas ze ta funkce ma vratit int

Rozumím, ten typing jsem tam zapomněla. Takže správné řešení bude smazat ten typing a do returnu dát proměnou, která obsahuje tuhle zprávu?

udelej proste print("u win") a return
a typing bude None

Děkuju, hotovo

ExperimentalHypothesis · 2024-02-03T07:37:16Z

1] guessing game/main.py

+    # getting a number of player
+    while True:
+        try:
+            player_num = int(input("Enter whole number: "))


ten try je zbutecne velky, idealne je obalit jen tu cast kodu kde cekas tu vyjimku

Opraveno, jak dodělám všechny opravy, postnu.

ExperimentalHypothesis · 2024-02-03T07:40:47Z

2] loto/main.py

+    Funktion return a sorted list of numbers.
+    """
+    # gettint a list of numbers
+    player_nums = []


lepsi je set protoze delas memebership test s pouzitim in operatoru. Set je na tuhle operaci lepe staveny, ma konstatni casovou komplexitu pro ziskani prvku, zatimco list linearni - ty listy jsou tady kratke ale i tak je set vzdy lepsi

ExperimentalHypothesis · 2024-02-03T07:41:07Z

2] loto/main.py

+    The function uses randint() and gets a list 
+    of six original numbers in the range 1-49.
+    """
+    draw_nums = []


tady by byl lepsi taky set, ze stejneho duvoadu jako vyse

ExperimentalHypothesis · 2024-02-03T07:42:48Z

2] loto/main.py

+
+    # evaluation of results
+    hits = 0
+    for num in player_nums:


tady pak delas dvojity loop prave proto, ze mas ty data v listech a ne v setech. Takze tohle pobezi v kvdardatickem case zatimco kdyzbys mela sety tak muze bezet v linernim case.

jak rikam, ty listy jsou tu male, takze to tolik nevadi, ale obecne kdyz mas v kodu IN tak je lepsi to chcekovat oproti setu

ExperimentalHypothesis · 2024-02-03T07:43:28Z

3] guessing game 2/main.py

+        return answer
+    else:
+        print("Wrong value, try again...") 
+        get_answer()


to delas rekurzivne jo? to je teda fikany :) Potencialne se ti ale muze stat, ze na to crashne cely program kdyz bude nekdo dostatecne dlouho zadavat chybne hodnoty, protoze se bude kupit kupit rekurzivni volani funkci na sebe a to ma nejkay limit

Jj, učili mě, že se máme vyvarovat tomu, abychom psali stejný kód více krát, tak mi to přišlo nejjednodušší. Mám to přepsat?

no to bys mela. anebo musis osetrit ze to nevyhodi eror.

ale to nemusi opakovat. Staci to misto do rekurze dat do loopu preci

ExperimentalHypothesis · 2024-02-03T07:45:09Z

3] guessing game 2/main.py

+
+    while num_of_tries:
+        # calculation of the quessed number
+        guess = int((max-min) / 2) + min


ne cast na int, ale udelat flor div //

ExperimentalHypothesis · 2024-02-03T07:46:54Z

3] guessing game 2/main.py

+            num_of_tries -= 1
+        elif answer == "you quessed":
+            print("I won!")
+            num_of_tries = 0


pracovat s numer of tries a mit to jako kontrolni promenou v loopu je trochu divny - proc to delas takhle?

Přijde mi to nejkratší podmínka pro while, aby vracela True, pokud má hráč ještě pokusy a False, pokud je už vyčerpal. V téhle části jsem zvažovala, jestli nepoužít break, ale použití num_of_tries = 0 mi přišlo pro ostatní čtenáře mého kódu čitelnější, že se pořád držím stejného principu, že jestli while loop jede závisí na téhle proměnné.

ne, to neni ze ma pokusy nebo vycerpal

ten algortmus sam o sobe zajisti, ze nemuze prekrocit 10 pokusu na 1000 protoze bezi v logaritmicke komlexite a log(1000) = 10.

ty tam vubec tohle nemas kontrolovat

ExperimentalHypothesis · 2024-02-03T07:47:13Z

4] guessing game 3/main.py

+guess = 0
+
+def calculation(minimum=int, maximum=int) -> int:
+    return int((maximum-minimum) / 2) + minimum


flor div //

ExperimentalHypothesis · 2024-02-03T07:48:11Z

4] guessing game 3/main.py

+
+@app.route("/too_low", methods=["GET", "POST"])
+def too_low():
+    global min_num, max_num, guess


uff .. gobalni promenne jsou dost bad practise, to se moc neuc. To se dela opravdu jen v pripadech kde se musi neco hodne ohnout.

Chápu, je mi jasný, že je to řešení poněkud na čuňátko... ale já prostě nemohla přijít na to, jak jinak to udělat. Neměl bys prosím nějakou radu?

no ja myslim ze mas hlavne udelat jen jedno tu routu @app.route("/", methods=["GET", "POST"])
a v ni mit celou tu logiku, ktera je v podstate stejna jako v tom predeslem ukolu, jenom je to obalene do HTML.

ExperimentalHypothesis · 2024-02-03T12:06:14Z

5] 2001/main.py

+    math_mark = ""
+
+    # first format check
+    if "D" not in entered_value:


cela ta logika na parsovani vstupu je super komplextni :)

to jde udelat celkme jednoduse pres regex

O Regex slyším prvně, to se teď hned nestihnu doučit, tak jsem opravila bez toho.

ExperimentalHypothesis · 2024-02-03T12:06:39Z

5] 2001/main.py

+    # first split + additional check
+    first_cut = entered_value.split("D")
+    if len(first_cut) != 2:
+        return error_allert


proc vraci nejaky string? normalne vyhod vyjimku

ExperimentalHypothesis · 2024-02-03T12:07:37Z

5] 2001/main.py

+            math_mark = "+"
+            second_cut = second_cut_check(first_cut[1], math_mark)
+            if second_cut == None:
+                return error_allert


ojojoj tady je trojite zanoreni.. to je opravdu velmi velmi komplikovane. ttakle to nemuzes delat

ExperimentalHypothesis · 2024-02-03T14:25:44Z

5] 2001/main.py

+    Pokud ano, vrátí hodnotu hodu za použití funkce 
+    calculation(). Pokud ne, vrátí chybové hlášení. 
+    """
+    error_dice = "WRONG DICE: a correct dice - D3, D4, D6, D8, D10, D12, D20, D100"


takle se to ale nedela nejake errory, k tomuhle s epouzivaji vyjimky.

ExperimentalHypothesis · 2024-02-03T14:25:59Z

5] 2001/main.py

+    Pokud ano, vrátí hodnotu hodu za použití funkce 
+    calculation(). Pokud ne, vrátí chybové hlášení. 
+    """
+    error_dice = "WRONG DICE: a correct dice - D3, D4, D6, D8, D10, D12, D20, D100"


takle se to ale nedela nejake errory, k tomuhle s epouzivaji vyjimky.

ExperimentalHypothesis · 2024-02-03T14:26:58Z

5] 2001/main.py

+        try: 
+            rolls = int(first_cut[0])
+        except ValueError:
+            return error_value


tady bys mela spis napsat raise a vyhodit nejakou tu vyjimku

ExperimentalHypothesis · 2024-02-03T14:27:49Z

5] 2001/main.py

+
+    # výpočet hodu s modifikátorem
+    if modifier:
+        return eval(str(calculation(rolls, dice)) + math_mark + str(modifier))


neni potreba delat eval na string kdyz ti ta funkce vrti int se kterym jde hned pocitat

Diana-Prusova added 6 commits January 30, 2024 11:39

test připojení

be1ffea

first example finish

8fa64c0

vypracováno druhé cvičení

1020070

vypracována další cvičení

44de903

oprava cvičení 4

b651e2d

oprava čtvrtého cvičení

89376fc

ExperimentalHypothesis reviewed Feb 3, 2024

View reviewed changes

Diana-Prusova added 4 commits February 3, 2024 09:54

Oprava prvního a druhého cvičení

ce5e942

opraveno třetí cvičení

9874229

oprava prvního cvičení

bda1fa7

vypracováno páté cvičení

433d0df

ExperimentalHypothesis reviewed Feb 3, 2024

View reviewed changes

Diana-Prusova added 2 commits February 3, 2024 14:29

oprava pátého cvičení

97bcbb6

snad už poslední oprava pátého cvičení

fae5946

ExperimentalHypothesis reviewed Feb 3, 2024

View reviewed changes

vypracováno čtvrté cvičení

fce33c5

Diana-Prusova added 2 commits February 3, 2024 19:59

oprava proměnných ve čtvrtém cvičení

746f445

Přidána mock exam.

89adde1

Conversation

Diana-Prusova commented Feb 2, 2024

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

ExperimentalHypothesis Feb 3, 2024 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

ExperimentalHypothesis Feb 3, 2024 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

ExperimentalHypothesis Feb 3, 2024 •

edited

Loading

ExperimentalHypothesis Feb 3, 2024 •

edited

Loading