【Python】 既に値が入っているリストの初期化について

Question

いつもお世話になっております。

早速本題に入らせていただきます。

まず現在のコードを提示させて頂きます。

---

test_dict = {}
test_list = []
for i, item in enumerate(queryset):
    # 処理...
    if 条件１:
        test_list.append(item)
    elif 条件２:
        # 辞書にlistを格納
        test_dict[item.name] = test_list

        # test_listを再利用するために初期化
        test_list = []  # ★警告が出る
        test_list.append(item)

上記コードの ★ 部分の警告

This list creation could be rewritten as a list literal 
Inspection info: This inspection detects situations 
                 when list creation could be rewritten with list literal. 

---

上記のように、for文中で for文の外側に位置するlistを初期化 して再利用したいのですが、
現状では警告が出てしまいます。
（※ 警告は出ますが、正常に動作はしています。）

test_list.clear() では辞書に格納しておいたlistの中身まで消えてしまいますし、どうにか警告を出さずに初期化できないかなと悩んでおります。

お時間ある方で list の初期化方法 について知識をお持ちの方いらっしゃいましたらご助力頂けますと幸いです。

Comments

[LouiS0616]

test_dict["item.name"] = test_list は test_dict[item.name] = test_list の誤植でしょうか？

[azuapricot]

あ、その通りです、修正します

Answer 1

これ、PyCharmですよね。
Alt+Enterでやれることの一覧が出てきて、そこから Replace list creation を選択すれば修正してくれたのでは?

たぶんAlt+Shift+Enterでも直に修正してくれそうに思います。

This list creation could be rewritten as a list literal

リストの生成はリストリテラルで書き直せるよ!
ですね。

Comments

[azuapricot]

うわほんとだ....ｳﾜｧ.....ほんとだぁ.....Alt + Enter 押して候補選択したら
LouiS0616さんが回答してくれてるのとまったく同じコードになりました...
Pycharmの知識も全然たりませんね...なるほど...

Answer 2

半分勘なので外していたらごめんなさい。次のように書けば警告が消えませんか？

test_list = [item]

Comments

[azuapricot]

ﾊｯ....... 消えました.....!!!
ポンコツなのでその発想はなかったです....

[LouiS0616]

誤：リストの再代入が問題
正：冗長なリストの生成が問題　なのかなぁ、と。

前者を問題視しているなら this variable may be rewritten after とか言いそうな気がします。(英語ニガテなので適当です)
そしてinspectionもfor文の前の test_list = [] に付くでしょう。

[azuapricot]

なるほど...　良い書き方ないかなぁと思ってましたが勉強になりました、ありがとうございました！

Answer 3

test_list.clear()

では？

Comments

[azuapricot]

質問本文にも記載してありますが、それだと辞書に格納済みのリストの値まで消えてしまうのでだめです。

[otn]

失礼しました。タイトルと最終行に注意を奪われすぎました。