AttributeError: 'int' object has no attribute 'price' как исправить?

Question

[Kryptonit]

Возникла проблема. Я так понял не видится контекст вызова. Мне нужно чтобы код сортировал по возрастанию массив объектов и хочу именно с контекстом self реализовать. суть программы - даётся файл из строк нужно отсортировать по возрастанию, я реализую через объект, каждая строка разбивается на 3 части и добавляется в массив объектов, затем сортируется по атрибуту price.

import operator
file = open('26.txt', 'r')
A=[0, 0, '']*401
lines = file.readlines()
class elem():
    def __init__(self, a, b, c):
        self.volume = b
        self.price = a
        self.letter = c
    
for i in range(1, len(lines)):
    a, b, c = lines[i].split(' ')
    A[i] = elem(a, b, c)
print(A[2].price)
A.sort(key = operator.attrgetter('price'))
print(A[2].price)

Answer 1

[Vindicar]

Команда
A=[0, 0, '']*401
даст список вида [0, 0, '', 0, 0, '', .........]
Т.е. число элементов списке будет 401 * 3.
Зачем вообще выделять список заранее?
Сделай его пустым A = [] и делай A.append(....) для каждой прочитанной строки.

Comments

[Kryptonit]

спасибо, заработало, просто раньше с С++ работал там чуть чуть по-другому массивы работают

[Vindicar]

Питоновский list - это ближе к C++ vector, чем к сишному массиву. Он сам о памяти заботится и перевыделяет, когда надо.

[Kryptonit]

Vindicar, тут ещё вопрос такой: я пытаюсь сравнить атрибут объекта со строкой, но каждый раз он не хочет быть равен true, т.е. я вывожу A[i].letter # на выводе 'A' типа я это стираю и пишу A[i].letter=='A' и выводит false, ну точнее ничего не выводит, что прошу у него, типы данных везде такие же какие я указал выше, вот как тогда сравнить атрибут с символом одного типа?

[o5a]

Kryptonit, так и сравнивать. Если что-то не то получилось, приведите полный код, похоже опять что-то не то написали. Тянуть багаж C++ в Python хорошим не заканчивается как правило, получаются костыли.

[Vindicar]

Kryptonit, погоди, погоди...
A[i].letter=='A' даст на выходе булево выражение. Само по себе это ничего не выведет, разумеется.
Если сделать print(A[i].letter=='A'), получишь в стандартном выводе "True" или "False".
А в твоём случае - не с кодировкой ли проблемы? Русская А и латинская A не равны.

[Kryptonit]

Vindicar, ни так ни так не выводит

import operator #библиотека для сортировки списка объектов по атрибуту
file = open('26.txt', 'r') 
count = 0
A=[]
money = 4000000
lines = file.readlines()
class elem(): # класс объекта создаваемого из элементов строки для дальнейшей сортировки 
    def __init__(self, a, b, c): # метод с контекстом класса для обработки разбиваемой строки 
        self.volume = b
        self.price = a
        self.letter = c
    
for i in range(1, len(lines)):
    a, b, c = lines[i].split(' ')
    A.append(elem(a,b,c))# забиваем массив объектами класса elem() попутна заполняя каждый объект
A.sort(key = operator.attrgetter('price')) # сортируем массив по цене 
for i in range(0, 10):
    if(A[i].letter=='А'):
        print('hey')
        money-=(A[i].price*A[i].volume)
print(money)

[Kryptonit]

Kryptonit, мне дальше надо со значениями работать каждого объекта, а условие не проходит нижнее ни при какой раскладке

[Vindicar]

Kryptonit, а почему
for i in range(0, 10):
у вас список сортируется по возрастанию цены, а потом вы в 10 самых дешевых товарах ищете товары, у которых letter - буква А.
1. Такие точно есть?
2. letter у вас точно однобуквенные? Или вам нужно чтобы начинались с А?
3. lines[i].split(' ') - а строки в lines[i] содержат переводы строки или нет?

[Kryptonit]

Да, я выводил точно есть в первых 10 их 4. Вроде да, строки в тхт имеют вид

322 17 A
103 38 A
460 19 B
456 69 B

(скопировал без изменений), перевод строки содержат потому что всё в массив забивается как нужно.
При чём самое интересное, что при выводе A[i].letter выводится одна буква без пробелов. а в условие не проходит, магия какая то

[Kryptonit]

Vindicar, где то подвох

[Vindicar]

Kryptonit, выведи repr(A[i].letter)

[Kryptonit]

Vindicar,

'B\n'
'B\n'
'A\n'
'A\n'
'A\n'
'B\n'
'A\n'
'A\n'
'B\n'
'B\n'

[Vindicar]

Ну а я что говорил? Переводы строк
'A\n' != 'A'
после
a, b, c = lines[i].split(' ')
сделай
c = c.rstrip()
чтобы обрезать все пробельные символы с конца.
Или, если перевод строки нужен, то сравнивай с 'A\n'

[Vindicar]

А вообще, посмотри в сторону модуля csv, может, получится его прикрутить под твою задачу?

[Kryptonit]

Vindicar, Что-то я лоханулся на простом) в С работал с этим а тут на тебе. Спасибо

Answer 2

[Andy_U]

У вас длина списка А превышает число прочитанных строк. Вот в хвосте и остаются не перезаписанные нули и пустые строки.

Comments

[Kryptonit]

я длину А регулировал ничего не изменилось

[Andy_U]

Kryptonit, Сколько строк у вас в файле?

[Kryptonit]

Andy_U, 400

[Andy_U]

А длина A какая? 3*400 небось?

[Kryptonit]

да не должно быть я на 5 пробовал маску массива умножать

[Kryptonit]

то есть если по этой логике 5*3 = 15, а там в любом случае перезапись идёт до крайнего элемента

[Andy_U]

Kryptonit, Чем вопросы задавать, не проще ли перед входом в цикл длины lines и А вывести на консоль?