Паразитизм при заполнении List<Search>

@Samidare · Регистрация: 29.10.2010

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

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
        public static Search[] Search(string s)
        {
            string[] node_l = { "model", "manufacturer", "designations", "color", "types_Video", "video_ati" ,
                             "video_nvidia", "diagonal", "cpu_intel", "cpu_amd", "ram", "port", "hdd", "wireless",
                             "bluetooth","os","description"};
            string[] search = s.Split('/');
            CL.Search t = new Search();
            List<Search> ts = new List<Search>();
            int count = 0;
            List<int> i = new List<int>();
            foreach (string sk in search)
            {
                if (sk != "Не важно")
                {
                    i.Add(count);
                }
                count++;
            }
            bool f = false;
            foreach (XElement el in xDoc.Root.Elements("model_laptop").Elements("laptop"))
            {
                foreach (int e in i)
                {
                    if (el.Attribute(node_l[e + 1]).Value.ToLower() == search[e].ToLower())
                    {
                        f = true;
                    }
                    else
                    {
                        f = false;
                        break;
                    }
                }
                if (f)
                {
                    t.model = el.Attribute(node_l[0]).Value;
                    t.description = el.Attribute(node_l[16]).Value;
                    ts.Add(t);
                }
            }
            return ts.ToArray();
        }

Суть проблемы заключается в том, что при каждой "истинной" итерации foreach в списке List<Search> ts, добавляется новый элемент, но при этом все предыдущие элементы принимают значение последнего. Как можно это исправить, что-бы предыдущие элементы не изменялись?
P.S.: вышеизложенный код, это попытка создания поиска по xml-файлу, а т.к. опыт программирования в C# у меня 3 недели, приму на заметку все "облегчения" кода для реализации поиска.

@ArtOfLife · 30.10.2010, 00:26

Если хотите увидеть хоть один ответ по делу, перепишите функцию так, чтобы хотя бы было понятно, что там происходит. Привыкайте называть переменные своими именами, а не произвольными буквами. Код абсолютно не читабелен. Еще вам следует обратить внимание на ваш foreach. Как минимум в одном месте рано или поздно вылетит эксепшен выхода за пределы границ массива. Плюс ко всему, при имеющемся коде формирования флага, можно получить совершенно непредсказуемые результаты.
Собственно предлагаю вам полностью преобразить функцию для улучшения читабельности, а также - привести пример обращения к данной функции: какую строку передаете? Да и что хранит в себе объект Search?

@Samidare · 30.10.2010, 08:35 **[ТС]**

Понятно.

Добавлено через 33 минуты
Понятно.

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
        public static Search[] Search(string szSearchIn)
        {
            //заполняется массив названиями атрибутов елемента "Laptop" 
            string[] arrNodeList = { "model", "manufacturer", "designations", "color", "types_Video", "video_ati" ,
                             "video_nvidia", "diagonal", "cpu_intel", "cpu_amd", "ram", "port", "hdd", "wireless",
                             "bluetooth","os","description"};
            //разделяем строку на подстроки
            string[] arrSearch = szSearchIn.Split('/');
            CL.Search Search = new Search();
            List<Search> listSearch = new List<Search>();
            int nCount = 0;
            List<int> listCount = new List<int>();
            //Перебираем все параметры поиска, для иключения ненужных(имеющих значение "Не важно")
            foreach (string szSearch in arrSearch)
            {
                if (szSearch != "Не важно")
                {
                    listCount.Add(nCount);
                }
                nCount++;
            }
            //устанавливаем значение флага "ложь"
            bool f = false;
            //перебираем все элементы типа "laptop" в xml-файле
            foreach (XElement Element in xDoc.Root.Elements("model_laptop").Elements("laptop"))
            {
                foreach (int nElement in listCount)
                {
                    //если значение аттрибута элемента совпадает с поисковым, то флаг принимает значение "истина",
                    // и если хоть один не совпадает, флаг принимает значение "ложь" и прерывает перебор.
                    if (Element.Attribute(arrNodeList[nElement + 1]).Value.ToLower() == arrSearch[nElement].ToLower())
                    {
 
                        f = true;
                    }
                    else
                    {
                        f = false;
                        break;
                    }
                }
                //если флаг всё-таки остался "истинным" то добавляется новая запись в listSearch
                if (f)
                {
                    Search.model = Element.Attribute(arrNodeList[0]).Value;
                    Search.description = Element.Attribute(arrNodeList[16]).Value;
                    listSearch.Add(Search);
                }
            }
        }

