Python. Почему мне нужно обойти список дважды, чтобы удалить все пустые элементы?

Question

Здравствуйте. Я хочу избавиться от пустых слов в списке.

file_obj = open('D:\sentences.txt.txt', 'r')

lower_text = ''
text = file_obj.readlines()
for line in text:
    lower_text += line.lower()

words = re.split('[^a-z]', lower_text)

a = 0
for word in words:
    for word in words:
        if word == '':
            words.remove('')
            a += 1
print(a)   
print(words)

Я открываю файл с текстом. Сохраняю его в переменную.
Считываю файл по строкам. Сохраняю в строку. Привожу их к нижнему регистру.
Разбиваю строку по словам в список.
Тут я хочу убрать "пустые слова" (''). Использую переменную а, чтобы показать удаленные пустые элементы списка. НО, если проходить один раз по списку, то не удаляет все пустые элементы. Почему?

Screenshots:

Скрин текстового файла. Не нашел как прикрепить С:
Я обхожу список один раз и удаляется только 91/129 пустых элементов.
Если обойти два раза все пустые элементы удаляются.

Нельзя удалять элементы из списка, который в данный момент используется в цикле — andreymal, Oct 26 '17 at 22:38

pinguin · Answer 1 · 2017-10-26T21:04:18.623

0

Весь этот цикл:

for word in words:
    for word in words:
        if word == '':
            words.remove('')
            a += 1

Можно заменить на простое:

words = list(filter(lambda word: word != '', words))

Здесь можно почитать о функции filter

edited Oct 26 '17 at 21:04

answered Oct 26 '17 at 20:56

pinguin

1,873

Да. действительно все работает. Спасибо) А не могли бы почему мой код работает не так? – hnf Oct 26 '17 at 21:06
Честно, не знаю. Возможно что-то происходит не так во время "сплита". Попробуйте делать так как посоветовал @AlexanderZonov. И чаще старайтесь использовать встроенные функции Python. – pinguin Oct 26 '17 at 21:12
@hnf и проголосуйте за ответ если он Вам помог – pinguin Oct 26 '17 at 21:20
С радостью, но не хватает репутации( – hnf Oct 26 '17 at 21:27
@hnf стрелочка наверх слева от ответа и кнопка "принять ответ", вы их не видите? – 0xdb Oct 27 '17 at 05:55
2

можно заменить на words = list(filter(None, words)) – godva Oct 27 '17 at 11:18

score 0 · Answer 2 · answered Oct 26 '17 at 21:05

0

Используйте метод read(), чтобы загрузить сразу весь файл в переменную.
Используйте регулярное выражение [^a-z]+ вместо [^a-z] в с функцией split.

answered Oct 26 '17 at 21:05

Окей. спасибо :) – hnf Oct 26 '17 at 21:07

score 0 · Accepted Answer · answered Oct 27 '17 at 07:06

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

for word in words[:]:
    if word == '':
        words.remove('')
        a += 1

(words[:] обыкновенная практика как из списка сделать его копию.)

vadim vaduxa · Answer 4 · 2017-10-27T07:27:44.913

0

with open('D:\sentences.txt.txt') as f:
    words = list(filter(bool, map(str.lower, re.split('[^a-z]', f.read()))))

edited Oct 27 '17 at 07:27

answered Oct 27 '17 at 07:19

vadim vaduxa

8,897

Python. Почему мне нужно обойти список дважды, чтобы удалить все пустые элементы?

4 Answers4