関数のIf/else文が機能しない

Question

ユーザーとパスワードが等しい場合にログインボタンをクリックすると、「ログイン成功/失敗」テキストが表示されます。

私が抱えている問題は、ユーザーとパスワードが何であっても、失敗したメッセージの代わりに常に「成功」​​メッセージが表示されることです。

私は.getを使用して、ユーザーの入力を取得し、変数に保存しています。相続人

私のコード：

from tkinter import * 

root = Tk() 

label_name = Label(root, text="Username", fg="black") 
label_password = Label(root, text="Password", fg="black") 
input_name = Entry(root) 
name = input_name.get() 
input_password = Entry(root, show="*") 
password = input_password.get() 

label_name.grid(row=0, column=0, sticky=E) 
label_password.grid(row=1, column=0, sticky=E) 

input_name.grid(row=0, column=1) 
input_password.grid(row=1, column=1) 

checkbox = Checkbutton(root, text="Remember me") 
checkbox.grid(columnspan=2, sticky=W) 

login_button = Button(root, text="Login") 
login_button.grid(row=1, column=2) 


login_success = Label(root, text="Welcome", fg="black") 
login_fail = Label(root, text="login fail", fg="black") 


def login(event): 
    if name == password: 
     for widget in root.winfo_children(): 
      widget.grid_forget() 
     login_true = Label(root, text="Welcome", fg="black") 
     login_true.grid() 
    elif name != password: 
     for widget in root.winfo_children(): 
      widget.grid_forget() 
     login_false = Label(root, text="login fail", fg="black") 
     login_false.grid() 


login_button.bind("<Button-1>", login) 

root.mainloop() 

EDIT： は、私はまた、 "コマンド" 属性を使用しますが、同じ結果でした：

def login(): 
    name = input_name.get() 
    password = input_password.get() 
    if name == password: 
     for widget in root.winfo_children(): 
      widget.grid_forget() 
     login_true = Label(root, text="Welcome", fg="black") 
     login_true.grid() 
    elif name != password: 
     for widget in root.winfo_children(): 
      widget.grid_forget() 
     login_false = Label(root, text="login fail", fg="black") 
     login_false.grid() 


login_button = Button(root, text="Login", command=login) 
login_button.grid(row=1, column=2) 


root.mainloop() 

をEDIT2： は、新しい条件

を追加することによって修正されました

def login(): 

    if input_name.get() == "" or input_password.get() == "": 
     for widget in root.winfo_children(): 
      widget.grid_forget() 
     login_false = Label(root, text="login fail", fg="black") 
     login_false.grid() 
    elif input_name.get() == input_password.get(): 
     for widget in root.winfo_children(): 
      widget.grid_forget() 
     login_true = Label(root, text="Welcome", fg="black") 
     login_true.grid() 
    elif input_name.get() != input_password.get(): 
     for widget in root.winfo_children(): 
      widget.grid_forget() 
     login_false = Label(root, text="login fail", fg="black") 
     login_false.grid() 

Answer 1

それは常にtrueを返す理由があります。 1つはコードの本体に、もう1つは機能内にありますlogin()。 nameとpasswordは、login()の両方とも、割り当てられた値を持たないので、両方ともNoneと同じです。彼らは常に平等であり、常にTrueを返すことを意味します。

変更以下の機能：<Button-1>ため.bind()を使用して

def login(): 
    if input_name.get() == input_password.get(): 
     for widget in root.winfo_children(): 
      widget.grid_forget() 
     login_true = Label(root, text="Welcome", fg="black") 
     login_true.grid() 
    elif input_name.get() != input_password.get(): 
     for widget in root.winfo_children(): 
      widget.grid_forget() 
     login_false = Label(root, text="login fail", fg="black") 
     login_false.grid() 

---

Buttonウィジェットにはひどいアイデアになる傾向があります。サイドノートでは

from tkinter import * 

root = Tk() 

def login(): 
    print("Login triggered") 

login_button = Button(root, text="Login", command=login) 
login_button.pack() 

root.mainloop() 

---

、あなたは同時に任意のパラメータを渡す必要がある場合は、：Buttonウィジェットがクリックされたときにコールバックを得ると呼ばれるために使用することができcommandと呼ばれる属性が組み込まれていますこれを行う最も簡単な方法は、Pythonの無名関数lambdaです。これは以下のように行うことができます：

from tkinter import * 

root = Tk() 

def login(variable): 
    print("Login triggered") 

login_button = Button(root, text="Login", command=lambda: login("variable")) 
login_button.pack() 

root.mainloop() 

Answer 2

私は決してtkinterを使用しませんでしたが、あなたはca guiを初期化している間にgetメソッドが呼び出されます。 （name = input_name.get()） 代わりに、ユーザーがloginメソッド内のボタンをクリックしたときにのみこれを呼び出す必要があります。あなたは2つのローカル名前空間にnameとpasswordを宣言しているので、

from tkinter import * 

root = Tk() 

label_name = Label(root, text="Username", fg="black") 
label_password = Label(root, text="Password", fg="black") 
input_name = Entry(root) 
input_password = Entry(root, show="*") 

label_name.grid(row=0, column=0, sticky=E) 
label_password.grid(row=1, column=0, sticky=E) 

input_name.grid(row=0, column=1) 
input_password.grid(row=1, column=1) 

checkbox = Checkbutton(root, text="Remember me") 
checkbox.grid(columnspan=2, sticky=W) 

login_button = Button(root, text="Login") 
login_button.grid(row=1, column=2) 


login_success = Label(root, text="Welcome", fg="black") 
login_fail = Label(root, text="login fail", fg="black") 


def login(event): 
    name = input_name.get() 
    password = input_password.get() 
    if name == password: 
     for widget in root.winfo_children(): 
      widget.grid_forget() 
     login_true = Label(root, text="Welcome", fg="black") 
     login_true.grid() 
    elif name != password: 
     for widget in root.winfo_children(): 
      widget.grid_forget() 
     login_false = Label(root, text="login fail", fg="black") 
     login_false.grid() 


login_button.bind("<Button-1>", login) 

root.mainloop()