Python if-else mess

Question

msgをよりクリーンな方法で作成できますか？各イベント（Ping、Pullリクエスト、Issue、Issueコメント、Repo、Create、Delete、プルrequsetレビュー、Push、Commitコメント）には、イベントをチェックし、アクションに従ってメッセージを作成するif節があります。

data = request.json 
    event = request.headers['X-Github-Event'] 
    msg = "" 

    ... 

    # Pull request 
    elif event == "pull_request": 
     if data['action'] == "opened": 
      msg = PullRequest(data).opened() 
     elif data['action'] == "closed": 
      msg = PullRequest(data).closed() 
     elif data['action'] == "assigned": 
      msg = PullRequest(data).assigned() 

    # Issue 
    elif event == "issues": 
     if data['action'] == "opened": 
      msg = Issue(data).opened() 
     elif data['action'] == "reopened": 
      msg = Issue(data).reopened() 
     elif data['action'] == "closed": 
      msg = Issue(data).closed() 
     elif data['action'] == "labeled": 
      msg = Issue(data).labeled() 
     elif data['action'] == "assigned": 
      msg = Issue(data).assigned() 
    ... 

Answer 1

lambda:Noneの背後にある考え方は、それがdata['action']かの呼び出し可能なデフォルトの略ということである要約

elif event == "pull_request": 
    getattr(PullRequest(data), data['action'], lambda:None)() 
elif event == "issues": 
    getattr(Issue(data), data['action'], lambda:None)() 

で

getattr(PullRequest(data), data['action'], lambda:None)() 

は、実際にPullRequest(data)またはIssue(data)の方法ではありません。

---

それともif-else文が気に入らない場合は、

callables = { 
    "pull_request":PullRequest, 
    "issues":Issue, 
} 

getattr(callables[event](data), data['action'], lambda:None)() 

Answer 2

のようなものは、適切なオブジェクトに文字列をマップdict sの条件文を交換してください。 （これは、あなたのメソッドの名前と一致する必要はありません調べる気に文字列で、カナックが提案しているものの一般化である。）

functions = { 
    "pull_requests": { 
     "opened" : methodcaller("opened"), 
     "closed" : methodcaller("closed"), 
     "assigned" : methodcaller("assigned") 
    }, 
    "issues": { 
     "opened" : methodcaller("opened"), 
     "reopened" : methodcaller("reopened"), 
     "closed" : methodcaller("closed"), 
     "assigned" : methodcaller("assigned"), 
     "labeled": methodcaller("labeled") 
    } 
} 

classes = { 
    "pull_requests": PullRequest, 
    "issues": Issue 
} 

obj = classes[event] 
msg = functions[event][data['action']](obj)