Ошибка при выборе, изменении, удалении и обновлении записей в БД

@vazovski · Регистрация: 29.10.2014

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

Не работают кнопки, ошибки каждой приведены ниже на скриншотах.
привожу код программы и проект в целом (см.архив)
помогите исправить ошибки

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
using System;
using System.Collections.Generic;
using System.Linq;
using System.Text;
using System.Threading.Tasks;
using System.Data.OleDb;
using Domain;
using System.Data;
 
namespace Session
{
    public class Broker
    {
        OleDbConnection connection;
        OleDbCommand command;
 
        private void connectTo()
        {
            connection = new OleDbConnection(@"Provider=Microsoft.Jet.OLEDB.4.0;Data Source=C:\Users\Я\Documents\Visual Studio 2012\Projects\Exp\Database.mdb");
            command = connection.CreateCommand();
        }
        public Broker()
        {
            connectTo();
        }
        public void Insert(Person p)
        {
            try
            {
                command.CommandText = "INSERT INTO TPersons (FirstName, LastName) VALUES(" + p.FirstName + "," + p.LastName + ")";
                command.CommandType = CommandType.Text;
                connection.Open();
                command.ExecuteNonQuery();
 
 
            }
            catch (Exception)
            {
                throw;
 
            }
            finally
            {
            }
            if (connection != null)
            {
 
                connection.Close();
            }
        }
 
        public List<Person> FillComboBox()
        {
            List<Person> personsList = new List<Person>();
 
 
            try
            {
                command.CommandText = "SELECT * FROM TPersons";
                command.CommandType = CommandType.Text;
                connection.Open();
 
                OleDbDataReader reader = command.ExecuteReader();
                while (reader.Read())
                {
                    Person p = new Person();
                    p.Id = Convert.ToInt32(reader["ID"].ToString());
                    p.FirstName = reader["FirstName"].ToString();
                    p.LastName = reader["LastName"].ToString();
 
                    personsList.Add(p);
                }
                return personsList;
 
 
            }
            catch (Exception)
            {
                throw;
 
            }
            finally
            {
 
                if (connection != null)
                {
 
                    connection.Close();
                }
            }
 
        }
        public void Update(Person oldPerson, Person newPerson)
        {
            try
            {
                command.CommandText = "UPDATE TPersons SET FirstName="+newPerson.FirstName+" LastName="+newPerson.LastName+" WHERE ID="+oldPerson.Id;
                command.CommandType = CommandType.Text;
                connection.Open();
 
                command.ExecuteNonQuery();
 
            }
            catch (Exception)
            {
                throw;
 
            }
            finally
            {
 
                if (connection != null)
                {
 
                    connection.Close();
                }
            }
 
        }
        public void Delete(Person p)
        {
            try
            {
                command.CommandText = "DELETE FROM TPersons WHERE ID= "+p.Id;
                command.CommandType = CommandType.Text;
                connection.Open();
 
                command.ExecuteNonQuery();
 
            }
            catch (Exception)
            {
                throw;
 
            }
            finally
            {
 
                if (connection != null)
                {
 
                    connection.Close();
                }
            }
        }
 
 
    }
}

@FallenAngel76ru · 03.11.2014, 23:44

1) Так, запихни код сначала в местный конструктор c# для упрощения просмотра кода.
2) Где выборка у тебя идёт(SELECT, сделай так:

C#
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
connection.Open();
using (OleDbCommand command_1 = new OleDbCommand("SELECT * FROM TPerson", connection))
{
using (OleDbDataReader reader = command_1.ExecuteReader())
{
while(reader.Read())
{
Person p = new Person();
p.Id = Convert.ToInt32(reader[0]);
p.FirstName = reader[1].ToString();
p.LastName = reader[2].ToString();
personsList.Add(p);
}
}
}
return personsList;

3) Где INSERT и UPDATE - передавай параметры для добавления/изменения, причём указывай так

C#
1
VALUES("'" + p.FirstName + "', '" + p.LastName + "'");

4) Ну, и где DELETE - то же самое - указывай так

C#
1
"DELETE FROM TPersons WHERE ID= '" + p.Id + "'";

@vazovski · 04.11.2014, 00:01 **[ТС]**

а Вы запускали программу после внесения этих изменений?
у меня по-прежнему не работает

@FallenAngel76ru · 04.11.2014, 00:08

Максимум, что смог сделать - это данные поправки.
Если адаптируешь под Framework 4.0 программу, то может гляну. с 4.5 не могу работать, ибо нет нужного софта(и не поставить).
И да, ошибки те же лезут ?

@vazovski · 04.11.2014, 13:18 **[ТС]**

да...ошибки те же

Добавлено через 2 минуты
в одном и том же месте постоянно

C#
1
2
3
4
5
6
7
8
catch (Exception)
            {
                throw;
 
            }
            finally
            {
            }

@pincet · 04.11.2014, 14:53

Ясен перец

C#
1
"INSERT INTO TPersons (FirstName, LastName) VALUES(" + p.FirstName + "," + p.LastName + ")";

не взлетает. Текстовые данные принято в кавычки заключать.
А так-то параметры пользуй - не будет траблов таких

