Remove all occurrences of strings from list

Question

Someone please tell me there is a cleaner way to write this!? I want to remove all instances of 'Features & Benefits'. There must be a cleaner way to do this for strings in a list.

lov_headers = [x.find('displayname').text for x in soup.find_all('attributedefinition')]


if 'Part number' in lov_headers:
    lov_headers.remove('Part number')
if 'Product Features' in lov_headers:
    lov_headers.remove('Product Features')
if 'Packaging' in lov_headers:
    lov_headers.remove('Packaging')
if 'Features & Benefits' in lov_headers:
    lov_headers.remove('Features & Benefits')
if 'Features & Benefits' in lov_headers:
    lov_headers.remove('Features & Benefits')
if 'Features & Benefits' in lov_headers:
    lov_headers.remove('Features & Benefits')
if 'Features & Benefits' in lov_headers:
    lov_headers.remove('Features & Benefits')
if 'Features & Benefits' in lov_headers:
    lov_headers.remove('Features & Benefits')
if 'Features & Benefits' in lov_headers:
    lov_headers.remove('Features & Benefits')
if 'Features & Benefits' in lov_headers:
    lov_headers.remove('Features & Benefits')
if 'Features & Benefits' in lov_headers:
    lov_headers.remove('Features & Benefits')
if 'Features & Benefits' in lov_headers:
    lov_headers.remove('Features & Benefits')
if 'Features & Benefits' in lov_headers:
    lov_headers.remove('Features & Benefits')
if 'Features & Benefits' in lov_headers:
    lov_headers.remove('Features & Benefits')

...please learn about loops. – Aran-Fey Apr 11 '18 at 22:22 — Aran-Fey, Apr 11 '18 at 22:22
`[x for x in lov_headers if x != 'Features & Benefits']` – user3483203 Apr 11 '18 at 22:22 — user3483203, Apr 11 '18 at 22:22

score 1 · Answer 1 · answered Apr 11 '18 at 22:24

Put what you want to remove into a set(), then use a list comprehension to filter out the results that you don't want:

to_remove = set(['Part number', 'Features & Benefits', 'Product Features', 'Packaging'])

new_list = [i for i in lov_headers if i not in to_remove]

score 1 · Answer 2 · answered Apr 11 '18 at 22:25

1

You can use a while loop:

while 'Features & Benefits' in lov_headers:
    lov_headers.remove('Features & Benefits')

answered Apr 11 '18 at 22:25

DYZ

55,249
10
64
93

1

The time complexity on this is not good at all. – user3483203 Apr 11 '18 at 22:26
2

Agree. But if the OP wants to remove few select headers, the time complexity does not matter. Downloading the document would take way more time than searching through a list. – DYZ Apr 11 '18 at 22:28

Remove all occurrences of strings from list

2 Answers2