正确的空检查

Question

我正在实例化一个 Associate 对象，并从我的主窗体内的 txtboxes 为其分配属性。空值检查的最佳做法是什么？是在分配之前使用 if 语句检查每个属性还是有更好的方法？这是我的代码：

Associate updateAssociate = new Associate();

updateAssociate.AssocID = txtAssocId.Text;
updateAssociate.FirstName = txtFname.Text;
updateAssociate.LastName = txtLname.Text;
updateAssociate.HireDate = Convert.ToDateTime(txtHireDate.Text);
updateAssociate.ContractEndDate = Convert.ToDateTime(txtContractEnd.Text);
updateAssociate.TerminationDate = Convert.ToDateTime(txtTerminationDate.Text);
updateAssociate.FullPartTimeID = cboFullPart.SelectedText;
updateAssociate.PrimaryRole = cboPRole.SelectedText;

Answer 1

根据您对问题的评论：

如果它是一个文本框，那么它将是 .Text 属性，我想在将它们分配给对象之前检查空值或空白值

您可以在分配时使用空值合并运算符来检查空值：

updateAssociate.AssocID = txtAssocId.Text ?? string.Empty;

或：

updateAssociate.AssocID = txtAssocId.Text ?? someDefaultValue;

这样，如果txtAssocId.Text 是null，那么您将定义的默认值分配给对象属性而不是null。

虽然我不完全确定TextBox 的.Text 属性会成为 null 而不是空字符串。也许您想同时检查两者？：

updateAssociate.AssocID = string.IsNullOrEmpty(txtAssocId.Text) ? someDefaultValue : txtAssocId.Text;

Answer 2

在 C# 6 中，它将是 null-conditional operator。

updateAssociate.AssocID = txtAssocId?.Text;

在以前的 c# 版本中，您可以编写一个方法来消除代码重复。像这样的：

public static T CheckNull<T>(Func<T> canBeNull) where T : class
    {
        try
        {
            return canBeNull();
        }
        catch (NullReferenceException)
        {
            return default(T);
        }
    }

并像这样使用它

updateAssociate.AssocID = CheckNull(() => txtAssocId.Text);

然后你可以将任何可以抛出空引用的代码包装到 lambda 中，将它传递给这个方法，而不再为它烦恼。