Pull to refresh

Comments 2

Быстро проглядел, но, чтобы я точно сделал:
- Перешёл бы с argv на argparse
- Добавил бы typing
- Можно убрать ненужный импорт
- Можно переработать if ... continue

спасибо, я не профессиональный програмист, я занимаюсь поддержкой файрволов, я вынужден был написать эти программы по необходимости для решения конкретной задачи с файрволом Juniper SRX.

  • с argparse согласен

  • typing - в каком месте? тут есть str переменные и есть list of str, думаю там правильно конвертируются типы, как можно улучшить?

  • как можно переработать if ... continue

Sign up to leave a comment.

Articles