开关盒不能正常工作

时间:2015-01-15 08:17:54

标签: java android switch-statement

我在下面发布的代码中,当我将字段ip留空/空并将值提供给其他字段时,toast始终会给出KATimer is invalid or missing消息。

我希望看到toast显示一条消息,指示空字段,但下面的代码,如果有任何字段为空,则总是显示KATimer is invalid or empty

为什么会发生这种情况,我错过了什么

代码:

btnStubView_Connect

btnStubView_Connect.setOnClickListener(new OnClickListener() {

        @Override
        public void onClick(View v) {
            // TODO Auto-generated method stub
            if (isValidMQTTConfigs(etStubView_ip) && 
                    isValidMQTTConfigs(etStubView_port) && 
                    isValidMQTTConfigs(etStubView_ClientID) &&
                    isValidMQTTConfigs(etStubView_KATimer)) {

                Log.d(TAG, "@btnStubView_ConnectListener(): all entries are valid");
                setCSession(cbStubView_CS.isChecked()); // set the current state of the cleanSession checkBox.
                addToContentValues();
                Log.d(TAG, "@btnStubView_ConnectListener(): all entries added toContentValues");

            } else {
                Log.w(TAG, "@btnStubView_ConnectListener(): one or more entry(s) is invalid or left blank.");
            }
        }
    });

isValidMQTTConfigs

protected boolean isValidMQTTConfigs(View view) {
    // TODO Auto-generated method stub
    boolean valid = false;
    String viewName = "";

    switch(view.getId()) {
    case R.id.etSubView_ip:
        viewName = "IP";
        if (isDuly( ((EditText) view).getText().toString())) {
            this.setIP(((EditText) view).getText().toString());
            return valid = true;
        }

    case R.id.etSubView_port:
        viewName = "Port";
        if (isDuly( ((EditText) view).getText().toString())) {
            this.setPort(((EditText) view).getText().toString());
            return valid = true;
        }

    case R.id.etSubView_clientID:
        viewName = "clientID";
        if (isDuly( ((EditText) view).getText().toString())) {
            this.setClienID(((EditText) view).getText().toString());
            return valid = true;
        }

    case R.id.etSubView_KATimer:
        viewName = "KAtimer";
        if (isDuly( ((EditText) view).getText().toString())) {
            this.setKATimer(((EditText) view).getText().toString());
            return valid = true;
        }
    }

    Log.w(TAG, "@checkMQTTConfigs(): " + viewName + " is invalid or missing");
    Toast.makeText(getActivity(), viewName + " is invalid or missing", Toast.LENGTH_SHORT).show();
    return valid;
}

isDuly

private boolean isDuly(String text) {
    // TODO Auto-generated method stub
    if (text.trim().equals("")) {
        return false;
    } else {
        return true;
    }
}

2 个答案:

答案 0 :(得分:4)

在每break;之后,您没有使用case导致以下情况执行,即使您不希望这样做。例如,这是正确的: -

switch(int){ 
   case 1:
   break;
   case 2:
   break;
 }

这会导致意外的输出虽然没有错: -

switch(int){ 
   case 1:
   case 2:
 }

在执行案例1后,省略break将导致执行案例2。

答案 1 :(得分:1)

你似乎错过了几个休息陈述。

Switch(X){
case 1: doOne();
case 2: doTwo();
case 3: doThree();
}

瀑布流确保如果X为2,则执行doTwo和doThree。如果X为1,则将执行所有三种方法。

如果只想调用链接方法,请将代码更改为:

Switch(X){
case 1: doOne(); break;
case 2: doTwo(); break;
case 3: doThree(); break;
}