Подключение не было закрыто. Подключение открыто

@dariuzz · Регистрация: 30.04.2023

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

Когда открываю форму, ввожу данные и кликаю на кнопку button1 ( Это кнопка отвечает за ввод данных в бд ), появляется окно "Подключение не было закрыто. Подключение открыто"
Прошу помочь мне с исправлением ошибки

Внизу прилагаю код

C#
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
131
132
133
134
135
136
137
138
139
140
141
142
143
144
145
146
147
148
149
150
151
152
153
using System;
using System.Collections.Generic;
using System.ComponentModel;
using System.Data;
using System.Drawing;
using System.Linq;
using System.Text;
using System.Threading.Tasks;
using System.Windows.Forms;
using System.Data.SqlClient;
 
namespace Прокат_Инструментов
{
    public partial class Tools : Form
    {
        public Tools()
        {
            InitializeComponent();
        }
        SqlConnection Con = new SqlConnection(@"Data Source=(LocalDB)\MSSQLLocalDB;AttachDbFilename=C:\Users\dariuz\Documents\ToolsRentaldb.mdf;Integrated Security=True;Connect Timeout=30");
        private void populate()
        {
            Con.Open();
            string query = "select * from ToolsTbl";
            SqlDataAdapter da = new SqlDataAdapter(query, Con);
            SqlCommandBuilder builder = new SqlCommandBuilder(da);
            var ds = new DataSet();
            da.Fill(ds);
            ToolsDGV.DataSource = ds.Tables[0];
            Con.Close();
        }
        // внесение инструментов в таблицу Tools.cs
        private void button1_Click(object sender, EventArgs e)
        {
            try
            {
                if (RegNumTb.Text == "" || ToolNameTb.Text == "" || BrandTb.Text == "" || PriceTb.Text == "" ||QuantityTb.Text == "")
                    MessageBox.Show("Не во всех полях есть данные");
                else
                {
                    Con.Open();
 
                    // Проверка, существует ли уже такой продукт
                    string checkQuery = "SELECT COUNT(*) FROM ToolsTbl where RegNum=" + RegNumTb.Text + ";";
                    using (SqlCommand checkCmd = new SqlCommand(checkQuery, Con))
                    {
                        checkCmd.Parameters.AddWithValue("RegNum", RegNumTb.Text);
 
                        int count = Convert.ToInt32(checkCmd.ExecuteScalar());
 
                        if (count > 0)
                        {
                            MessageBox.Show("Продукт с таким ID уже существует!");
                            Con.Close();
                        }
 
                        else
                        {
                            Con.Close();
                            Con.Open();
                            string query = "insert into ToolsTbl values(" + RegNumTb.Text + ", " + ToolNameTb.Text + ", '" + BrandTb.Text + "','" + PriceTb.Text + "'," + QuantityTb.Text + ")";
                            SqlCommand cmd = new SqlCommand(query, Con);
                            cmd.ExecuteNonQuery();
                            MessageBox.Show("Продукт добавлен успешно!");
                            Con.Close();
                            populate();
                        }
                    }
                }
            }
            catch (Exception ex)
            {
                MessageBox.Show(ex.Message);
            }
        }
 
        private void Tools_Load(object sender, EventArgs e)
        {
            populate();
        }
 
        private void button7_Click(object sender, EventArgs e)
        {
            Application.Exit();
        }
        // удаление инструментов
        private void button3_Click(object sender, EventArgs e)
        {
            try
            {
                if (RegNumTb.Text == "")
                {
                    MessageBox.Show("Выберите продукт для удаления");
                }
                else
                {
                    Con.Open();
                    string query = "delete from ToolsTbl where RegNum=" + RegNumTb.Text + "";
                    SqlCommand cmd = new SqlCommand(query, Con);
                    cmd.ExecuteNonQuery();
                    MessageBox.Show("Продукт удален Успешно!");
                    Con.Close();
                    populate();
                }
            }
            catch (Exception Myex)
            {
                MessageBox.Show(Myex.Message);
            }
        }
 
        private void ToolsDGV_CellContentClick(object sender, DataGridViewCellEventArgs e)
        {
            RegNumTb.Text = ToolsDGV.SelectedRows[0].Cells[0].Value.ToString();
            ToolNameTb.Text = ToolsDGV.SelectedRows[0].Cells[1].Value.ToString();
            BrandTb.Text = ToolsDGV.SelectedRows[0].Cells[2].Value.ToString();
            PriceTb.Text = ToolsDGV.SelectedRows[0].Cells[3].Value.ToString();
            QuantityTb.Text = ToolsDGV.SelectedRows[0].Cells[4].Value.ToString();
        }
 
        private void button2_Click(object sender, EventArgs e)
        {
            if (RegNumTb.Text == "" || ToolNameTb.Text == "" || BrandTb.Text == "" || PriceTb.Text == "" || QuantityTb.Text == "")
            {
                MessageBox.Show("Недостающая информация");
            }
            else
            {
                try
                {
                    Con.Open();
                    string query = "update ToolsTbl set ToolName= " + ToolNameTb.Text + ", Brand= '" + BrandTb.Text + "', Price= '" + PriceTb.Text+"', Quantity = '" + QuantityTb.Text + "' where RegNum=" + RegNumTb.Text + "";
                    SqlCommand cmd = new SqlCommand(query, Con);
                    cmd.ExecuteNonQuery();
                    MessageBox.Show("Инструмент успешно изменен");
                    Con.Close();
                    populate(); 
                }
                catch (Exception Myex)
                {
                    MessageBox.Show(Myex.Message);
                }
            }
        }
 
