Как правильно обернуть в try except?

Question

[Yrets169]

Добрый вечер, как правильно заключить код в блок try except для корректной работы?
Фрагмент кода который открывает раздел в реестре и добавляет его содержимое в список list_keys_reg.
Except в данном случае срабатывает независимо есть такой раздел в реестре или нету.
Как правильно отловить ошибку?

import winreg
list_keys_reg = []
try:
   access_registry = winreg.ConnectRegistry(None, winreg.HKEY_CURRENT_USER)
   access_key = winreg.OpenKey(access_registry, r"Software\\JavaSoft\\Prefs\\jetbrains")
   i = 0
   while True:
      x = winreg.EnumKey(access_key, i)
      list_keys_reg.append(x)
      i += 1
except:
     print("не удается найти раздел")

Comments

[Алан Гибизов]

А except всегда одинаковый?

[Yrets169]

Алан Гибизов, раздел либо есть либо нету, третьего не дано

[mikeyuriev]

Yrets169, Уберите try/except и посмотрите, где у вас программа валится. Скорее всего - на вызове winreg.EnumKey(). Хотя я не знаток винды и виндового реестра.

[kisaa]

Вообще, у вас с отступами какая-то фигня на приведенном фрагменте: print в блоке except должен быть на уровне while в блоке try

Upd.:
А, ну правильно, у вас выхода из while нет, вываливается по эксепшену только когда не может найти (i+1)-ый раздел.

[Yrets169]

mikeyuriev, программа валится, если указать несуществующий раздел реестра, о чем собственно ошибка и предупреждает

[Yrets169]

kisaa, нет, except в любом случае вываливается (кроме того случая где цикл бесконечный), если раздела в реестре нету, то try пропускается, если есть, то в список list_keys_reg добавляются подразделы указанной ветки реестра и после чего except отрабатывает

[Yrets169]

kisaa, "print в блоке except должен быть на уровне while в блоке try" на самом деле так и есть, print из except находится на уровне while блока try, почему здесь так отобразило - непонятно, суть заключается в том, что от расположения print ничего не меняется

[Алан Гибизов]

Yrets169, except - это ошибка вообще. Надо ловить не except, а конкретную ошибку, а остальные не ловить. А вы своим try except ловите любую ошибку и не видите, что в ней.
Сделайте хотя бы

try:
    …
except Exception as e:
    print(e)

[Yrets169]

Алан Гибизов, это работает так же, но если убрать цикл из try, то все работает правильно

except Exception as e:
    print(e)

[Алан Гибизов]

Yrets169, вы не показали, как именно «так же» работает. Там должно хоть что-то печататься при ошибке. И при разных ошибках разное печататься.

[mikeyuriev]

Yrets169, можете что-то такое попробовать:

import winreg

try:
    access_registry = winreg.ConnectRegistry(None, winreg.HKEY_CURRENT_USER)
    access_key = winreg.OpenKey(access_registry, r"Software\\JavaSoft\\Prefs\\jetbrains")
except OSError:
    print("не удается найти раздел")

list_keys_reg = []
i = 0
while True:
    try:
        x = winreg.EnumKey(access_key, i)
  	list_keys_reg.append(x)
  	i += 1
    except OSError:
        break

[Yrets169]

mikeyuriev, этот вариант подходит, но я бы сделал вот так:

import winreg

try:
    access_registry = winreg.ConnectRegistry(None, winreg.HKEY_CURRENT_USER)
    access_key = winreg.OpenKey(access_registry, r"Software\\JavaSoft\\Prefs\\jetbrains")
except OSError:
    print("не удается найти раздел")

if 'access_key' in locals():
    list_keys_reg = []
    i = 0
    while True:
        try:
            x = winreg.EnumKey(access_key, i)
            list_keys_reg.append(x)
            print(list_keys_reg)
            i += 1
        except OSError:
            break

Answer 1

[Yrets169]

немного модифицировал код mikeyuriev, добавив условие
if 'access_key' in locals(): будет самым оптимальным вариантом

import winreg

try:
    access_registry = winreg.ConnectRegistry(None, winreg.HKEY_CURRENT_USER)
    access_key = winreg.OpenKey(access_registry, r"Software\\JavaSoft\\Prefs\\jetbrains")
except OSError:
    print("не удается найти раздел")

if 'access_key' in locals():
    list_keys_reg = []
    i = 0
    while True:
        try:
            x = winreg.EnumKey(access_key, i)
            list_keys_reg.append(x)
            print(list_keys_reg)
            i += 1
        except OSError:
            break