Как перейти к следующему элементу во вложенном списке? Python

Question

Работа с несколькими списками, итерация по каждому из них. Вот фрагмент кода:

self.links = [] 
self.iter=iter(self.links) 
for tgt in self.links: 
    for link in self.mal_list: 
     print(link) 
     if tgt == link: 
      print("Found Suspicious Link: {0}".format(tgt)) 
      self.count += 1 

     else: 
      self.count += 1 
      self.crawl(self.iter.next()) 

Его переход на следующий элемент в списке ссылок, просто отлично. Для списка подписи вредоносных программ я попытался использовать аналогичный элемент iter, но я не совсем уверен, что это даже лучший способ, и если бы это было поместить его в мой код, чтобы каждая ссылка, которая была скрыта из списка, сравнивалась с каждый элемент в списке вредоносных программ. Перед циклом открывается следующий элемент в списке ссылок. Какие-либо предложения?

Answer 1

Существенный способ, которым вы это выполняете, хорошо, но он будет медленным.

Попробуйте вместо этого:

for tgt in links: 
     if tgt in mal_links: 
      # you know that it's a bad link 
     else: 
      crawl(tgt) 

Я не понимаю, почему вы держите два итератора, идущие по списку. Это приведет к ошибке, потому что вы не вызываете next на self.iter в случае обнаружения вредоносной ссылки. В следующий раз, когда tgt не является плохой ссылкой, когда вы вызываете next, он будет продвигаться к ранее обнаруженной плохой ссылке, и вы сканируете это. Есть ли какая-то причина, по которой вы чувствуете необходимость перешагнуть через две копии итератора вместо одного?

Кроме того, ваш первоначальный код будет сканировать страницу один раз за каждый раз, когда он не определен как равный данной ссылке вредоносного ПО. Это может привести к некоторым сердитым веб-мастерам в зависимости от того, насколько велик ваш список.

Answer 2

Не уверен, что вы пытаетесь спросить, но вы можете упростить свой код. Хотя это не обязательно.

self.links = [] 
self.non_malware_link = [link for link in self.links if link not in self.mal_list] 
results = map(self.crawl, self.non_malware_link) 

О некоторых вопросах с кодом:

1. self.count точно так же, как LEN (self.links)

Помимо значения self.count, каждая вещь иначе похоже, что он делает то, что нужно.

Answer 3

Поиск элемент внутри списка медленна, если это то, что вы пытаетесь сделать, то использовать dict или в set вместо list для self.mal_list:

mal_list = set(self.mal_list) 
for tgt in self.links: 
    if tgt in mal_list: 
     print("Found Suspicious Link: {0}".format(tgt)) 
     self.count += 1 
    else: 
     self.count += 1 
     self.crawl(self.iter.next()) 

или, если вы можете имеют self.links как установлено:

mal_list = set(self.mal_list) 
links = set(self.links) 
detected = links.intersection(mal_list) 
for malware in detected: 
    print("Found Suspicious Link: {0}".format(tgt)) 
    self.count += 1