ボタン内のvoid呼び出しにオペレータを修正するにはどうすればよいですか？

Question

私は初心者のC＃devで、何かを忘れています。 （！ValidateCleared（actionsChecked））コードがオペレータを投げてエラーになりました！ voidには適用できません。この例でこれを修正する方法がわかりません。私はそれをboolにして休憩を使用しようとしましたが、アクションは期待どおりに動作しませんでした。それは1つの行動だけをチェックしました。

protected void btnSubmit_Click(object sender, EventArgs e) 
{ 
    ValidateActionSelected();enter code here 
    ValidateCleared(actionsChecked); 

    if (!ValidateCleared(actionsChecked)) 
    { 
     if (String.Equals(ddlActionsAndDocuments.SelectedItem.Text.ToString(), "XXX YYY") || String.Equals(ddlActionsAndDocuments.SelectedItem.Text.ToString(), "XXX ZZZ")) 
     { 
      reqEffectiveDate.ErrorMessage = ""; 
     } 

     if (ddlActionsAndDocuments.SelectedValue == ActionTypes.XXXYYY || 
      ddlActionsAndDocuments.SelectedValue == ActionTypes.XXXYYYDenial || 
      ddlActionsAndDocuments.SelectedValue == ActionTypes.XXXzzz) 
     { 
      ValidateXXXYYY(actionsChecked, ddlActionsAndDocuments.Text); 
     } 

     if (ddlActionsAndDocuments.SelectedValue == InsuranceActionTypes.WWW || 
      ddlActionsAndDocuments.SelectedValue == InsuranceActionTypes.YYYZZZ || 
      ddlActionsAndDocuments.SelectedValue == InsuranceActionTypes.YYYWWW || 
      ddlActionsAndDocuments.SelectedValue == InsuranceActionTypes.YYYWaived) 
     { 
      ValidateCertificate(actionsChecked, ddlActionsAndDocuments.Text); 
     } 
    } 
} 

private void ValidateCleared(List<xxCaseEntity> actionsChecked) 
{ 

    //bool error = false; 

    foreach (xxCaseEntity ACTIONyy in actionsChecked) 
    { 
     if (ACTIONyy.XXStatusCode == 40 || ACTIONyy.XXStatusCode == 45) 
     { 
      //error = true; 
      DisplayErrorMessage("FR Action Cannot Apply to " + ACTIONyy.CaseIdentifier); 

      actionsChecked.Remove(ACTIONyy); 
      //break; 
     } 

    } 
    return; 

    //return error; 
} 

Answer 1

あなたは `ValidateClearedを（使用したい場合は、固定コードは、それがむしろvoid``より `bool`を返すかのように、`、それは実際にはかなり `bool`を返す作ってみてください）

protected void btnSubmit_Click(object sender, EventArgs e) 
    { 
     ValidateActionSelected();enter code here 
     ValidateCleared(actionsChecked); 

     if (!ValidateCleared(actionsChecked)) 
     { 
      if (String.Equals(ddlActionsAndDocuments.SelectedItem.Text.ToString(), "XXX YYY") || String.Equals(ddlActionsAndDocuments.SelectedItem.Text.ToString(), "XXX ZZZ")) 
      { 
       reqEffectiveDate.ErrorMessage = ""; 
      } 

      if (ddlActionsAndDocuments.SelectedValue == ActionTypes.XXXYYY || 
       ddlActionsAndDocuments.SelectedValue == ActionTypes.XXXYYYDenial || 
       ddlActionsAndDocuments.SelectedValue == ActionTypes.XXXzzz) 
      { 
       ValidateXXXYYY(actionsChecked, ddlActionsAndDocuments.Text); 
      } 

      if (ddlActionsAndDocuments.SelectedValue == InsuranceActionTypes.WWW || 
       ddlActionsAndDocuments.SelectedValue == InsuranceActionTypes.YYYZZZ || 
       ddlActionsAndDocuments.SelectedValue == InsuranceActionTypes.YYYWWW || 
       ddlActionsAndDocuments.SelectedValue == InsuranceActionTypes.YYYWaived) 
      { 
       ValidateCertificate(actionsChecked, ddlActionsAndDocuments.Text); 
      } 
     } 
    } 

    private bool ValidateCleared(List<xxCaseEntity> actionsChecked) 
    { 
     List<xxCaseEntity> removeItems = new List<xxCaseEntity>(); 
     foreach (xxCaseEntity ACTIONyy in actionsChecked) 
     { 
      if (ACTIONyy.XXStatusCode == 40 || ACTIONyy.XXStatusCode == 45) 
      { 
       DisplayErrorMessage("FR Action Cannot Apply to " + ACTIONyy.CaseIdentifier); 

       //actionsChecked.Remove(ACTIONyy); 
       removeItems.Add(ACTIONyy); 
      } 
     } 
     foreach(xxCaseEntity ACTIONyy in removeItems) 
     { 
      actionsChecked.Remove(ACTIONyy); 
     } 
     return true; 
    }