Фильтровать элементы из списка на основе их, содержащие условия спама

Question

Итак, я создал скрипт, который скрежет некоторых сайтов и строит список результатов. Каждый результат имеет следующую структуру:

result = {'id': id, 
      'name': name, 
      'url': url, 
      'datetime': datetime, 
     } 

Я хочу, чтобы отфильтровать результаты из списка результатов, основанных на спам термы в названии. Я определил следующую функцию, и, кажется, фильтровать определенные результаты, но не все из них:

def filterSpamGigsList(theList): 
    index = 0 
    spamTerms = ['paid','hire','work','review','survey', 
       'home','rent','cash','pay','flex', 
       'facebook','sex','$$$','boss','secretary', 
       'loan','supplemental','income','sales', 
       'dollars','money'] 
    for i in theList: 
     for y in spamTerms: 
      if y in i['name'].lower(): 
       theList.pop(index) 
       break   
      index += 1 
    return theList 

Любой ключ, почему это не может быть отфильтровывая все результаты, содержащие эти термины спам? Может быть, мне нужно вызвать .split() на имя после вызова .lower(), поскольку некоторые из имен являются фразами?

Answer 1

Я думаю, у вас возникла проблема с заменой объекта на место, как итерация над ним, как предложил Якуб.

Невероятным способом было бы вернуть новый список. Я бы разделил это на две функции для удобочитаемости:

def is_spam(value): 
    spam_terms = ['paid','hire','work','review','survey', 
       'home','rent','cash','pay','flex', 
       'facebook','sex','$$$','boss','secretary', 
       'loan','supplemental','income','sales', 
       'dollars','money'] 
    for term in spam_terms: 
     if term in value.lower(): 
      return True 
    return False 

def filter_spam_gigs_list(results): 
    return [i for i in results if not is_spam(i['name'])]