Как создать такой цикл?

Question

[alina_yadtskaya]

Есть такая конструкция:

items = json.loads(z_api.call('/v3/statistic/ab/'))
f = open("blacklist.txt", "r+")
f.readlines()
for elem in items['stats']:
    if elem["sip"] in f:
        pass
    else:
        s = str(elem["sip"] + '\n')
        f.write(s)
f.close()

Я пытаюсь вытащить значение sip итерацией и добавить его в blacklist.txt, если его там раньше не было. Почему ничего не происходит? Файл пустой остается. В последующем планирую к нему обращаться и сверять. Если sip который будет в цикле, будет присутствовать в этом файле, то будет pass, если нет, то добавлять.

Comments

[iddqda]

s = str(elem["sip"] + '\n')
т.е. ты в курсе про '\n' ?
вот только не там его прибавляешь

ну и что мешает использовать debug()?
это гораздо быстрее чем вопросы сюда оформлять

Answer 1

[o5a]

Приведите пример данных. Возможно запросы не соответствуют структуре.
Из того, что есть, вижу 2 ошибки
s = str(elem["sip"] + '\n')
если elem["sip"] не текстовый, то подобная конструкция выдаст ошибку, т.к. str не правильно использован.

f.readlines()
...
    if elem["sip"] in f:

проверка не будет работать, т.к. f пустой, readlines() уже перемотал указатель в конец файла.

Comments

[alina_yadtskaya]

Элемент текстовый! Хорошо. Я так понял, что речь про a = f.readlines()?

[o5a]

Да, только их еще надо избавить от символов переноса строки. Можно так
a = f.read().splitlines()

Answer 2

[soremix]

if elem["sip"] in f:

Вы проверяете наличие элемента не в тексте файла, а в объекте open

f = open("blacklist.txt", "r+")
data = f.readlines()
if elem["sip"] in data:
  # ...

Ну и если ничего не планируется больше в if, то тогда уж можно сделать

if elem["sip"] not in data:
    f.write(elem['sip'] + '\n')

Comments

[alina_yadtskaya]

Спасибо!) Теперь он добавлят строки в файл и ничего не сравнивает. Почему все же не работает часть кода? Ведь после первой записи, он должен сравнить. Или нужно еще раз его открывать через open в 'a' и там через итерацию сравнивать?
Скину еще раз полностью, который уже работает. Возможно, так неудобно читать.

items = json.loads(z_api.call('/v3/statistic/ab/'))
f = open("blacklist.txt", "r+")
data = f.readlines()
for elem in items['stats']:
    if elem["sip"] not in data:
        f.write(elem["sip"] + '\n')
    elif elem["sip"] in data:
        pass
f.close()

[soremix]

alina_yadtskaya, в смысле не сравнивает? Строки в readlines будут с \n, если у вас данные без них, то и\n нужно обрезать,

items = json.loads(z_api.call('/v3/statistic/ab/'))
f = open("blacklist.txt", "r+")
data = [line.strip() for line in f.readlines()]
for elem in items['stats']:
    if elem["sip"] not in data:
        f.write(elem["sip"] + '\n')
f.close()