我不想调用任何函数用户,请单击默认选项。现在,我的问题是我应该将其留空还是使用 return 或其他方式?
if (myPref.getString("font_style_key", "default").equals("default")){
//return;
}else {
if (myPref.getString("font_style_key", "default").equals("font_1")){
textView.setTypeface(...);
}else if (myPref.getString("font_style_key", "default").equals("font_2")){
textView.setTypeface(...);
}
答案 0 :(得分:1)
否定条件是有道理的,这样就消除了else
子句:
if (!myPref.getString("font_style_key", "default").equals("default")) {
if (myPref.getString("font_style_key", "default").equals("font_1")) {
textView.setTypeface();
} else if (myPref.getString("font_style_key", "default").equals("font_2")) {
textView.setTypeface();
}
}
答案 1 :(得分:1)
看起来这是switch
语句的完美用例。您将基于一个特定的值有几个选项,并且通常是更简洁的代码
String fontStyle = myPref.getString("font_style_key", "default");
switch(fontStyle) {
case "font_1":
textView.setTypeface(...);
break;
case "font_2":
textView.setTypeface(...);
break;
// We could add this line if we wanted some default behaviour
default:
...
}
此外,请注意多次调用myPref.getString("font_style_key", "default")
总是会返回相同的值。您可以将其存储在变量中并节省一些时间