tkinterで計算結果が表示されるが、答えが間違っている

Question

環境　
Windows１０
python
anaconda

tkinterで計算させる簡易的なプログラムを作っています。
作っては見たのですが、二つ目の計算の部分が実際に電卓をたたいて導いた回答とは違う回答になってしまいます。
変数番号が違うのでしょうか？
ご教授お願い致します。

import tkinter as tk

# ボタンを押したときの処理 --- (*1)
def calc_bmi():
    # 計算
    h = float(textHeight.get()) 
    w = float(textWeight.get())
    bmi = (1000* h) / (3.14*w)
    
    per = int(bmi)
    # 結果をラベルに表示
    s = " 回転速度 ｓ{0} ".format(per)
    labelResult['text'] = s




# ウィンドウを作成 --- (*2)
win = tk.Tk()
win.title("送り・回転速度・計算")
win.geometry("300x300")

# 部品を作成 --- (*3)
labelHeight = tk.Label(win, text=u'周速:')
labelHeight.pack()

textHeight = tk.Entry(win)
textHeight.insert(tk.END, '200')
textHeight.pack()

labelWeight = tk.Label(win, text=u'刃物径:')
labelWeight.pack()

textWeight = tk.Entry(win)
textWeight.insert(tk.END, '10')
textWeight.pack()

labelResult = tk.Label(win, text=u'---')
labelResult.pack()

calcButton = tk.Button(win, text=u'計算')
calcButton["command"] = calc_bmi
calcButton.pack()

# ウィンドウを動かすwin.mainloop()


#ここから自作プログラム

def calc_gg():
    # 計算
    f = float(textHeight1.get()) 
    z = float(textWeight1.get())
    n = float(textWeight1.get())

    fk = f * z * n
    
    gg = int(fk)
    # 結果をラベルに表示
    t = " 送り速度 F{0} ".format(gg)
    labelResult1['text'] = t

#部品
labelWeight1 = tk.Label(win, text=u'1刃当たりの送り:')
labelWeight1.pack()

textHeight1 = tk.Entry(win)
textHeight1.insert(tk.END, '0.1')
textHeight1.pack()

labelWeight1 = tk.Label(win, text=u'刃数:')
labelWeight1.pack()

textHeight1 = tk.Entry(win)
textHeight1.insert(tk.END, '2')
textHeight1.pack()

labelWeight1 = tk.Label(win, text=u'回転数:')
labelWeight1.pack()

textHeight1 = tk.Entry(win)
textHeight1.insert(tk.END, '5000')
textHeight1.pack()

#計算結果

labelResult1 = tk.Label(win, text=u'---')
labelResult1.pack()

calcButton1 = tk.Button(win, text=u'計算')
calcButton1["command"] = calc_gg
calcButton1.pack()


# ウィンドウを動かす
win.mainloop()

Comments

[magichan]

二つ目の計算　は具体的にどのような式で導かれるのでしょうか？
[1刃当たりの送り] x [回転数] x [回転数]
ですか？

[town]

2つ目は、1刃当たりの送り×刃の数×回転数です！

Answer 1

変数番号が違うのでしょうか？

はいそのとおりです。
間違いは多々ありまして

１．一刃あたりの送りと刃数と回転数を入力するEntry Boxが全てtextHeight1という変数を使用しているので、前者２つが上書きされる（結果 textHeight には回転数が入っております）
２．にも関わらず、f(一刃あたりの送り)をtextHeight1 から取得している
３．のこり２つのzとnをtextWeight1から取得しようとしているが、そんな名前の変数はない

あたりを直す必要があります。

まず変数ですが、変数の名前と実際の機能と合っているようには見えませんので、その辺を直しつつ　LabelやEntry Boxが格納されている変数名がユニークになるように（変数名を考えるのが大変なのであればとりあえずはLabel1, Label2のような連番の数値でもよいので）修正してみてください。

---

【一応変更例を追記】
とりあえず、ラベルが格納されている変数を label+数値 EntryBoxが格納されている変数を text+数値としてコードを修正したサンプルを載せておきますので参考にしてください。

import tkinter as tk

# ボタンを押したときの処理 --- (*1)
def calc_bmi():
    # 計算
    h = float(textHeight.get())
    w = float(textWeight.get())
    bmi = (1000* h) / (3.14*w)

    per = int(bmi)
    # 結果をラベルに表示
    s = " 回転速度 ｓ: {0} ".format(per)
    labelResult['text'] = s

# ウィンドウを作成 --- (*2)
win = tk.Tk()
win.title("送り・回転速度・計算")
win.geometry("300x400")

# 部品を作成 --- (*3)
labelHeight = tk.Label(win, text=u'周速:')
labelHeight.pack()

textHeight = tk.Entry(win)
textHeight.insert(tk.END, '200')
textHeight.pack()

labelWeight = tk.Label(win, text=u'刃物径:')
labelWeight.pack()

textWeight = tk.Entry(win)
textWeight.insert(tk.END, '10')
textWeight.pack()

labelResult = tk.Label(win, text=u'---')
labelResult.pack()

calcButton = tk.Button(win, text=u'計算')
calcButton["command"] = calc_bmi
calcButton.pack()

# ウィンドウを動かすwin.mainloop()


#ここから自作プログラム

def calc_gg():
    # 計算
    f = float(text1.get())
    z = float(text2.get())
    n = float(text3.get())

    fk = f * z * n

    gg = int(fk)
    # 結果をラベルに表示
    t = " 送り速度 F: {0} ".format(gg)
    labelResult1['text'] = t

#部品
label1 = tk.Label(win, text=u'1刃当たりの送り:')
label1.pack()

text1 = tk.Entry(win)
text1.insert(tk.END, '0.1')
text1.pack()

label2 = tk.Label(win, text=u'刃数:')
label2.pack()

text2 = tk.Entry(win)
text2.insert(tk.END, '2')
text2.pack()

label3 = tk.Label(win, text=u'回転数:')
label3.pack()

text3 = tk.Entry(win)
text3.insert(tk.END, '5000')
text3.pack()

#計算結果

labelResult1 = tk.Label(win, text=u'---')
labelResult1.pack()

calcButton1 = tk.Button(win, text=u'計算')
calcButton1["command"] = calc_gg
calcButton1.pack()

# ウィンドウを動かす
win.mainloop()

Comments

[town]

詳しくありがとうございます！！これを参考にしながら、復習したいと思います！
ちなみになんですが、これだけ変数があると頭が混乱してしまうのですが、どう言ったことに注意しながら進めているのでしょうか？

テキストエディタはVisualStudioを使っています。

[magichan]

今回は変数名がユニークになること第一にした為、とりあえず変数名末に数値を振りましたが、基本的には「わかりやすい名前をつける」ことが一番かと思います。
変数名や関数名などは、言語毎・所属する団体毎・作者毎に様々な命名規則や命名ポリシーが存在する為ここで簡単に述べることはできませんが、例えば '刃数'のラベル格納する変数名を”label_for_blade_number”などにすることで、第３者がコードを見た際にこの変数が何のための変数なのかが一目でわかるようになりますので、作者本人も混乱なくコードを実装できるのではないかと思います。