Во-первых, не надо обходить список, создавая диапазон по длине списка. Можно просто:
for s in socket_list:
...
Во-вторых, как я уже писал в прошлом вопросе, нельзя изменять список, по которому идёт итерация. Можно вести отдельный список сбойных сокетов и удалить их после цикла. Наконец, в вашем случае лучше использовать множества, а не списки. Подытоживая всё написанное:
closed_sockets = set()
for s in sockets:
try:
# Операции с сокетом
except socket.error as e:
# Какие-либо дополнительные проверки типа ошибки
closed_sockets.add(s)
sockets -= closed_sockets # При условии, что sockets - множество
UPDATE: Судя по ошибке RuntimeError: Set changed size during iteration
, когда изменений множества в самом цикле не происходит, его изменяет другой поток. Поэтому для итерации стоит создать копию, а изменение глобальной переменной блокировать:
closed_sockets = set()
for s in set(sockets):
try:
# Операции с сокетом
except socket.error as e:
# Какие-либо дополнительные проверки типа ошибки
closed_sockets.add(s)
with threading.Lock():
sockets -= closed_sockets
Не забудьте добавить такую же блокировку в том месте, где в множество добавляются элементы.
arr = [x for x in arr if нужное-условие-по-x]
– andreymal 27 июн '17 в 10:40