Python3 problem z obluga wyjatku zwiazanego z string

Question

Czesc napisalem metode w pewnej klasie do ewidencji. A w tej metodzie oblusge wyjatku w ktorym parametr bedacy zbiorem slownikow jest podany w nieprawidlowym formacie.

https://pastebin.com/UVBE74Gq

Kod wyglada nastepujaco jednak mam problem poniewaz wpisujac nawet 'krowa' wyjatek nie jest wyrzucany a pozniej mam jakies glupoty w ewidencji.

Wiecie jak ten problem rozwiazac w moim przypadku?

Z gory dzieki za pomoc

Answer 1

Zacznijmy od tego, że tego się nie da czytać. Używaj zmienne i unikaj range() w pętlach. Nie wiem też czemu zamieniasz dicty na dict()y..

A ogólnie to problem jest pewnie tu:

if rzecz == 'a' or 'b' or 'c':

Bo to jest równoważne temu:

if (rzecz == 'a') or 'b' or 'c': # zawsze prawdziwe

Prawdopodobnie powinno być tak...

if rzecz in ('a', 'b', 'c'):

Ale patrząc na ten kod, zgaduję że to sam pomysł jest zły (sprawdzasz czy pierwszy klucz jest poprawny, po czym od razu zapisujesz? Wygląda dziwnie.).

Comments

Wielkie dzieki za pomoc podzialalo :D a co do zapisywania to faktycznie to co robilem bylo bez sensu.

https://pastebin.com/0eFcrwkw chyba wyglada juz lepiej i co wazne dziala jak powinno

---

Dalej jest trochę mało optymalnie. Umieść mały kod z tym fragmentem (minimalna okrojona klasa). Coś podpowiem.

---

Wciąż to się słabo czyta - czemu nie napiszesz zwykłych pętli po kontenerach?

def set_steps(self, steps):
    for step in steps:
        for key in step:
            if key not in ('sold/bought','sold','step_amount','exec_date','Stock','pair','price'):
                raise ValidationError("Wrong argument dict_list syntax") # SyntaxError sluzy do czego innego, lepiej miec wlasne wyjatki