pythonで型のエラー

Question

前提・実現したいこと

pythonで自動販売機のモデルみたいなのを作っています。しかしinputの型はstring だと知っていても型についてのエラーが出てしまいます。どうしたらよろしいでしょうか？

発生している問題・エラーメッセージ

---------------------------------------------------------------------------
NameError                                 Traceback (most recent call last)
<ipython-input-4-5d5168a40431> in <module>
     38             return returnOrNot
     39 
---> 40 moneyPut = decide('a')
     41 # __________________________________________________________________
     42 #自動販売機の飲み物の種類をタプル型で表示

NameError: name 'decide' is not defined

該当のソースコード

from datetime import date
#todayに日付を入れる
today = date.today()
#Drinkクラスを作成
class Drink:
    def __init__(self, name, price, count):
        self.name = name
        self.price = price
        self.count = count
        self.lamp = False
#お金を返却させるための関数
    def returnMoney(self, moneyToReturn):
        if int(moneyPut) > 1:
            return  str(moneyToReturn) + "円が返却されます。"
            self.moneyPut = '0'
#ランプ表示の関数
    def SoldOutLamp(self):
        if self.count == 0:
            self.lamp = True
        else:
            self.lamp = False
            
        if self.lamp == False:
            return 'can buy'
        else:
            return 'sold out'
    def decide(self, i):
        if i == 'a':
            moneyPut = int(input('put money'))
            return moneyPut
        elif i == 'b':
            liquid = int(input('what to drink? 0:cola, 1:water, 2:tea'))
            return liquid
        else:
            returnOrNot=int(input('返金しますか？　0:Yes, 1:No  :'))
            return returnOrNot  

moneyPut = decide('a')
# __________________________________________________________________
#自動販売機の飲み物の種類をタプル型で表示
cola = Drink('cola', 150, 40)
water = Drink('water', 100, 3)
tea = Drink('tea', 140, 0)

#これらをdrinksでまたタプルにする
drinks = (cola, water, tea)
#ここでお金を入れる
# decide('a')

#ここで飲み物を選ぶ
liquid = decide('b')

やりたいこと

●目標_______________________________
1. 必要金額が投入されている場合に、飲み物ボタンが押されると缶を出す
2. 売切れランプの追加（売切れの時は購入できない）
もし可能ならなぜエラーが出てきてしまうのか教えてもらうことって可能でしょうか？
本当にお願いします！
またクラスの中の関数を使って出されたliquidという数字を使ってdrinks[liquid].name のような事ってできますか？

### 補足情報（FW/ツールのバージョンなど）
python 3.73です。

Comments

[can110]

「NameError: name 'decide' is not defined」エラーが発生します。
第三者が実行、検証できるコードを提示ください。

[t_obara]

自動販売機モデルというのであれば、自動販売機クラスする方が分かりやすいかと思います。各Drinkの個数や自動販売機に投入された金額の把握、何が購入されたかなどをそのクラスに管理させるのが良いかと思います。
また、decideを引数で処理を分ける理由が無いですし、名称と処理内容に齟齬が出るのでは無いでしょうか。3つの関数にした方が分かりやすくなるかと思います。

Answer 1

おそらく，以下のliquid変数がNoneであることが原因であると思われます．

liquid = decide('b')

decide関数をみてみると，標準入力から数字を受け取った際に何も値を返していないので，ここを修正することで正しい挙動に近づくと思われます．
以下のように関数で何らかの値を返すように変更してみてください．

def decide(self, i):
  """
  何らかの処理
  """
  
  return liquid

Comments

[NatsukiKataoka]

from datetime import date
#todayに日付を入れる
today = date.today()
#Drinkクラスを作成
class Drink:
def __init__(self, name, price, count):
self.name = name
self.price = price
self.count = count
self.lamp = False
#お金を返却させるための関数
def returnMoney(self, moneyToReturn):
if int(moneyPut) > 1:
return str(moneyToReturn) + "円が返却されます。"
self.moneyPut = '0'
#ランプ表示の関数
def SoldOutLamp(self):
if self.count == 0:
self.lamp = True
else:
self.lamp = False

if self.lamp == False:
return 'can buy'
else:
return 'sold out'

def decide(self, i):
if i == 'a':
moneyPut = int(input('put money'))
return moneyPut
elif i == 'b':
liquid = int(input('what to drink? 0:cola, 1:water, 2:tea'))
return liquid
else:
returnOrNot=int(input('返金しますか？　0:Yes, 1:No :'))
return return

# class ForReceipt:
# def __init__(self, name, price, count):
# super().__init__(name, price, count) #親クラスのコンストラクタを実行

class ReceiptFor(Drink):
def message1(self):
message1 = (drinks[liquid].name + 'を買うためにお金を' + str(moneyPut) + "入れました。")
return message1
def message2(self, letter):
if letter == 'a':
message2 = ('しかし'+ drinks[liquid].name +'を買うためのお金が足りないため買えませんでした。レシートは以上です。')
return message2
elif letter == 'b':
message2 = ("あなたの残高は{}円です。".format(str(moneyPut)))
return message2
def message3(self, letter):
if letter == 'a':
message3 = 'なにもなし'
return message3
else:
message3 = ('しかし'+ drinks[liquid].name +'を買うための在庫がないため買えませんでした。レシートは以上です。')
return message3
def message4(self, letter):
if letter == 'a':
message4 = 'なにもなし'
return message4
else:
message4 = (returnMoney(moneyPut))
return message4

moneyPut = (decide('a'))
# __________________________________________________________________
#自動販売機の飲み物の種類をタプル型で表示
cola = Drink('cola', 150, 40)
water = Drink('water', 100, 3)
tea = Drink('tea', 140, 0)

#これらをdrinksでまたタプルにする
drinks = (cola, water, tea)
#ここでお金を入れる
# decide('a')

#ここで飲み物を選ぶ
liquid = decide('b')

すみませんこれでもうまくいかなくて....もし構わなければ教えてもらう事ってきますか？

[noconocolib]

手元ではどういったエラーになりますか？
ちなみにdecide関数がDrinkクラスの中に定義されてしまっているので，このコードの呼び出し方だとNameError: name 'decide' is not definedなどと言ったエラーになりそうです．

これは，decide関数をDrinkクラスの外で定義することで回避できそうです．

Answer 2

def decide()で値(liquid、moneyPut)を返していないのでエラーになっていると思います。