が、私はこのようになります画面を持って冗長

Question

を取り除くために

public class PowersScreen extends Screen { 

public PowersScreen(Game game) { 
    super(game); 
    // TODO Auto-generated constructor stub 
} 

Graphics g = game.getGraphics(); 

public static int choice1 = 3; 

int canvasW = g.canvasWidth(); 
int canvasH = g.canvasHeight(); 
Point centerOfCanvas = new Point(canvasW/2, canvasH/2); 
int rectW = 100; 
int rectH = 100; 
int left = centerOfCanvas.x - (rectW/2); 
int top = centerOfCanvas.y - (rectH/2); 
int right = centerOfCanvas.x + (rectW/2); 
int bottom = centerOfCanvas.y + (rectH/2); 

static Image chosen_one; 

static String chosen_text = "hi"; 

int left_side_popcornX = g.canvasWidth()/4 - (Assets.happyPopcorn.getWidth()/2); 
int right_side_popcornX = 3 * g.canvasWidth()/4 - (Assets.happyPopcorn.getWidth()/2); 

static String back = App.getContext().getString(R.string.back); 
static String next = App.getContext().getString(R.string.next); 
static String choose_pwrup_1 = App.getContext().getString(R.string.choose_pwrup_1); 
static String pwr1 = App.getContext().getString(R.string.power1); 
static String pwr2 = App.getContext().getString(R.string.power2); 
static String pwr3 = App.getContext().getString(R.string.power3); 
static String pwr4 = App.getContext().getString(R.string.power4); 
static String pwr5 = App.getContext().getString(R.string.power5); 
static String pwr6 = App.getContext().getString(R.string.power6); 
static String pwr7 = App.getContext().getString(R.string.power7); 
static String pwr8 = App.getContext().getString(R.string.power8); 

Rect rect1 = new Rect(0, 0, g.canvasWidth(), g.canvasHeight()/8); 
Rect rect2 = new Rect(left, top+100, right, bottom+100); 
Rect rect3 = new Rect(left, top-100, right, bottom-100); 
Rect info = new Rect(left_side_popcornX-10, (6 * g.canvasHeight()/8), right_side_popcornX + Assets.happyPopcorn.getWidth() + 10, (7 * g.canvasHeight()/8)); 

@Override 
public void update(float deltaTime) { 
    Graphics g = game.getGraphics(); 
    List<TouchEvent> touchEvents = game.getInput().getTouchEvents(); 

    int len = touchEvents.size(); 
    for (int i = 0; i < len; i++) { 
     TouchEvent event = touchEvents.get(i); 
     if (event.type == TouchEvent.TOUCH_UP) { 

     int x2 = event.x; 
     int y2 = event.y; 

     if (rect2.contains(x2, y2)) { 
      game.setScreen(new MainMenuScreen(game)); 
     } 

     if (rect3.contains(x2, y2)) { 
      game.setScreen(new PowersScreen2(game)); 
     } 

     if (x2 >= left_side_popcornX && x2 < (left_side_popcornX + Assets.happyPopcorn.getWidth()) 
       && y2 >= 2 * g.canvasHeight()/8 && y2 < (2 * g.canvasHeight()/8 + Assets.happyPopcorn.getHeight())) { 
      //first popcorn 
      choice1 = 3; 
     } 

     if (x2 >= left_side_popcornX && x2 < (left_side_popcornX + Assets.happyPopcorn.getWidth()) 
       && y2 >= 3 * g.canvasHeight()/8 && y2 < (3 * g.canvasHeight()/8 + Assets.happyPopcorn.getHeight())) { 
      //second popcorn 
      choice1 = 6; 
     } 
     if (x2 >= left_side_popcornX && x2 < (left_side_popcornX + Assets.happyPopcorn.getWidth()) 
       && y2 >= 4 * g.canvasHeight()/8 && y2 < (4 * g.canvasHeight()/8 + Assets.happyPopcorn.getHeight())) { 
      //third popcorn 
      choice1 = 9; 
     } 
     if (x2 >= left_side_popcornX && x2 < (left_side_popcornX + Assets.happyPopcorn.getWidth()) 
       && y2 >= 5 * g.canvasHeight()/8 && y2 < (5 * g.canvasHeight()/8 + Assets.happyPopcorn.getHeight())) { 
      //fourth popcorn 
      choice1 = 12; 
     } 
     if (x2 >= right_side_popcornX && x2 < (right_side_popcornX + Assets.happyPopcorn.getWidth()) 
       && y2 >= 2 * g.canvasHeight()/8 && y2 < (2 * g.canvasHeight()/8 + Assets.happyPopcorn.getHeight())) { 
      //sixth popcorn 
      choice1 = 15; 
     } 
     if (x2 >= right_side_popcornX && x2 < (right_side_popcornX + Assets.happyPopcorn.getWidth()) 
       && y2 >= 3 * g.canvasHeight()/8 && y2 < (3 * g.canvasHeight()/8 + Assets.happyPopcorn.getHeight())) { 
      //seventh popcorn 
      choice1 = 18; 
     } 
     if (x2 >= right_side_popcornX && x2 < (right_side_popcornX + Assets.happyPopcorn.getWidth()) 
       && y2 >= 4 * g.canvasHeight()/8 && y2 < (4 * g.canvasHeight()/8 + Assets.happyPopcorn.getHeight())) { 
      //eighth popcorn 
      choice1 = 21; 
     } 
     if (x2 >= right_side_popcornX && x2 < (right_side_popcornX + Assets.happyPopcorn.getWidth()) 
       && y2 >= 5 * g.canvasHeight()/8 && y2 < (5 * g.canvasHeight()/8 + Assets.happyPopcorn.getHeight())) { 
      //ninth popcorn 
      choice1 = 24; 
     }      
     } 
    } 
} 

@Override 
public void paint(float deltaTime) { 
    // TODO Auto-generated method stub 
    Paint paint = new Paint(); 
    paint.setTextSize(20); 
    paint.setTextAlign(Align.CENTER); 
    paint.setColor(Color.YELLOW); 
    paint.setTypeface(Assets.myFont); 
    paint.setStyle(Style.STROKE); 

    g.clearScreen(Color.rgb(51, 51, 255)); 

    g.drawRectText(choose_pwrup_1, rect1, paint); 
    g.drawImage(PowersScreen.getDecodedChoice(), (g.canvasWidth()/2) - (CustomizeScreen.getDecodedChoice().getWidth()/2), g.canvasHeight()/8); 
    g.drawRectText(back, rect2, paint); 
    g.drawRectText(next, rect3, paint); 

    g.drawRectText(chosen_text, info, paint); 
    g.drawRect(info, paint); 

    g.drawImage(Assets.powerup_speedup, left_side_popcornX, 2 * g.canvasHeight()/8); 
    g.drawImage(Assets.powerup_stopall, left_side_popcornX, 3 * g.canvasHeight()/8); 
    g.drawImage(Assets.powerup_clearall, left_side_popcornX, 4 * g.canvasHeight()/8); 
    g.drawImage(Assets.powerup_doublepoints, left_side_popcornX, 5 * g.canvasHeight()/8); 
    g.drawImage(Assets.powerup_regainheart, right_side_popcornX, 2 * g.canvasHeight()/8); 
    g.drawImage(Assets.powerup_nolostheart, right_side_popcornX, 3 * g.canvasHeight()/8); 
    g.drawImage(Assets.powerup_nobadpopcorn, right_side_popcornX, 4 * g.canvasHeight()/8); 
    g.drawImage(Assets.powerup_stopboss, right_side_popcornX, 5 * g.canvasHeight()/8); 
} 

@Override 
public void pause() { 
    // TODO Auto-generated method stub 

} 

@Override 
public void resume() { 
    // TODO Auto-generated method stub 

} 

@Override 
public void dispose() { 
    // TODO Auto-generated method stub 

} 

@Override 
public void backButton() { 
    // TODO Auto-generated method stub 
    game.setScreen(new PowersExplainScreen(game)); 
} 

public static Image getDecodedChoice() { 
    if (choice1 == 3) { 
     chosen_one = Assets.powerup_speedup; 
     chosen_text = pwr1; 
    } 
    if (choice1 == 6) { 
     chosen_one = Assets.powerup_stopall; 
     chosen_text = pwr2; 
    } 
    if (choice1 == 9) { 
     chosen_one = Assets.powerup_clearall; 
     chosen_text = pwr3; 
    } 
    if (choice1 == 12) { 
     chosen_one = Assets.powerup_doublepoints; 
     chosen_text = pwr4; 
    } 
    if (choice1 == 15) { 
     chosen_one = Assets.powerup_regainheart; 
     chosen_text = pwr5; 
    } 
    if (choice1 == 18) { 
     chosen_one = Assets.powerup_nolostheart; 
     chosen_text = pwr6; 
    } 
    if (choice1 == 21) { 
     chosen_one = Assets.powerup_nobadpopcorn; 
     chosen_text = pwr7; 
    } 
    if (choice1 == 24) { 
     chosen_one = Assets.powerup_stopboss; 
     chosen_text = pwr8; 
    } 
    return chosen_one; 
} 
} 

