Проблема в проге

@Devilion123 · Регистрация: 29.10.2015

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

Не могу понять, почему в данном случае программа выполняется неправильно + на моем ПК в визуалке она вылетает, а на остальных работает нормально. Задача - написать текстовый квест. Проблема на данный момент такая: При выборе пункта номер 2 пишется не тот пункт, который нужен + у кого-то может вылетать (не знаю, проблема это визуалки или в самом коде, но такое тоже бывает)

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
#include <iostream>
#include <stdlib.h>
using namespace std;
int main()
{
    setlocale(LC_ALL, "rus");
    int a, step = 7;
    cout << "Вы встали с кровати. Вам нужно сделать чай и бежать на пары. У вас есть 6 шагов, а иначе вы не успеете. \nЧто вы сделаете в первую очередь? \n1. Поставить чайник с водой кипятиться \n2. Приготовить кружку с чайным пакетиком и сахаром сразу \n";
    cin >> a;
    if (a = 1)
    {
        int c;
        cout << "Налив воду в чайник и поставив его на плиту, вы решили сразу проверить, есть ли у вас в запасе сахар и чайные пакетики \n Найдя все, что вам нужно, вы предварительно положили это в кружку\n";
        step = step - 2;
        cout << "Шагов осталось  " << step << endl;
        cout << "Сделав все дела на этом этапе, вы решили добавить в свой чай что-нибудь еще. Что вы хотите добавить? \n 1. Лимон \n 2. Молоко \n 3. Мед \n 4. Варенье \n";
        cout << "Шагов осталось  " << step << endl;
        cin >> c;
        switch (c)
        {
        case 1:
            cout << "Лимона не оказалось у вас под рукой, поэтому вы прошли до холодильника. Не найдя его там, вы вернулись на кухню. Вы просто не заметили его за салфеткой...\n";
            step = step - 2;
            cout << "Шагов осталось  " << step << endl;
            cout << "Собрав все ингредиенты, вы положили их в кружку и залили кипятком. Чай готов!\n";
            step = step - 2;
            cout << "Шагов осталось  " << step << endl;
            break;
        case 2:
            cout << "Найдя в холодильнике молоко, вы поставили его рядом с кружкой\n";
            step = step - 1;
            cout << "Шагов осталось  " << step << endl;
            cout << "Собрав все ингредиенты, вы положили их в кружку и залили кипятком, а затем и молоком. Чай готов!\n";
            step = step - 2;
            cout << "Шагов осталось  " << step << endl;
            break;
        case 3:
            cout << "Вы захотели добавить немного медку в свой чай. Поискав его какое-то время, вы обнаружили, что он закончился :(\n";
            step = step - 2;
            cout << "Шагов осталось  " << step << endl;
            cout << "Собрав все ингредиенты, вы положили их в кружку и залили кипятком. Чай готов!\n";
            step = step - 2;
            cout << "Шагов осталось  " << step << endl;
            break;
        case 4:
            cout << "Вам повезло, потому что варенье уже стояло рядом\n";
            step = step - 1;
            cout << "Шагов осталось  " << step << endl;
            cout << "Собрав все ингредиенты, вы положили их в кружку и залили кипятком. Чай готов!\n";
            step = step - 2;
            cout << "Шагов осталось  " << step << endl;
            break;
        }
    }
 
    else if (a = 2)
    {
        int b;
        cout << "Вы заметили, что сахар и чайные пакетики закончились, поэтому вам надо их найти, чтобы пополнить запасы. Вы долго искали их и потеряли много времени\n";
        step = step - 3;
        cout << "После нахождения сахара и чайных пакетиков вы поставили чайник на плиту, чтобы вскипятить воду \n";
        cout << "Сделав все дела на этом этапе, вы решили добавить в свой чай что-нибудь еще. Что вы хотите добавить? \n1. Лимон \n2. Молоко \n3. Мед \n4. Варенье \n";
        cin >> b;
        switch (b)
        {
        case 1:
            cout << "Лимона не оказалось у вас под рукой, поэтому вы прошли до холодильника. Не найдя его там, вы вернулись на кухню. Вы просто не заметили его за салфеткой...\n";
            step = step - 2;
            cout << "Шагов осталось  " << step << endl;
            cout << "Собрав все ингредиенты, вы положили их в кружку и залили кипятком. Чай готов!\n";
            step = step - 2;
            cout << "Шагов осталось  " << step << endl;
            break;
        case 2:
            cout << "Найдя в холодильнике молоко, вы поставили его рядом с кружкой\n";
            step = step - 1;
            cout << "Шагов осталось  " << step << endl;
            cout << "Собрав все ингредиенты, вы положили их в кружку и залили кипятком, а затем и молоком. Чай готов!\n";
            step = step - 2;
            cout << "Шагов осталось  " << step << endl;
            break;
        case 3:
            cout << "Вы захотели добавить немного медку в свой чай. Поискав его какое-то время, вы обнаружили, что он закончился :(\n";
            step = step - 2;
            cout << "Шагов осталось  " << step << endl;
            break;
        case 4:
            cout << "Вам повезло, потому что варенье уже стояло рядом\n";
            step = step - 1;
            cout << "Шагов осталось  " << step << endl;
            cout << "Собрав все ингредиенты, вы положили их в кружку и залили кипятком. Чай готов!\n";
            step = step - 2;
            cout << "Шагов осталось  " << step << endl;
            break;
        }
    }
}