        private void button4_Click(object sender, EventArgs e)
        {
            this.Hide();
            AdminMainForm main = new AdminMainForm();
            main.Show();
        }
    }
}

@Ludwig Valentin · 30.11.2023, 00:38

Это дело обрабатывается в каком-то другом месте.
Чтобы записывать данные в БД (базу данных), надо сначала открыть соединение с БД.
Создавать соединение - Con и открывать его лучше сразу перед сохранением данных.
Также надо ввести проверку, соединился ли ты с БД, или строка подключения неверна.
Подвесить блок try - catch на открытие соединения.
После выполнения всех операций с БД закрыть соединение.

@Usaga · 30.11.2023, 06:00

Сообщение от dariuzz

SqlConnection Con = new SqlConnection(@"Data Source=(LocalDB)\MSSQLLocalDB;AttachDbFi lename=C:\Users\dariuz\Documents\ToolsRe ntaldb.mdf;Integrated Security=True;Connect Timeout=30");

Сообщение от dariuzz

Con.Close();
Con.Open();

Не надо вот такой вот фигнёй заниматься. Создавай подключение только там, где оно нужно, открывай, используй оператор using и выбрасывай ссылку. Не храни ссылку на подключение в виде поля класса. И не будет проблем.

Примерно так:

C#
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
        private void button1_Click(object sender, EventArgs e)
        {
            try
            {
                if (RegNumTb.Text == "" || ToolNameTb.Text == "" || BrandTb.Text == "" || PriceTb.Text == "" ||QuantityTb.Text == "")
                    MessageBox.Show("Не во всех полях есть данные");
                else
                {
                    using var conn = new SqlConnection(строка подключения);
                    conn.Open();
 
                    // Проверка, существует ли уже такой продукт
                    string checkQuery = "SELECT COUNT(*) FROM ToolsTbl where RegNum=" + RegNumTb.Text + ";";
                    using (SqlCommand checkCmd = new SqlCommand(checkQuery, conn))
                    {
                        checkCmd.Parameters.AddWithValue("RegNum", RegNumTb.Text);
 
                        int count = Convert.ToInt32(checkCmd.ExecuteScalar());
 
                        if (count > 0)
                        {
                            MessageBox.Show("Продукт с таким ID уже существует!");
                            Con.Close();
                        }
 
                        else

Если не сохранять состояние (особенно там, где это на фиг не надо), то и проблем с его неконсистентностью (толи открыто подключение, толи закрыто) не будет.

@dariuzz · 30.11.2023, 20:51 **[ТС]**

Всем спасибо за помощь, моя ошибка была исправлена, была проблема с кавычками

Новые блоги и статьи Все статьи Все блоги /
делаю науч статью по влиянию грибов на сукцессию anaschu 13.03.2026 прикрепляю статью	SDL3 для Desktop (MinGW): Создаём пустое окно с нуля для 2D-графики на SDL3, Си и C++ 8Observer8 10.03.2026 Содержание блога Финальные проекты на Си и на C++: hello-sdl3-c. zip hello-sdl3-cpp. zip Результат:	Установка CMake и MinGW 13.1 для сборки С и C++ приложений из консоли и из Qt Creator в EXE 8Observer8 10.03.2026 Содержание блога MinGW - это коллекция инструментов для сборки приложений в EXE. CMake - это система сборки приложений. Здесь описаны базовые шаги для старта программирования с помощью CMake и. . .	Как дизайн сайта влияет на конверсию: 7 решений, которые реально повышают заявки Neotwalker 08.03.2026 Многие до сих пор воспринимают дизайн сайта как “красивую оболочку”. На практике всё иначе: дизайн напрямую влияет на то, оставит человек заявку или уйдёт через несколько секунд. Даже если у вас. . .
Модульная разработка через nuget packages DevAlt 07.03.2026 Сложившийся в . Net-среде способ разработки чаще всего предполагает монорепозиторий в котором находятся все исходники. При создании нового решения, мы просто добавляем нужные проекты и имеем. . .	Модульный подход на примере F# DevAlt 06.03.2026 В блоге дяди Боба наткнулся на такое определение: В этой книге («Подход, основанный на вариантах использования») Ивар утверждает, что архитектура программного обеспечения — это структуры,. . .	Управление камерой с помощью скрипта OrbitControls.js на Three.js: Вращение, зум и панорамирование 8Observer8 05.03.2026 Содержание блога Финальная демка в браузере работает на Desktop и мобильных браузерах. Итоговый код: orbit-controls-threejs-js. zip. Сканируйте QR-код на мобильном. Вращайте камеру одним пальцем,. . .	SDL3 для Web (WebAssembly): Синхронизация спрайтов SDL3 и тел Box2D 8Observer8 04.03.2026 Содержание блога Финальная демка в браузере. Итоговый код: finish-sync-physics-sprites-sdl3-c. zip На первой гифке отладочные линии отключены, а на второй включены:. . .

@Ludwig Valentin Valechka-Думай о Высоком 171 / 290 / 26 Регистрация: 11.08.2022 Сообщений: 3,713
	30.11.2023, 00:38
	Это дело обрабатывается в каком-то другом месте. Чтобы записывать данные в БД (базу данных), надо сначала открыть соединение с БД. Создавать соединение - Con и открывать его лучше сразу перед сохранением данных. Также надо ввести проверку, соединился ли ты с БД, или строка подключения неверна. Подвесить блок try - catch на открытие соединения. После выполнения всех операций с БД закрыть соединение. 0

@dariuzz 0 / 0 / 0 Регистрация: 30.04.2023 Сообщений: 4
	30.11.2023, 20:51 [ТС]
	Всем спасибо за помощь, моя ошибка была исправлена, была проблема с кавычками 0