Что не так в приведенном коде?

@ninnnnnnr · Регистрация: 17.01.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
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
173
174
175
176
177
178
179
180
181
182
183
184
185
186
187
188
189
190
191
192
193
194
195
196
197
198
199
200
201
202
203
204
205
206
207
208
209
210
211
212
213
214
215
216
217
218
219
220
221
222
223
224
225
226
227
228
229
230
231
232
233
234
235
236
237
238
239
240
241
242
243
244
245
246
247
248
249
250
251
252
253
254
255
256
257
258
259
260
261
262
263
264
265
266
267
268
269
270
271
272
273
274
275
276
277
278
279
280
281
282
using System;
using System.Collections.Generic;
using System.Linq;
using System.Text;
 
namespace ClassTesting
{
 
    class Program
    {
        static void Main(string[] args)
        {
 
        }
    }
 
    public class Man
    {
        // конструктор класса (данная функция вызывается 
        // при создании нового экземпляра класса 
        public Man(string _name)
        {
            // переменная, которая будет хранить команду пользователя 
            string user_command = "";
 
            // бесконечный цикл 
            bool Infinity = true;
            // пустой (раный null) экземпляр класса Man 
            Man SomeMan = null;
 
            while (Infinity) // пока Infinity равно true 
            {
                // приглашение ввести команду 
                System.Console.WriteLine("Пожалуйста, введите команду");
 
                // получение строки (команды) от пользователя 
                user_command = System.Console.ReadLine();
 
                // обработка команды с помощью оператора ветвления 
                switch (user_command)
                {
 
                    // если user_command содержит строку exit 
                    case "exit":
                        {
                            Infinity = false;
                            // теперь цикл завершиться, и программа завершит свое выполнение 
                            break;
                        }
 
                    // если user_command содержит строку help 
                    case "help":
                        {
                            System.Console.WriteLine("Список команд:");
                            System.Console.WriteLine("---");
 
                            System.Console.WriteLine("create_man : команда создает человека, (экземпляр класса Man)");
                            System.Console.WriteLine("kill_man : команда убивает человека");
                            System.Console.WriteLine("talk : команда заставляет человека говорить (если создан экземпляр класса)");
                            System.Console.WriteLine("go : команда заставляет человека идти (если создан экземпляр класса)");
 
                            System.Console.WriteLine("---");
                            System.Console.WriteLine("---");
 
                            break;
                        } // если команду определить не удалось 
 
                    default:
                        {
                            System.Console.WriteLine("Ваша команда не определена, пожалуйста, повторите снова");
                            System.Console.WriteLine("Для вывода списка команд введите команду help");
                            System.Console.WriteLine("Для завершения программы введите команду exit");
                            break;
                        }
 
 
 
                    case "create_man":
                        {
                            // проверяем, создан ли уже какой либо человек 
                            if (SomeMan != null)
                            {
                                // человек уже существует. 
                                // убиваем его 
                                // (это не обязательная операция синтаксиса языка 
                                // а всего лишь каприз автора кода :), вы можете пропустить эту строку) 
 
                                SomeMan.Kill();
 
                            } // просим ввести имя человека 
 
                            System.Console.WriteLine("Пожалуйста, введите имя создаваемого человека ");
 
                            // получаем строку введенную пользователем 
                            user_command = System.Console.ReadLine();
 
                            // создаем новый объект в памяти, в качестве параметра 
                            // передаем имя человека 
                            SomeMan = new Man(user_command);
 
                            // сообщаем о создании 
                            System.Console.WriteLine("Человек успешно создан ");
 
                            break;
 
                        }
                    case "kill_man":
                        {
                            // проверяем, что объект существует в памяти 
                            if (SomeMan != null)
                            {
                                // вызываем фукнцию сметри 
                                SomeMan.Kill();
                            }
                            break;
                        }
                    case "talk":
                        {
                            // проверяем, что объект существует в памяти 
                            if (SomeMan != null)
                            {
                                // вызываем функцию разговора 
                                SomeMan.Talk();
                            }
                            else // иначе 
                            {
                                System.Console.WriteLine("Человек не создан. Команда не может быть выполнена");
                            }
 
                            break;
                        }
 
                    case "go":
                        {
 
                            // проверяем, что объект существует в памяти 
                            if (SomeMan != null)
                            {
                                // вызываем функцию передвижения 
                                SomeMan.Go();
                            }
                            else
                            {
                                System.Console.WriteLine("Человек не создан. Команда не может быть выполнена");
                            }
 
                            break;
 
                        }
 
                        // получаем имя человека из входного параметра 
                        // конструктора класса 
                        Name = _name;
                        // экземпляр жив 
                        isLife = true;
 
                        // генерируем возраст от 15 до 50 
                        Age = (uint)rnd.Next(15, 51);
                        // и здоровье, от 10 до 100% 
                        Health = (uint)rnd.Next(10, 101);
 
                }
            }
        }
            // экземпляр класса Random 
        // для генерации случайных чисел 
 
