Всегда выполняется ветка else

@leha_4333 · Регистрация: 15.02.2025

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

Я делаю крестики-нолики на c++ и столкнулся с проблемой что вне зависимости от значения происходит ветка else
Вот код:

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
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
131
132
133
134
135
136
137
138
139
140
141
142
143
144
145
146
147
148
149
150
151
152
153
154
155
156
157
158
159
160
161
162
163
164
165
166
167
168
169
170
171
172
#include <Windows.h> 
#include <iostream> 
#include <string> 
using namespace std; 
int main() 
{ 
    byte victory = 0; 
    string test[3][3] = { 
        {" ", " ", " "}, 
        {" ", " ", " "}, 
        {" ", " ", " "} 
    }; 
    byte team = 1; 
    
    
    while (victory == 0) 
    { 
        if (team == 1) 
        { 
            byte mesto; 
            cout << test[0][0] + "|" + test[0][1] + "|" + test[0][2] << endl << test[1][0] + "|" + test[1][1] + "|" + test[1][2] << endl << test[2][0] + "|" + test[2][1] + "|" + test[2][2] << endl; 
            cout << "Enter:" << endl; 
            cout << "1 for cross in highest left cage" << endl; 
            cout << "2 for cross in highest middle cage" << endl; 
            cout << "3 for cross in highest right cage" << endl; 
            cout << "4 for cross in middle left cage" << endl; 
            cout << "5 for cross in middle middlecage" << endl; 
            cout << "6 for cross in middle right cage" << endl; 
            cout << "7 for cross in lowest left cage" << endl; 
            cout << "8 for cross in lowest middle cage" << endl; 
            cout << "9 for cross in lowest right cage" << endl; 
            cin >> mesto; 
            if(mesto == 1 && test[0][0] == " ") 
            { 
                test[0][0] = "X"; 
            } else if (mesto == 2 && test[0][1] == " ") 
            { 
                test[0][1] = "X"; 
            } else if (mesto == 3 && test[0][2] == " ") 
            { 
                test[0][2] = "X"; 
            } else if (mesto == 4 && test[1][0] == " ") 
            { 
                test[1][0] = "X"; 
            } else if (mesto == 5 && test[1][1] == " ") 
            { 
                test[1][1] = "X"; 
            } else if (mesto == 6 && test[1][2] == " ") 
            { 
                test[1][2] = "X"; 
            } else if (mesto == 7 && test[2][0] == " ") 
            { 
                test[2][0] = "X"; 
            } else if (mesto == 8 && test[2][1] == " ") 
            { 
                test[2][1] = "X"; 
            } else if (mesto == 9 && test[2][2] == " ") 
            { 
                test[2][2] = "X"; 
            } else 
            { 
                cout << "There is no such place on the board or this place is already busy" << endl; 
            } 
            team = 2; 
        }
 
                if (team == 2)  
                {
 
            byte mesto;  
 
            cout << test[0][0] + "|" + test[0][1] + "|" + test[0][2] << endl << test[1][0] + "|" + test[1][1] + "|" + test[1][2] << endl << test[2][0] + "|" + test[2][1] + "|" + test[2][2] << endl;  
 
            cout << "Enter:" << endl;  
 
            cout << "1 for cross in highest left cage" << endl;  
 
            cout << "2 for cross in highest middle cage" << endl;  
 
            cout << "3 for cross in highest right cage" << endl;  
 
            cout << "4 for cross in middle left cage" << endl;  
 
            cout << "5 for cross in middle middlecage" << endl;  
 
            cout << "6 for cross in middle right cage" << endl;  
 
            cout << "7 for cross in lowest left cage" << endl;  
 
            cout << "8 for cross in lowest middle cage" << endl;  
 
            cout << "9 for cross in lowest right cage" << endl;  
 
            cin >> mesto;  
 
            if (mesto == 1 && test[0][0] == " ")  
 
            {  
 
                test[0][0] = "O";  
 
            }  
 
            else if (mesto == 2 && test[0][1] == " ")  
 
            {  
 
                test[0][1] = "O";  
 
            }  
 
            else if (mesto == 3 && test[0][1] == " ")  
 
            {  
 
                test[0][1] = "O";  
 
            }  
 
            else if (mesto == 4 && test[0][1] == " ")  
 
            {  
 
                test[0][1] = "O";  
 
            }  
 
            else if (mesto == 5 && test[0][1] == " ")  
 
            {  
                test[0][1] = "O";  
            }  
            else if (mesto == 6 && test[0][1] == " ")  
 
            {  
 
                test[0][1] = "O";  
 
            }  
            else if (mesto == 7 && test[0][1] == " ")  
            {  
                test[0][1] = "O";  
            }  
 
            else if (mesto == 8 && test[0][1] == " ")  
 
            {  
 
                test[0][1] = "O";  
 
            }  
 
            else if (mesto == 9 && test[0][1] == " ")  
 
            {  
 
                test[0][1] = "O";  
 
            }  
 
            else  
            {  
                cout << "There is no place on the board or this place is already busy" << endl;  
            }  
 
            team = 1;  
 
        }  
 
    }  
    return 0;  
}

