如何處理多個Android方法？

Question

好的，問題是。在我的Android應用程序中，我有兩個單獨的活動供選項和主要活動使用。主要活動中有一個地方，它檢查選項的變化並應用樣式。它看起來像這樣：

if (prefs.getBoolean("opt_changed", true)) { 
     Theme = prefs.getInt("theme", Theme); 
     Font = prefs.getInt("font", Font); 
     Size = prefs.getInt("size", Size); 

     SetApplicableStyle(this, Theme, Font, Size); 

     /** Setting opt_changed to false. */ 
     SharedPreferences.Editor editor = prefs.edit(); 
     editor.putBoolean("opt_changed", false); 
     editor.commit(); // apply changes 
    } 

SetApplicableStyle方法，這裏所說的，看起來是這樣的：

public void SetApplicableStyle (DTypeActivity dTypeActivity, int Theme, int Font, int Size) { 
    // Retrieving the EditText and the View as objects 
    final EditText edit_text = (EditText) findViewById(R.id.editText1); 
    final View main_view = (View) findViewById(R.id.mainview); 

    // Setting the theme 
    switch(Theme){ 
    case 1: 
     SetThemeLight (this); 
    break; 
    case 2: 
     SetThemeBlue (this);   
    break; 
    case 3: 
     SetThemeDark (this); 
    break; 
    } 

    // Setting the font 
    switch(Font){ 
    case 1: 
     SetFontSans (this); 
    break; 
    case 2: 
     SetFontSerif (this);   
    break; 
    case 3: 
     SetFontMono (this); 
    break; 
    } 

    // Setting the size 
    switch(Size){ 
    case 1: 
     SetSizeSm (this); 
    break; 
    case 2: 
     SetSizeMd (this);  
    break; 
    case 3: 
     SetSizeBg (this); 
    break; 
    } 
} 

的，它的Set[Something][Somewhat]方法的例子，有SetThemeLight一個：

public void SetThemeLight (DTypeActivity dTypeActivity) { 
     final EditText edit_text = (EditText) findViewById(R.id.editText1); 
     final View main_view = (View) findViewById(R.id.mainview); 
     main_view.setBackgroundDrawable(getResources().getDrawable(R.drawable.grey_background)); 
     edit_text.getBackground().setAlpha(0); 
     edit_text.setTextColor(getResources().getColor(R.color.DrText)); 

} 

我的問題涉及在這個簡單的應用程序中使用的方法的數量。我一直在考慮減少代碼量並實施SetApplicableStyle方法。現在我正在考慮擺脫Set[Something][Somewhat]是否合適，並將它們的線路直接連接到SetApplicableStyle交換機的情況。我主要關心的是方法的數量，但我知道，巨大的方法也是一種不好的做法。這裏更好的解決方案是什麼？全部源代碼可用here。

Answer 1

我假設你複製了SetThemeX方法中的大部分代碼。因此，我建議引入，抓住主題精髓的一類，並使用：

class MyTheme { 
    public int background; 
    public int alpha; 
    public int color; 
    public MyTheme(int background, int alpha, int color) { 
     this.background = background; 
     this.alpha = alpha; 
     this.color = color; 
    } 
} 

進行一個方法來設置你的主題：

public void setTheme(DTypeActivity dTypeActivity, MyTheme theme) { 
    final EditText edit_text = (EditText) findViewById(R.id.editText1); 
    final View main_view = (View) findViewById(R.id.mainview); 
    main_view.setBackgroundDrawable(getResources().getDrawable(theme.background)); 
    edit_text.getBackground().setAlpha(theme.alpha); 
    edit_text.setTextColor(getResources().getColor(theme.color)); 
} 

，並保持地圖中的某個地方你存儲這些主題：

Map<Integer, MyTheme> themes = new HashMap<>(); 
themes.put(1, new MyTheme(R.drawable.grey_background, 0, R.color.DrText)); 
// put other themes 

在你SetApplicableStyle方法，你可以然後只需使用

public void SetApplicableStyle (DTypeActivity dTypeActivity, int theme, int font, int size) { 
    setTheme(dTypeActivity, themes.get(theme); 
    // set font and size similarly 
}