[DunkLord]

#python
#naukaprogramowania

Dlaczego nie mogę tego zrobić w ten sposób?
Funkcja ma zwrócić ciąg wejściowy, z tym, że każdy element który jest taki sam jak 1 ma być zamieniony na $ (oprócz 1 elementu).

Po uruchomieniu dostaję błąd: TypeError: string indices must be integers.

Kod i cały błąd tutaj : https://pastebin.com/bmHjRMSU

help (╯︵╰,)

[tenji]

@DunkLord: bo i to jest znak a nie index. Musisz uzyc for index, char in enumerate(ciag)

[pshemeq]

@DunkLord:
for i in ciag:
nie iteruje Ci po indeksach, a po literach. Jak chcesz indeksy to zrób

for indeks, wartosc in enumerate(ciag):

i potem sprawdzaj co tam trzeba

[croppz]

@DunkLord: for i in text iteruje po znakach (czyli typ string). Możesz użyć na przykład

for i, char in enumerate(text):

[DunkLord]

@tenji: @pshemeq: @croppz: potem juz w srodku jeśli chcę porównać czy dany znak jest równy z 1 to muszę to zrobić w sposób ciag[???]==ciag[0]?
co w miejscu ??? dać?

[croppz]

@DunkLord: i

[pshemeq]

wartosc == ciag[0]

[DunkLord]

@pshemeq:
def zamiana(ciag):
for i,char in enumerate(ciag):
if(i!=0):
if(char==ciag[0]):
ciag[i]=="$"
return ciag

print(zamiana("roweror"))

to mi zwraca "roweror", a powinno być "rowe$o$".

co jeszcze źle robię?

dzięki za pomoc :D

[croppz]

@DunkLord: = to przypisanie, == to porównanie (zwraca true lub false). I z tego co pamiętam to znaków w stringu nie można było podmieniać po indeksach (text[i] = '$' wywali błąd). Pokombinuj z funkcją str.replace albo rozbij to na listę, pozmieniaj co chcesz i potem złącz.

[DunkLord]

@croppz: o kurła/ jedno = sie za dużo. to już mój błąd, niedopatrzenie. dzięki. popróbuję jeszcze.
rozwiaązałem to w ten sposób, jak to teraz jeszcze złączyć tak by wyświetliło cały ciąg, a nei taką teblice?

def zamiana(ciag):
wynik=[]
for i,char in enumerate(ciag):
if(i==0):
wynik.append(ciag[i])
if(i!=0):
if(char==ciag[0]):
wynik.append("$")
else:
wynik.append(ciag[i])
return wynik

print(zamiana("roweror"))

zwraca ['r', 'o', 'w', 'e', '$', 'o', '$']

[DunkLord]

@croppz: o dobra, znalazłem sam rozwiązanie. return ''.join(wynik)

[Budek24]

@DunkLord: a dlaczego nie użyć funkcji replace i operatora slice?

s[0] + s[1:].replace(s[0],'$')

[DunkLord]

a dlaczego nie użyć funkcji replace i operatora slice?

@Budek24: bo tego nie znałem :D

[Budek24]

@DunkLord: No to teraz znasz :D

[DunkLord]

@Budek24: to teraz będę wiedział :D