C＃中if / else的条件行为不符合预期

Question

我在用C＃编写的if / else语句中遇到怪异的行为。 我是C＃的新手，也是使用Visual Studio Pro 2017的ASP.NET的新手。

我正在为Web应用程序添加功能，以允许通过用户通过Mudassar Khan上载的.CSV文件将SQL数据上传到表中

一切与此都很好。 我现在正在尝试在上传之前验证数据。我使用的是与上一篇文章相同的.CSV，但其中引入了一些“错误”条件。

1,John,Hammond,United States
2,Shouldbreak,India
3,JohnMathews,Ireland
4,Robert,Schidner,Russia,sdfsdfdf,sdfsdfsdf,

在下面的代码中，您将看到我正在测试，以确保通过设置变量中所需的列数并将其与的每行中发现的逗号数进行比较，以确保每行中的列数均正确。 CSV。

我想要的验证如下：

1. 如果列数正确，请添加到dataTable中。
2. 如果没有正确的列数。
   1. 显示包含错误消息的ASP控件。
   2. 如果列太多。
      1. 将一个字符串添加到错误消息，其中包括行号和不正确的列数。
   3. 如果列太少。
      1. 将一个字符串添加到错误消息变量，其中包括行号和不正确的列数。

我还在单独的ASP控件中跟踪和显示在每行中找到的逗号数，以进行调试。

以下是我的代码隐藏文件中的代码：

//Upload and save the file
string csvPath = Server.MapPath("~/_uploadedDataCSV/") + Path.GetFileName(FileUpload1.PostedFile.FileName);
FileUpload1.SaveAs(csvPath);

DataTable dt = new DataTable();

int columnCount = 4; // Expected number of columns
int commas = 0; // Set comma count to 0;
int tableRow = 0; // set current table row to 0;
Count.Text = string.Empty;
string errMessage = string.Empty;

dt.Columns.AddRange(new DataColumn[4] // Add expected number of columns to table
{
    // Add column Header and expected (c#)datatype 
    new DataColumn("Id", typeof(int)),
    new DataColumn("First Name", typeof(string)),
    new DataColumn("Last Name", typeof(string)),
    new DataColumn("Country",typeof(string))
});

string csvData = File.ReadAllText(csvPath); // Creates String of all content in CSV file
foreach (string row in csvData.Split('\n')) // Do the following for each row in the file
{
    // Start Validation Logic
    // - - - - - - - - - - - - - - - - - - - -

    tableRow++;

    // Count commas in row
    foreach (char ch in row)
    {
        if (ch.Equals(','))
        {
            commas++;
        }
    }

    if (!string.IsNullOrEmpty(row)) // if the row is not empty
    {
        if (commas == columnCount) // if commas in row equals the number of columns expected
        {
            dt.Rows.Add();
            int i = 0;
            foreach (string cell in row.Split(','))
            {
                dt.Rows[dt.Rows.Count - 1][i] = cell;
                i++;
            }
        }
        //Begin error reporting for row
        // - - - - - - - - - - - - - - - - - - - -
        else if (commas != columnCount)
        {
            err.Visible = true; // show error message element
            err.Text = errMessage; // set text of error message element to value of errMessage variable
            if (commas >= columnCount) // if too many columns in row
            {
                errMessage+= $"Row {tableRow} has too many columns <br />";
            }
            else if (commas < columnCount) // if not enough columns in row
            {
                errMessage+= $"Row {tableRow} does not have enough columns <br />";
            }
        }
    }

    Count.Text+= commas.ToString() + "|";
    commas = 0;
}

我遇到的问题是，当一行中的列过多时，我没有收到与此情况相关的错误消息。 调试控制器会正确显示逗号的数量，如果数量太少，则会正确显示消息，而不会显示太多消息。 当我逐步调试器中的代码时，报告所有行的变量（逗号/ columnCount）已更正。

有什么建议么？ 如果有任何遗漏，我很乐意提供更多信息。

我尝试删除过少列的情况，但仍然无法显示过多列的消息。 我也尝试过反转检查顺序，但是给出了相同的输出。 我已尝试根据此讨论进行重建。

Answer 1

所有学分都应该归@Mike McCaughan所有，他的评论绝对正确。 我只想举一个迈克所说的例子：

string a = string.Empty;
string b = a;
a += "3";
Console.WriteLine(b); //result in empty string in console

回到你的问题：

string errMessage = string.Empty;
err.Text = errMessage;
errMessage+= $"Row {tableRow} has too many columns <br />";
Console.WriteLine(err.Text); //result in empty string in console

发生这种情况的原因是因为c＃中字符串的不变性。 意味着对字符串对象的任何更改都将导致创建新的对象。 您可以在此处找到更多信息： 字符串对象的不变性