Pythonでfizzbuzz。入力した値の型で条件分岐をしたい。

Question

前提・実現したいこと

python3でtkinterを使って、数字を入力するとfizzbuzzを判定するプログラムを作っています。

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

str,floatを入力、またはなにも価を入れないでボタンを押すと下記のようなエラーが出ます。
最後の''には入力したstr,floatが入ります。
値を入れない場合は''です。

Exception in Tkinter callback
Traceback (most recent call last):
  File "C:\Users\ryota01\Anaconda3\envs\tensorflow13\lib\tkinter\__init__.py", line 1699, in __call__
    return self.func(*args)
  File "tkfizzbuzz.py", line 36, in <lambda>
    Button1 = tkinter.Button(text=u'FizzBuzz', command=lambda: fizzbuzz(Entry1.get()))
  File "tkfizzbuzz.py", line 15, in fizzbuzz
    n = int(n)
ValueError: invalid literal for int() with base 10: ''

該当のソースコード

#!/usr/bin/env python
# -*- coding: utf-8 -*-

import tkinter
import sys

root = tkinter.Tk()
root.title(u'Fizz Buzz')

#definition
def fizzbuzz(n):
    fzbz = "Fizz Buzz"
    fz = "Fizz"
    bz = "Buzz"
    n = int(n)
    if n % 15 == 0:          
        resltfizbuz(fzbz)
    elif n % 3 == 0:
        resltfizbuz(fz)
    elif n % 5 == 0:
        resltfizbuz(bz)
    else:
        resltfizbuz(n)
    Entry1.delete(0,tkinter.END)

def resltfizbuz(reslt):
    Listbox1.insert(tkinter.END,reslt)


Entry1 = tkinter.Entry(width=20)
Entry1.pack()

Listbox1 = tkinter.Listbox()
Listbox1.pack()

Button1 = tkinter.Button(text=u'FizzBuzz', command=lambda: fizzbuzz(Entry1.get()))
Button1.pack()

root.mainloop()

試したこと

nがintの時だけfizzbuzzを判定させるために、fizzbuzz(n)を下記のように変えると何を入力してボタンを押しても反応しなくなりました。

intの場合はizzzbuzzを判定し、それ以外はEntry1.delete(0,tkinter.END)を処理するようにするにはどうすればいいですか？

def fizzbuzz(n):
    fzbz = "Fizz Buzz"
    fz = "Fizz"
    bz = "Buzz"
    if type(n) == int:
        n = int(n)
        if n % 15 == 0:          
            resltfizbuz(fzbz)
        elif n % 3 == 0:
            resltfizbuz(fz)
        elif n % 5 == 0:
            resltfizbuz(bz)
        else:
            resltfizbuz(n)
    else:
        return 0
    Entry1.delete(0,tkinter.END)

補足情報（FW/ツールのバージョンなど）

ここにより詳細な情報を記載してください。

Answer 1

nが空のときにスキップしたいのなら、if n == '': returnすればよいです。

nがintの時だけfizzbuzzを判定させるために、fizzbuzz(n)を下記のように変えると何を入力してボタンを押しても反応しなくなりました。

あくまでもnは文字列ですので、type(n) == intは常に偽です。
次のように書くのはありですね。

def fizzbuzz(n):
    ...
    try:
        n = int(n)
    except ValueError:
        return;

    ...
    # これ以降はnが整数であることを確信して書いてよい

Comments

[mkgrei]

正規表現を使うと元のコードっぽいことができますが、try-exceptの方が考えることが少なくていいですね。

[LouiS0616]

Pythonのカジュアルに例外を投げまくる姿勢は慣れるまでは違和感ありましたが、やっぱ楽ですね。

[sai99]

思い通りの職ができました。ありがとうございます。