        private Random rnd = new Random();
 
        // закрытые члены, которые нельзя изменить 
        // извне класса
 
        // строка, содержащая имя 
        private string Name;
 
        // беззнаковое целое число, содержащая возраст 
        private uint Age;
 
        // беззнаковое целое число, отражающее уровень здоровья 
        private uint Health;
 
        // булево, означающее жив ли данный человек 
        private bool isLife;
 
        // заготовква функции "говорить" 
        public void Talk()
        {
            // генерируем случайное число от 1 до 3 
            int random_talk = rnd.Next(1, 4);
 
            // объявляем переменную, в которой мы будем хранить 
            // строку 
 
            string tmp_str = "";
            // в зависимости от случ значения random_talk 
            switch (random_talk)
            {
 
                case 1: // если 1 - называем свое имя 
                    { // генерируем текст сообщения 
                        tmp_str = "Привет, меня зовут " + Name + ", рад познакомиться";
                        // завершаем оператор выбора 
                        break;
                    }
 
                case 2: // возраст 
                    {
                        // генерируем текст сообщения 
                        tmp_str = "Мне " + Age + ". А тебе?";
                        // завершаем оператор выбора 
                        break;
                    }
 
                case 3: // говорим о своем здоровье 
                    {
 
                        // в зависимости от параметра здоровья 
                        if (Health > 50)
                            tmp_str = "Да я здоров как бык!";
                        else
                            tmp_str = "Со здоровьем у меня хреново, дожить бы до " + (Age + 10).ToString();
 
                        // завершаем оператор выбора 
                        break;
                    }
                    // выводим в консоль сгенерированное сообщение 
                    System.Console.WriteLine(tmp_str);
            }
 
        }
 
 
 
        // заготовка функции "идти" 
        public void Go()
        {
            // если объект жив 
            if (isLife == true)
            {
                // если показатель более 40 
                // считаем объект здоровым 
                if (Health > 40)
                {
                    // генерируем строку текста 
                    string outString = Name + " мирно прогуливается по городу";
                    // выводим в консоль 
                    System.Console.WriteLine(outString);
                }
                else
                {
                    // генерируем строку текста 
                    string outString = Name + " болен и не может гулять по городу";
                    // выводим в консоль 
                    System.Console.WriteLine(outString);
                }
 
            }
            else
            {
                // генерируем строку текста 
                string outString = Name + " не может идти, он умер";
                System.Console.ReadKey();
            }
 
        }
 
        // функция, возвращающая показатель - жив ли данный человек. 
        public bool IsAlive()
        {
            // возращаем значение, к которому мы не можем 
            // обратиться напрямую из вне класса, 
            // так как оно имеет статус private 
            return isLife;
        }
        public void Kill()
        {
            // устанавливаем значение isLife (жив) 
            // в false... 
            isLife = false;
        }
    }
}

@tarasalk · 17.01.2015, 17:14

1)код должен быть в теге CSHARP
2)нет отступов.
3)нет задания или хоть какого то описания
Хватит для начала?)

@Prog_maker · 17.01.2015, 18:09

в мейне нет ни одного действия