私は何をしたいユーザーを持っています使用したい電源をタップします（その電源の説明が画面の下部に表示され、現在の選択が画面上部に表示されます）。その後、2番目の電源をオンにしてから3番目の電源をオンにすることができる「次へ」を押すことができます。彼らがパワーアップを選択するたびに、そのパワーアップを再び選択するオプションが消えるはずです（パワーアップの絵がなくなるはずです）。また、前の電源を再選択するために「戻る」を押す必要があります。

私は今、それぞれの電源を扱う3つの異なるクラスを持つことでこれを達成しています。他の2つは、intオプションのオプションを除いて、これとまったく同じように見えます（数値は異なるので、どの電源が選択されているかを知る必要があります）。だから、多くの冗長性があり、スペースを使い果たします。私はすべてが1つのクラスで扱われるべきであるように感じます。これをやり遂げるにはどうすればいいですか？

Answer 1

他の2つは、int choiceオプション（数字が異なるので、どの電源が選択されているかを知る必要があります）を除いて、このようになります。

ここには2つのオプションがあります。それを行うには

---

一つの方法は、クラスが作成されたときに設定されているものとして、「int型の選択」、およびクラスは、一般的にそれを参照するパラメータ化することです。

public class PowerupHandler { 
    private int choice; 
    public PowerupHandler(int choice) { 
     this.choice = choice; 
    } 
} 

---

それを行うための別の方法は同じことを行いabstractクラスを作成することであるが、代わりにサブクラスはただの介入なしのコンストラクタに数値を設定し

public abstract class PowerupHandlerAbstract { 
    private int choice; 
    public PowerupHandlerAbstract(int choice) { 
     this.choice = choice; 
    } 
} 
... 
public class PowerupHandlerTypeOne extends PowerupHandlerAbstract { 
    public PowerupHandlerTypeOne() { 
     super(3); 
    } 
} 

私の個人を（スキップしている読者にはちょっと意味があるので、設定にもよりますが）抽象的なルートになります。