Что такое более аккуратный, более pythonic способ сделать следующее перечисление?

Question

for row, instrument in enumerate(instruments): 
    for col, value in enumerate(instrument): 
     self.table.SetValue(row, col, value) 

Answer 1

Вы упомянули в комментариях, что этот цикл является частью асинхронной функции (с точки зрения скрученной структуры). В этом случае вы не хотите блокировать в течение длительного времени:

from twisted.internet import task 

for i, row in enumerate(instruments): 
    task.coiterate(self.table.SetValue(i, j, v) for j, v in enumerate(row)) 

Таким образом, все строки назначаются параллельно.

Примечание:

• Берегитесь позднее связывание для i и row. В этом случае используйте (lambda i=i, row=row: ...)().
• task.coiterate() использует глобальный объект, поэтому одновременно может быть несколько обновлений таблиц (это может быть не то, что вы хотите).

---

Вот @SilentGhost' answer (удалена):

self.table = instruments

Потому что это то, что вы, кажется, делать.

И замечание @ [Бен Хьюз] Я имею в виду:

Мне нужно позвонить SetValue в явной форме (ее на PyGridTableBase) для каждого значения - , как этот код вызывается через витой отложенного метода - мой мозг не так много хорошо перекручивание/перечисления в аккуратном пути ..... - Бен Хьюз

Answer 2

Одним из вариантов является использование списка понимание:

[self.table.setValue(row, col, value) 
    for row, instrument in enumerate(instruments) 
     for col, value in enumerate(instrument)] 

Не уверен, если это какая-то аккуратнее или более вещий ... Но это еще один способ выразить петлю.

Можно утверждать, что понимание списка более четкое, так как элемент «действие» (setValue) помещается первым/вверху. Вместо того, чтобы быть похороненным внутри петли.

EDIT:

Еще один способ, с помощью вспомогательной функции и выражение генератора:

def loop(iterator): 
    for item in iterator: pass 

loop(self.table.setValue(row, col, value) 
    for row, instrument in enumerate(instruments) 
     for col, value in enumerate(instrument)) 

Вспомогательная функция может быть встроенной в любой(), если вы знаете наверняка, что тело цикла никогда не будет оценивать True в булевом контексте (any() остановит итерацию на первом True).

Answer 3

То, что вы вызываете row, не является строкой, это индекс строки. instrument - это строка. Кроме того:

Если единственный инструмент, который у вас есть это SetValue(row_index, column_index, value) метод и этот метод делает больше, чем помочь повторить структуру instruments кропотливо и instruments сбор не является, как ваш код описывает, то нет лучше путь. Это уже намного лучше, чем

#WARNING: BAD PRACTICE! DON'T USE THIS CODE! 
for row_index in xrange(len(instruments)): 
    instrument = instruments[row_index] 
    for column_index in xrange(len(instrument)): 
     self.table.SetValue(row_index, column_index, instrument[column_index]) 

или любые извращения.

В противном случае вы можете быть заинтересованы в import copy; self.table = copy.copy(instruments) или даже (как SilentGhost предложил) self.table = instruments

Answer 4

Принципиальный вопрос не касается этого цикла.

Фундаментальные вопросы таковы:

1) Где это instruments структура пришли, и почему вам нужно реорганизовать его?

2) Что это такое? self.table Структура, по которой вы звоните SetValue?

3) Что вы собираетесь делать с этой структурой self.table?

Пока вы не ответите на эти вопросы, ваш образец кода Python не имеет контекста, в котором его можно оценить. Это всего лишь код.