Python3 Tkinter エントリの未入力チェックを実装したいです

Question

Tkinterでエントリを複数個作成します。今回は5個作成します。

ユーザはエントリにデータを入力します。

入力しなければならない個数は決まっていません。
ただし、最低1つは入力します。
入力は上のエントリから順番にします。

そのため、エラー（抜け防止）は

• データが1つも入力されていないとき
• 未入力エントリがあり、それより下のエントリに入力があるとき

に発生するようにしたいです。

例えば、下のようにエントリ[]に対し入力した場合

[a]
[]
[]
[d]
[]

2番目と3番目が入力されていないので、この2つはエラーの対象です。
最低1つ入力 かつ 抜けなしならいいので、上の例では5番目はエラーの対象になりません。

そのため、エラーは「2番目と3番目に入力漏れあり」と出るのが理想です。
まあ2番目だけでもいいです。
3番目だけっていうのは違う気がします（ユーザからしたら、一つずつのエラー表示なら「せめて上から指摘しろよ」って感じになりそう）。
テキストボックスの色を変えるとかでもいいです。
とりあえず「○番目」か分かればいいです。

一応判別するためのプログラムを自分で作ってはみたのですが、上からではなく下からチェックしてしまっているからか、上の例だと3番目のエラー表示になってしまいます。

それが下のコードです。
chk()の部分で抜け防止のためのチェックをしています。
一度 5個全てのエントリをget()してから、上から最下入力済みエントリまで比較する方法も考えてはみたのですが、処理が多いような気もしました。（get()してから比較すると2周してるため）

チェック部分の処理効率を重視した場合にほかにどんなチェック方法が考えられますか。

import tkinter as tk

def chk():
    global entries
    if entries[0].get() == "":
        print("1番目が未入力")
        return False
    for i in range(4):
        if entries[i].get() == "":
            if entries[i + 1].get() != "":
                print("{}番目が未入力".format(i + 1))
                return False
    return True

def pushed(e):
    if chk() == True:
        print("o：未入力なし")
    else:
        print("x：未入力あり")

win = tk.Tk()
win.geometry("300x200")

entries = []

for i in range(5):
    entry = tk.Entry(win)
    entry.grid()
    entries.append(entry)

button = tk.Button(win, text="click", width=10, default="active")
button.grid()

button.bind("<ButtonRelease>", pushed)

win.mainloop()

Answer 1

1,4 番目に入力がある場合、2,3番目をエラーとして報告する関数

サンプル入力: [1, 0, 0, 1, 0]
サンプル出力: [0, 1, 1, 0, 0]

となるような関数を考えると良いですよ。

Demo on repl.it

Comments

[person]

この方がソースを書くときに、直感的にわかりやすいですね。

Answer 2

疑問点

ただし、最低1つは入力します。

とありましたが、これはどれか一つという事でしょうか？
掲題のコードを見た感じだと1番目(1番上)のエントリーのみ絶対に入力を促している様にも伺えました。

そしてfor文でウィジェットをリストに格納するならば配置していない状態でリストに格納すべきです。
その後に再度for文で一つずつ配置してください。

def chk():
	global entries
	for i, entry in enumerate(entries):
		result = entry.get()
		if result == '':
			print('{}番目が未入力'.format(i+1))
			return False
	return True

entries = [tk.Entry(win) for _ in range(5)]
[entry.grid() for entry in entries]

またエントリーの色を変える場合には
pushed関数の中身も変えてしまう必要はありますが
chk関数内でreturnはさせずに以下にする事で色を変える事も出来ます。

def chk():
	global entries
	for i, entry in enumerate(entries):
		result = entry.get()
		if result == '':
			print('{}番目が未入力'.format(i+1))
			entry['background'] = 'pink'
		else:
			entry['background'] = 'white'

その他には、処理にもよりますが
エントリーへの入力が終わった際にボタンを押して次の処理を行うなどといった場合には、
予めボタンにはstate=disabledを設定しておき
Entryに対し「値が入力されていた場合にボタンのstate=enabledを付与する」
といった処理で関連付けることによって、チェックを可能にする方法も。

Comments

[person]

◆回答
>> ただし、最低1つは入力します。
> とありましたが、これはどれか一つという事でしょうか？
> 掲題のコードを見た感じだと1番目(1番上)のエントリーのみ絶対に入力を促している様にも伺えました。

エントリは上から順に入力するので、結果的に1番上のエントリは絶対に入力します。

◆質問
> for文でウィジェットをリストに格納するならば配置していない状態でリストに格納すべきです。
> その後に再度for文で一つずつ配置してください。

そうすべき理由は何でしょうか。

[nto]

>> そうすべき理由は何でしょうか。
失礼しました、コメント欄の為内包表記で失礼しますが
entries = [tk.Entry(win).grid() for i in range(5)] と
.grid()して配置した状態までをリストに格納しているのかと勘違いしておりました。
掲題のコードはリストへの格納と配置は別々で処理が行われていましたね。
entries = []
for i in range(5):
　　entry = tk.Entry(win).grid()
　　entries.append(entry)
と格納してしまうと後から.get()で入力した内容が取得できなくなってしまうからです。

[person]

これで取得できるのですが、ダメでしょうか？

import tkinter as tk

def returnPressed(e):
global entries
print("----------")
for i in range(len(entries)):
print(entries[i].get())
print("----------")

win = tk.Tk()

entries = []
for i in range(5):
entry = tk.Entry(win)
entry.grid()
entries.append(entry)

win.bind("<Return>", returnPressed)

win.mainloop()

[nto]

先に申し上げた通り、entry.grid()で配置した状態でリストに格納しているものと勘違いしており、掲題のコードでは配置していない状態でwidgetのみを格納している様ですので問題ありません。
万が一にentry.grid()を格納していた場合には取得ができなくなる為と記載しております。
お確かめください。

[person]

理解しました。