pythonのリストオブジェクトを使う際の初期化する理由

Question

以下のようなコードがあったとして、なぜ空のリストを事前にいれて初期化する必要があるのでしょうか？
テストコードも初期化処理(test_list = []とhoge_list = [])の箇所をコメントアウトしても何もエラーが出ません。本当に書くべき必要があるものなのでしょうか？

def function():
    test_list = []
    if 'test' in body['scope']:
        test_list = is_unique(body['scope']['test'])

    hoge_list = []
    if 'hoge' in body['scope']:
        hoge_list = is_unique(body['scope']['hoge'])


    for test in test_list:
       ...

    for hoge in hoge_list:
       ...

return hogehoge

Answer 1

'test' in body['scope']や'hoge' in body['scope']がFalseだった場合、すぐ下でNameError: name '~~' is not definedになります。その対策です。

つまるところ、こう書いているのと同じです。

    if 'test' in body['scope']:
        test_list = is_unique(body['scope']['test'])
    else:
        test_list = []

    if 'hoge' in body['scope']:
        hoge_list = is_unique(body['scope']['hoge'])
    else:
        hoge_list = []

これなら納得がいくのでは？

Answer 2

初期化処理(test_list = []とhoge_list = [])の箇所をコメントアウトしても何もエラーが出ません。本当に書くべき必要があるものなのでしょうか？

次のような理由です。
ブロック内に処理が進まなかったときに NameError が発生します。

>>> lst = []
>>>
>>> dst = []
>>> for e in lst:
...     dst = [e]
...
>>> print(dst)
[]

>>> lst = []
>>>
>>> for e in lst:
...     dst = [e]
...
>>> print(dst)
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
NameError: name 'dst' is not defined

追記：
うっかりfor文の例を挙げてしまいましたが、事情は同じです。
ブロック内で変数を宣言する場合は、常に『ブロックに入らなかった場合』を考慮する必要があります。

Comments

[sequelanonymous]

> 常に『ブロックに入らなかった場合』を考慮する必要があります。
腑に落ちました。ありがとうございます。

Answer 3

初期化がない場合、

if 'test' in body['scope']

この条件がfalse のとき、

for test in test_list:

ここでエラーとなってしまいます