Добавление элемента в список, если таковой отсутствует

@manumozz · Регистрация: 16.04.2015

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

У меня есть текстовый файл. В нём присутствуют названия компаний. Мне нужно создать список компаний, находить их по ключевому слову в тексте, затем сравнивать с уже имеющимся списком и, если она уже есть выводить её индекс в списке, если же нет - добавлять в список. Никак не пойму как строить логику.

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
public void Create_companies_list (ref int i)
        {
            for (i += 0; i < all_lines_articles.Count; i++)
            {
                line = all_lines_articles[i];
                if (line.Contains(DAL.Keywords.KEY_LIST_COMPANIES))
                {
                    for (i += 2; i < all_lines_articles.Count; i++)
                    {
                        line = all_lines_articles[i];
                        bool line_is_not_empty = line.Length > 2;
                        if (line_is_not_empty)
                        {
                            string company_name = line;
                            for (int company_index = 0; company_index < companies_list.Count; company_index++)
                            {
                                string company_current_name = companies_list[company_index].Company_name;
                                if (company_name == company_current_name)
                                {
                                    break;//вот здесь нужно выйти из двух циклов сразу.
                                }
                            }
                            this.companies_list.Add(new Company(company_name));
                        }
                        else
                        {
                            //return;
                            this.Create_companies_list(ref i);
                        }
                    }
                }
            }
        }

Как сделать так, чтобы при совпадении программа выходила сразу из двух циклов. Потому что, если она прервёт внутренний цикл сравнения, то это ничего не меняет, она по завершению всё равно добавит в список компанию не смотря на то, что такая в списке есть. А надо, чтобы наружный цикл запустился заново и взял следующее имя компании.

@ZX-Spectrum · 17.10.2015, 10:50

Объявите в начале метода bool flag = false;
Во внутреннем цикле перед break поставьте flag = true;
Во внешнем цикле перед его закрывающей скобкой или там где нужен выход напишите инструкцию
if (flag == true) break;
В начале внешнего цикла поставьте flag = false;

@Даценд · 17.10.2015, 10:55

C#
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
bool breaked = false;
for (int company_index = 0; company_index < companies_list.Count; company_index++)
{
   string company_current_name = companies_list[company_index].Company_name;
   if (company_name == company_current_name)
   {
      breaked=true;      
      break;//вот здесь нужно выйти из двух циклов сразу.
   }
}
if (breaked)
{
   continue;
}
this.companies_list.Add(new Company(company_name))

@Optiqfide · 17.10.2015, 17:04

Самое простое - это использовать оператор goto. Просто передать управление на операторы после цикла

@manumozz · 17.10.2015, 17:17 **[ТС]**

мне после первого, применённого мной goto, преподаватель спросил: "Желаю ли я, чтобы он прекратил со мной общаться?". Учитывая, что он дал добро на то, чтобы стать моим репетитором, баловаться с этим оператором рискованно ))))

@Optiqfide · 17.10.2015, 17:18

Ну это наверно единственный адекватный вариант его использования))

@manumozz · 17.10.2015, 17:18 **[ТС]**

пытаюсь прицепить bool флаги по первому варианту... надеюсь справлюсь.

@Optiqfide · 17.10.2015, 17:22

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
public void Create_companies_list(ref int i)
        {
            bool flag = false;
            for (i += 0; i < all_lines_articles.Count; i++)
            {
                line = all_lines_articles[i];
                if (line.Contains(DAL.Keywords.KEY_LIST_COMPANIES))
                {
                    for (i += 2; i < all_lines_articles.Count; i++)
                    {
                        line = all_lines_articles[i];
                        bool line_is_not_empty = line.Length > 2;
                        if (line_is_not_empty)
                        {
                            string company_name = line;
                            for (int company_index = 0; company_index < companies_list.Count; company_index++)
                            {
                                string company_current_name = companies_list[company_index].Company_name;
                                if (company_name == company_current_name)
                                {
                                    flag = true;//
                                    break;//вот здесь нужно выйти из двух циклов сразу.
                                }
                            }
                            if (flag) {flag = false; break;}//
                            this.companies_list.Add(new Company(company_name));
                        }
                        else
                        {
                            //return;
                            this.Create_companies_list(ref i);
                        }
                    }
                }
            }
        }

@manumozz · 17.10.2015, 17:28 **[ТС]**

Да, это практически то, что нужно итого заработал вот такой код

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 void Create_companies_list (ref int i)
        {
            bool flag = false;
            for (i += 0; i < all_lines_articles.Count; i++)
            {
                line = all_lines_articles[i];
                if (line.Contains(DAL.Keywords.KEY_LIST_COMPANIES))
                {
                    for (i += 2; i < all_lines_articles.Count; i++)
                    {
                        flag = false;
                        line = all_lines_articles[i];
                        bool line_is_not_empty = line.Length > 2;
                        if (line_is_not_empty)
                        {
                            string company_name = line;
                            for (int company_index = 0; company_index < companies_list.Count; company_index++)
                            {
                                string company_current_name = companies_list[company_index].Company_name;
                                if (company_name == company_current_name)
                                {
                                    flag = true;
                                    break;
                                }
                            }
                            if (flag == true)
                            {
                                continue;
                            }
                            else
                            {
                                this.companies_list.Add(new Company(company_name));
                            }
                        }
                        else
                        {
                            this.Create_companies_list(ref i);
                        }
                    }
                }
            }
        }

