Выборка из БД возвращает только одно значение

@Illuminated · Регистрация: 13.12.2010

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

Нужно вытянуть из таблицы параметры, а вытаскивает только 1 значение.Нужна помошь:Вот код:Где натупил подскажите:

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
public static void ViewDatchik()
        {
 
            con.Open();
            List<String> datch = new List <String> ();
            com.CommandText = String.Format ("SELECT * FROM datchik WHERE id = '1'");
            NpgsqlDataReader reader;
            reader = com.ExecuteReader ();
            while (reader.Read())
            {
 
 
                             datch.Add (Convert.ToString(reader.GetValue(reader.GetOrdinal("s_number"))));
                             datch.Add (Convert.ToString(reader.GetString (reader.GetOrdinal ("type")))); 
                             datch.Add (Convert.ToString (reader.GetString (reader.GetOrdinal ("parametr")))); 
                              datch.Add (Convert.ToString (reader.GetString (reader.GetOrdinal ("status")))); 
                              datch.Add (Convert.ToString (reader.GetString (reader.GetOrdinal ("start_datchik")))); 
                              datch.Add (Convert.ToString (reader.GetInt32 (reader.GetOrdinal ("id_kategory"))));
                
 
            }
 
            reader.Close ();
            for (int i = 1; i<datch.Count; i++) 
            {
                     Console.WriteLine ("+-{0}", datch [i]);
 
                                com.CommandText = String.Format ("SELECT * FROM datchik WHERE ID = '{1}'");
                reader = com.ExecuteReader();
                while (reader.Read())
                {
 
                                   Console.WriteLine ("+-{0}", reader.GetString (reader.GetOrdinal ("s_number")));
                                   Console.WriteLine ("+-{1}", reader.GetString (reader.GetOrdinal ("type")));
                                   Console.WriteLine ("+-{2}", reader.GetString (reader.GetOrdinal ("parametr")));
                                   Console.WriteLine ("+-{3}", reader.GetString (reader.GetOrdinal ("status")));
                                   Console.WriteLine ("+-{4}", reader.GetString (reader.GetOrdinal ("start_dtchik")));
                                   Console.WriteLine ("+-{5}", reader.GetString (reader.GetOrdinal ("id_kategory")));
                }
                
                
            }
            con.Close();

Добавлено через 1 минуту
Вот так он выглядел до того, как я его переделал на то, что выше))))

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
{
 
            con.Open();
            List<String> datch = new List <String> ();
            com.CommandText = String.Format ("SELECT * FROM datchik");
            NpgsqlDataReader reader;
            reader = com.ExecuteReader ();
            while (reader.Read())
            {
 
                datch.Add (Convert.ToString(reader.GetValue(reader.GetOrdinal("s_number"))));
                datch.Add (Convert.ToString(reader.GetString (reader.GetOrdinal ("type")))); 
                datch.Add (Convert.ToString (reader.GetString (reader.GetOrdinal ("parametr")))); 
                datch.Add (Convert.ToString (reader.GetString (reader.GetOrdinal ("status")))); 
                datch.Add (Convert.ToString (reader.GetString (reader.GetOrdinal ("start_datchik")))); 
                datch.Add (Convert.ToString (reader.GetInt32 (reader.GetOrdinal ("id_kategory"))));
                //s = Convert.ToString(reader.GetInt32(reader.GetOrdinal("id")));
 
            }
 
            reader.Close ();
            for (int i = 0; i<datch.Count; i++) 
            {
                Console.WriteLine ("+-{0}", datch [i]);
 
                com.CommandText = String.Format ("SELECT * FROM datchik");
                reader = com.ExecuteReader();
                while (reader.Read())
                {
                    Console.WriteLine ("+-{0}({0}) ({1}) ({2}) ({3}) ({4}) ({5}) ({6})", reader.GetString (reader.GetOrdinal ("s_number")), reader.GetString (reader.GetOrdinal ("type")),reader.GetString (reader.GetOrdinal ("parametr")), reader.GetString (reader.GetOrdinal ("status")), reader.GetString (reader.GetOrdinal ("start_dtchik")), reader.GetString (reader.GetOrdinal ("id_kategory")), reader.GetValue(reader.GetOrdinal ("ID")));
                }
                Console.WriteLine ("|");
                reader.Close ();
            }
            con.Close();
 
 
 
 
        }

@adm_loro · 28.11.2012, 22:34

C#
1
com.CommandText = String.Format ("SELECT * FROM datchik WHERE ID = '{1}'");

вот 100 % косячная cтрока, скорее всего ожидалось:

C#
1
com.CommandText = String.Format ("SELECT * FROM datchik WHERE ID = '{0}'", id);

