TypeError: writelines() требует итеративного аргумента

Question

Я делал простой скрипт, который находит панель администратора веб-сайта.
Я смог сделать это с помощью интерфейса командной строки, но я также хотел сделать простой gui с помощью easygui.
я часы делаем это, но я получаю эту ошибку в строке 33:
«TypeError: writelines требует итерации аргумента»

Вот мой код:

import os, httplib 
import easygui as eg 

admpagelist = ['admin/','administrator/','admin1/','admin2/'] 

title = 'Test Easygui' 
msg='Enter Your Target: (without http://) ' 
value = [] 
value = eg.enterbox(msg,title) 
value = value.replace('Enter Your Target: (without http://)','') 

collectornf = [] 
collectorf = [] 

try: 
    con = httplib.HTTPConnection(value) 
    con.connect() 
except: 
    eg.msgbox('Host is offline or invalid url! ') 

for adm in admpagelist: 
    adm = '/%s' %adm 
    host = value + adm 
    con = httplib.HTTPConnection(value) 
    con.connect() 
    request = con.request('GET',adm) 
    response = con.getresponse() 
    if response.status == 200: 
     collectorf = collectorf.append(str(host)) 
     found = open('C:/Users/Andi/Desktop/found.txt','w') 
     found.writelines(collectorf) 
     found.close() 
    else: 
     collectornf = collectornf.append(str(host)) 
     notfound = open('C:/Users/Andi/Desktop/notfound.txt','w') 
     notfound.writelines(collectornf) 
     notfound.close() 

Answer 1

Вашего вопрос в том, что вы делаете collectorf = collectorf.append(str(host)).

list.append() изменяет список на месте и возвращает None. Затем вы назначаете значение None и переписываете список.

Просто сделай collectorf.append(str(host)) вместо:

if response.status == 200: 
    filename = "found.txt" 
    data = collectorf 
else: 
    filename = "notfound.txt" 
    data = collectornf 
data.append(str(host)) 
with open('C:/Users/Andi/Desktop/' + filename,'w') as found: 
    found.writelines(data) 

(Refactored, чтобы избежать копирования кодирования/вставить и использовать некоторые из советов, который я разместил в комментариях).

Обратите внимание, что для записи данных после цикла не будет много смысла, а не повторно.

Answer 2

Я бы сохранил файлы открытыми. Постоянные открытия и закрытие кажутся для меня большими накладными расходами.

with open('C:/Users/Andi/Desktop/found.txt','w') as found, open('C:/Users/Andi/Desktop/notfound.txt','w') as notfound: # before 2.7, write this as 2 with statements 
    for adm in admpagelist: 
     adm = '/%s' % adm 
     host = value + adm 
     con = httplib.HTTPConnection(value) 
     con.connect() 
     request = con.request('GET', adm) 
     response = con.getresponse() 
     if response.status == 200: 
      targetlist = collectorf 
      targetfile = found 
     else: 
      targetlist = collectornf 
      targetfile = notfound 
     targetlist.append(str(host)) 
     targetfile.write(str(host) + '\n') 
     # if you are paranoid, you can as well do: 
     # targetfile.flush() 

with материал отвечает за потом снова закрыть файл.

Если вам не нужны списки после этого, вы можете их опустить, поскольку мы записываем файлы в каждый цикл.