0

Пытаюсь обработать ситуацию, когда java не установлена:

try:
  java_version = subprocess.check_output(['java', '-version'], stderr=subprocess.STDOUT)
  #print(java_version)
except Exception as e:
  java_version = '0'

Корректно ли я это делаю?

1
  • у внешнего кода нет явных признаков того, что выдернуть версию не получилось. лучше напрямую возвращать None в случае exit code != 1, да и исключение ловить вряд ли стоит - насколько понимаю, оно будет выброшено только в случае, если все пойдет совсем на юг, и об этом конечный пользователь должен будет как-то узнать (e.g. сейчас у вас может вообще не запуститься процесс, но об этом никто не узнает, все будут считать, что джава не установлена)
    – etki
    7 апр 2017 в 22:10

1 ответ 1

0

Если вы хотите прочитать вывод команды java -version, то код корректный. Насколько в целом такой подход правильный зависит как эта информация позже используется (может вы хотите: update-alternatives --list java).

Вариация на тему:

#!/usr/bin/env python3
from subprocess import check_output, STDOUT

try:
    java_version = check_output(['java', '-version'], stderr=STDOUT)
except FileNotFoundError:   # java command is not found 
    java_version = None  

CalledProcessError можно не ловить, а позволить по стеку подняться, так как java -version не должна ненулевой статус возвращать в обычной ситуации (java отсутствует или java присутствует и на настоящую java команду указывает).

java_version либо None либо байты здесь.

Ваш ответ

By clicking “Отправить ответ”, you agree to our terms of service and acknowledge that you have read and understand our privacy policy and code of conduct.

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.