Python if文の複数条件式をスマートにする方法

Question

以下のようなif文における複数の条件式をスマート（if文が長くならないように、見やすく、またmodel、factoryの種類が増減した際、容易に修正できるように）にするためにリスト参照の他に良い方法はありますでしょうか？
※PythonのVersionの制約は2.6になります。

model = 'A001'
factory = 'Osaka'

#if文に複数の条件式が含まれる。今後もmodel、factoryの種類は増える。
if model == 'A001' or model == 'B001' or model == 'C001':
    print('good quality model')
else:
    print('bad quality model')

if model == 'A001' or model == 'B001':
     print('new model')
else:
    print('old model')

if model == 'A001':
    print('high price')
else:
    print('low price')

if model == 'A001' and (factory == 'Osaka' or factory == 'Tokyo'):
    print('zaiko a few')
else:
    print('zaiko a lot of')

#if文の複数条件をリスト参照によりスマートにする。model、factoryが増える場合、リストに追加する。
good_quality_list = ['A001','B001','C001']
new_model_list = ['A001','B001']
high_price_list = ['A001']
zaiko_a_few_list = ['Osaka','Tokyo']

if model in good_quality_list:
    print('good quality model')
else:
    print('bad quality model')

if model in new_model_list:
     print('new model')
else:
    print('old model')

if model in high_price_list:
    print('high price')
else:
    print('low price')

if model in high_price_list and (factory in zaiko_a_few_list):
    print('zaiko a few')
else:
    print('zaiko a lot of')

ご教授の程、よろしくお願いいたします。

Comments

[chirimen]

現状のコードのどういう点が「スマートでない」と感じていて、どのような方向で改善したいと考えているかについて、考えているところを書いた方がよいでしょう。

model や factory の増え方や管理方法、条件が増えるのかどうか、にもよるかと思います。

[Zuishin]

ハードコーディングするのではなく、データとしてデータベースやテキストファイルなど外部で提供するのが良いと思います。

[Zuishin]

質問が編集されましたが、スマートでないのは if でハードコーディングされているからです。それに拘る限り、ダサくて保守性が低くなるのはどうしようもありません。あきらめてください。

Answer 1

リストではなく辞書になりますが、以下のようにすることも可能です。

Model = {   "A001":["GQ","NM","HP"],
            "B001":["GQ","NM","LP"],
            "C001":["GQ","OM","LP"],
            "D001":["BQ","OM","LP"],
        }
Zaiko = {   "A001-Osaka":"FZ",
            "A001-Tokyo":"FZ",
            "A001-Nagoya":"LZ",
            "B001-Osaka":"LZ",
            "B001-Tokyo":"LZ",
            "B001-Nagoya":"LZ",
            "C001-Osaka":"LZ",
            "C001-Tokyo":"LZ",
            "C001-Nagoya":"LZ",
            "D001-Osaka":"LZ",
            "D001-Tokyo":"LZ",
            "D001-Nagoya":"LZ",
        }

Abbr = {    "GQ":"good qualty model",
            "BQ":"bad quality model",
            "NM":"new model",
            "OM":"old model",
            "HP":"high price",
            "LP":"low price",
            "FZ":"zaiko a few",
            "LZ":"zaiko a lot of",
        }


def print_model(model,factory):
    global Model
    global Zaiko
    global Abbr
    if model not in Model:
        print(model + " not found")
        return
    key = model + "-" + factory
    if key not in Zaiko:
        print(key + " not found")
        return
    print("model=" + model + " factory=" + factory)
    ab = Model[model][0]    #qualty
    print(Abbr[ab])
    ab = Model[model][1]    #new or old
    print(Abbr[ab])
    ab = Model[model][2]    #price
    print(Abbr[ab])
    ab = Zaiko[key]         #zaiko
    print(Abbr[ab])



model = "A001"
factory = "Osaka"
print_model(model,factory)

model = "B001"
factory = "Tokyo"
print_model(model,factory)

実行結果
model=A001 factory=Osaka
good qualty model
new model
high price
zaiko a few
model=B001 factory=Tokyo
good qualty model
new model
low price
zaiko a lot of
（Python 2.7.5で確認。2.6では確認していません。）

尚、Zuishinさんが指摘されているように、データをテキストファイル（CSV）に格納し、
それを読み込んで、辞書に格納するのが、汎用的な方法になるかと思います。
その場合、各辞書対応のCSVファイルはいかのように定義すれば良いかと。（一例です）

Model用
#model,quality,new or old,price
A001,GQ,NM,HP
B001,GQ,NM,LP
C001,GQ,OM,LP
D001,BQ,OM,LP

Zaiko用
#model,Osaka,Tokyo,Nagoya
A001-Osaka,FZ
A001-Tokyo,FZ
A001-Nagoya,LZ
B001-Osaka,LZ
B001-Tokyo,LZ
B001-Nagoya,LZ
C001-Osaka,LZ
C001-Tokyo,LZ
C001-Nagoya,LZ
D001-Osaka,LZ
D001-Tokyo,LZ
D001-Nagoya,LZ

Abbr用
#abbr,full
GQ,good quality model
BQ,bad quality model
NM,new model
OM,old model
HP,high price
LP,low price
FZ,zaiko a few
LZ,zaiko a lot of