パラメータを持たないメソッドを含むリファクタリングswitch文

Question

現在、私のコードをリファクタリングしてブロックされています。私はswitchステートメントをリファクタリングできることを知っていますが、私はそうする経験が不足しています:(パターンの大部分とスイッチを扱う方法の大部分は、メソッドの異なるパラメータの管理に焦点が当てられています。私はそれについて何ができるかのアイデアは？本当に任意のヒントをお願い申し上げます。

public void manageMenuAction(MenuItem item, GiftListPresenterViewConnector activityConnector) { 
    switch (item.getItemId()) { 
     case R.id.nav_your_account: 
      activityConnector.goToEditAccount(); 
      break; 
     case R.id.help: 
      activityConnector.goToHelpScreen(); 
      break; 
     case R.id.nav_about: 
      createInfoDialog(); 
      break; 
     case R.id.nav_exit: 
      activityConnector.exitApp(); 
      break; 
     case R.id.nav_logout: 
      clearLocalData(); 
      activityConnector.logout(); 
      break; 
    } 
} 

を私はjavaの1.7での作業、アンドロイドをコードしています。GiftListPresenterViewConnectorがインタフェースである。

Answer 1

を理論的に、あなたはジェネリックを実装できますMenuItem.idの動作でMenuItemがこの汎用インスタンスを保持し、manageMenuActionに入るとMenuItem.getItemId().executeBehavior()。

（あなたはまた、代わりにidの一般的なインスタンスを保持するのジェネリックMenuItemを作ることができる）

このidはおそらく単純なIntegerで、スイッチケースはおそらくだろうので、正直なところ、私はそれをしないだろうそれを行う最も単純な方法

Answer 2

私に際立っている項目は、activityConnectorを通過しない唯一のパスだからcreateInfoDialogです。 GiftListPresenterViewConnectorのメソッドを作成する場合は、このメソッド全体をそのクラスに移動することができ、それを引数として取る必要はありません。しかし私は他のコメント作成者に同意します。このメソッドは正常に見えます。

Answer 3

もう1つの選択肢として、地図を使用することができます。

1. キーは
2. 値が

が次にあなたが

callbacks.get(item.getItemId()).doCallback(); 

のようなコードを書くことができ、いくつかの一般的なコールバックオブジェクトになり、あなたのアイテムIDになるしかし、それもそうですビット "far-fetched";おそらくあまり多くの "アンドロイド様式"ではありません。

Answer 4

'command'デザインパターンの使用を検討してくださいhttps://en.wikipedia.org/wiki/Command_pattern