@tarasalk · 17.01.2015, 18:14

Многие комментарии излишни, больше путают, чем реально о чем то говорят.
Привязка класса к консоли сужает область его применения. Зачем вообще делать всю игру в классе? С таким же успехом класс можно было вообще не создавать.
Приписывать System к консольным командам излишне.
Команды(create_man, help, exit...) не имеют какого то общего места хранения. Надо бегать по всему коду искать их, а если там переименовать что то, то опять же в нескольких местах. С возрастом и здоровьем та же проблема(158 строчка). Такие вещи надо выносить в переменные.
Функция говорить: не очевидно что за генерация чисел 1 от 3(189 стр). Параметры не динамические, каждый раз когда я буду добавлять сообщение, мне придется менять диапазон в этом генераторе.
238 стр. не обязательно писать if(isLife == true), можно просто (isLife)

Изобилие switch и if тоже сбивает столку. Если без них никак не обойтись, лучше выносить их содержимое в другие места. Например

C#
1
2
3
case 'help':
   showHelp();
   break;

Читабельность в разы повышается.

@Prog_maker · 17.01.2015, 18:27

tarasalk, Хорошие замечания )

@ninnnnnnr · 18.01.2015, 00:51 **[ТС]**

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

@Lexeq · 18.01.2015, 01:43

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

но в консоле нет никакого текста, в чем может быть проблема ?

Вам же написали

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

в мейне нет ни одного действия

@Lynatik001 · 18.01.2015, 01:50

хочется взять и послать заново читать основы програмирования

@ninnnnnnr · 19.01.2015, 00:16 **[ТС]**

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
173
174
175
176
177
178
179
180
181
182
183
184
185
186
187
188
189
190
191
192
193
194
195
196
197
198
199
200
201
202
203
204
205
206
207
208
209
210
211
212
213
214
215
216
217
218
219
220
221
222
223
224
225
226
227
228
229
230
231
232
233
234
235
236
237
238
239
240
241
242
243
244
245
246
247
248
249
250
251
252
253
254
255
256
257
258
259
260
261
262
263
264
265
266
267
268
269
270
271
272
273
274
275
276
277
278
279
280
281
282
283
using System; 
using System.Collections.Generic; 
using System.Linq; 
using System.Text;
 
namespace ClassTesting
{
 