Добавлено через 17 минут
Ну и вообще этот код очень плох. Что он делает то я понял, а вот что он должен делать - нет.
1. У Вас везде IDisposable объекты, но Вы не используете конструкцию using(){}, чем рискуете при возникновении исключения не закрыть подключение к БД, конечно сборщик мусора таки закроет его, но это произойдет далеко не сразу.
2. Лучше использовать запросы с параметрами, чем строить запрос используя string.Format.
3. Convert.ToString (reader.GetString (reader.GetOrdinal ("parametr"))), а это вообще АД, возникает только один вопрос: ЗАЧЕМ?

@Illuminated · 28.11.2012, 22:58 **[ТС]**

Должно вывести на консоль что-то типо такого:
id s_number type parametr и.т.д
1 02 ДТТ температура +++

Добавлено через 18 минут
Вытаскивает данные из таблички на консоль в текстовом формате

@adm_loro · 29.11.2012, 18:58

вот как я бы на начальном этапе разработки решал бы эту задачу:

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
154
155
156
157
158
159
160
161
162
163
164
165
166
167
168
169
170
171
172
173
174
175
176
177
178
179
180
181
182
183
184
185
186
187
188
189
190
191
192
193
194
195
196
197
198
199
200
201
202
203
204
205
206
207
208
209
210
211
212
213
214
215
216
217
218
219
220
221
222
223
224
225
using System;
using System.Collections.Generic;
using System.Data.Common;
using System.Threading;
using System.Data;
using System.Data.SqlClient;
 
namespace DbManager
{
    class Program
    {
        static void Main()
        {
            try
            {
                const string connectionString = @"*********заменить на свою***************";
                using (var cnn = LDbConnection.OpenConnection(connectionString))
                {
                    var dathick = new DatchikDbScope(cnn);
                    //Test #1
                    var info = dathick.GetDatchikInfoAll();
                    if(info.Count > 0)
                        PrintResult(info);
                    //Test #2
                    var info2 = dathick.GetDatchikInfo(2);
                    if (info.Count > 0)
                        PrintResult(info2);
                }
            }
            catch(Exception ex)
            {
                Console.WriteLine(ex);
            }
        }
 
        private static void PrintResult(IEnumerable<DatchikInfo> info)
        {
            foreach(var line in info)
            {
                Console.WriteLine(line);
                Console.WriteLine("=============================");
            }
        }
    }
 
    /// <summary>
    /// Класс умеет безопасно открывать подключение к SQL Server
    /// Установлен таймаут на подключение и 2 попытки с интервалами в 3 секунды.
    /// </summary>
    public class LDbConnection : IDisposable
    {
        private const int AttemptsCount = 2; //кол-во попыток
        private const int AttemptsDelay = 3000; //задержка между попытками
        private const int ConnectingTimeout = 15000; //сколько будем ждать подключения
 
        private readonly ManualResetEvent _connectionReady = new ManualResetEvent(false);
        private readonly string _connectionString;
 
        public LDbConnection(string connectionString)
        {
            _connectionString = connectionString;
        }
 
        public static DbConnection OpenConnection(string connectionString)
        {
            using(var cnn = new LDbConnection(connectionString))
            {
                return cnn.ObtainConnection();
            }
        }
 
        public DbConnection ObtainConnection()
        {
            var connection = new SqlConnection(_connectionString); // Заменить на используемый Вами класс Connection
            for (var i = 0; i < AttemptsCount; ++i)
            {
                try
                {
                    if (DoTryToObtainConnection(connection))
                        return connection;
                }
                catch (Exception ex)
                {
                    if (i == AttemptsCount - 1)
                        Console.WriteLine(ex.Message);//тут лучше в лог писать
                }
                Thread.Sleep(AttemptsDelay);
            }
            throw new Exception("SQL server is not available");
        }
 
        private bool DoTryToObtainConnection(DbConnection connection)
        {
            try
            {
                _connectionReady.Reset();
                connection.StateChange += ConnectionStateChange;
                connection.Open();
 
                if (!_connectionReady.WaitOne(ConnectingTimeout, false))
                    return false;
 
                return connection.State == ConnectionState.Open;
            }
            finally
            {
                connection.StateChange -= ConnectionStateChange;
            }
        }
 
        private void ConnectionStateChange(object sender, StateChangeEventArgs e)
        {
            var connection = (SqlConnection)sender;
            if (connection.State != ConnectionState.Connecting && connection.State != ConnectionState.Executing &&
                connection.State != ConnectionState.Fetching)
                _connectionReady.Set();
        }
 
        public void Dispose()
        {
            _connectionReady.Dispose();
        }
    }
 
