In my opinion this is poor because it could be made much more clear with an if statement:
if (school != null) {
myLabel.Text = school.SchoolName;
}
else {
myPanel.Visible = false;
}
That will certainly avoid using exception handling unnecessarily and make the code's meaning very obvious.