@JohnBlack123 · 21.02.2019, 22:07

1. Вы забыли return 0 в конце main, без него та же Visual Studio не сможет скомпилировать Ваш код.
2. Почему в условиях Вы проверяете равна ли переменная чему-либо используя одно равно? Это ошибка замените его на два равно :

C++
1
if(var == value) ...

3. Используйте вместо этой конструкции var=var-1 декремент : var--.
А вместо var=var-2 такую : var-=2
4. Тип int можно заменить на short, так как числа Вы вводите именно в таком диапазоне.
Немного не ясно, почему в некоторых случаях Вы вычитаете из var 2 или 3, а в других 1. Объясните свою мысль.
Посмотрел на скорую руку, пусть уважаемые знатоки присмотрятся, может еще найдутся ошибки.

Добавлено через 7 минут
Devilion123, как по мне, так основная ошибка, как я написал ранее тут :

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

C++
1
if (a = 1)

и тут :

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

C++
1
else if (a = 2)

Грубая ошибка заменить на :

C++
1
2
if (a == 1) ...
else if (a == 2) ...

@Devilion123 · 21.02.2019, 22:23 **[ТС]**

Добавлено через 2 минуты
Да, насчет "==" спасибо, но есть проблема, потому что после выбора, который записан в кейсе, программа все равно вылетает. Может system("pause") поможет?

@JohnBlack123 · 21.02.2019, 22:26

Если программа вылетает и Вы не успеваете даже посмотреть на ее результат
в конце ее выполнения, значит нужно добавить паузу или ожидание нажатия клавиши.
system ("pause") подойдет

@New man · 21.02.2019, 22:49

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

3. Используйте вместо этой конструкции var=var-1 декремент : var--.
А вместо var=var-2 такую : var-=2

Какая разница? Пусть пишет так, как ему удобнее.

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

4. Тип int можно заменить на short, так как числа Вы вводите именно в таком диапазоне.

Нафига? Экономия на спичках?

Devilion123, я бы описал логику Вашей программы в виде графа.
Это сильно упростит код и не даст превратить его в кучу спагетти.
Что-то в этом роде:

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
struct StoryState{
    bool final = false;
    std::string action;
    std::unordered_map<int, StoryState*> next_states; // Ключ — ввод, значение — новое состояние, которое обрабатываем.
};
StoryState GreetingState;
StoryState FirstSide;
StoryState SecondSide;
StoryState Finish;
 
