Как избежать NRE при чтении тэга кнопки C#

Question

(всхлипывая от разочарования)

Как избежать раздражающего NRE в этом случае? Результат кода правильный, но я хочу избавиться от NRE.

фон:

Есть некоторые кнопки на форме, некоторые из которых представляют собой место классной комнаты. Другие кнопки не имеют значения, потому что мне нужны только сидения, и только кнопки сидений имеют метки, которые имеют ценность.

Итак, я просматриваю все кнопки и читаю их теги, если он не пуст, а затем держите эту кнопку в списке. Остальные игнорируются.

Однако, когда код запускается, NRE появляется в четвертой строке, строка начинается с «try».

foreach (Button seatbutton in this.Controls) 
{ 
    string betta; 
    try { betta = seatbutton.Tag.ToString(); } 
    catch { betta = ""; } 

    if (!string.IsNullOrEmpty(betta)) 
    { 
    seatbuttons.Add(seatbutton); 
    } 
} 

Это самый короткий, самый простой пример такого типа NRE в моем коде. Есть еще несколько.

Я искал в Интернете, и большинство ответов относятся к следующим темам: «Плохие привычки кодирования вызвали это».

Как вы, наверное, можете сказать, я совершенно новичок в этом деле, и у меня даже не было времени на создание привычек. Можете ли вы помочь? Возможно, с некоторыми советами по ХОРОШЕМУ кодированию?

T_T спасибо!

Answer 1

Вам необходимо проверить, есть ли tagnull, прежде чем вы назовете метод ToString(). Что-то вроде этого:

string betta = seatbutton.Tag == null ? "" : seatbutton.Tag.ToString(); 
if (betta == "") { 
    seatbuttons.Add(seatbutton); 
} 

Если вы используете последнюю версию VS/C#, то вы можете альтернативно использовать новый оператор:

string betta = seatbutton.Tag?.ToString(); 
if (!string.IsNullOrEmpty(betta)) { 
    seatbuttons.Add(seatbutton); 
}