お互いに頼っているが同じ結果につながる2つの条件を単純化するには？

Question

私は整数年を受け取る関数を持っていますが、文字列'ALL'を渡すことができ、何かを返すこともできます。

私は今、このコードの醜い部分を持っている：

if type(year) != str or (type(year) == str and year.upper() != 'ALL'): 
    total_results = self.filterResultsByYear(total_results, year, year2) 

結果は現在の年に、デフォルトでフィルタリングされ、そして他の年によってフィルタリングすることができますが、ユーザーがフィルタリングないしたい場合、彼らが持っていますその年の「すべて」を渡す。

私が上記の忌み嫌いを書いた理由は、私がちょうどif year.upper() != 'ALL'を持っていれば、私は整数を渡すとTypeErrorが得られます。もし私がif type(year) != str and year.upper() != 'ALL'を置くと、私はまだ同じエラーが出ます。上記のコードは本当に醜いと私はそれをもっとpythonicにしたいと思った。これを行うにはどんなツールが必要ですか？ total_resultsとyear2があり、あなたがそれらを処理したいのか内容に応じて

Answer 1

：

try: 
    year = int(year) 
    total_results = self.filterResultsByYear(total_results, year, year2) 
except ValueError: 
    if not isinstance(year, (str, unicode)): 
     raise # Not string, unicode or coercible to an integer. 
    if year.lower() == 'all': 
     # Your logic here. 
    else: 
     # String but not 'all'. Exception handling. 

さようなら方法は、1がisinstance(object, class)またはisinstance(object, (class 1, class 2, ...))使用するクラスの等価性をチェックします。