Неконтролируемый Флажок = текстовое поле со значением 0

Question

У меня есть этот код:

for (int i = 0; i < ListView2.Items.Count; i++) 
{ 
    int a; 
    Int32.TryParse(((TextBox)ListView2.Items[i].FindControl("BrankyTextBox")).ToString(), out a); 

    if (((CheckBox)ListView2.Items[i].FindControl("UcastCheckBox")).Checked=false && 
     a > 0) 
    { 
RegisterStartupScript("alertBox2", "<script type='text/javascript'>alert('Error');</script>"); 
    } 
} 

Я хочу, чтобы, когда флажок снят значением текстового поля должно быть только 0. Но этот код только меняет все флажки в ListView, чтобы остановить ... У вас есть идея, как его решить?

Answer 1

Вы можете очистить это немного, пока вы на него:

for (Int32 i = 0; i < ListView2.Items.Count; i++) 
{ 
    Int32 a; 
    Int32.TryParse(((TextBox)ListView2.Items[i].FindControl("BrankyTextBox")).Text, out a); 

    if (!((CheckBox)ListView2.Items[i].FindControl("UcastCheckBox")).Checked && a > 0) 
    { 
     RegisterStartupScript("alertBox2", "<script type='text/javascript'>alert('Error');</script>"); 
    } 
} 

Другими словами, используйте not operator, который !, чтобы показать обратный тест. myVariable == false эквивалентно простому написанию !myVariable.

Answer 2

У вас было только одно = вместо == для сравнения.

if (((CheckBox)ListView2.Items[i].FindControl("UcastCheckBox")).Checked == false || a > 0) 
{ 
    RegisterStartupScript("alertBox2", "<script type='text/javascript'>alert('Error');</script>"); 
} 

Answer 3

Вы задаете значение true для флажков в инструкции if. Вам нужно использовать double =. Как это:

if (((CheckBox)ListView2.Items[i].FindControl("UcastCheckBox")).Checked==false || 
    a > 0) 

Answer 4

Ваш if не делать то, что вы ожидаете

if (((CheckBox)ListView2.Items[i].FindControl("UcastCheckBox")).Checked=false 
     || a > 0) 

Вы назначая false к Checked собственности, и это утверждение всегда false. В настоящее время, ваш if может быть переписан, чтобы быть

if (false || a > 0) 

Что явно не то, что вы хотите. Добавить = знак, что делает его

if (((CheckBox)ListView2.Items[i].FindControl("UcastCheckBox")).Checked == false 
     || a > 0) 

Чтобы исправить условное выражение. Другое предложение здесь было бы поменять условия, извлекая выгоду из логического короткого ciruiting:

if (a > 0 || ((CheckBox)ListView2.Items[i].FindControl("UcastCheckBox")).Checked == false) 

В этом случае, когда a > 0 является true не будут оцениваться другие условия, в результате чего (чуть-чуть!) Более высокой производительности. См. this other question для получения информации о коротком замыкании.