Не могу понять как правильно определить переменную в файле и сделать запись?

Question

[metalexs]

Не могу понять как правильно сделать так, чтобы определить наличие в файле "id.txt", "ID" взятый из переменной "a" и если в строках его нет, то добавить запись. Я сделал вот такую конструкцию, но в любом случает срабатывает else.

a = input("Введите id: ")

   with open('id.txt') as f:
        n = 0
        for line in f:
            n += 1
            if a in line:

                print(a, "есть совпадение", n)
            
            else:
                print("нет совпадения")
                file = open("id.txt", "a")
                file.write(f"{a} \n")
                file.close()

Помогите разобраться.

Answer 1

[Александр Нестеров]

Вообще, я бы советовал использовать как минимум json, который можно преобразовать в структуры данных python. Ну или базу данных. Для таких случаев они больше подходят.
Но тут уже решайте сами...
Касательно вопроса:
Ну, давайте подумаем логически. Есть файл вида:

22
12
146
34

Искомый id - 146. То есть 3-я строка. Разумеется, что поскольку ни 1, 2 и 4 строки не подходят - будет срабатывать else.
Выход из этой ситуации - вынести запись в файл из цикла в отдельный блок условия. Я бы, например, добавил булеву переменную перед циклом со значением False и в блоке if a in line: менял на True

Comments

[metalexs]

Если вам не сложно можете написать как должно выглядеть? Я совсем еще "зеленый" =)

[Александр Нестеров]

metalexs,
Сейчас перечитал и понял, что затупил, вариант со счетчиком не подходит. Лучше использовать "флаг" (переменную со значением True/False)

a = input("Введите id: ")

with open('id.txt') as f:
    n = 0
    has_id = False # По умолчянию указывает, что id не найден
    for line in f:
        n += 1
        if a in line:
            has_id = True          
            break # Завершает цикл


if has_id: # Условие будет верно, если is_id НЕ равно 0, False, None или не является пустым списком/словарем
    print(a, "есть совпадение", n)
else:
    print("нет совпадения")
    file = open("id.txt", "a")
    file.write(f"{a} \n")
    file.close()

[metalexs]

Александр Нестеров, Попробовал еще вот так и тоже работает

f = open("id.txt")
    if a in f.read():
        print("true")

    else:
        print("false")
        f = open("id.txt", "a")
        f.write(f"{a} \n")
        f.close()

А лучше f = open("id.txt") так делать или with open('id.txt') as f: ? Что правильней?

[Александр Нестеров]

metalexs,
Тут правильнее сравнивать:

f = open("id.txt")
...
f.close()

и
with open('id.txt') as f
В таком случае - чисто вопрос вкуса.
Разница в том, что во втором случае используется контекстный менеджер, при использовании которого файл закрывается сам.

[metalexs]

Александр Нестеров, Спасибо большое за помощь.