Finální verze projektu by A21015 · Pull Request #208 · turecekt/vs_project

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

+# 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

+# 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

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


Suggested change

class Prevod:

def __init__(self):

pass

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

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


Suggested change

@staticmethod

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

+            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

+
+    # 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

+
+        # 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

+    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

+    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

Conversation

A21015 commented Jan 29, 2022

Uh oh!

turecekt left a comment

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

turecekt left a comment

Choose a reason for hiding this comment

Uh oh!

A21015 commented Jan 31, 2022

Uh oh!

Choose a reason for hiding this comment

Uh oh!

A21015 commented Feb 1, 2022

Uh oh!

turecekt left a comment

Choose a reason for hiding this comment

Uh oh!

turecekt left a comment

Choose a reason for hiding this comment

Uh oh!

turecekt left a comment

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!

2 participants