класс Search:

C#
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
using System;
using System.Collections.Generic;
using System.Linq;
using System.Text;
 
namespace CL
{
    public class Search
    {
        //модель
        public string model;
        //описание
        public string description;
        public override string ToString()
        {
            return this.model;
        }
    }
}

пример передаваемой строки szSearchIn: "Asus/нетбук/чёрный/встроенное/Не важно/Не важно/Не важно/Не важно/Не важно/1024 мб/Не важно/160 гб/Не важно/Не важно/Windows XP Home"
P.S.: как удалить вышестоящий пост? нечаянно отправил.

@ArtOfLife · 30.10.2010, 13:39

По порядку.

C#
1
if (Element.Attribute(arrNodeList[nElement + 1]).Value.ToLower() == arrSearch[nElement].ToLower()) {/*...*/}

Зачем вы прибавляете единицу? На глаз видно, что если на вход поступит строка, в которой присутствуют все элементы (без атрибута "Не важно"), вы получите эксепшен выхода за границы массива, т.к. вряд ли программа найдет элемент arrNodeList с индексом arrNodeList.Length. Нумерация ведь с нуля начинается.
Далее... Механизм формирования флага. Вы ведь понимаете, что вы будете автоматически исключать правильные варианты, пока первые значения строки arrSearch будут равны "Не важно"?! Если я правильно понял, вы осуществляете запись в список при любом из совпадений, т.е. это - ключевой параметр. Тогда зачем вам флаг нужен? Делайте так:

C#
1
2
3
4
5
6
7
8
9
10
                foreach (int nElement in listCount)
                {
                    if (Element.Attribute(arrNodeList[nElement]).Value.ToLower() == arrSearch[nElement].ToLower())
                    {
                        Search.model = Element.Attribute(arrNodeList[0]).Value;
                        Search.description = Element.Attribute(arrNodeList[16]).Value;
                        listSearch.Add(Search);
                        break;
                    }
                }

Если же я неправильно понял вас, то опишите, по какому конкретно критерию вы осуществляете поиск. Ваш алгоритм?

@Samidare · 30.10.2010, 14:33 **[ТС]**

По порядку:
arrNodeList отличается от arrSearch тем, что в нём на 2 элемента больше(самый первый и самый последний) которые не участвуют в поиске.
В arrNodeList

Code
1
"model", "manufacturer", "designations", "color", "types_Video", "video_ati" ,"video_nvidia", "diagonal", "cpu_intel", "cpu_amd", "ram", "port", "hdd", "wireless", "bluetooth","os","description"

в arrSearch

Code
1
"manufacturer", "designations", "color", "types_Video", "video_ati" , "video_nvidia", "diagonal", "cpu_intel", "cpu_amd", "ram", "port", "hdd", "wireless", "bluetooth","os"

именно из-за этого и добавляется 1.
Формирование флага возможно и не обычное, но оно справляется с главной задачей, а именно отсеять записи в которых значение атрибутов(за исключением тех, что были "Не важно") не совпадают с поисковыми. Т.е. необходимо находить строки с полным соответствием (за исключением атрибутов с "Не важно"). Поиск выполняется нормально, но при добавлении новой записи в List все его элементы становятся одинаковыми с последним, это и есть основная проблема.

@m0nax · 30.10.2010, 15:15

Search.model = Element.Attribute(arrNodeList[0]).Value;
Search.description = Element.Attribute(arrNodeList[16]).Value;
listSearch.Add(Search);

в этом куске проблема, что бы элементы были разные (и независимые)нужно создавать новый Search и добавлять его в список, а не менять значения в старом
это же ссылочный тип...тема уже разжевана везде где только можно

