Python перебирает список

1

Я пытаюсь перебрать каждый элемент в списке. Но итератор не проходит через все объекты. Вот мой код функции (dirs и root происходит от os.walk)

def remove_hidden_dirs(dirs,root):
    """
    Function remove hidden directories . .. in unix
    """
    logging.debug("\n\n remove hidden" )
    logging.debug('Start removing hidden dirs for %s',root)
    logging.debug("Length of dirs %s",len(dirs))
    logging.debug('File list before removing')
    dirs.sort()
    for d in dirs:
        logging.debug(d)
    i=0
    for d in dirs:
        i=i+1
        logging.debug("Iterating over %s",d)
        if d[0:1] == '.' or d[0:2] =='..':
            dirs.remove(d)
            logging.debug("Dir %s is being removed because of being hidden dir",d)
        else:
            logging.debug("Dir %s is NOT being removed because of being hidden dir",d)
    logging.debug("Iterate on %s", i)
    logging.debug('File list after removing')
    for d in dirs:
        logging.debug(d)
    logging.debug('end remove hidden files\n\n')
    return dirs

И вот часть моего log.file

DEBUG:root:Start removing hidden dirs for /home/artur/
DEBUG:root:Length of dirs 38
DEBUG:root:File list before removing
DEBUG:root:.adobe
DEBUG:root:.android
DEBUG:root:.cache
DEBUG:root:.config
DEBUG:root:.dbus
...
DEBUG:root:Iterate on 22 dirs
DEBUG:root:File list after removing
DEBUG:root:.adobe
DEBUG:root:.cache
DEBUG:root:.dbus
...

Заранее благодарим за помощь

Теги:
list
iteration

2 ответа

4
Лучший ответ

Очень просто: вы удаляете текущий элемент из dirs во время итерации по нему. Это означает, что вы пропустите два вперед в следующем цикле. См. здесь.

Если бы я сказал, что неясно, скажите, что вы перебираете этот список чисел:

[1, 2, 3, 4, 5, 6...]
 ^

Это состояние списка изначально; то 1 удаляется, и цикл переходит ко второму элементу списка:

[2, 3, 4, 5, 6...]
    ^

Поскольку вы удалили 1 и переместились вперед одним из списка, теперь вы находитесь в 3, а не 2.

[2, 4, 5, 6...]

Затем вы удаляете 3... и т.д.

Поделиться
  • 1
    Быстрое решение состоит в том, чтобы дублировать список, разрезая его: для d в dirs [:]
  • 0
    Что еще более важно: это означает, что цикл пропускает определенные записи, потому что они все сместились на одно место.
Показать ещё 4 комментария
9

Не изменяйте списки во время итерации по ним. Вместо этого создайте новый список, содержащий только те элементы, которые вы хотите сохранить.

dirs = [d for d in dirs if not d.startswith('.')]

Является эффективным и понятным для этого способом.

Поделиться

Ещё вопросы

Сообщество Overcoder
Наверх
Меню