@SmallEvil · 15.02.2025, 21:04

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

byte mesto;

Что такое byte ?
Подозреваю что это однобайтовое целое, что при вводе такого значения ввод интерпретируется как символ а не число.
То есть вы вводите символ при запросе, а сравниваете с числом в условиях.

@TheCalligrapher · 15.02.2025, 21:14

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

вне зависимости от значения происходит ветка else

Какая "ветка else"? У вас в коде множество "веток else". О чем речь?

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

byte victory = 0;

Но этот код невозможно даже скомпилировать. О каком "происходит ветка else" вообще может идти речь?

@Alex1126 · 15.02.2025, 21:18

Вы когда нолик ставите у вас проверяется свободное место только test[0][1] а должны разные.

@SmallEvil · 15.02.2025, 22:11

leha_4333, ещё добавлю, что вот таких условий стоит избегать. Придумывать способы соотносить одно с другим.
В вашем случае нужно соотнести ввод от пользователя (1-9) и индексы ячейки массива.
Это, например, можно сделать след. образом:

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
int mesto = 1;
// ...
cin >> mesto;
mesto = mesto - 1; // для более естественного расчета
int row = mesto / 3;
int col  = mesto % 3;
 
if (test[row][col] == " ")
   test[row][col] = (team == 1 ? "X" : "O"); // устанавливаем крестик или нолик соответственно
else
   cout << "There is no place on the board or this place is already busy" << endl;
// после хода и проверки на победу, меняем ход 
team = (team == 1 ? 2 : 1);

И это всё вместо множества условий в двух блоках.
Практикуйтесь. Удачи.

Добавлено через 2 минуты
p.s. Так же не помешает добавить проверку ввода, что введено число в диапазоне [1, 9].

Новые блоги и статьи Все статьи Все блоги /
Музыка, написанная Искусственным Интеллектом volvo 04.12.2025 Всем привет. Некоторое время назад меня заинтересовало, что уже умеет ИИ в плане написания музыки для песен, и, собственно, исполнения этих самых песен. Стихов у нас много, уже вышли 4 книги, еще 3. . .	От async/await к виртуальным потокам в Python IndentationError 23.11.2025 Армин Ронахер поставил под сомнение async/ await. Создатель Flask заявляет: цветные функции - провал, виртуальные потоки - решение. Не threading-динозавры, а новое поколение лёгких потоков. Откат?. . .	Поиск "дружественных имён" СОМ портов Argus19 22.11.2025 Поиск "дружественных имён" СОМ портов На странице: https:/ / norseev. ru/ 2018/ 01/ 04/ comportlist_windows/ нашёл схожую тему. Там приведён код на С++, который показывает только имена СОМ портов, типа,. . .	Сколько Государство потратило денег на меня, обеспечивая инсулином. Programma_Boinc 20.11.2025 Сколько Государство потратило денег на меня, обеспечивая инсулином. Вот решила сделать интересный приблизительный подсчет, сколько государство потратило на меня денег на покупку инсулинов. . . .	Ломающие изменения в C#.NStar Alpha Etyuhibosecyu 20.11.2025 Уже можно не только тестировать, но и пользоваться C#. NStar - писать оконные приложения, содержащие надписи, кнопки, текстовые поля и даже изображения, например, моя игра "Три в ряд" написана на этом. . .
Мысли в слух kumehtar 18.11.2025 Кстати, совсем недавно имел разговор на тему медитаций с людьми. И обнаружил, что они вообще не понимают что такое медитация и зачем она нужна. Самые базовые вещи. Для них это - когда просто люди. . .	Создание Single Page Application на фреймах krapotkin 16.11.2025 Статья исключительно для начинающих. Подходы оригинальностью не блещут. В век Веб все очень привыкли к дизайну Single-Page-Application . Быстренько разберем подход "на фреймах". Мы делаем одну. . .	Фото: Daniel Greenwood kumehtar 13.11.2025	Расскажи мне о Мире, бродяга kumehtar 12.11.2025 — Расскажи мне о Мире, бродяга, Ты же видел моря и метели. Как сменялись короны и стяги, Как эпохи стрелою летели. - Этот мир — это крылья и горы, Снег и пламя, любовь и тревоги, И бескрайние. . .	PowerShell Snippets iNNOKENTIY21 11.11.2025 Модуль PowerShell 5. 1+ : Snippets. psm1 У меня модуль расположен в пользовательской папке модулей, по умолчанию: \Documents\WindowsPowerShell\Modules\Snippets\ А в самом низу файла-профиля. . .

@Alex1126 152 / 136 / 26 Регистрация: 12.12.2020 Сообщений: 1,114
	15.02.2025, 21:18
	Вы когда нолик ставите у вас проверяется свободное место только test[0][1] а должны разные. 0

Всегда выполняется ветка else

Решение