Comments 2
Быстро проглядел, но, чтобы я точно сделал:
- Перешёл бы с argv на argparse
- Добавил бы typing
- Можно убрать ненужный импорт
- Можно переработать if ... continue
спасибо, я не профессиональный програмист, я занимаюсь поддержкой файрволов, я вынужден был написать эти программы по необходимости для решения конкретной задачи с файрволом Juniper SRX.
с argparse согласен
typing - в каком месте? тут есть str переменные и есть list of str, думаю там правильно конвертируются типы, как можно улучшить?
как можно переработать if ... continue
Sign up to leave a comment.
Как управлять тысячами строк правил Firewall, на примере Juniper SRX