Code not working as expected.

[an_droid]

The code below works for the single line of code (#), but not when broken up into 3 different assignments (##). Where am I going wrong?

# ActivePython 2.7.10.12 on Windows 7
# want to create a list of lists 
# [["one","two","three"],["four","five","six"]]
# out of the list all_entries
# works for the single line of code (#), but
# alternate code of individual indices (##) gives 
# [["four","five","six"],["four","five","six"]]


all_entries = ["one","two","three","four","five","six"]

bookmark_entry = [0] * 3

bookmark_list = []

limit = len(all_entries)

for i in range(0,limit,3):
  
  print "i = :" + str(i)    ### just for debugging purposes

  #bookmark_entry = [all_entries[i],all_entries[i + 1],all_entries[i + 2]]   # as single line

  bookmark_entry[0] = all_entries[i]         ##
  print bookmark_entry[0]

  bookmark_entry[1] = all_entries[i + 1]     ##
  print bookmark_entry[1]

  bookmark_entry[2] = all_entries[i + 2]     ##
  print bookmark_entry[2]

  bookmark_list.append(bookmark_entry)

  print "bookmark_entry" + str(bookmark_entry)

print "bookmark list " + str(bookmark_list)

[nilamo]

In the first, you're creating a new list with three elements. In the second, you're setting the first, then second, then third element of an undefined variable.

Instead of bookmark_entry[0] = something, try bookmark_entry.append(something)... and also define the variable somewhere first.

[an_droid]

I thought

bookmark_entry = [0] * 3

defined and declared bookmark _entry to be a list with 3 elements?

[nilamo]

I didn't see that line. Ok, so what's the error you're getting?

[an_droid]

I think I worked it out. I've certainly declared one array, but then I'm appending the same actual list object twice! Thanks for your help anyway.