学校项目-当 nchar 设置为 9 位时，为什么我的 8 位数据会被插入？

Question

这是一个学校项目，我感谢你的帮助。预期系统行为：当 txt 文件中的 SSN 列不等于 9 位时，不应将包含该列的行插入到数据库中。实际行为：当 SSN 列不等于 9 位时，它仍将其插入到数据库中。要插入的列：Last name，First name，MI，SSAN和Payrate。SSAN 的存储过程代码：@SSAN nchar(9)这是我在包含我的 SQL 的类中执行 SSAN 的方式：cmdSQL.Parameters.Add(new SqlParameter("@SSAN", SqlDbType.NChar));cmdSQL.Parameters["@SSAN"].Direction = ParameterDirection.Input;cmdSQL.Parameters["@SSAN"].Value = SSAN;这是我在主程序中验证它的方法：if (strFields[3].Length == 9){        blnOK = true;}else{    blnOK = false;    Console.WriteLine("Error: Unable to Parse SSAN!");}这是我从主程序调用数据库类的方式。  intRetCode = clsDatabase.InsertEmployee(strFields[2], strFields[0], strFields[1], strFields[3],                    Convert.ToDecimal(strFields[4]));问题：如何确保如果 SSAN 不等于 9 位，它所包含的所有行的数据都不会插入到数据库中？

Answer 1

您的问题出现在您的IsValidRecord. 您有一个变量 ，blnOK您可以来回切换它。

变量blnOK以 开头true，然后strFields[0]被检查。然后你设置blnOK为trueor false，这很好。但是，您随后检查strFields[1]，并继续设置blnOK为true或false基于该值，而不管之前的检查是什么。这将贯穿您的所有领域。最终结果是 的结果IsValidRecord将始终是您检查的最后一个字段的结果（在本例中为strFields[4]），无论其他检查的结果如何。

解决此问题的简单方法是，return blnOK;无论何时将其值更改为false，这都会导致函数“短路”，甚至不检查任何其他字段。失败的第一个字段将使整批字段失败。

如果要检查所有字段，则可以检查blnOK每次检查中的状态，并且仅在先前检查中true未设置时才将其设置为false。

处理此问题的另一种方法是为每个字段设置一个变量，并返回所有值的集合。就像是：

private static Boolean IsValidRecord(String[] strFields)

{

    Boolean blnOK = true;

    Boolean bln0OK = true;

    Boolean bln1OK = true;

    Boolean bln2OK = true;

    Boolean bln3OK = true;

    Boolean bln4OK = true;

    Decimal decCheck;

    //Check all fields that must be validated

    if (String.IsNullOrWhiteSpace(strFields[0]))

    {

        Boolean bln0OK = false;

        Console.WriteLine("Error: Unable to Parse First Name!");

    }

    else

    {

        bln0OK = true;

    }

    if (String.IsNullOrWhiteSpace(strFields[1]))

    {

        bln1OK = false;

        Console.WriteLine("Error: Unable to Parse Middle Initial!");

    }

    else

    {

        bln1OK = true;

    }

    if (String.IsNullOrWhiteSpace(strFields[2]))

    {

        bln2OK = false;

        Console.WriteLine("Error: Unable to Parse Last Name!");

    }

    else

    {

        bln2OK = true;

    }

        if (strFields[3].Length == 9)

        {

                bln3OK = true;

        }

        else

        {

            bln3OK = false;

            Console.WriteLine("Error: Unable to Parse SSAN!");

        }

    if (strFields[4] == null)

    {

        bln4OK = false;

    }

    else

    {

        if (Decimal.TryParse(strFields[4], out decCheck))

        {

            bln4OK = true;

        }

        else

        {

            bln4OK = false;

            Console.WriteLine("Error: Unable to Parse PayRate!");

        }

    }

    blnOK = bln0OK && bln1OK && bln2OK && bln3OK && bln4OK; 

    return blnOK;

}

blnOK = bln0OK && bln1OK && bln2OK && bln3OK && bln4OK;有效，因为仅当两个操作数都为 时才&&返回。如果其中任何一个值为，则结果将为。truetruefalsefalse