Как получить в f-строке актуальное значение переменной?

Question

[nihi1ist]

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

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

Код

total_sum = 0


def check_string(string):
    try:
        result = eval(string)
        return result
    except SyntaxError:
        while True:
            need_fix = input(f'Обнаружена ошибка в строке: {string}\tХотите исправить? ').lower()
            if need_fix == 'да':
                corrected_string = input('Введите исправленную строку: ')
                result = check_string(corrected_string)
                return result
            elif need_fix == 'нет':
                break
            else:
                print('Ответ не распознан, повторите ввод.')
                continue


with open('calc.txt', 'r') as tasks:
    for line in tasks:
        task = line[:-1] if '\n' in line else line
        task_result = check_string(task)
        if not task_result:
            continue
        total_sum += task_result
    print()
    print(f'Сумма результатов: {total_sum}')

P.S. Буду благодарен за советы по оптимизации.

Comments

[Алан Гибизов]

Надо уточнить, какие данные попадают на вход, что вы ожидаете на выходе и что получилось.
Я протестировал в меру своего понимания на файле

2+2
2+3
2+
2+4

(Последняя строка пустая)
и получил результат:

Старт 1
Обнаружена ошибка в строке: 2+	Хотите исправить? да
Введите исправленную строку: 2+1
Обнаружена ошибка в строке: 	Хотите исправить? нет

Сумма результатов: 18

Старт 2
Обнаружена ошибка в строке: 2+	Хотите исправить? нет
Обнаружена ошибка в строке: 	Хотите исправить? нет

Сумма результатов: 15

[nihi1ist]

Алан Гибизов, тестировал на таких данных:

100 + 34
10 +* 3
20 -* 2
23 / 4

Считает исходя из предложенного примера верно:

Обнаружена ошибка в строке: 10 +* 3 Хотите исправить? Нет
Обнаружена ошибка в строке: 20 -* 2 Хотите исправить? Да
Введите исправленную строку: 20 - 2

Сумма результатов: 157.75

[Алан Гибизов]

nihi1ist, тогда что не так? Надо конкретизировать вопрос.

[nihi1ist]

Алан Гибизов, когда вводишь некорректную строку, в тексте input() отображается изначальная строка. Хотелось бы, чтобы отображалась текущая строка, введенная некорректно.

Answer 1

[Алан Гибизов]

total_sum = 0


def check_string(string):
    try:
        result = eval(string)
        return result
    except SyntaxError:
        while True:
            need_fix = input(f'Обнаружена ошибка в строке: {string}\tХотите исправить? ').lower()
            if need_fix == 'да':
                string = input('Введите исправленную строку: ')
                result = check_string(string)
                return result
            elif need_fix == 'нет':
                break
            else:
                print('Ответ не распознан, повторите ввод.')
                continue


with open('calc.txt', 'r') as tasks:
    for line in tasks:
        task = line[:-1] if '\n' in line else line
        task_result = check_string(task)
        if not task_result:
            continue
        total_sum += task_result
    print()
    print(f'Сумма результатов: {total_sum}')

Замечу в скобках

Но мне в принципе не нравится такая структура программы, я бы делал по-другому. Рекурсивный вызов функции не оправдан и усложняет понимание.
Функцию надо бы разбить минимум на две, лучше три маленьких.
Не люблю все эти break и циклы while True.
Вечерком попробую сделать, сейчас занят.

Comments

[Totem]

Как бы ты сделал без "break и цикла while True" и без рекурсии?

Просто метод которым ты бы действовал

[Алан Гибизов]

Totem, ну ты бы ещё через год спросил :) я уже забыл всё и снова вчитываться в код :)))

В общих чертах: у while должно быть явное условие, а не выход по break.
То же и про рекурсию: она тут используется для чего? Чтобы вернуться в начало функции с новой строкой и вновь проверить, сработает ли eval. Ну так можно же всё это обернуть в ещё один while и явно проверять, если eval сработал - выставили признак и цикл завершается. Любой школьник поймёт прогу.

Явное лучше неявного.

[Алан Гибизов]

Totem, и, конечно, каждую из подзадач вытащить в отдельную функцию. Получение строки - это одно. Проверка её - другое. Суммирование результатов - третье. Работа с файлом - четвертое. И т.п.

[Totem]

Алан Гибизов, все понял, спасибо уважаемый