    /// <summary>
    /// Делает всю полезную работу, т.е. обращается к бд с запросами.
    /// </summary>
    public class DatchikDbScope
    {
        const string query_base = "SELECT [id],[s_number],[type],[parametr],[status],[start_datchik],[id_kategory] FROM datchik";
        private readonly DbConnection _connection;
        public DatchikDbScope(DbConnection connection)
        {
            _connection = connection;
        }
 
        public List<DatchikInfo> GetDatchikInfoAll()
        {
            var items = new List<DatchikInfo>();
            using (var cmd = _connection.CreateCommand())
            {
                cmd.CommandType = CommandType.Text;
                cmd.CommandText = query_base;
                using(var reader = cmd.ExecuteReader())
                {
                    while (reader.Read())
                    {
                        items.Add(Row2DatchikInfo(reader));
                    }
                }
            }
            return items;
        }
 
        public List<DatchikInfo> GetDatchikInfo(int id)
        {
            var items = new List<DatchikInfo>();
            using (var cmd = _connection.CreateCommand())
            {
                cmd.CommandType = CommandType.Text;
                cmd.CommandText = query_base + " WHERE id = @id"; // '@id' это параметр работает для SQL Server, а вот для других бд например sqlite нужно писать ':id'
                var pid = MakeParam("@id", id);
                cmd.Parameters.Add(pid);
                using (var reader = cmd.ExecuteReader())
                {
                    while (reader.Read())
                    {
                        items.Add(Row2DatchikInfo(reader));
                    }
                }
            }
            return items;
        }
 
        private static SqlParameter MakeParam(string name, object value)
        {
            return new SqlParameter { ParameterName = name, Value = value ?? DBNull.Value };
        }
        //считываем строку с проверкой каждого поля на NULL
        private static DatchikInfo Row2DatchikInfo(IDataRecord reader)
        {
            var id = reader["id"] == Convert.DBNull ? 0 : (int)reader["id"];
            var s_number = reader["s_number"] == Convert.DBNull ? 0 : (int) reader["s_number"];
            var type = reader["type"] == Convert.DBNull ? string.Empty : (string) reader["type"];
            var parametr = reader["parametr"] == Convert.DBNull ? string.Empty : (string) reader["parametr"];
            var status = reader["status"] == Convert.DBNull ? string.Empty : (string) reader["status"];
            var start_datchik = reader["start_datchik"] == Convert.DBNull ? DateTime.MinValue : (DateTime)reader["start_datchik"];
            var id_kategory = reader["id_kategory"] == Convert.DBNull ? 0 : (int)reader["Id_kategory"];
            return new DatchikInfo(id, s_number, type, parametr, status, start_datchik, id_kategory);
         }
 
 
    }
 
    /// <summary>
    /// Это данные отдой строки из таблицы
    /// </summary>
    public class DatchikInfo
    {
        public int Id { get; set; }
        public int S_Number { get; set; }
        public string Type { get; set; }
        public string Parametr { get; set; }
        public string Status { get; set; }
        public DateTime Start_datchik { get; set; }
        public int Id_kategory { get; set; }
        
        public DatchikInfo(int id, int s_number, string type, string parametr, string status, DateTime start, int id_kat)
        {
            Id = id;
            S_Number = s_number;
            Type = type;
            Parametr = parametr;
            Status = status;
            Start_datchik = start;
            Id_kategory = id_kat;
        }
 
        //вот тут уж как Вам душе угодно можно будет выводить
        public override string ToString()
        {
            return string.Format("id:{0}  \r\n s_number:{1}  \r\n type:{2}  \r\n parametr:{3}  \r\n status:{4}  \r\n start_datchik:{5}  \r\n id_kategory:{6}  \r\n", Id, S_Number, Type, Parametr, Status, Start_datchik, Id_kategory);
        }
    }
}

@Illuminated · 29.11.2012, 22:36 **[ТС]**

Спасибо огромное за ответ!!!Профессионально написано, буду разбирать(еще раз благодарочка!!!)

Новые блоги и статьи Все статьи Все блоги /
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. Программа предоставляет более. . .

@Illuminated 0 / 0 / 0 Регистрация: 13.12.2010 Сообщений: 33
	28.11.2012, 22:58 [ТС]
	Должно вывести на консоль что-то типо такого: id s_number type parametr и.т.д 1 02 ДТТ температура +++ Добавлено через 18 минут Вытаскивает данные из таблички на консоль в текстовом формате 0

@Illuminated 0 / 0 / 0 Регистрация: 13.12.2010 Сообщений: 33
	29.11.2012, 22:36 [ТС]
	Спасибо огромное за ответ!!!Профессионально написано, буду разбирать(еще раз благодарочка!!!) 0