Ошибка при использовании метода setPixel класса QImage, Python Опции

[zuze]

Какие левые результаты?

Ой сам виноват, ступил, тут только одно значение неправильное было, а именно i доходило до 1, а не до 0.
Переделал

for i in range(491, -1, -1):

[zuze]

Вот тут появилсась такая загвостка.

# Глобальная переменная
bf = []

# В функции 1
for j in range(0, 720, 1):
     for i in range(491, -1, -1):
          bf = struct.unpack("B", fp.read(1))

# В функции 2
for j in range(0, 720, 1):
     for i in range(491, -1, -1):
          im1.setPixel(j, i, QtGui.QColor(bf[0], bf[0], bf[0], 255).rgba())
          print bf[0]

В функции 1 всё впорядке, а вот в функции 2 bf[0] выводит постоянно 21 и это совершенно не верно.
Почему так происходит?

[lanz]

bf обновляется только в первой функции.
Во второй функции используется последнее считанное в bf значение.

Зачем делать два одинаковых цикла в разных местах?

[zuze]

Зачем делать два одинаковых цикла в разных местах?

У меня проблемы с торможениями программы и я ищу узкие места. Хотел bf использовать глобально в другой функции, но видемо это не возможно. Наверно в таких случаях надо список использовать?

Сообщение отредактировал zuze - 26.3.2013, 10:38

[lanz]

Это возможно, только нужно добавлять в массив.

bf = []

def f1():
    global bf
    for ...
        bf.append ([])
        for ...            
            bf[i].append(struct.unpack("B", fp.read(1))[0])
def f2():
    global bf
    for ...
        for ...            
            im1.setPixel (j, i, QtGui.QColor(bf[i][j], bf[i][j], bf[i][j] ...

print выполняется очень долго. Его надо убрать из внутреннего цикла при замере производительности.

Сообщение отредактировал lanz - 26.3.2013, 10:52

[zuze]

print выполняется очень долго. Его надо убрать из внутреннего цикла при замере производительности.

Это сомо собой, это я для проверки того, что там ерунда выходит, а так я конечно уберу.

Я сделал так:

for j in range(0, 720, 1):
     bf.append([])
     for i in range(491, -1, -1):
          bf[i].append(struct.unpack("B", fp.read(1))[0])

Выскочила ошибка: IndexError: list index out of range

[lanz]

bf[i].append(struct.unpack("B", fp.read(1))[0])

Надо заменить i на j, поскольку мы добавляем в j-ый список.

Кстати тут надо согласовать индексы потому что список заполняется с начала, а используется с конца.

[zuze]

Кстати тут надо согласовать индексы потому что список заполняется с начала, а используется с конца.

Я думал это надо сделать вот таким образом

bf = []

def f1():
    global bf
    for j in range(0, 720, 1):
         bf.append([])
         for i in range(491, 0, -1):
              bf[j].append(struct.unpack("B", fp.read(1))[0])
def f2():
    global bf
    for j in range(0, 720, 1):
         for i in range(491, -1, -1):
              im1.setPixel(j, i, QtGui.QColor(bf[-i][j], bf[-i][j], bf[-i][j], 255).rgba())

Я в функции 2 при задании i в bf указал минус, то есть bf[-i][j], вроде это тогда с конца спаиска. Но это не помогло.
Ошибка: IndexError: list index out of range

[lanz]

Ну да, заполняем 491 элемент:

for i in range(491, 0, -1):

А считываем 492:

for i in range(491, -1, -1):

Кроме того последний элемент будет -1 а не -0, а первый -492, поэтому в установке пикселей надо:

for i in range(492, 0, -1):

[zuze]

Я сделал так:

bf = []

def f1():
    global bf
    for j in range(0, 720, 1):
         bf.append([])
         for i in range(491, 0, -1):
              bf[j].append(struct.unpack("B", fp.read(1))[0])
def f2():
    global bf
    for j in range(0, 720, 1):
         for i in range(491, 0, -1):
              im1.setPixel(j, i, QtGui.QColor(bf[-i][j], bf[-i][j], bf[-i][j], 255).rgba())

Ошибка: IndexError: list index out of range

Сообщение отредактировал zuze - 26.3.2013, 13:23