Обнаружен недостижимый код в программе

@BrainShaker01 · Регистрация: 23.10.2015

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

Сегодня услышал интересную задачу от друга, решил попробовать его реализовать.

Суть задачи в следующем:
В небольшом отеле 10 комнат. Каждого из пришедших заселяют в случайно выбранную комнату.
Если в случайно выбранной комнате уже есть постоялец, всех постояльцев выгоняют из отеля.
Вопрос:
Сколько итераций понадобится чтобы все 10 комнат были заполнены лишь одним постояльцем в каждой комнате?

Решение делал следующим образом:
Сделать рандомный генератор;
Зачислять +1 в выбранную рандомную комнату;
Делать проверку после каждого зачисления;
Если после проверки в комнате более 1 человека, то:
- итерацию прекратить;
- итерация +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
 
            Random R = new Random();
            int i, b, n = 1, d = 0;
            int[] A = new int[10];
 
            link0:
            for (i = 0; i < 10; i++)
                A[i] = 0;
            
            link2:
            d = 0;
            b = R.Next(0,9);
            A[b]++;
            for (i = 0; i < 10; i++)
            {
                if (A[i] > 1)
                {
                    Console.WriteLine("Iteration number {0}", n);
                    Console.WriteLine("Ended up with 2 people in room #{0}", i+1);
                    n++;
                    goto link0;
                }
                else if (A[i] == 1)
                {
                    d++;
                    if (d == 10)
                        goto link1;
                }
                goto link2;
            }
            link1:
            Console.WriteLine("Iteration number {0} ended up with 1 people in each room!", n);
            Console.ReadLine();

Проблема вылезла в проверке номеров на наличие постояльцев (т.е. чисел больше единицы в массиве). В коде это 14 строка. Ошибка под названием "Обнаружен недостижимый код". Когда я просматривал программу с помощью точки остановы, то заметил, что программа попросту пропускала цикл. В результате программа зацикливается и сбрасывает все данные массива лишь при A[0]>1.

Как можно исправить программу, чтобы она целиком просматривала цикл, а не проходила сквозь него?

@Aael · 06.06.2016, 20:28

BrainShaker01, не используйте goto, не проверяйте число постояльцев после зачисления. Их нужно проверять ДО зачисления (после выбора номера комнаты) и только в той комнате куда вы хотите приписать нового жителя.

По программе:

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
 
            Random R = new Random();
            int i, b, n = 1, d = 0;
            int[] A = new int[10];
 
            link0:
            for (i = 0; i < 10; i++)
                A[i] = 0;
            
            link2:
            d = 0;
            b = R.Next(0,10); // ДО 10
            A[b]++;
            for (i = 0; i < 10; i++)
            {
                if (A[i] > 1)
                {
                    Console.WriteLine("Iteration number {0}", n);
                    Console.WriteLine("Ended up with 2 people in room #{0}", i+1);
                    n++;
                    goto link0;
                }
                else if (A[i] == 1)
                {
                    d++;
                    if (d == 10)
                        goto link1;
                }
                
            }
            goto link2; // ПОСЛЕ ПРОВЕРКИ ВСЕХ КОМНАТ
            link1:
            Console.WriteLine("Iteration number {0} ended up with 1 people in each room!", n);
            Console.ReadLine();

@EvilFromHell · 06.06.2016, 20:31

Даже не смотря внимательно на код видно сразу goto link2;, то есть безусловный выход с цикла на первой итерации при любых условиях. Причем переход выше цикла. Оттуда и недостижимый код, и зацикливание. Вообще не стоит слишком много лепить goto в программе. Оно чаще всего надо для выхода с вложенных циклов, а злоупотребление им приводит к тому, что код тяжело читать.

@DataPlanner · 06.06.2016, 21:52

Эта задача по графам, решайте ее соответственно

@BrainShaker01 · 07.06.2016, 10:55 **[ТС]**

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

b = R.Next(0,10); // ДО 10

Тут вы не правы, у меня как раз комнаты нумеруются от 0 до 9, в обратном случае может выйти ошибка "Index out of range exception".

@EvilFromHell · 07.06.2016, 10:59

BrainShaker01, читаем MSDN:

maxValue
Исключенный верхний предел создаваемого случайного числа.Значение maxValue должно быть больше или равно 0.

Новые блоги и статьи Все статьи Все блоги /
Валидация и контроль данных табличной части документа перед записью Maks 22.04.2026 Алгоритм из решения ниже реализован на примере нетипового документа, разработанного в КА2. Задача: контроль и валидация данных табличной части документа перед записью с учетом регламента компании. . .	Отчёт о затраченных материалах за определенный период с макетом печатной формы Maks 21.04.2026 Отчёт из решения ниже размещён в конфигурации КА2. Задача: разработка отчёта по затраченным материалам за определённый период, с возможностью вывода печатной формы отчёта с шапкой и подвалом. В. . .	Отчёт о спецтехнике находящейся в ремонте Maks 20.04.2026 Отчёт из решения ниже размещен в конфигурации КА2. Задача: отобразить спецтехнику, которая на данный момент находится в ремонте. Есть нетиповой документ "Заявка на ремонт спецтехники" который. . .	Памятка для бота и "визитка" для читателей "Semantic Universe Layer (Слой семантической вселенной)" Hrethgir 19.04.2026 Сгенерировано для краткого описания по случаю сборки и компиляции скелета серверного приложения. И пусть после этого скажут, что статьи сгенерированные AI - туфта и не интересно. И это не реклама -. . .
Запрет удаления строк ТЧ документа при определённом условии Maks 19.04.2026 Алгоритм из решения ниже реализован на примере нетипового документа "Аккумуляторы", разработанного в конфигурации КА2. У данного документа есть ТЧ, в которой в зависимости от прав доступа. . .	Модель заражения группы наркоманов alhaos 17.04.2026 Условия задачи сформулированы тут Суть: - Группа наркоманов из 10 человек. - Только один инфицирован ВИЧ. - Колются одной иглой. - Колются раз в день. - Колются последовательно через. . .	Мысли в слух. Про "навсегда". kumehtar 16.04.2026 Подумалось тут, что наверное очень глупо использовать во всяких своих установках понятие "навсегда". Это очень сильное понятие, и я только начинаю понимать край его смысла, не смотря на то что давно. . .	My Business CRM MaGz GoLd 16.04.2026 Всем привет, недавно возникла потребность создать CRM, для личных нужд. Собственно программа предоставляет из себя базу данных клиентов, в которой можно фиксировать звонки, стадии сделки, а также. . .

@EvilFromHell 979 / 874 / 350 Регистрация: 26.04.2012 Сообщений: 2,647
	06.06.2016, 20:31
	Даже не смотря внимательно на код видно сразу `goto link2;`, то есть безусловный выход с цикла на первой итерации при любых условиях. Причем переход выше цикла. Оттуда и недостижимый код, и зацикливание. Вообще не стоит слишком много лепить goto в программе. Оно чаще всего надо для выхода с вложенных циклов, а злоупотребление им приводит к тому, что код тяжело читать. 0

@DataPlanner 158 / 187 / 48 Регистрация: 25.11.2013 Сообщений: 978
	06.06.2016, 21:52
	Эта задача по графам, решайте ее соответственно 0