Python3 Tkinter 決まった形式の文字列のみ入力可能にしたい

Question

"No.*"という形式の文字列のみエントリに入力できるようにしたいです。
後からエントリの文字列を拾って比較するというよりは、形式が違えば入力そのものをできないようにしたいです。

とりあえず、期待動作できるようにはなりましたが、ほかにいい方法はありますか？

スマートな方法であれば、そちらを採用したいです。
また、以下のやり方で良くない部分があれば指摘してほしいです。

import tkinter as tk
from tkinter import ttk

def func(p):
    # エントリにインサートする文字列を "No.*" に制限（*は0文字以上の任意の文字）。
    # IndexError を起こさないために、len(p) >= n and p[n-1] == "*" とする。
    # 上記について、and は前者がFalseの場合、後者を見ないため。
    # ちなみに、or は前者がTrueの場合、後者を見ない。

    jouken = \
        p == "" or \
        len(p) == 1 and p[0] == "N" or \
        len(p) == 2 and p[1] == "o" or \
        len(p) == 3 and p[2] == "." or \
        len(p) > 3 and True

    return jouken

if __name__ == "__main__":
    win = tk.Tk()
    win.geometry("300x300+0+0")

    sv = tk.StringVar()

    vcmd = win.register(func)

    entry = ttk.Entry(win, textvariable=sv, validatecommand=(vcmd, "%P"), validate="key")
    entry.grid()

    win.mainloop()

# 参考：https://python.keicode.com/advanced/tkinter-widget-entry-validate.php

Answer 1

以下のやり方で良くない部分があれば指摘してほしいです。

良くないというか、改善可能なポイントはあります。

• len() が1文字入力毎に、毎回最大4回呼ばれる (改善可能な点)
• 検査する文字列を変更する時に、コードを変更することになる。(保守性の問題）

---

• 入力文字が prefix で始まる → 入力文字の startswith メソッドで検査
• prefix の入力中 ("N", "No") は、prefix.startswith で検査

def func(p, prefix="No."):
    return p.startswith(prefix) or prefix.startswith(p)


if __name__ == "__main__":

    import unittest

    class TestInputValidataion(unittest.TestCase):
        def test_inputNo_ok(self):
            self.assertTrue(func(""))
            self.assertTrue(func("N"))
            self.assertTrue(func("No"))
            self.assertTrue(func("No."))
            self.assertTrue(func("No.100"))

        def test_inputNo_ng(self):
            self.assertFalse(func("no."))
            self.assertFalse(func("No,"))
            self.assertFalse(func(" No."))
    
    unittest.main()

• 単体テストを作っておくことで、変更時のテストがやり易くなります。

続けて番号入力も検査するなら、他には正規表現もありますが、
入力中の状態の表現が少し複雑になりがちなので、
prefix.startswith での検査方法と合わせて使うと良いです。

• check point: プレフィクスの "No." の表現を

　別のモノに変えたい場合、変更が一カ所で済む。

Comments

[person]

回答ありがとうございます。

関数（len()）が複数回呼ばれる点については考えていませんでした。
変数に入れると、定義した変数分メモリなどを消費すると思ったので使わなかったのですが、関数を複数回呼ぶと微小とはいえ時間がかかりそうですね。以後、気をつけようと思います。

startswith()については盲点でした。
nto様のご指摘にあるカーソル移動時の入力問題についてもこれで対応できそうなので、startswith()を使うことにします。

Answer 2

現状だと、No.と入力した後、1~3文字目にカーソルを合わせると
その他の文字に自由に変更できる様な状態になってしまっている為
厳格に先頭の「No.」を固定させたいならばもう少し条件の見直しが必要かもしれません。

	if len(p) <= 3:
		if p in ['N', 'No', 'No.']:
			return True
		else:
			return False
	else:
		if p[0:3] == 'No.':
			return True
		else:
			return False

Comments

[person]

回答ありがとうございます。
確かにカーソル移動させた時に違う文字が打てたり、その後末尾から文字を消していくと消せなくなったりするのは問題ですね。

teamikl様の回答にあるstartswith()を使って、そうならないようにします。