@vazovski · 05.11.2014, 16:27 **[ТС]**

"INSERT INTO TPersons (FirstName, LastName) VALUES("'" + p.FirstName + "', '" + p.LastName + "'");

выдает ошибку в этом месте "Превышение допустимого числа символов в символьной константе".
Как это исправить?

@pincet · 05.11.2014, 17:08

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

Как это исправить?

послушай взрослых, используй параметры

@FallenAngel76ru · 05.11.2014, 17:25

Именно.
Пробуй с параметрами.
Как делал здесь с параметрами, так и тут сделай.
к примеру в параметр @FirstName передай своё p.FirstName с соответствующими в таблице типами данных.

@vazovski · 05.11.2014, 17:48 **[ТС]**

таким образом?
"INSERT into TPersons (FirstName, LastName)" + "values(" + p.FirstName + "," + p.LastName + ")";

@FallenAngel76ru · 05.11.2014, 19:13

Проверить не на чем, потому пробуй сам:

C#
1
2
3
4
5
6
7
8
using (OleDbCommand cmd = new OleDbCommand("INSERT INTO TPersons(FirstName, LastName) VALUES(@FirstName, @LastName)", connection))
{
     cmd.Parameters.Add("@FirstName", OleDbType.VarChar, 100);
     cmd.Parameters["@FirstName"].Value = p.FirstName;
     cmd.Parameters.Add("@LastName", OleDbType.VarChar, 100);
     cmd.Parameters["@LastName"].Value = p.LastName;
     cmd.ExecuteNonQuery();
}

Новые блоги и статьи Все статьи Все блоги /
http://iceja.net/ сервер решения полиномов iceja 18.01.2026 Выкатила http:/ / iceja. net/ сервер решения полиномов (находит действительные корни полиномов методом Штурма). На сайте документация по API, но скажу прямо VPS слабенький и 200 000 полиномов. . .	Первый деплой lagorue 16.01.2026 Не спеша развернул своё 1ое приложение в kubernetes. А дальше мне интересно создать 1фронтэнд приложения и 2 бэкэнд приложения развернуть 2 деплоя в кубере получится 2 сервиса и что-бы они. . .	Расчёт переходных процессов в цепи постоянного тока igorrr37 16.01.2026 / * Дана цепь постоянного тока с R, L, C, k(ключ), U, E, J. Программа составляет систему уравнений по 1 и 2 законам Кирхгофа, решает её и находит: токи, напряжения и их 1 и 2 производные при t = 0;. . .	Восстановить юзерскрипты Greasemonkey из бэкапа браузера damix 15.01.2026 Если восстановить из бэкапа профиль Firefox после переустановки винды, то список юзерскриптов в Greasemonkey будет пустым. Но восстановить их можно так. Для этого понадобится консольная утилита. . .
Изучаю kubernetes lagorue 13.01.2026 А пригодятся-ли мне знания kubernetes в России?	Сукцессия микоризы: основная теория в виде двух уравнений. anaschu 11.01.2026 https:/ / rutube. ru/ video/ 7a537f578d808e67a3c6fd818a44a5c4/	WordPad для Windows 11 Jel 10.01.2026 WordPad для Windows 11 — это приложение, которое восстанавливает классический текстовый редактор WordPad в операционной системе Windows 11. После того как Microsoft исключила WordPad из. . .	Classic Notepad for Windows 11 Jel 10.01.2026 Old Classic Notepad for Windows 11 Приложение для Windows 11, позволяющее пользователям вернуть классическую версию текстового редактора «Блокнот» из Windows 10. Программа предоставляет более. . .

@vazovski 0 / 0 / 0 Регистрация: 29.10.2014 Сообщений: 24
	04.11.2014, 00:01 [ТС]
	а Вы запускали программу после внесения этих изменений? у меня по-прежнему не работает 0

@FallenAngel76ru 24 / 24 / 18 Регистрация: 11.03.2012 Сообщений: 175
	04.11.2014, 00:08
	Максимум, что смог сделать - это данные поправки. Если адаптируешь под Framework 4.0 программу, то может гляну. с 4.5 не могу работать, ибо нет нужного софта(и не поставить). И да, ошибки те же лезут ? 0

@vazovski 0 / 0 / 0 Регистрация: 29.10.2014 Сообщений: 24
	05.11.2014, 16:27 [ТС]
	"INSERT INTO TPersons (FirstName, LastName) VALUES("'" + p.FirstName + "', '" + p.LastName + "'"); выдает ошибку в этом месте "Превышение допустимого числа символов в символьной константе". Как это исправить? 0

@FallenAngel76ru 24 / 24 / 18 Регистрация: 11.03.2012 Сообщений: 175
	05.11.2014, 17:25
	Именно. Пробуй с параметрами. Как делал здесь с параметрами, так и тут сделай. к примеру в параметр @FirstName передай своё p.FirstName с соответствующими в таблице типами данных. 0

@vazovski 0 / 0 / 0 Регистрация: 29.10.2014 Сообщений: 24
	05.11.2014, 17:48 [ТС]
	таким образом? "INSERT into TPersons (FirstName, LastName)" + "values(" + p.FirstName + "," + p.LastName + ")"; 0