最終的な文字列変数の使用を避ける

Question

オーバーライドされたメソッドの1つのアプリケーションに対して、このGUIを作成しましたactionPerformedcommandにfinalを挿入する必要があります。この変数は内部クラスで使用されるためです。この問題を回避する方法はありますか？

@Override 
public void actionPerformed(ActionEvent e) 
{ 
    final String command = e.getActionCommand(); 

    if(command.equals("add")) 
    { 
     new Thread(new Runnable(){ 

      @Override 
      public void run() 
      { 
       // ... code skipped ... // 
       ous.writeObject(command); 
       // ... code skipped ... // 
      } 
     }).start(); 
    } 

    else if(command.equals("remove")) 
    { 
     new Thread(new Runnable(){ 

      @Override 
      public void run() 
      { 
       // ... code skipped ... // 
       ous.writeObject(command); 
       // ... code skipped ... // 
      } 
     }).start(); 
    } 
} 

私は、コードの先頭に

String command = null; 

を宣言する問題を解決しようとしました。

Answer 1

いいえ、これはJavaの機能です。クロージャの実装における限界です。クロージャ変数は最終的なものである必要があります。

Answer 2

Runnableを実装する別のクラスを作成し、コンストラクタにStringを渡して、そのオブジェクトをスレッド作成に使用できます。次のようになります。

class ThreadTask implements Runnable { 
    private String command; 

    public ThreadTask(String command) { 
     this.command = command; 
    } 

    public void run { 
     // 
    } 
} 

また、IMOのコードを読みやすくします。

Answer 3

まず、内部クラスにcommandは必要ありません。インスタンスを作成するまでには、確かにcommandの値が分かるので、実際の値を使用することもできます。

私は、明らかに反復されたコードを適切な非匿名クラスで抽出し、適切なcommandを渡します。それは、あなたがcommandについて最終的な心配を止めることができるように、デザインを十分に変更するでしょう。