Grep

[Krock21]

No description provided.

[yurii-litvinov]

Имело смысл сделать пуллреквест в ветку CLI, иначе дифф показывает вообще все исходники.

В условии просили ещё обоснование выбора библиотеки. В Питоне есть из чего повыбирать, ну и вообще, это задача на борьбу с образом мышления "фигак-фигак --- и в продакшн".

Ещё на неправильные параметры (например, grep -A -1 param bash_builtins.py) надо ругаться.

[Krock21]

Добавил проверку на отрицательность -А
Обоснования argparse и правда, не было.
Я его выбрал т.к. видел, что почти все его используют, и т.к. он по умолчанию идёт в питоне.
Ну и люди советуют на stackoverflow: https://stackoverflow.com/questions/20063/whats-the-best-way-to-parse-command-line-arguments

Какие ещё могут быть неправильные параметры, не очень понимаю

[Krock21]

В ходе пересмотра своего кода я понял, что он какой-то ужасно отстойный и не позволяет контролировать всё, что хотелось бы.
Слишком не универсальный. Не могу указывать stderr для команд и даже код возврата обрабатывать.

[yurii-litvinov]

Насчёт других ошибок:

grep -A main cli.py
Exception in thread Thread-2:
Traceback (most recent call last):
  File "C:\Tools\Python38\lib\threading.py", line 932, in _bootstrap_inner
    self.run()
  File "C:\Tools\Python38\lib\threading.py", line 870, in run
    self._target(*self._args, **self._kwargs)
  File "C:\Students\sd-bachelors\Krock21rus-bash\bash_builtins.py", line 140, in grep_function
    after_parameter = int(parsed_args.get('A'))
ValueError: invalid literal for int() with base 10: 'main'

Понятно, что произошло, но всё равно такое лучше пользователю не показывать. Примерно так же обстоят дела с несуществующим файлом.

[yurii-litvinov]

Ага, теперь всё ок, зачтена