@Samidare · 30.10.2010, 15:42 **[ТС]**

to m0nax, спасибо, действительно проблема в этом.
PS: не ругайся, я третью неделю изучаю C#

Новые блоги и статьи Все статьи Все блоги /
SDL3 для Web (WebAssembly): сборка C/C++ проекта из консоли 8Observer8 30.01.2026 Содержание блога Если вы откроете примеры для начинающих на официальном репозитории SDL3 в папке: examples, то вы увидите, что все примеры используют следующие четыре обязательные функции, а. . .	Установка Emscripten SDK (emsdk) и CMake на Windows для сборки C и C++ приложений в WebAssembly (Wasm) 8Observer8 30.01.2026 Чтобы скачать Emscripten SDK (emsdk) необходимо сначало скачать и уставить Git: Install for Windows. Следуйте стандартной процедуре установки Git через установщик. Система контроля версиями Git. . .	Подключение Box2D v3 к SDL3 для Android: физика и отрисовка коллайдеров 8Observer8 29.01.2026 Содержание блога Box2D - это библиотека для 2D физики для анимаций и игр. С её помощью можно определять были ли коллизии между конкретными объектами. Версия v3 была полностью переписана на Си, в. . .	Инструменты COM: Сохранение данный из VARIANT в файл и загрузка из файла в VARIANT bedvit 28.01.2026 Сохранение базовых типов COM и массивов (одномерных или двухмерных) любой вложенности (деревья) в файл, с возможностью выбора алгоритмов сжатия и шифрования. Часть библиотеки BedvitCOM Использованы. . .
Загрузка PNG с альфа-каналом на SDL3 для Android: с помощью SDL_LoadPNG (без SDL3_image) 8Observer8 28.01.2026 Содержание блога SDL3 имеет собственные средства для загрузки и отображения PNG-файлов с альфа-каналом и базовой работы с ними. В этой инструкции используется функция SDL_LoadPNG(), которая. . .	Загрузка PNG с альфа-каналом на SDL3 для Android: с помощью SDL3_image 8Observer8 27.01.2026 Содержание блога SDL3_image - это библиотека для загрузки и работы с изображениями. Эта пошаговая инструкция покажет, как загрузить и вывести на экран смартфона картинку с альфа-каналом, то есть с. . .	Влияние грибов на сукцессию anaschu 26.01.2026 Бифуркационные изменения массы гриба происходят тогда, когда мы уменьшаем массу компоста в 10 раз, а скорость прироста биомассы уменьшаем в три раза. Скорость прироста биомассы может уменьшаться за. . .	Воспроизведение звукового файла с помощью SDL3_mixer при касании экрана Android 8Observer8 26.01.2026 Содержание блога SDL3_mixer - это библиотека я для воспроизведения аудио. В отличие от инструкции по добавлению текста код по проигрыванию звука уже содержится в шаблоне примера. Нужно только. . .

@ArtOfLife 145 / 145 / 26 Регистрация: 09.10.2009 Сообщений: 261
	30.10.2010, 00:26
	Если хотите увидеть хоть один ответ по делу, перепишите функцию так, чтобы хотя бы было понятно, что там происходит. Привыкайте называть переменные своими именами, а не произвольными буквами. Код абсолютно не читабелен. Еще вам следует обратить внимание на ваш foreach. Как минимум в одном месте рано или поздно вылетит эксепшен выхода за пределы границ массива. Плюс ко всему, при имеющемся коде формирования флага, можно получить совершенно непредсказуемые результаты. Собственно предлагаю вам полностью преобразить функцию для улучшения читабельности, а также - привести пример обращения к данной функции: какую строку передаете? Да и что хранит в себе объект Search? 0

@Samidare 0 / 0 / 0 Регистрация: 29.10.2010 Сообщений: 4
	30.10.2010, 15:42 [ТС]
	to m0nax, спасибо, действительно проблема в этом. PS: не ругайся, я третью неделю изучаю C# 0