書籍『退屈なことはPythonにやらせよう』の演習問題(P127,5.6.2)からの質問

Question

#書籍『退屈なことはPythonにやらせよう』の演習問題(P127,5.6.2)からの質問です。(下に原文アリ)

書籍『退屈なことはPythonにやらせよう』の演習問題(P127,5.6.2)からの質問で、エラー(AttributeError: 'NoneType' object has no attribute 'items')の直し方が分からず困っています。

#演習問題は以下です(原文ママ)
ファンタジーゲームの持ち物リスト用にリストから辞書に移す関数ドラゴンに勝って獲得したアイテムを、次のような文字列のリストで表すとします。

dragon_loot = ['金貨', '手裏剣', '金貨', '金貨', 'ルビー']

持ち物リストにアイテムを追加する add_to_inventory(inventory, added_items) という関数を書きなさい。ただし、引数inventoryは、前述のようなプレイヤーの持ち物リストを表し、引数 added_items は dragon_loot のようなリストだとします。関数 add_to_inventory() は、更新した辞書を返すようにしてください。リスト added_items には、同じアイテムが複数回出現しうることに注意しましょう。次のようなコードになると思います。

def add_to_inventory(inventory, added_items):
＃ここにコードを書きます。
inv = {'金貨': 42, 'ロープ': 1}
dragon_loot = ['金貨', '手裏剣', '金貨', '金貨', 'ルビー']
inv = add_to_inventory(inv, dragon_loot)
display_inventory(inv)

前述の display_inventory (※後述)を使えば、次のように表示されるでしょう。

持ち物リスト:
45 金貨
1　ロープ
1　ルビー
1 手裏剣
アイテム総数: 48

#エラーの出ているプログラム
これが私の書いたプログラムです。
原文中の金貨をgold medals, 手裏剣をdirks, ルビーをrubies, ロープをropesと置き換えて書いています。

def add_to_inventory(inventory, added_items):
    for added_items in dragon_loot:
        inventory.setdefault(added_items, 0)
        inventory[added_items] = inventory[added_items] + 1

def display_inventory(inventory):
    print('List of My Inventory:')
    item_total = 0
    for k, v in inventory.items():
        print(str(v) + ' ' + k)
        item_total = item_total + v
    print('Item Total: ' + str(item_total))

dragon_loot = ['gold medals', 'dirks', 'gold medals', 'gold medals', 'rubies']
inv = {'gold medals': 42, 'ropes': 1}

inv = add_to_inventory(inv, dragon_loot)
display_inventory(inv)

これを出力すると、以下のようになってしまいます。

List of My Inventory:
Traceback (most recent call last):
File "C:\Users\ifari\Desktop\python\test#3.py", line 19, in <module>
display_inventory(inv)
File "C:\Users\ifari\Desktop\python\test#3.py", line 10, in display_inventory
for k, v in inventory.items():
AttributeError: 'NoneType' object has no attribute 'items'

※『演習問題は以下です(原文ママ)』で(※後述)とした display_inventory のコードは、プログラム中の

def display_inventory(inventory):
    print('List of My Inventory:')
    item_total = 0
    for k, v in inventory.items():
        print(str(v) + ' ' + k)
        item_total = item_total + v
    print('Item Total: ' + str(item_total))

inv = {'gold medals': 42, 'ropes': 1}

display_inventory(inv)

の部分で、こちらは正常に作動しました。

#別パターンで正常に出力されたプログラム
こちらも私が書いたプログラムで、書籍中で示されていた形とは違うパターンで書いてみたプログラムなのですが、こちらは正常に

持ち物リスト:
45 金貨
1　ロープ
1　ルビー
1 手裏剣
アイテム総数: 48

と出力されました。

def add_to_inventory(inventory, added_items):
    for added_items in dragon_loot:
        inventory.setdefault(added_items, 0)
        inventory[added_items] = inventory[added_items] + 1
    print('List of My Inventory:')
    item_total = 0
    for k, v in inventory.items():
        print(str(v) + ' ' + k)
        item_total = item_total + v
    print('Item Total: ' + str(item_total))

dragon_loot = ['gold medals', 'dirks', 'gold medals', 'gold medals', 'rubies']
inv = {'gold medals': 42, 'ropes': 1}

add_to_inventory(inv, dragon_loot)

しかし、書籍中で示されていた形とは違うので、もし書籍中で示されていた形で正しくプログラムされていたらどういったものになるのか、教えていただきたいです。

Comments

[y_waiwai]

このままではコードが読めないので、質問を編集し、<code>ボタンを押し、出てくる’’’の枠の中にコードを貼り付けてください

Answer 1

処理の内容は追ってませんが、エラーの原因は問題文より、

＞関数 add_to_inventory() は、更新した辞書を返すようにしてください。

原因: add_to_inventory()が戻り値を返していないので、invがNoneになる

inv = add_to_inventory(inv, dragon_loot)

---

解決策:

add_to_inventory関数の末尾で、inventory辞書を返すとエラーはなくなります。

return inventory

もしくは、辞書を返さないのなら
問題には背きますが、inv = で変数を上書きしないようにする。

# 辞書自体は更新されているので、正常なデータが表示されます。

add_to_inventory(inv, dragon_loot)
display_inventory(inv)

Comments

[XocvOrmaz]

それで解決しました!! 本当にありがとうございました!!