    class Program
    {
        static void Main(string[] args)
        {
            // переменная, которая будет хранить команду пользователя 
            string user_command = "";
 
            // бесконечный цикл 
            bool Infinity = true;
            // пустой (раный null) экземпляр класса Man 
            Man SomeMan = null; 
 
            while (Infinity) // пока Infinity равно true 
            {
                // приглашение ввести команду 
                System.Console.WriteLine("Пожалуйста, введите команду");
 
                // получение строки (команды) от пользователя 
                user_command = System.Console.ReadLine();
 
            }
            // обработка команды с помощью оператора ветвления 
            switch (user_command)
            {
                case "create_man":
                    {
                        // проверяем, создан ли уже какой либо человек 
                        if (SomeMan != null)
                        {
                            // человек уже существует. 
                            // убиваем его 
                            // (это не обязательная операция синтаксиса языка 
                            // а всего лишь каприз автора кода :), вы можете пропустить эту строку) 
 
                            SomeMan.Kill();
 
                        } // просим ввести имя человека 
 
                        System.Console.WriteLine("Пожалуйста, введите имя создаваемого человека ");
 
                        // получаем строку введенную пользователем 
                        user_command = System.Console.ReadLine();
 
                        // создаем новый объект в памяти, в качестве параметра 
                        // передаем имя человека 
                        SomeMan = new Man(user_command);
 
                        // сообщаем о создании 
                        System.Console.WriteLine("Человек успешно создан ");
 
                        break;
 
                    }
                case "kill_man":
                    {
                        // проверяем, что объект существует в памяти 
                        if (SomeMan != null)
                        {
                            // вызываем фукнцию сметри 
                            SomeMan.Kill();
                        }
                        break;
                    }
                case "talk":
                    {
                        // проверяем, что объект существует в памяти 
                        if (SomeMan != null)
                        {
                            // вызываем функцию разговора 
                            SomeMan.Talk();
                        }
                        else // иначе 
                        {
                            System.Console.WriteLine("Человек не создан. Команда не может быть выполнена");
                        }
 
                        break;
                    }
 
                case "go":
                    {
 
                        // проверяем, что объект существует в памяти 
                        if (SomeMan != null)
                        {
                            // вызываем функцию передвижения 
                            SomeMan.Go();
                        }
                        else
                        {
                            System.Console.WriteLine("Человек не создан. Команда не может быть выполнена");
                        }
 
                        break;
 
                    } 
 
 
                // если user_command содержит строку exit 
                case "exit":
                    {
                        Infinity = false;
                        // теперь цикл завершиться, и программа завершит свое выполнение 
                        break;
                    }
 
                // если user_command содержит строку help 
                case "help":
                    {
                        System.Console.WriteLine("Список команд:");
                        System.Console.WriteLine("---");
 
                        System.Console.WriteLine("create_man : команда создает человека, (экземпляр класса Man)");
                        System.Console.WriteLine("kill_man : команда убивает человека");
                        System.Console.WriteLine("talk : команда заставляет человека говорить (если создан экземпляр класса)");
                        System.Console.WriteLine("go : команда заставляет человека идти (если создан экземпляр класса)");
 
                        System.Console.WriteLine("---");
                        System.Console.WriteLine("---");
 
                        break;
                    } // если команду определить не удалось 
 
                default:
                    {
                        System.Console.WriteLine("Ваша команда не определена, пожалуйста, повторите снова");
                        System.Console.WriteLine("Для вывода списка команд введите команду help");
                        System.Console.WriteLine("Для завершения программы введите команду exit");
                        break;
                    }
 
            }
 
        }
 
        public class Man
        {
            // конструктор класса (данная функция вызывается 
            // при создании нового экземпляра класса 
            public Man(string _name)
            {
                // получаем имя человека из входного параметра 
                // конструктора класса 
                Name = _name;
                // экземпляр жив 
                isLife = true;
 
                // генерируем возраст от 15 до 50 
                Age = (uint)rnd.Next(15, 51);
                // и здоровье, от 10 до 100% 
                Health = (uint)rnd.Next(10, 101);
 
            }
            // экземпляр класса Random 
            // для генерации случайных чисел 
 
            private Random rnd = new Random();
 
            // закрытые члены, которые нельзя изменить 
            // извне класса
 
            // строка, содержащая имя 
            private string Name;
 
            // беззнаковое целое число, содержащая возраст 
            private uint Age;
 
            // беззнаковое целое число, отражающее уровень здоровья 
            private uint Health;
 
            // булево, означающее жив ли данный человек 
            private bool isLife;
 
            // заготовква функции "говорить" 
            public void Talk()
            {
                // генерируем случайное число от 1 до 3 
                int random_talk = rnd.Next(1, 4);
 
                // объявляем переменную, в которой мы будем хранить 
                // строку 
 
                string tmp_str = "";
                // в зависимости от случ значения random_talk 
                switch (random_talk)
                {
 
                    case 1: // если 1 - называем свое имя 
                        { // генерируем текст сообщения 
                            tmp_str = "Привет, меня зовут " + Name + ", рад познакомиться";
                            // завершаем оператор выбора 
                            break;
                        }
 
                    case 2: // возраст 
                        {
                            // генерируем текст сообщения 
                            tmp_str = "Мне " + Age + ". А тебе?";
                            // завершаем оператор выбора 
                            break;
                        }
 
                    case 3: // говорим о своем здоровье 
                        {
 
                            // в зависимости от параметра здоровья 
                            if (Health > 50)
                                tmp_str = "Да я здоров как бык!";
                            else
                                tmp_str = "Со здоровьем у меня хреново, дожить бы до " + (Age + 10).ToString();
 
                            // завершаем оператор выбора 
                            break;
                        }
 
                }
                // выводим в консоль сгенерированное сообщение 
                System.Console.WriteLine(tmp_str);
 
 
            }
 
