поиск строки в файле с помощью python faliure

Question

поиск строки в файле с помощью python faliure

Я использую этот код для поиска писем в конкретном файле и записи их в другой файл. Я использовал оператор «in», чтобы убедиться, что письмо не дублируется. Но этот код не выполняется после строки для строки в строке f: . Может ли кто-нибудь указать на ошибку, которую я здесь сделал?

tempPath = input("Please Enter the Path of the File\n")
temp_file = open(tempPath, "r")
fileContent = temp_file.read()
temp_file.close()

pattern_normal = re.compile("[-a-zA-Z0-9._][email protected][-a-zA-Z0-9_]+.[a-zA-Z0-9_.]+")

pattern_normal_list = pattern_normal.findall(str(fileContent))

with open('emails_file.txt', 'a+') as f:            
    for item in pattern_normal_list:            
        for line in f:
            if line in item:
                print("duplicate")
            else:
                print("%s" %item)
                f.write("%s" %item)
                f.write('\n')

0

добавлено 07 Февраль 2013 в 11:03 автор direndd отредактировано 09 Февраль 2013 в 11:34

Просмотры: 3

источник

nl ja de

Проблема в том, что при первом запуске этот emails_file.txt будет пустым, поэтому строка не будет читать, поэтому вы никогда не сможете ее добавить. @Torxed показывает вам решение.

добавлено 07 Февраль 2013 в 11:16, автор sotapme, источник

2 ответы

Видеокурс по Java для начинающих

Получи видеокурс по Java для начинающих с упражнениями

Видеокурс "Создание приложений для Android"

Получи множество видеоуроков по Android сейчас

Новое решение:

tempPath = input("Please Enter the Path of the File\n")
temp_file = open(tempPath, "r")
fileContent = temp_file.read()
temp_file.close()

pattern_normal = re.compile("[-a-zA-Z0-9._][email protected][-a-zA-Z0-9_]+.[a-zA-Z0-9_.]+")

addresses = list(set(pattern_normal.findall(str(fileContent))))
with open('new_emails.txt', 'a+') as f:
    f.write('\n'.join(addresses))

Я думаю, что ваша логика была неправильной, это работает:

addresses = ['[email protected]', '[email protected]']

with open('emails_file.txt', 'a+') as f:
    fdata = f.read()
    for mail in addresses:
        if not mail in fdata:
            f.write(mail + '\n')

Не прочитав много кода, похоже, что вы зацикливаете строки по очереди, проверяя, существует ли в строке адрес, который вы также зацикливаете, если вы не добавляете к нему свой адрес электронной почты? Но в 99% из 100 строк адрес не будет в строке, поэтому вы получите нежелательное дополнение.

Вывод моего фрагмента кода:

[[email protected] ~]$ cat emails_file.txt 
[email protected]
[email protected]
[[email protected] ~]$ python test.py 
[[email protected] ~]$ cat emails_file.txt 
[email protected]
[email protected]
[email protected]
[[email protected] ~]$

1

добавлено 07 Февраль 2013 в 11:08 автор Torxed

источник

Я понял, что моя логика ошибочна. Но я изменил, поскольку вы сказали, что он пишет дубликаты записей. Исходный файл, в котором я получаю электронные письма, содержит дубликаты записей. Этот код записывает их все в файл emails.txt. Измененный фрагмент кода: с открытым ('emails_file.txt', 'a +') как f: fdata = f.read() для элемента в pattern_normal_list: если элемент не в fdata: print ("% s"% item) f. write ("% s"% item) f.write ('\ n')

добавлено 07 Февраль 2013 в 11:33, автор direndd, источник

Добавлена обновленная версия, в которой вы 1 читаете файл, содержащий douplicate адреса электронной почты. Затем отфильтруйте это .. подождите минуту .. erm ..

добавлено 07 Февраль 2013 в 11:38, автор Torxed, источник

автор Rafał Łużyński · Answer 1 · 2013-02-07T11:13Z

for line in f:

Не следует ли сначала вызвать f.readlines ()?

lines = f.readlines()
for line in lines:

Проверь это.

2

добавлено 07 Февраль 2013 в 11:13 автор Rafał Łużyński

источник

Вам не нужно использовать оператор с open() как f , он по существу сбрасывается до, f является файловым дескриптором, и если вы повторяете этот дескриптор файла, вы получите одну строку на время по умолчанию в Python.

добавлено 07 Февраль 2013 в 11:14, автор Torxed, источник

Достаточно справедливо, это не требование, но должно быть в описанном использовании :) Но опять же, вы правы, +1

добавлено 07 Февраль 2013 в 11:21, автор Torxed, источник

@Torxed: но только один раз; OP пытается просмотреть весь файл для каждого элемента в списке, который не будет работать без seek() (хотя я согласен, что этот ответ может быть более ясным).

добавлено 07 Февраль 2013 в 11:18, автор geoffspear, источник

@Torxed: для этого вам не нужен оператор with . Открытие файла с помощью f = open (filename) и чтение с помощью для строки в f . Конечно, использование с всегда является хорошей идеей.

добавлено 07 Февраль 2013 в 11:19, автор Matthias, источник