Member 12876243 Ответов: 3

Как заставить мой список возвращать отдельные строки

У меня есть два списка, заполненных базой данных. При выборе элемента в первом списке он заполняет второй список значениями, связанными с этим элементом.

Проблема, с которой я столкнулся, заключается в том, что если в первом списке выбран другой элемент, то первый выбранный элемент дублирует заполненный элемент во втором списке.

Извините, что не могу показать пример.

Что я уже пробовал:

private void Area_SelectionChanged(object sender, SelectionChangedEventArgs e)
        {
            if (Area.SelectedItems.Count < 1)
            {
                AreaPostCode.Items.Clear();
                AreasSelected.Clear();
                return;
            }
            AreasSelected.Text = "";
            foreach (object area in Area.SelectedItems)
            {
                AreasSelected.Text += (AreasSelected.Text == "" ? "" : ", ") + area.ToString();
            }

            DataView view = new DataView();
            DataTable dt = new DataTable();
            //SqlDataAdapter adpt = new SqlDataAdapter("SELECT PostCode,Town FROM tblAllPostCodes WHERE Town IN(" + string.Join(",", Area.SelectedItems.Cast<string>().Select(si => "'" + si.ToString() + "'").ToArray()) + ")", sqlConTwo);
            SqlDataAdapter adpt = new SqlDataAdapter("SELECT DISTINCT SUBSTRING(PostCode, 0, 5) AS PostCode, Town FROM tblAllPostCodes WHERE Town IN(" + string.Join(",", Area.SelectedItems.Cast<string>().Select(si => "'" + si.ToString() + "'").ToArray()) + ")", sqlConTwo);

            adpt.Fill(dt);

            foreach (DataRow dr in dt.Rows)
            {
                AreaPostCode.Items.Add(dr["PostCode"].ToString());
            }
         

        }

Richard MacCutchan

Вам нужно очистить список AreaPostCode перед добавлением дополнительных элементов.

3 Ответов

Рейтинг:
1

Patrice T 

SqlDataAdapter adpt = new SqlDataAdapter("SELECT DISTINCT SUBSTRING(PostCode, 0, 5) AS PostCode, Town FROM tblAllPostCodes WHERE Town IN(" + string.Join(",", Area.SelectedItems.Cast<string>().Select(si => "'" + si.ToString() + "'").ToArray()) + ")", sqlConTwo);

Не обязательно решение вашего вопроса, но у вас есть еще одна проблема.
Никогда не создавайте SQL-запрос путем объединения строк. Рано или поздно вы сделаете это с помощью пользовательских вводов, и это откроет дверь к уязвимости под названием "SQL-инъекция", она опасна для вашей базы данных и подвержена ошибкам.
Одна кавычка в имени - и ваша программа выйдет из строя. Если пользователь вводит имя, например "Брайан О'Коннер", может привести к сбою вашего приложения, это уязвимость SQL-инъекции, и сбой-это наименьшая из проблем, вредоносный пользовательский ввод, и он продвигается к командам SQL со всеми учетными данными.
SQL-инъекция - Википедия[^]
SQL-инъекция[^]
Атаки SQL-инъекций на примере[^]
PHP: SQL-инъекция - руководство пользователя[^]
Шпаргалка по предотвращению инъекций SQL - OWASP[^]
Как я могу объяснить SQL-инъекцию без технического жаргона? - Обмен Стеками Информационной Безопасности[^]

Рейтинг:
0

OriginalGriff

Не делай этого так! Никогда не объединяйте строки для построения команды SQL. Это оставляет вас широко открытыми для случайной или преднамеренной атаки SQL-инъекции, которая может уничтожить всю вашу базу данных. Вместо этого всегда используйте параметризованные запросы.

Когда вы объединяете строки, вы вызываете проблемы, потому что SQL получает такие команды, как:

SELECT * FROM MyTable WHERE StreetAddress = 'Baker's Wood'
Цитата, добавленная пользователем, завершает строку в том, что касается SQL, и вы получаете проблемы. Но могло быть и хуже. Если я приду и наберу вместо этого: "x';DROP TABLE MyTable;--", то SQL получит совсем другую команду:
SELECT * FROM MyTable WHERE StreetAddress = 'x';DROP TABLE MyTable;--'
Которые SQL видит как три отдельные команды:
SELECT * FROM MyTable WHERE StreetAddress = 'x';
Совершенно правильный выбор
DROP TABLE MyTable;
Вполне допустимая команда "удалить таблицу" 
--'
А все остальное-это комментарии.
Так оно и происходит: выбирает любые совпадающие строки, удаляет таблицу из базы данных и игнорирует все остальное.

Поэтому всегда используйте параметризованные запросы! Или будьте готовы часто восстанавливать свою БД из резервной копии. Вы ведь регулярно делаете резервные копии, не так ли?

Member 12876243

На этом этапе нет необходимости часто создавать резервные копии. У меня есть резервная копия базы данных. Я могу перейти на параметризованный без проблем.

OriginalGriff

Затем сделайте это на протяжении всего вашего приложения в качестве приоритета.

Рейтинг:
0

MarcusCole6833

Что касается вышеизложенного, то уточнение SQL-это лучшее решение, но за пределами лучшего SQL вы можете использовать объекты system.data, чтобы сделать вывод отличным или решение на основе linq. Что касается вышеупомянутого уточнения SQL, то это лучшее решение, это всегда последнее последнее решение.

Помимо кода, Вы можете использовать linq с потоками данных и использовать метод Distinct.

LINQ to DataSet - ADO.NET | Microsoft Docs[^]

Перечислимый.Самостоятельного Метода (Системы.По LINQ) | Майкрософт Документы[^]


DataTable dt = new DataTable();
            dt.Columns.Add(new DataColumn("name", typeof(string)));

            List<string> names = new List<string>();

            names.Add("Peter");
            names.Add("Peter");
            names.Add("Paul");
            names.Add("Paul");
            names.Add("Matthew");
            names.Add("John");
            names.Add("Rodney");
            names.Add("Derrick");
            names.Add("Albert");
            names.Add("Denzel");

            foreach(string s in names)
            {
                DataRow dr = dt.NewRow();
                dr["name"] = s;
                dt.Rows.Add(dr);
            }
            DataView view = new DataView(dt);
            DataTable distinctValues = view.ToTable(true, "name");


ВТОРОЗАКОНИЕ = 

Peter
Peter
Paul
Paul
Matthew
John
Rodney
Derrick
Albert
Denzel



уникальное значение=

Peter
Paul
Matthew
John
Rodney
Derrick
Albert
Denzel