[Marek1991]

#programowanie #python Jak to poprawić, żeby było dobrze?

if userchoice not in (optionsstr.split(",") or commands or choices):

Chodzi mi o True gdy user_choice nie jest w żadnej z tych list. Na początku miałem to rozwinięte i porównywałem z każdą listą, ale chciałbym to skrócić.

[L.....k]

rozbij to na 3 osobne warunki (user not in optionstr or user not in commands or...) albo rozpakuj te listy w jedną

[Kura_Wasylisa]

@Marek1991 jak dlugie te listy to lepiej będzie w set zamienic

[Marek1991]

@Lunatik: No właśnie wcześniej miałem rozbite i wyglądało to mega amatorsko. Myślałem, że da się jakoś sprytnie skrócić.
@Kura_Wasylisa:

[Marek1991]

@Kura_Wasylisa: @Lunatik: dobra zrobiłem to tak i #!$%@? jajca

if userchoice not in optionsstr.split(",") + commands + choices:

[L.....k]

@Marek1991: i to i to jest czytelne
ale jak chcesz łączyć te 3 listy w jedną, to przypisz je do jakiejś zmiennej która ma sensowną nazwę

Np
invalid_choices = a + b + c

[Oo-oO]

@Marek1991: Dla kompletności (choć też jestem zdania że x+y+z jest spoko)
if not any(user_choice in L for L in (options, choices, commands))

[Marek1991]

@Lunatik: ale to jest dodatkowa linia kodu, która nic nie daje

[L.....k]

@Marek1991: a jest jakiś konkurs na używanie jak najmniejszej liczby linii kodu czy o czymś nie wiem?

[Marek1991]

@Lunatik: kod musi być pareto optymalny

[L.....k]

@Marek1991: kod musi być przede wszystkim czytelny i zrozumiały