Чего от меня ждет PyCharm?

Question

[Александр]

Доброго вечера,

Народ, подскажите пожалуйста, чего от меня хочет PyCharm?
Есть кусок кода по созданию логгера для записи событий в Windows EventLog:

if name is 'nt':
    try:
        from logging.handlers import NTEventLogHandler
        win_log_handler = NTEventLogHandler(appname='update_aduser', logtype='Application')
        win_log_handler.setFormatter(log_formatter)
        win_logger = logging.getLogger('win_logger')
        win_logger.addHandler(win_log_handler)
    except ImportError:
        write_log(file_logger, logging.WARNING, "cannot import 'NTEventLogHandler' module")

PyCharm подчеркивает строку и пишет такое вот предупреждение:

Comments

[Дмитрий]

Картинка маленька и не кликабельная. Сделайте так чтоб ошибку можно было увидеть !!! ;)

[Александр]

Я делаю лучше. =)
'NTEventLogHandler' in try block with 'except ImportError' should also be fefined in except block
This inspection detects names that should resolve but don't. Due to dynamic dispatch and duck typing, this is possible in a limited but useful number of cases. Tol-level and class-level items are supported better than instance items.

Answer 1

[tsarevfs]

stackoverflow.com/a/35901372/1762922

Comments

[Александр]

Да, я читал и думаю, что плохо понял суть. Он вроде как хочет, чтобы я не просто написал except, но еще и описал, что должно произойти при этом. Проблема в том, что это не обязательный модуль, лог будет писаться так же и в файл и для выполнения программы не принципиальна запись именно в Windows EventLog.

[tsarevfs]

Александр:
Написать после except ImportError:
NTEventLogHandler = none
не помогает?

[Александр]

tsarevfs: помогает. ) Перестает ругаться, только я сути не понял.

[tsarevfs]

Александр: Вас просили как-то определить имя NTEventLogHandler во второй ветке. Самый тупой вариант -- присвоить None.

[Александр]

tsarevfs: ага, хорошо... Но выше суть я правильно описал?

[javedimka]

Александр: Если я понял суть, то в твоем случае это действительно не важно и такое присваивание необходимо, если потом, где-нибудь в середине кода, тебе нужно будет выполнить какое-нибудь действие с модулем - если он был импортирован или выполнить другое действие, если твой блок try ушел в исключение.
Т.е. Что-то типо этого

try:
import module1
except ImportError:
module1 = None
... Какой-нибудь код ...

if module1:
var = module1.attribute
else:
pass

Не оборачивать же все в try/except

[javedimka]

Александр Т.е. за пределами блока try/except узнать, а был ли модуль вообще импортирован? И в зависимости от этого выполнить действие.

[Александр]

javedimka: хорошо. ) Пусть будет так. Спасибо. =)