Ctrl →

johniek_comp · Июнь 1, 2012 18:58:19

Не понимаю в чем ошибка:

for i,c in enumerate(array):
        if c[i] > c[i+1]:

TypeError: ‘int’ object has no attribute ‘__getitem__’

Отредактировано johniek_comp (Июнь 1, 2012 19:18:54)

Cryo · Июнь 1, 2012 19:34:12

По-видимому, в array содержатся объекты, имеющие тип ‘int’ - числа
__getitem__ - специальный метод, позволяющий получить элемент по индексу или ключу

>>> s = 'python'
>>> s.__getitem__(0)
'p'
>>> s[0]
'p'
>>> n = 1025
>>> n.__getitem__(3)
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
AttributeError: 'int' object has no attribute '__getitem__'

Нужно понимать, что твоя переменная с - это на каждой итерации каждый последующий элемент в массиве array, но не сам array:

>>> for i, c in enumerate([10, 20, 30, 40, 50]):
...     i, type(c)
... 
(0, <type 'int'>)
(1, <type 'int'>)
(2, <type 'int'>)
(3, <type 'int'>)
(4, <type 'int'>)

Можно было написать так:

>>> array = [1,5,3,8,9,0,4,3]
>>> for i in range(len(array)):
...     try:
...         array[i] > array[i+1]
...     except IndexError:
...         pass

Отредактировано Cryo (Июнь 1, 2012 20:25:16)

reclosedev · Июнь 1, 2012 21:29:09

Cryo
Можно было написать так:

Проще без try:

for i in range(len(array) - 1):
    ...

Андрей Светлов · Июнь 1, 2012 22:59:46

enumerate

asilyator · Июнь 1, 2012 23:33:02

johniek_comp
Не понимаю в чем ошибка:
for i,c in enumerate(array):
        if c[i] > c[i+1]:
TypeError: ‘int’ object has no attribute ‘__getitem__’

В c у тебя будет не array, а его элемент. Надо так:

for i,c in enumerate(array):
        if array[i] > array[i+1]:

Или лучше

prev = array[0]
for i in array:
    if prev > i:
        #...
    prev = i

Вообще я не очень люблю обращение по индексу там, где без него можно обойтись, например, с foreach.

Отредактировано asilyator (Июнь 1, 2012 23:34:32)

fata1ex · Июнь 2, 2012 00:04:47

asilyator, зачем повторять то, что уже было сказано выше?

Cryo
Нужно понимать, что твоя переменная с - это на каждой итерации каждый последующий элемент в массиве array, но не сам array

asilyator
В c у тебя будет не array, а его элемент

Зачем предлагать неработающий код?

>>> array = [1,2,3]
>>> for i,c in enumerate(array):
...         if array[i] > array[i+1]:
...             pass
...             
Traceback (most recent call last):
  File "<input>", line 2, in <module>
IndexError: list index out of range

А глядя на это, сразу хочется посоветовать почитать базовую литературу по программированию. Так не пишут.

prev = array[0]
for i in array:
    if prev > i:
        #...
    prev = i

asilyator · Июнь 2, 2012 02:40:38

fata1ex
asilyator, зачем повторять то, что уже было сказано выше?

Написал быстрее, чем прочитал тред, что дальше?

fata1ex
А глядя на это, сразу хочется посоветовать почитать базовую литературу по программированию. Так не пишут.

Эт почему?

py.user.next · Июнь 2, 2012 10:01:29

asilyator
Эт почему?

элемент сравнивается с самим собой

наверху написан вариант с range()
ещё один добавлю

>>> array = [1, 2, 3, 4, 5, 1, 2, 3, 4, 5]
>>> for i in range(1, len(array)):
...     print(array[i - 1] > array[i])
... 
False
False
False
False
True
False
False
False
False
>>>

FishHook · Июнь 2, 2012 11:24:49

ИМХО самое православное решение будет таким

a = [1, 2, 3, 4, 5, 1, 2, 3, 4, 5,7]
for p, n in  zip(a[:-1], a[1:]):
...     print p>n
False
False
False
False
True
False
False
False
False
False

fata1ex · Июнь 2, 2012 13:58:56

asilyator, плюс к замечанию py.user.next то, что в вашем варианте нам приходится держать в голове больше деталей, чем при варианте с доступом по индексу. И это в коде, где всего несколько строк. Подобные вещи довольно быстро захламляют программу.

FishHook, решение красивое, но слишком сложное :)

Отредактировано fata1ex (Июнь 2, 2012 13:59:07)

Python-сообщество

Уведомления

#1 Июнь 1, 2012 18:58:19

цикл for

#2 Июнь 1, 2012 19:34:12

цикл for

#3 Июнь 1, 2012 21:29:09

цикл for

#4 Июнь 1, 2012 22:59:46

цикл for

#5 Июнь 1, 2012 23:33:02

цикл for

#6 Июнь 2, 2012 00:04:47

цикл for

#7 Июнь 2, 2012 02:40:38

цикл for

#8 Июнь 2, 2012 10:01:29

цикл for

#9 Июнь 2, 2012 11:24:49

цикл for

#10 Июнь 2, 2012 13:58:56

цикл for

Board footer