            // заготовка функции "идти" 
            public void Go()
            {
                // если объект жив 
                if (isLife == true)
                {
                    // если показатель более 40 
                    // считаем объект здоровым 
                    if (Health > 40)
                    {
                        // генерируем строку текста 
                        string outString = Name + " мирно прогуливается по городу";
                        // выводим в консоль 
                        System.Console.WriteLine(outString);
                    }
                    else
                    {
                        // генерируем строку текста 
                        string outString = Name + " болен и не может гулять по городу";
                        // выводим в консоль 
                        System.Console.WriteLine(outString);
                    }
 
                }
                else
                {
                    // генерируем строку текста 
                    string outString = Name + " не может идти, он умер";
                    System.Console.WriteLine(outString);
                }
 
 
            }
            public void Kill()
            {
                // устанавливаем значение isLife (жив) 
                // в false... 
                isLife = false;
            }
 
            // функция, возвращающая показатель - жив ли данный человек. 
            public bool IsAlive()
            {
                // возращаем значение, к которому мы не можем 
                // обратиться напрямую из вне класса, 
                // так как оно имеет статус private 
                return isLife;
            }
 
        }
 
    }
}

Я подправил код, но все равно программа до конца не работает, в чем может быть ошибка ?

@Lexeq · 19.01.2015, 00:25

Что именно не работает?

@ninnnnnnr · 19.01.2015, 00:51 **[ТС]**

в консоле выводит текст: пожалуйста введите команду, но что бы я не вводил, постоянно выводит этот текст

@Lexeq · 19.01.2015, 01:05

ninnnnnnr,
Потому что у вас этот запрос находится в бесконечном цикле.

C#
1
2
3
4
5
            while (Infinity) // пока Infinity равно true, а оно всегда равно true
            {
                System.Console.WriteLine("Пожалуйста, введите команду");
                user_command = System.Console.ReadLine();
            }

Дальше этого выполнение не пойдет.

@ninnnnnnr · 19.01.2015, 17:34 **[ТС]**

Спасибо, исправил. Появилась другая проблема: не работают команды talk и kill_man, в чем может быть проблема ?

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

@tarasalk 1992 / 1216 / 440 Регистрация: 13.06.2013 Сообщений: 4,115
	17.01.2015, 17:14
	1)код должен быть в теге CSHARP 2)нет отступов. 3)нет задания или хоть какого то описания Хватит для начала?) 0

@Prog_maker 459 / 403 / 153 Регистрация: 23.01.2011 Сообщений: 1,054
	17.01.2015, 18:09
	в мейне нет ни одного действия 0

@Prog_maker 459 / 403 / 153 Регистрация: 23.01.2011 Сообщений: 1,054
	17.01.2015, 18:27
	tarasalk, Хорошие замечания ) 1

@ninnnnnnr 0 / 0 / 0 Регистрация: 17.01.2015 Сообщений: 18
	18.01.2015, 00:51 [ТС]
	Спасибо за комментарии. Вы ни могли указать в коде на ошибку, у меня отладка проводится без ошибок, но в консоле нет никакого текста, в чем может быть проблема ? 0

@Lynatik001 48 / 40 / 15 Регистрация: 28.09.2012 Сообщений: 818
	18.01.2015, 01:50
	хочется взять и послать заново читать основы програмирования 1

@Lexeq 1151 / 743 / 483 Регистрация: 21.01.2014 Сообщений: 1,903
	19.01.2015, 00:25
	Что именно не работает? 0

@ninnnnnnr 0 / 0 / 0 Регистрация: 17.01.2015 Сообщений: 18
	19.01.2015, 00:51 [ТС]
	в консоле выводит текст: пожалуйста введите команду, но что бы я не вводил, постоянно выводит этот текст 0

@ninnnnnnr 0 / 0 / 0 Регистрация: 17.01.2015 Сообщений: 18
	19.01.2015, 17:34 [ТС]
	Спасибо, исправил. Появилась другая проблема: не работают команды talk и kill_man, в чем может быть проблема ? 0