import string

letters = list(string.ascii_lowercase)

dic = {}
lista = []
strin=()
final=[]
for id, i in enumerate(letters):
    dic[id]=i

def alphabet_position(text):
    for i in text:
           strin=' '.join([str(k+1)+v for k,v in dic.items() if v == i.lower()])

(Jan-30-2021, 02:24 AM)bowlofred Wrote: In the first place, why are you building a dictionary that maps from position to letter, but then using it to look from letter to position? My first thought is that you should build the dictionary in that direction and use it naturally. What this program does is use the dictionary like a list. It would be simpler to just loop through the original string.ascii_lowercase than loop through the dictionary.

Next, while alphabet_position loops over the characters in the input, it overwrites strin each time through the loop. So only the final character is kept in that variable.

Finally, the function has no return call. So if you are calling it, you will always get back None

def alphabet_position(text):
    for i in text:
        lista.append([k+1 for k,v in dic.items() if v == i.lower()])
        strin=' '.join([str(k+1)+v for k,v in dic.items() if v == i.lower()])

(Jan-30-2021, 02:29 AM)karabakh Wrote: When I add the ids to some list, the list is not being overwritten.

full_string = ""
for s in list:
    full_string += s  # Keeps the existing portion each time through the loop.

(Jan-30-2021, 03:52 AM)bowlofred Wrote:

(Jan-30-2021, 02:29 AM)karabakh Wrote: When I add the ids to some list, the list is not being overwritten.

That's because you're using a method of the list that modifies it, like list.append(). What you're not doing is reassigning the list each time through like list = [new_item].

With strings, you'd instead assign the existing part and the new part to a variable. Something like:

full_string = ""
for s in list:
    full_string += s  # Keeps the existing portion each time through the loop.