Не добавляются данные в таблицу БД

@elektroforez · Регистрация: 15.04.2018

Author24 — интернет-сервис помощи студентам

Пытаюсь поочередно добавить данные в 3 таблицы БД: улица, адрес и постоянные клиенты. В улицу и адрес ничего не сохраняется, хотя программа заходит в блок (я чекала). В клиента инфа сохраняется, но с неправильным кодом адреса(последним в таблице, но предполагается, что последним будет только что добавленный). Я не могу понять почему именно в эти две таблицы не добавляются данные, хоть я и делала все одинаково. Уверенна, что просто упускаю что-то мелкое и глупое, помогите, пожалуйста.

C#

private void button1_Click(object sender, EventArgs e)
        {
            try 
            {
                int clientCode = this.vievClientDataGridView.Rows.Count;
                string name = textBox2.Text;
                string surname = textBox1.Text;
                string phone = textBox3.Text;
                int adressCode = this.pizzaDataSet.Адреса.Rows.Count+1;
                string street = textBox4.Text;
                string building = textBox5.Text;
                string enter = textBox6.Text;
                int streetType;
                //с парсингом valueMember какая-то фигня, поэтому так
                switch (comboBox1.SelectedIndex)
                {
                    case 0: streetType = 1; break;
                    case 1: streetType = 2; break;
                    case 2: streetType = 3; break;
                    case 3: streetType = 4; break;
                    case 4: streetType = 5; break;
                    default: streetType = 1; break ;
 
                }
                int streetCode = this.pizzaDataSet.Вулиця.Rows.Count + 1;
                bool streetExist = false;
                bool adressExist = false;
 
                //проверяю, есть ли уже улица в базе
          /*      for (int i = 0; i < pizzaDataSet.Вулиця.Rows.Count - 1; i++)
                {
                    if (street.Equals(pizzaDataSet.Вулиця.Rows[i][1].ToString()) && streetType.ToString().Equals((pizzaDataSet.Вулиця.Rows[i][2].ToString())))
                    {
                        streetCode = i + 1;
                        streetExist = true;
                    }
                }*/
                conn = new SqlConnection();
                conn.ConnectionString = "integrated security=SSPI;data source=ADMIN\\SQLEXPRESS; persist security info=false; initial catalog=Pizza";
                conn.Open();
 
                if (!streetExist)
                {
                    if (name.Equals("") || surname.Equals("") || phone.Equals("") || street.Equals("") || building.Equals("")) throw new EmptyFieldException("Не всі поля заповнені");
                    SqlCommand myCommand = conn.CreateCommand();
 
                    // сперва добавляю улицу
 
                    myCommand.CommandText = "insert into Вулиця(КодВулиці, Назва, КодТипуВулиці) values(@streetCode, @street, @streetType)";
                    myCommand.Parameters.Add("@streetCode", SqlDbType.Int, 4);
                    myCommand.Parameters.Add("@street", SqlDbType.NVarChar, 50);
                    myCommand.Parameters.Add("@streetType", SqlDbType.Int, 4);
                    myCommand.Parameters["@streetCode"].Value = streetCode + 1;
                    myCommand.Parameters["@street"].Value = street;
                    myCommand.Parameters["@streetType"].Value = streetType;
                    this.вулицяTableAdapter.Fill(this.pizzaDataSet.Вулиця);
                    MessageBox.Show("Street" + streetCode + "added");
                }
                //проверяю, есть ли адрес в базе
           /*     for (int i = 0; i < pizzaDataSet.Адреса.Rows.Count - 1; i++) 
                {
                    if (streetCode.ToString().Equals(pizzaDataSet.Адреса.Rows[i][3].ToString()) && building.Equals(pizzaDataSet.Адреса.Rows[i][1])
                                && enter.Equals(pizzaDataSet.Адреса.Rows[i][2])) 
                    {
                        adressExist = true;
                        adressCode = i + 1;
                    }
                }*/
 
                if (!adressExist)
                {
                    // добавляю адрес
                    SqlCommand myCommand2 = conn.CreateCommand();
                    myCommand2.CommandText = "insert into Адреса(КодАдреси, Будівля, [Під'їзд], КодВулиці) values(@adresCode, @building, @enter, @streetCode)";
                    myCommand2.Parameters.Add("@adressCode", SqlDbType.Int, 4);
                    myCommand2.Parameters.Add("@building", SqlDbType.NVarChar, 4);
                    myCommand2.Parameters.Add("@enter", SqlDbType.NVarChar, 4);
                    myCommand2.Parameters.Add("@streetCode", SqlDbType.Int, 4);
                    myCommand2.Parameters["@adressCode"].Value = adressCode+1;
                    myCommand2.Parameters["@building"].Value = building;
                    myCommand2.Parameters["@enter"].Value = enter;
                    myCommand2.Parameters["@streetCode"].Value = streetCode;
          //          this.адресаTableAdapter.Fill(this.pizzaDataSet.Адреса);
                    MessageBox.Show("adress " + adressCode + " added");
                }
                // и наконец-то добавляю клиента
                SqlCommand myCommand3 = conn.CreateCommand();
                myCommand3.CommandText = "insert into ПостійніКлієнти(КодКлієнта, НомерТелефону, Знижка, КодАдреси, [Ім'я], Прізвище) " +
                                            "values(@clientCode, @phone, @discount, @adressCode, @name, @surname)";
                myCommand3.Parameters.Add("@clientCode", SqlDbType.Int, 4);
                myCommand3.Parameters.Add("@phone", SqlDbType.NVarChar, 12);
                myCommand3.Parameters.Add("@discount", SqlDbType.Decimal, 3);
                myCommand3.Parameters.Add("@adressCode", SqlDbType.Int, 4);
                myCommand3.Parameters.Add("@name", SqlDbType.NVarChar, 20);
                myCommand3.Parameters.Add("@surname", SqlDbType.NVarChar, 20);
                myCommand3.Parameters["@clientCode"].Value = clientCode;
                myCommand3.Parameters["@phone"].Value = phone;
                myCommand3.Parameters["@discount"].Value = 0.05;
                myCommand3.Parameters["@adressCode"].Value = adressCode;
                myCommand3.Parameters["@name"].Value = name;
                myCommand3.Parameters["@surname"].Value = surname;
 
                int uspIzm = myCommand3.ExecuteNonQuery();
                if (uspIzm != 0) MessageBox.Show("Зміни внесені", "Зміна запису");
                else MessageBox.Show("Не вдалося внести зміни", "Зміна запису");
            }
            catch (EmptyFieldException ex)
            {
                MessageBox.Show(ex.Message, "Застереження", MessageBoxButtons.OK, MessageBoxIcon.Warning);
            }
            catch (Exception ex)
            {
                MessageBox.Show(ex.ToString());
            }
            finally
            {
                conn.Close();
            }
            this.vievClientTableAdapter.Fill(this.pizzaDataSet.VievClient);
            textBox1.Clear();
            textBox2.Clear();
            textBox3.Clear();
            textBox4.Clear();
            textBox5.Clear();
            textBox6.Clear();
        }

