New to coding, Using the zip() function to create Diret and getting weird results

[Shagamatula]

Hey guys im new to coding doing a mature student diploma, im doing my 3rd project for the course and im getting results that don’t make sense. Please can someone tell me why the directory on the second pass is different? If you put this code in Python Tutor you will see that dirct_1 on the second pass starts in 10's when it should start at 1.

What am I trying to do?
Generate a directory containing for eg.. a:1 key and 1 as value, next would be b:2 with 2 as value and so on. So basically it’s a battleship grid reference.

Issue: On the second iteration the value starts at 10 and next run at 20.
Anny and all help welcome...

letters = []


dict_1 = {}
def go():
    for i in range(97, 107):
        letters.append(chr(i))
        numbers = (list(range(1, 3)))
        print(numbers)
        final = []
        i = 0
    for i in (numbers):
        for content in (letters):
            num = content + ":" + str(i)
            alpha_num = num.strip()
            final.append(alpha_num)
            
    values = (list(range(1, 201)))
    dict_1 = dict(zip(final, values))
    
    
    print(f"Final_zip =  {final}")
    print(f"Values zip =  {values}")
    print(f"Dict_1 =  {dict_1}")
go()
go()

[bowlofred]

Use the python bbcode (the python button in the message box) to enclose your code.

Unless there's a reason, please just use the letters directly rather than ascii codes.

for letter in "ABCDEFGH":
    ...

Instead of

for i in range(97, 107):
    letter = chr(i)

But the actual problem you're running into is because letters is a global variable and the function is just appending to it. So each time, the list gets longer.

And since the letters repeat, it will add each one to the dict, but only the last one stays.

As letters isn't used outside the function, it shouldn't be there. Put it inside the function and that behavior will disappear.

[menator01]

Not sure I understand what it is your wanting but, Here is an example of a grid. Can be modified to store x,y in a dict.

from string import ascii_uppercase

letters = [letter for letter in ascii_uppercase[:9]]
numbers = [i+1 for i in range(len(letters))]

print(f"   {'  '.join(letters)}")

for n in numbers:
    dot = f' . '*len(letters)
    print(n, dot, sep=' ', end='\n')

output

Output:
   A  B  C  D  E  F  G  H  I
1  .  .  .  .  .  .  .  .  .
2  .  .  .  .  .  .  .  .  .
3  .  .  .  .  .  .  .  .  .
4  .  .  .  .  .  .  .  .  .
5  .  .  .  .  .  .  .  .  .
6  .  .  .  .  .  .  .  .  .
7  .  .  .  .  .  .  .  .  .
8  .  .  .  .  .  .  .  .  .
9  .  .  .  .  .  .  .  .  .

[deanhystad]

Neither letters or dict_1 should be defined outside the function go()

This:

    for i in range(97, 107):
        letters.append(chr(i))
        numbers = (list(range(1, 3)))
        print(numbers)
        final = []
        i = 0
    for i in (numbers):
        for content in (letters):
            num = content + ":" + str(i)
            alpha_num = num.strip()
            final.append(alpha_num)

Can be replaced by:

final = []
for number in (1, 2):  # More consice than range(1, 3)
    for letter in "abcdefghij"  # Easier to understand than range(97, 107)
        final.append(f"{letter}:{number}")

Or even a 1 line comprehension.

[Shagamatula]

Neither letters or dict_1 should be defined outside the function go()

This:

    for i in range(97, 107):
        letters.append(chr(i))
        numbers = (list(range(1, 3)))
        print(numbers)
        final = []
        i = 0
    for i in (numbers):
        for content in (letters):
            num = content + ":" + str(i)
            alpha_num = num.strip()
            final.append(alpha_num)

Can be replaced by:

final = []
for number in (1, 2):  # More consice than range(1, 3)
    for letter in "abcdefghij"  # Easier to understand than range(97, 107)
        final.append(f"{letter}:{number}")

Or even a 1 line comprehension.

[Shagamatula]

Wow guys, thanks so much for all the advice ive been going mad trying to figure this out. But I see now from all you advice that its my logic, im learning and quicker now because of you guys, Thanks a million, heading out for day but when I get back ill have another crack at it. Thanks again everyone.

[Shagamatula]

Hey guys im new to coding doing a mature student diploma, im doing my 3rd project for the course and im getting results that don’t make sense. Please can someone tell me why the directory on the second pass is different? If you put this code in Python Tutor you will see that dirct_1 on the second pass starts in 10's when it should start at 1.

What am I trying to do?
Generate a directory containing for eg.. a:1 key and 1 as value, next would be b:2 with 2 as value and so on. So basically it’s a battleship grid reference.

Issue: On the second iteration the value starts at 10 and next run at 20.
Anny and all help welcome...

letters = []


dict_1 = {}
def go():
    for i in range(97, 107):
        letters.append(chr(i))
        numbers = (list(range(1, 3)))
        print(numbers)
        final = []
        i = 0
    for i in (numbers):
        for content in (letters):
            num = content + ":" + str(i)
            alpha_num = num.strip()
            final.append(alpha_num)
            
    values = (list(range(1, 201)))
    dict_1 = dict(zip(final, values))
    
    
    print(f"Final_zip =  {final}")
    print(f"Values zip =  {values}")
    print(f"Dict_1 =  {dict_1}")
go()
go()