Finální verze projektu #208

A21015 · 2022-01-29T18:08:25Z

No description provided.

turecekt

Doplnit unit testy aby procházel pytest a opravit formátování aby procházel kontrola flake8

turecekt · 2022-01-30T11:18:33Z

projekt_pucek_2021/22/main.py

+# Deklarace proměnné typu string,
+# která následně určuje, jestli se má program ukončit
+zadani = " "
+
+# Cyklus, při kterém program běží, dokud není na konci zadáno A pro ukončení
+while zadani != 'a':
+    rimZad = ""
+    arZad = 0
+    rim = 0
+
+    # Cyklus, při kterém běží input do doby, než uživatel zadá validní hodnotu
+    while rim < 1 or rim > 3999:
+        rimZad = input("Zadej římské: ")
+        x = rimZad.isnumeric()
+
+        # Obsahuje-li římské zadání číslo, jedná se o chybu
+        if x == 1:
+            print("Nezadal jsi římské číslo.")
+
+        # Pokud je vše zadáno správně, program pokračuje dál
+        else:
+            if rimZad.islower():
+                rimZad = rimZad.upper()
+            rim = Prevod().naar(rimZad)
+
+            # Pokud se výsledek nachází v intervalu
+            # pro platnost, program vypíše výsledek
+            if 0 < rim < 4000:
+                print(rimZad + str(" = ") + str(rim))
+
+            # Je-li výsledná hodnota mimo interval
+            # pro platnost, program zahlásí chybu
+            else:
+                print("Chybné zadání.")
+
+    # Cyklus, který běží po dobu, dokud není správně zadané číslo
+    while arZad < 1 or arZad > 3999:
+        arZad = str(input("Zadej arabské: "))
+        x = arZad.isnumeric()
+
+        # Kontrola, jestli bylo správně zadáno číslo v zadání
+        if x == 1:
+            arZad = int(arZad)
+            # Převod proměnné typu string na integer
+
+            # Pokud se zadané číslo nenachází v
+            # platném rozsahu, program zahlásí chybu
+            if arZad < 1 or arZad > 3999:
+                print("Chybné zadání.")
+
+            # V opačném případě se provede převod
+            else:
+                print(str(arZad) + str(" = ") + Prevod().narim(arZad))
+
+        # Nebylo-li zadáno číslo, program zahlásí chybu
+        else:
+            print("Nezadal jsi číslo.")
+            arZad = 0
+            # Zápis hodnoty, aby při špatném zadání neskončil cyklus
+
+    # Ukončovací dialog programu
+    zadani = input("Přejete si ukončit program? (A/N)\n")
+
+    # Podmínka pro not-case-sensitive zadání pro ukončení
+    if zadani.isupper():
+        zadani = zadani.lower()
+        # Převod velkého písmena na malé


Toto dát pod podmínku if __name__ == "__main__"

turecekt

stále nejsou unit testy aby bylo dostatečné pokrytí kódu testy a opravené formátování aby procházela kontrola flake8

A21015 · 2022-01-31T20:42:34Z

Dobrý den, nevím co by mohlo být za problém, unit testy mi vždy projdou bez chyby.

turecekt · 2022-01-31T21:27:40Z

projekt_pucek_2021/22/main.py

+# Testovací příkazy, je možné přehlédnout..
+# print(Prevod().narim(3999))
+# print(Prevod().naar("MMMCMXCIX"))


Suggested change

# Testovací příkazy, je možné přehlédnout..

# print(Prevod().narim(3999))

# print(Prevod().naar("MMMCMXCIX"))

def test_prevod():

"""Testovací příkazy."""

assert Prevod().narim(3999) == "MMMCMXCIX"

assert Prevod().naar("MMMCMXCIX") == 3999

A21015 · 2022-02-01T22:23:20Z

Jaké jsou přesně kritéria, aby ten pytest prošel? Co všechno to kontroluje?

turecekt

Obecně pytest volá všechny funkce s podporovanými keywordy v názvu (např. prefix test_), které najde ve zdrojovém kódu projektu.
Funkce test_prevod pomocí dvou assertů kontrolovalo správnost návratů vámi implementovaných funkcí narim a naar pro vybranou hodnotu vstupního parametru.
Funkci test_prevod vraťte zpátky a povolte si provádění workflow Python checker (python_checker.yml) v actions, zde: https://github.com/A21015/vs_project/actions
Při každém odjetí do repozitáře se následně test automaticky spustí a v actions uvidíte průběh/výsledek.

turecekt

Ještě prosím přesunout soubor main.py do kořenového adresáře.

turecekt

Zkuste ještě zrušit tu třídu viz navrhované úpravy, stejně není využito OOP, mohlo by se to už vyjít do limitu pokrytí kódu testy 66% viz Checks/Actions. Pokud ne, tak proveďte ještě nějaký refaktoring kódu na zkrácení počtu řádků mimo funkce, které máte již pokryté testy.
Následně pak upravte chyby, které bude následně vracet flake8.

turecekt · 2022-02-04T17:17:08Z

main.py

+class Prevod:
+    def __init__(self):
+        pass