void InitStates()
{
    GreetingState.action = "Hello, man. Choose 0 if you want to join FirstSide, 1 to other";
    FirstSide.action = "You are revolter. If you want to betray your brothers, press 0, if you want to exit, press 100";
    SecondSide.action = "You are riot. If you want to change a side, press 0, if you want to exit, press 100";
    Finish.action = "You are ending game...";
 
    GreetingState.next_states = std::unordered_map<int, StoryState*>{
        {0, &FirstSide},
        {1, &SecondSide}};
    FirstSide.next_states = std::unordered_map<int, StoryState*>{
        {0, &SecondSide},
        {100, &Finish}};
    SecondSide.next_states = std::unordered_map<int, StoryState*>{
        {0, &FirstSide},
        {100, &Finish}};
    
     Finish.final = true;
}
 
int main()
{
     //Подготовка
     InitStates();
 
     // История
     StoryState* current_state = &GreetingState;
     do{
         // Сообщение
         std::cout<<current_state->action << std::endl;
         // Игра закончена?
         if(current_state->final) return 0; 
         int next;
         std::cin>>next;
         while(current_state->next_states.find(next)==current_state->next_states.end()) // Если неправильный ввод
         {
             std::cout<<"UNKNOWN INPUT"<<std::endl;
             std::cin>>next;
         }
         current_state = current_state->next_states[next]; // Переходим к следующему шагу истории
     } while(true);  
}

Добавлено через 1 минуту
Devilion123, ещё можно разделить сообщение истории и сообщение о командах. Можно данные вынести в текстовый файл какой-нибудь.

Новые блоги и статьи Все статьи Все блоги /
Программный контроль заполнения реквизита табличной части документа Maks 02.04.2026 Алгоритм из решения ниже реализован на примере нетипового документа "СписаниеМатериалов", разработанного в конфигурации КА2. Задача: реализовать контроль заполнения реквизита "ПричинаСписания". . .	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. Задача: при создании документов установить период списания автоматически. . .	Вывод данных в справочнике через динамический список Maks 01.04.2026 Реализация из решения ниже выполнена на примере нетипового справочника "Спецтехника" разработанного в конфигурации КА2. Задача: вывести данные из ТЧ нетипового документа. . .
Программное заполнения текстового поля в реквизите формы документа Maks 01.04.2026 Алгоритм из решения ниже реализован на нетиповом документе "ВыдачаОборудованияНаСпецтехнику" разработанного в конфигурации КА2, в дополнении к предыдущему решению. На форме документа создается. . .	К слову об оптимизации kumehtar 01.04.2026 Вспоминаю начало 2000-х, университет, когда я писал на Delphi. Тогда среди программистов на форумах активно обсуждали аккуратную работу с памятью: нужно было следить за переменными, вовремя. . .	Идея фильтра интернета (сервер = слой+фильтр). Hrethgir 31.03.2026 Суть идеи заключается в том, чтобы запустить свой сервер, о чём я если честно мечтал давно и давно приобрёл книгу как это сделать. Но не было причин его запускать. Очумелые учёные напечатали на. . .	Модель здравосоХранения 6. ESG-повестка и устойчивое развитие; углублённый анализ кадрового бренда anaschu 31.03.2026 В прикрепленном документе раздумья о том, как можно поменять модель в будущем

@Devilion123 1 / 1 / 0 Регистрация: 29.10.2015 Сообщений: 72
	21.02.2019, 22:23 [ТС]
	Добавлено через 2 минуты Да, насчет "==" спасибо, но есть проблема, потому что после выбора, который записан в кейсе, программа все равно вылетает. Может system("pause") поможет? 0

@JohnBlack123 215 / 162 / 52 Регистрация: 09.12.2017 Сообщений: 520
	21.02.2019, 22:26
	Если программа вылетает и Вы не успеваете даже посмотреть на ее результат в конце ее выполнения, значит нужно добавить паузу или ожидание нажатия клавиши. system ("pause") подойдет 0