On 2016-04-01 21:27, Fillmore wrote:

notorious pass by reference vs pass by value biting me in the backside
here. Proceeding in order.

I need to scan a list of strings. If one of the elements matches the
beginning of a search keyword, that element needs to snap to the front
of the list.
I achieved that this way:


     for i in range(len(mylist)):
          if(mylist[i].startswith(key)):
              mylist = [mylist[i]] + mylist[:i] + mylist[i+1:]

Since I need this code in multiple places, I placed it inside a function

def bringOrderStringToFront(mylist, key):

      for i in range(len(mylist)):
          if(mylist[i].startswith(key)):
              mylist = [mylist[i]] + mylist[:i] + mylist[i+1:]

and called it this way:

   if orderstring:
       bringOrderStringToFront(Tokens, orderstring)

right?
Nope, wrong! contrary to what I thought I had understood about how
parameters are passed in Python, the function is acting on a copy(!) and
my original list is unchanged.

I fixed it this way:

def bringOrderStringToFront(mylist, key):

      for i in range(len(mylist)):
          if(mylist[i].startswith(key)):
              mylist = [mylist[i]] + mylist[:i] + mylist[i+1:]
      return(mylist)

and:

if orderstring:
     Tokens = bringOrderStringToFront(Tokens, orderstring)

but I'm left with a sour taste of not understanding what I was doing
wrong. Can anyone elaborate? what's the pythonista way to do it right?

Python always passes a reference to the object, so the name "mylist" in the function is a local name that refers to the list that you passed.

When you say "mylist = something", you're just binding that local name to another object (i.e., it'll now refer to that object instead).

What you want to do it mutate the list itself. You can do that by replacing its elements with the new list you've created:

    mylist[:] = [mylist[i]] + mylist[:i] + mylist[i+1:]

--
https://mail.python.org/mailman/listinfo/python-list

Reply via email to