If-else语句和可见性 - android

Question

我的应用程序中有五个复选框以及五个进度条，并根据复选框调用可见性以使进度条可见/消失。所以在可见性方法中使用if-else语句还有其他选择，在这种情况下，在五个复选框中有无穷无尽的可能性，所以我必须编写无穷无尽的if语句，以便使进度条在所有可能性中可见/消失

例如 - 如果选中CheckBox1＆amp; checkbox2，则使progressbar2可见              如果选中checkbox1＆amp; checkbox3，则使progressbar2可见

换句话说，如果选中任何一个复选框，我想让进度条1可见，如果检查任何两个复选框，则可见进度条2等等。

import android.app.Activity;
import android.content.SharedPreferences;
import android.os.Bundle;
import android.preference.PreferenceManager;
import android.view.View;
import android.widget.CheckBox;
import android.widget.CompoundButton;
import android.widget.ProgressBar;

public class Progress extends Activity {

    ProgressBar progressBar1;
    ProgressBar progressBar2;

    CheckBox checkBox1;
    CheckBox checkBox2;

    SharedPreferences sharedPreferences;
    SharedPreferences.Editor editor;
    SharedPreferences setprefsd;

    @Override
    protected void onCreate(Bundle savedInstanceState) {
        super.onCreate(savedInstanceState);
        setContentView(R.layout.activity_progress);

        setprefsd = PreferenceManager.getDefaultSharedPreferences(getApplicationContext());
        progressBar1 = (ProgressBar) findViewById(R.id.progressBar1);
        progressBar2 = (ProgressBar) findViewById(R.id.progressBar2);
        checkBox1 = (CheckBox) findViewById(R.id.checkBox1);
        checkBox2 = (CheckBox) findViewById(R.id.checkBox2);

        if (setprefsd.getBoolean("FirstCheckBox", false) == true) {
            checkBox1.setChecked(true);
            updateProgressBars();
        }

        if (setprefsd.getBoolean("SecondCheckBox", false) == true) {
            checkBox2.setChecked(true);
            updateProgressBars();
        }
    }

    public void updateProgressBars() {
        progressBar1.setVisibility(View.GONE);
        progressBar2.setVisibility(View.GONE);

        if (checkBox1.isChecked() && checkBox2.isChecked()) {
            progressBar2.setVisibility(View.VISIBLE);
        } else if (checkBox1.isChecked()) {
            progressBar1.setVisibility(View.VISIBLE);
        }
    }
}

Answer 1

简短回答

您可以使用三元运算符使可见性更改更简单（而不是if）。

我想这里的方法是计算复选框的数量，然后设置可见性。您只需要像这样更改updateProgressBars()方法：

public void updateProgressBars() {
    int nbCheckboxes = 0;
    if (checkBox1.isChecked())
        nbCheckboxes++;
    if (checkBox2.isChecked())
        nbCheckboxes++;

    progressBar1.setVisibility(nbCheckboxes >= 1 ? View.VISIBLE : View.GONE);
    progressBar2.setVisibility(nbCheckboxes >= 2 ? View.VISIBLE : View.GONE);
}

---

其他评论

带有布尔表达式的IF语句

您使用的构造如：

if (myBooleanExpression == true)

不需要== true，因为您在此处使用布尔表达式。改为使用：

if (myBooleanExpression)

为变量使用数组

如果您使用具有相同目的的多个编号变量，则可能需要考虑使用数组。

public class Progress extends Activity {

    ProgressBar[] progressBars;
    CheckBox[] checkBoxes;

    SharedPreferences sharedPreferences;
    SharedPreferences.Editor editor;
    SharedPreferences setprefsd;

    @Override
    protected void onCreate(Bundle savedInstanceState) {
        super.onCreate(savedInstanceState);
        setContentView(R.layout.activity_progress);

        setprefsd = PreferenceManager.getDefaultSharedPreferences(getApplicationContext());

        progressBars = new ProgressBar[3]; // 3, or whatever number you have
        progressBars[0] = (ProgressBar) findViewById(R.id.progressBar1); 
        progressBars[1] = (ProgressBar) findViewById(R.id.progressBar2); 
        progressBars[2] = (ProgressBar) findViewById(R.id.progressBar3);

        checkBoxes = new CheckBox[3];  // 3 or whatever number you have
        checkBoxes[0] = (CheckBox) findViewById(R.id.checkBox1);
        checkBoxes[1] = (CheckBox) findViewById(R.id.checkBox2);
        checkBoxes[2] = (CheckBox) findViewById(R.id.checkBox3);

        checkBoxes[0].setChecked(setprefsd.getBoolean("FirstCheckBox", false));
        checkBoxes[1].setChecked(setprefsd.getBoolean("SecondCheckBox", false));
        checkBoxes[2].setChecked(setprefsd.getBoolean("ThirdCheckBox", false));

        updateProgressBars();
    }

    public void updateProgressBars() {
        int nbCheckBoxes = 0;
        for (CheckBox cb : checkBoxes) {
            if (cb.isChecked())
                nbCheckBoxes++;
        }

        for (int i = 0; i < progressBars.length; i++) {
            progressBars[i].setVisibility(nbCheckboxes > i ? View.VISIBLE : View.GONE);
        }
    }
}