@Usaga · 10.05.2018, 04:24

elektroforez, говнище несусветное, а не код.

Вы сохраняете только клиентов. Для адреса и улицы (а улица не адрес, да?) вы создаёте команду, но не выполняете её (ExecuteNonQuery).

Ну и придумывать внешний ключ на запись на клиенте путём прибавления к количеству записей единицы... Это вообще туши свет.

C#

1	int streetCode = this.pizzaDataSet.Вулиця.Rows.Count + 1;

Спросите человека, который это вам написал, о чём и чем он думал.

@Welcome20 · 10.05.2018, 06:48

Usaga, после увиденных с 5 - по 12 строку кода....мне страшно стало смотреть ниже....прочитав твой ответ, пожалуй на этом остановлюсь и закрою страницу =)

@elektroforez · 10.05.2018, 16:40 **[ТС]**

Спасибо, я просто впервые пытаюсь работать с базами. Объясните пж, что не так с прибавлением единицы?

@Usaga · 10.05.2018, 17:39

elektroforez, плохо в том, что так вообще не делается. Идентификаторы либо база должна генерировать (можно и через secuence), либо GUID-ы использовать.

Идентификаторы должны быть уникальными (на то они и идентификаторы, так ведь?), а если исходить из количества записей (как вы сделали), то уникальностью пахнуть не будет. Вот у вас пять записей, вы решили, что идентификатор следующей будете 5+1 = 6. Ок. Потом вы удалили одну запись и решили вставить следующую. Получается, что следующий идентификатор опять 6! Фигня, не находите?

@elektroforez · 11.05.2018, 00:02 **[ТС]**

Действительно бред
Спасибо, что пояснили

@Welcome20 54 / 48 / 31 Регистрация: 14.04.2015 Сообщений: 273
	10.05.2018, 06:48	3
	Usaga, после увиденных с 5 - по 12 строку кода....мне страшно стало смотреть ниже....прочитав твой ответ, пожалуй на этом остановлюсь и закрою страницу =) 0

@elektroforez 0 / 0 / 0 Регистрация: 15.04.2018 Сообщений: 3
	10.05.2018, 16:40 [ТС]	4
	Спасибо, я просто впервые пытаюсь работать с базами. Объясните пж, что не так с прибавлением единицы? 0

@Usaga 12073 / 8383 / 1280 Регистрация: 21.01.2016 Сообщений: 31,578
	10.05.2018, 17:39	5
	elektroforez, плохо в том, что так вообще не делается. Идентификаторы либо база должна генерировать (можно и через secuence), либо GUID-ы использовать. Идентификаторы должны быть уникальными (на то они и идентификаторы, так ведь?), а если исходить из количества записей (как вы сделали), то уникальностью пахнуть не будет. Вот у вас пять записей, вы решили, что идентификатор следующей будете 5+1 = 6. Ок. Потом вы удалили одну запись и решили вставить следующую. Получается, что следующий идентификатор опять 6! Фигня, не находите? 1

@elektroforez 0 / 0 / 0 Регистрация: 15.04.2018 Сообщений: 3
	11.05.2018, 00:02 [ТС]	6
	Действительно бред Спасибо, что пояснили 0