Suggested change

class Prevod:

def __init__(self):

pass

turecekt · 2022-02-04T17:17:22Z

main.py

+        pass
+
+    # Metoda pro převod z arabských čísel na římská
+    @staticmethod


Suggested change

@staticmethod

turecekt · 2022-02-04T17:17:35Z

main.py

+            return " "  # Vrátí zpět pouze mezeru
+
+    # Metoda pro převod z římských čísel na arabské
+    @staticmethod


Suggested change

@staticmethod

turecekt · 2022-02-04T17:21:42Z

main.py

+
+    # Cyklus, při kterém program běží, dokud
+    # není na konci zadáno A pro ukončení
+    while zadani != 'a':


Suggested change

while zadani != 'a':

while zadani.lower() != 'a':

turecekt · 2022-02-04T17:22:40Z

main.py

+
+        # Podmínka pro not-case-sensitive zadání pro ukončení
+        if zadani.isupper():
+            zadani = zadani.lower()
+            # Převod velkého písmena na malé


Suggested change

# Podmínka pro not-case-sensitive zadání pro ukončení

if zadani.isupper():

zadani = zadani.lower()

# Převod velkého písmena na malé

turecekt · 2022-02-04T17:23:43Z

main.py

+    assert Prevod().narim(3999) == 'MMMCMXCIX'
+    assert Prevod().narim(3888) == 'MMMDCCCLXXXVIII'
+    assert Prevod().narim(2421) == 'MMCDXXI'
+    assert Prevod().narim(15) == 'XV'


Suggested change

assert Prevod().narim(3999) == 'MMMCMXCIX'

assert Prevod().narim(3888) == 'MMMDCCCLXXXVIII'

assert Prevod().narim(2421) == 'MMCDXXI'

assert Prevod().narim(15) == 'XV'

assert narim(3999) == 'MMMCMXCIX'

assert narim(3888) == 'MMMDCCCLXXXVIII'

assert narim(2421) == 'MMCDXXI'

assert narim(15) == 'XV'

turecekt · 2022-02-04T17:24:16Z

main.py

+    assert Prevod().naar("MMMCMXCIX") == 3999
+    assert Prevod().naar("MMMDCCCLXXXVIII") == 3888
+    assert Prevod().naar("MMCDXXI") == 2421
+    assert Prevod().naar("XV") == 15


Suggested change

assert Prevod().naar("MMMCMXCIX") == 3999

assert Prevod().naar("MMMDCCCLXXXVIII") == 3888

assert Prevod().naar("MMCDXXI") == 2421

assert Prevod().naar("XV") == 15

assert naar("MMMCMXCIX") == 3999

assert naar("MMMDCCCLXXXVIII") == 3888

assert naar("MMCDXXI") == 2421

assert naar("XV") == 15

Finální verze projektu

8174af3

turecekt requested changes Jan 30, 2022

View reviewed changes

Úpravy pro funčknost unit testů

64f06d5

turecekt requested changes Jan 31, 2022

View reviewed changes

Úpravy pro funkčnost unit testů

a45aa50

turecekt requested changes Jan 31, 2022

View reviewed changes

Úpravy

b62154a

Úpravy, aby bylo využito co nejvíce částí kódu

7f64718

turecekt requested changes Feb 2, 2022

View reviewed changes

A21015 added 4 commits February 3, 2022 16:45

Test

2018f90

Test 2

f7d5318

Test 3

bc35ca5

Test 4

8f7ec24

turecekt requested changes Feb 3, 2022

View reviewed changes

Snad finální commit

3cd3f85

turecekt requested changes Feb 4, 2022

View reviewed changes

A21015 added 8 commits February 4, 2022 18:52

Finální úpravy - odstranění třídy pro metody, aj.

a979032

Test

0e308f8

Test 2

b893bd9

Test 3

05e4815

Refaktoring kódu

611ffd4

Podrobnější refaktoring kódu

5cf0874

Pokus o dosáhnutí hranice 66%

00130e6

Test

5aed2da

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Finální verze projektu #208

Finální verze projektu #208

A21015 commented Jan 29, 2022

turecekt left a comment

turecekt Jan 30, 2022

turecekt left a comment

A21015 commented Jan 31, 2022

turecekt Jan 31, 2022

A21015 commented Feb 1, 2022

turecekt left a comment

turecekt left a comment

turecekt left a comment

turecekt Feb 4, 2022

turecekt Feb 4, 2022

turecekt Feb 4, 2022

turecekt Feb 4, 2022

turecekt Feb 4, 2022

turecekt Feb 4, 2022

turecekt Feb 4, 2022

Finální verze projektu #208

Are you sure you want to change the base?

Finální verze projektu #208

Conversation

A21015 commented Jan 29, 2022

turecekt left a comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

turecekt left a comment

Choose a reason for hiding this comment

A21015 commented Jan 31, 2022

Choose a reason for hiding this comment

A21015 commented Feb 1, 2022

turecekt left a comment

Choose a reason for hiding this comment

turecekt left a comment

Choose a reason for hiding this comment

turecekt left a comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment