Проверка значений элементов в булевом массиве - C#

Question

Я пишу некоторую проверку ошибок и пытаюсь использовать логический массив для хранения истинных или ложных элементов, а затем мое заключительное условие анализирует через сохраненные элементы, чтобы определить, все верно в visual studio 2008. Theres, вероятно, более простой способ выполнить проверку ошибок, но может также изучить, как использовать массив. Вот то, что я до сих пор

bool[] checker = new bool[1]; // declared array...I think 

private void print_button_Click(object sender, EventArgs e) 
{ 
    if (authorbox.Text == "") 
    { 
    MessageBox.Show("Author field empty", "Required Entry"); 
    } 
    else 
    { 
    checker[0] = true; // assigning element to array correctly? 
    } 

    if (titlebox.Text == "") 
    { 
    MessageBox.Show("Title field Empty", "Required Entry"); 
    } 
    else 
    { 
    checker[1] = true; 
    } 

    // The part I am having trouble with basically if any of my array elements are 
    // false don't execute printing. Else go ahead and print. 
    if() 
    { 
    } 
    else 
    { 
    printPreviewDialog1.Document = printDocument1; 
    printPreviewDialog1.ShowDialog(); 
    } 
} 

Answer 1

вместо того, чтобы использовать массив было бы гораздо проще просто выйти из метода, как только обнаружена ошибка:

private void print_button_Click(object sender, EventArgs e) { 
    if (authorbox.Text == "") { 
    MessageBox.Show("Author field empty", "Required Entry"); 
    return; 
    } 

    if (titlebox.Text == "") { 
    MessageBox.Show("Title field Empty", "Required Entry"); 
    return; 
    } 

    printPreviewDialog1.Document = printDocument1; 
    printPreviewDialog1.ShowDialog(); 
} 

Answer 2

Ну это не идеальный способ для обработки ошибок, но вы можете использовать .Contains() метод.

if (checker.Contains(false)) 
{ 
    // Do Something 
} 
else 
{ 
    printPreviewDialog1.Document = printDocument1; 
    printPreviewDialog1.ShowDialog(); 
} 

Answer 3

Помимо всего прочего, вы должны сказать

bool[] checker = new bool[2]; 

если вы хотите массив, состоящий из 2-х элементов;) В данном случае массив, кажется, не делать слишком много смысла, потому что это немного запутывает вещи. Вы можете сделать то же самое только с одной логической переменной.

Answer 4

Я уверен, что метод Contains, предложенный NebuSoft, является расширением LINQ и поэтому недоступен в .NET 2.0. Однако вы можете использовать метод Array.IndexOf<T>, как это:

if (Array.IndexOf<bool>(checker, false) != -1) 
{ 
    // some element in the array is false 
} 
else 
{ 
    // no false in the array 
} 

Однако NebuSoft прав, утверждая, что это не самый лучший подход. Если вам интересно узнать больше, я буду рад обсудить его дальше.

Answer 5

Если вы используете .NET 3.5 вы можете использовать любой и все, чтобы увидеть, если какая-либо из булевых истинно, или если все они являются истинными:

if (checker.Any(x => x)) 

или:

if (checker.All(x => x)) 

Кроме того, если вам нужен массив из двух логических элементов, вы должны использовать new bool[2] не new bool[1]. Было бы проще использовать List<bool>.

Answer 6

Использование логических массивов аккумулировать один идти/не годен значение не является излишеством. Есть более полезные вещи, с которыми вы могли бы играть, чтобы получить массив массивов.

Ты лучше просто результаты операции AND ваших промежуточных проверок в значение, а затем проверить, что для истинно/ложно:

public bool CheckControls() 
{ 
    bool pass = true; 
    pass &= !string.IsNullOrEmpty(authorbox.Text)); 
    pass &= !string.IsNullOrEmpty(titlebox.Text)); 
    // if any of these are empty then pass is to false and stays that way. 
    return pass; 
} 

Если вам необходимо отслеживать, какие промежуточный тест не пройден, а затем рассмотреть используя целочисленные и предопределенные константы степеней двух. Здесь вы вместо этого проверяете нуль, если все хорошо. Это позволяет вам маскировать возвращаемое значение и накапливать любую комбинацию результатов теста. Если у вас менее 32 (или 64) тестов.

int AUTHORBOX = 2; 
    int TITLEBOX = 4; 
    int ISBNBOX = 8; 
    int PRICEBOX = 16; 

    public int AlternateCheck() 
    { 
     int temp = 0; 
     temp += string.IsNullOrEmpty(authorbox.Text) ? AUTHORBOX : 0; 
     temp += string.IsNullOrEmpty(titlebox.Text) ? TITLEBOX : 0; 
     temp += string.IsNullOrEmpty(isbnbox.Text) ? ISBNBOX : 0; 
     temp += string.IsNullOrEmpty(pricebox.Text) ? PRICEBOX : 0; 
     return temp; 
    }