PYTHON - Kopiowanie listy

Question

LNU PYTHON, 

mam problem z " if x[i] < 0 " oraz " if x[i] < 10 " nie mam pomysłu jak wyciągnąć wartość z listy.

 

do funkcji convert() przez parametr przekazano listę ocen testowanego produktu. W związku z wprowadzeniem dodatkowego systemu oceniania należy wartości w liście przekształcić w taki sposób, że wartości ujemne należy zamienić na wartość 0, zaś wartości większe od 10 na wartość 10. Funkcja powinna zwracać jedną listę, będącą połączeniem dwóch list: przekazanej przez parametr oraz zmodyfikowanej. 

Dodatkowy warunek: poradź sobie bez metody append(). 

Przykład:

Wejście: [ 5, -2, 2, 10, 12 ]
Wyjście: [ 5, -2, 2, 10, 12, 5, 0, 2, 10, 10 ]

Kod: 

def convert( x ):
    
    copy_x = list(x)
    
    for i in copy_x:
        
        if copy_x[i] < 0:
            copy_x[i] = 0
        elif copy_x[i] > 10: 
            copy_x = 10
            
    return x + copy_x

 

Answer 1

Błąd masz tutaj:

    for i in copy_x:
(...)
        if copy_x[i]

Pętla `for` iteruje się po elementach, nie indeksach. Jeśli chcesz używać indeksy (np do modyfikowania listy), potrzebujesz indeksy, które możesz uzyskać iterując się po range() lub enumerate().

(Tylko dla formalności zaznaczę że można to też rozwiązać bez indeksów i modyfikowania - budując nową listę od zera)

Comments

Dziękuje za podpowiedź, 

udało mi się rozwiązać to zadanie 

def convert( x ):
    
    copy_x = list(x)
    index = 0
    
    for i in copy_x:
        
        if copy_x[index] < 0:
            copy_x[index] = 0
        elif copy_x[index] > 10: 
            copy_x[index] = 10
            
        index += 1
        
    return x + copy_x

Jeżeli masz jeszcze jakieś uwagi do mojego kody napisz, z chęciom przeczytam :)

 

---

Tak jak mówiłem, zarówno range() jak i enumerate() by to zapisały jaśniej, bo nie trzeba ręcznie inkrementować licznika:

for index, e in enumerate(copy_x):
    if e < 0:
        copy_x[index] = 0

# lub

for index in range(len(copy_x)):
    if copy_x[index] < 0:
        copy_x[index] = 0

No i można to też rozwiązać w ogóle bez myślenia o indeksach i modyfikowaniu listy, bo można drugą listę zbudować od zera wołając na niej push() w pętli. (albo użyć list comprehension, ale zgaduję że jeszcze o nich nie słyszałeś)

---

Dziękuje, rozwiązanie z range() bardziej do mnie przemawia niż enumerate(). Dzięki temu mój kod będzie o wiele czystszy i bardziej intuicyjny. :)

Answer 2

Proponuję użycie map()

 

przykład  [ on-line ]

def convert( x ):
    return x + list(map((lambda x: 0 if x<0 else 10 if x>10 else x), x))

x = [ 5, -2, 2, 10, 12 ]

print('Wejście:', x)
print('Wyjście:', convert(x))

 

P.S. How to Use Python Lambda Functions

Comments

Dziękuje za twoją propozycję :)