Path: csiph.com!fu-berlin.de!uni-berlin.de!not-for-mail
From: Gary Herron <gherron@digipen.edu>
Newsgroups: comp.lang.python
Subject: Re: list reversal error
Date: Thu, 3 Mar 2016 15:15:19 -0800
Lines: 46
Message-ID: <mailman.172.1457047506.20602.python-list@python.org>
References: <8b3d06eb-0027-4396-bdf8-fee0cc9ff771@googlegroups.com>
Mime-Version: 1.0
Content-Type: text/plain; charset=windows-1252; format=flowed
Content-Transfer-Encoding: 7bit
User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:38.0) Gecko/20100101 Thunderbird/38.5.1
In-Reply-To: <8b3d06eb-0027-4396-bdf8-fee0cc9ff771@googlegroups.com>
Precedence: list
Xref: csiph.com comp.lang.python:104003

On 03/03/2016 02:51 PM, vlyamtsev@gmail.com wrote:
> i have list of strings "data" and i am trying to build reverse list data1
> data1 = []
> for i in range(len(data)):
>     j = len(data) - i
>     data1.append(data[j])
>
> but i have the following error:
> data1.append(data[j])
> IndexError: list index out of range
>   
> am i doing it wrong?
> Thanks

Look at the values (say with a print) you get from your line
     j = len(data) - i
You'll find that that produces (with a list of 4 elements for example) 
4,3,2,1 when in fact you want 3,2,1,0. Soo you really want
     j = len(data) - i -1


Better yet, use more of Python with
     data1 = list(reversed(data))

Or don't even make a new list, just reverse the original list in place
 >>> L=[1,2,3]
 >>> L.reverse()
 >>> L
[3, 2, 1]

Or even better, if you simply want to iterate through the original list, 
but in reverse order:
     for datum in reversed(data):
         ... whatever with datum ...
which wastes no time actually reversing the list, but simply loops 
through them back to front.


Gary Herron

-- 
Dr. Gary Herron
Department of Computer Science
DigiPen Institute of Technology
(425) 895-4418