то есть, внешний цикл должен всё-таки продолжится, просто пропустив повторяющуюся компанию. Большое спасибо за помощь.

@Optiqfide · 17.10.2015, 17:29

В условии, если используешь bool, не надо писать ==true. Просто flag, ибо он и так либо false либо true

@ZX-Spectrum · 18.10.2015, 08:47

Если Goto явно не грозит запутыванием кода, а наоборот упрощает его, то почему бы им и не воспользоваться в таком случае ? Хотя я в своей практике его никогда не использую. Придерживаюсь рекомендаций старых зубров программирования. И всегда получается обойтись без него.

Новые блоги и статьи Все статьи Все блоги /
Установка Qt Creator для C и C++: ставим среду, CMake и MinGW без фреймворка Qt 8Observer8 05.04.2026 Среду разработки Qt Creator можно установить без фреймворка Qt. Есть отдельный репозиторий для этой среды: https:/ / github. com/ qt-creator/ qt-creator, где можно скачать установщик, на вкладке Releases:. . .	AkelPad-скрипты, структуры, и немного лирики.. testuser2 05.04.2026 Такая программа, как AkelPad существует уже давно, и также давно существуют скрипты под нее. Тем не менее, прога живет, периодически что-то не спеша дополняется, улучшается. Что меня в первую очередь. . .	Отображение реквизитов в документе по условию и контроль их заполнения Maks 04.04.2026 Алгоритм из решения ниже реализован на примере нетипового документа "ПланированиеСпецтехники", разработанного в конфигурации КА2. Данный документ берёт данные из другого нетипового документа. . .	Фото всей Земли с борта корабля Orion миссии Artemis II kumehtar 04.04.2026 Это первое подобное фото сделанное человеком за 50 лет. Снимок называют новым вариантом легендарной фотографии «The Blue Marble» 1972 года, сделанной с борта корабля «Аполлон-17». Новое фото. . .
Вывод диалогового окна перед закрытием, если документ не проведён Maks 04.04.2026 Алгоритм из решения ниже реализован на примере нетипового документа "СписаниеМатериалов", разработанного в конфигурации КА2. Задача: реализовать программный контроль на предмет проведения документа. . .	Программный контроль заполнения реквизитов табличной части документа Maks 02.04.2026 Алгоритм из решения ниже реализован на примере нетипового документа "СписаниеМатериалов", разработанного в конфигурации КА2. Задача: 1. Реализовать контроль заполнения реквизита. . .	wmic не является внутренней или внешней командой Maks 02.04.2026 Решение: DISM / Online / Add-Capability / CapabilityName:WMIC~~~~ Отсюда: https:/ / winitpro. ru/ index. php/ 2025/ 02/ 14/ komanda-wmic-ne-naydena/	Программная установка даты и запрет ее изменения Maks 02.04.2026 Алгоритм из решения ниже реализован на примере нетипового документа "СписаниеМатериалов", разработанного в конфигурации КА2. Задача: при создании документов установить период списания автоматически. . .

@ZX-Spectrum 8 / 8 / 0 Регистрация: 09.11.2013 Сообщений: 55
	17.10.2015, 10:50
	Объявите в начале метода bool flag = false; Во внутреннем цикле перед break поставьте flag = true; Во внешнем цикле перед его закрывающей скобкой или там где нужен выход напишите инструкцию if (flag == true) break; В начале внешнего цикла поставьте flag = false; 1

@Optiqfide 2 / 2 / 3 Регистрация: 25.09.2012 Сообщений: 8
	17.10.2015, 17:04
	Самое простое - это использовать оператор goto. Просто передать управление на операторы после цикла 0

@manumozz 2 / 2 / 0 Регистрация: 16.04.2015 Сообщений: 73
	17.10.2015, 17:17 [ТС]
	мне после первого, применённого мной goto, преподаватель спросил: "Желаю ли я, чтобы он прекратил со мной общаться?". Учитывая, что он дал добро на то, чтобы стать моим репетитором, баловаться с этим оператором рискованно )))) 1

@Optiqfide 2 / 2 / 3 Регистрация: 25.09.2012 Сообщений: 8
	17.10.2015, 17:18
	Ну это наверно единственный адекватный вариант его использования)) 0

@manumozz 2 / 2 / 0 Регистрация: 16.04.2015 Сообщений: 73
	17.10.2015, 17:18 [ТС]
	пытаюсь прицепить bool флаги по первому варианту... надеюсь справлюсь. 0

@Optiqfide 2 / 2 / 3 Регистрация: 25.09.2012 Сообщений: 8
	17.10.2015, 17:29
	В условии, если используешь bool, не надо писать ==true. Просто flag, ибо он и так либо false либо true 1

@ZX-Spectrum 8 / 8 / 0 Регистрация: 09.11.2013 Сообщений: 55
	18.10.2015, 08:47
	Если Goto явно не грозит запутыванием кода, а наоборот упрощает его, то почему бы им и не воспользоваться в таком случае ? Хотя я в своей практике его никогда не использую. Придерживаюсь рекомендаций старых зубров программирования. И всегда получается обойтись без него. 1

Добавление элемента в список, если таковой отсутствует

Решение