Что не так с моим заявлением IF? - PullRequest
Новая
       30

Что не так с моим заявлением IF?

0 голосов
/ 07 октября 2010

Я создаю таблицу аудита, и у меня есть простые методы аудита Вставить и Удалить.Я немного застрял в методе обновления - мне нужно иметь возможность получить текущие значения в базе данных, новые значения в параметрах запроса и сравнить их, чтобы я мог ввести старые значения и измененные значения в таблицув базе данных.

Вот мой код: 

    protected void SqlDataSource1_Updating(object sender, SqlDataSourceCommandEventArgs e)
    {
        string[] fields = null;
        string fieldsstring = null;
        string fieldID = e.Command.Parameters[5].Value.ToString();
        System.Security.Principal.  WindowsPrincipal p = System.Threading.Thread.CurrentPrincipal as System.Security.Principal.WindowsPrincipal;
        string[] namearray = p.Identity.Name.Split('\\');
        string name = namearray[1];
        string queryStringupdatecheck = "SELECT VAXCode, Reference, CostCentre, Department, ReportingCategory FROM NominalCode WHERE ID = @ID";
        string queryString = "INSERT INTO Audit (source, action, itemID, item, userid, timestamp) VALUES (@source, @action, @itemID, @item, @userid, @timestamp)";
        using (SqlConnection connection = new SqlConnection("con string = deleted for privacy"))
        {
            SqlCommand commandCheck = new SqlCommand(queryStringupdatecheck, connection);
            commandCheck.Parameters.AddWithValue("@ID", fieldID);
            connection.Open();
            SqlDataReader reader = commandCheck.ExecuteReader();
            while (reader.Read())
            {
                for (int i = 0; i < reader.FieldCount - 1; i++)
                {
                    if (reader[i].ToString() != e.Command.Parameters[i].Value.ToString())
                    {
                        fields[i] = e.Command.Parameters[i].Value.ToString() + "Old value: " + reader[i].ToString();
                    }
                    else
                    {

                    }
                }
            }
                fieldsstring = String.Join(",", fields);

                reader.Close();

            SqlCommand command = new SqlCommand(queryString, connection);
            command.Parameters.AddWithValue("@source", "Nominal");
            command.Parameters.AddWithValue("@action", "Update");
            command.Parameters.AddWithValue("@itemID", fieldID);
            command.Parameters.AddWithValue("@item", fieldsstring);
            command.Parameters.AddWithValue("@userid", name);
            command.Parameters.AddWithValue("@timestamp", DateTime.Now);
            try
            {
                command.ExecuteNonQuery();
            }
            catch (Exception x)
            {
                Response.Write(x);
            }
            finally
            {
                connection.Close();
            }
        }
    }

У меня проблема в том, что массив fields [] ВСЕГДА равен нулю.Несмотря на то, что в окне отладки VS показано, что e.Command.Parameter.Value [i] и reader [i] отличаются, переменная fields выглядит так, как будто она никогда не вводится.

Спасибо

Ответы [ 3 ]

1 голос
/ 07 октября 2010

Вы никогда не устанавливаете для своего fields[] ничего, кроме нуля, поэтому оно является нулевым, когда вы пытаетесь получить к нему доступ.Вам нужно создать массив, прежде чем вы сможете присвоить ему значения.Попробуйте: 

           SqlDataReader reader = commandCheck.ExecuteReader();
           fields = new string[reader.FieldCount]
0 голосов
/ 07 октября 2010

Do fields = new string[reader.FieldCount], чтобы у вас был массив для назначения.Вы пытаетесь написать на null[0].

0 голосов
/ 07 октября 2010

Я не совсем понимаю, что вы здесь делаете, но если вы проводите аудит, почему бы вам просто не вставить каждое изменение в таблицу аудита вместе с отметкой времени?

...