如何处理多个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 
}