С Новым годом! Форум программистов, компьютерный форум, киберфорум
С++ для начинающих
Войти
Регистрация
Восстановить пароль
Карта форума Темы раздела Блоги Сообщество Поиск Заказать работу  
 
Рейтинг 4.73/15: Рейтинг темы: голосов - 15, средняя оценка - 4.73
1 / 1 / 0
Регистрация: 04.07.2013
Сообщений: 12
1

Прекращена работа программы proj.exe

03.09.2013, 01:41. Показов 2969. Ответов 6
Метки нет (Все метки)

Author24 — интернет-сервис помощи студентам
Доброго времени суток всем. У меня возникла неприятная проблемка. Я написал программу(даже не всю а только часть) которая, в последствии, будет что-то вроде морского боя на консоле. В процессе написания я ее запускал, чтобы проверить некоторые элементы, и она прекрасно работала. Но вот дойдя до определенного момента появилось вот что(см. скриншот). Идей почему это происходит и как с этим бороться у меня нет. Пользуюсь Dev-c++ 4.9.9.2. Система Win7 Домашняя расширенная. Если вдруг проблема может оказаться в написанном коде(ну мало ли, хотя компилируется нормально), то можете посмотреть его ниже(если хочется смотреть 270+ строк ). Если потребуется еще какая-нибудь информация по теме - спрашивайте, отвечу на все, на что смогу. Заранее спасибо всем, кто откликнется.
Кликните здесь для просмотра всего текста
Прототипы функций
C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
#include <cstdlib>
#include <iostream>
#include <string>
 
using namespace std;
 
class Sea
{
public:
    Sea(string);
    void setCorabli();
    void viewPole();
    int proverka(int, int, int);
private:
    void setCorabl(int);
    int pole[10][10];
    int poleVistrelov[10][10];
    string PlayerName;
    int numberOfShips[4];
    int coordinatiCorablei[20][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
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
#include <cstdlib>
#include <iostream>
#include "Sea.h"
 
using namespace std;
 
Sea::Sea(string name)
{
    for (int i3 = 1; i3 <= 4; i3++)
        numberOfShips[i3] = 0;
    PlayerName = name;
    for (int i = 0; i <= 10; i++)
    {
        for (int j = 0; j <= 10; j++)
            pole[i][j] = 0;
    }
    for (int i1 = 0; i1 <= 10; i1++)
    {
        for (int j1 = 0; j1 <= 10; j1++)
            poleVistrelov[i1][j1] = 0;
    }
    for (int i2 = 0; i2 <= 20; i2++)
    {
        for (int j2 = 0; j2 <= 2; j2++)
            coordinatiCorablei[i2][j2] = 0;
    }
}
 
void Sea::setCorabli()
{
    char num;
    int number = 0;
    cout << "The player " << PlayerName << "places the ships." << endl;
    while (number < 10)
    {
        cout << "\nEnter the size of the ship that you want to put." << endl;
        cin >> num;
        switch (num)
        {
            case '1':
                if(numberOfShips[1] < 4)
                {
                    setCorabl(1);
                }
                else
                    cout << "All ships of this size have been installed." << endl;
                break;
            case '2':
                if(numberOfShips[2] < 3)
                {
                    setCorabl(2);
                }
                else
                    cout << "All ships of this size have been installed." << endl;
                break;
            case '3':
                if(numberOfShips[3] < 2)
                {
                    setCorabl(3);
                }
                else
                    cout << "All ships of this size have been installed." << endl;
                break;
            case '4':
                if(numberOfShips[4] < 1)
                {
                    setCorabl(4);
                }
                else
                    cout << "All ships of this size have been installed." << endl;
                break;
            default:
                cout << "Incorrect size" << endl;
                break;
        }
        number = numberOfShips[1] + numberOfShips[2] + numberOfShips[3] + numberOfShips[4];
    }
    cout << PlayerName << " successeful placed his ships!" << endl;
}
 
void Sea::setCorabl(int size)
{
    int num = 1;
    int xy[2*size];
    cout << "Start placing the ships. Enter the -1 instead of the any coordinate to abort the installation ship." << endl;
    cout << "Where would you like to place the " << size << "-size ship?" << endl;
    while (num <= size)
    {
        cout << "Enter the x" << num << " coordinate." << endl;
        cin >> xy[2*num - 1];
        while (xy[2*num - 1] <= 0 || xy[2*num - 1] > 10)
        {
            if (xy[2*num - 1] == -1)
            {
                break;
            }
            else
            {
                cout << "Incorrect x" << num << " coordinate. Enter another." << endl;
                cin >> xy[2*num - 1];
            }
        }
        if (xy[2*num - 1] == -1)
        {
            cout << "The operation was canceled by the user." << endl;
            break;
        }
        cout << "Enter the y" << num << " coordinate." << endl;
        cin >> xy[2*num];
        while (xy[2*num] <= 0 || xy[2*num] > 10)
        {
            if (xy[2*num] == -1)
            {
                break;
            }
            else
            {
                cout << "Incorrect y" << num << " coordinate. Enter another." << endl;
                cin >> xy[2*num];
            }
        }
        if (xy[2*num] == -1)
        {
            cout << "The operation was canceled by the user." << endl;
            break;
        }
        if (proverka(num, xy[2*num - 1], xy[2*num]) == 1)
        {
            cout << "Successeful." << endl;
            num++;
        }
        else
        {
            cout << "Incorrect coordinates." << endl;
        }
    }
    if (num == size)
    {
        
        cout << size << "-size ship was successeful installed." << endl;
        numberOfShips[size]++;
    }
}
 
int Sea::proverka(int num, int x, int y)
{
    int successPoint = 0;
    if (num == 1)
    {
        for (int i = -1; i <= 1; i++)
        {
            for (int j = -1; j <= 1; j++)
            {
                if (pole[x + i][y + j] == 0)
                    successPoint++;
            }
        }
        if (successPoint == 9)
            return 1;
        else
            return 0;
    }
    else
    {
        if (num == 2)
        {
            for (int i = -1; i <= 1; i += 2)
            {
                for (int j = -1; j <= 1; j += 2)
                {
                    if (pole[x + i][y + j] == 0)
                        successPoint++;
                }
            }
            if (successPoint == 4)
            {
                for (int i1 = -1; i1 <= 1; i1 += 2)
                {
                    if (pole[x][y + i1] == 0)
                        successPoint++;
                    if (pole[x + i1][y] == 0)
                        successPoint++;
                }
                if (successPoint == 8)
                    return 1;
                else
                    return 0;
            }
            else
                return 0;
        }
        else
        {
            if (pole[x][y] == 0)
            {
                for (int i = -1; i <= 1; i += 2)
                {
                    for (int j = -1; j <= 1; j += 2)
                    {
                        if (pole[x + i][y + j] == 0)
                            successPoint++;
                    }
                }
                if (successPoint == 4)
                {
                    for (int i1 = -1; i1 <= 1; i1 += 2)
                    {
                        if (pole[x][y + i1] == 0)
                            successPoint++;
                        if (pole[x + i1][y] == 0)
                            successPoint++;
                    }
                    if (successPoint == 8)
                        return 1;
                    else
                        return 0;
                }
                else
                    return 0;
            }
        }
    }
}
 
void Sea::viewPole()
{
    for (int i = 0; i <= 10; i++)
    {
        for (int j = 0; j <= 10; j++)
        {
            if (j != 0 && i != 0)
            {
                if (pole[i][j] == 0)
                    cout << " _  ";
                else
                    cout << "    ";
            }
            else
            {
                cout << "    ";
            }
        }
        cout << endl;
        for (int j1 = 0; j1 <= 10; j1++)
        {
            if (j1 != 0)
            {
                if (i != 0)
                {
                    if (pole[i][j1] == 0)
                        cout << "|_| ";
                    else
                        cout << " O  ";
                }
                else
                    cout << " " << j1 << "  ";
            }
            else
            {
                if (i == 0)
                    cout << "    ";
                else
                {
                    if (i != 10)
                        cout << " " << i << "  ";
                    else
                        cout << " " << i << " ";
                }
            }
        }
        cout << endl;
    }
}
И, собственно, сам мейн
C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
#include <cstdlib>
#include <iostream>
#include "Sea.h"
 
using namespace std;
 
int main()
{
    string name;
    getline(cin, name);
    Sea Player1(name);
    Player1.setCorabli(); //вот именно после добавления этой штуки и началась эта проблема
    Player1.viewPole();
    system("PAUSE");
    return 0;
}
Миниатюры
Прекращена работа программы proj.exe  
0
Programming
Эксперт
94731 / 64177 / 26122
Регистрация: 12.04.2006
Сообщений: 116,782
03.09.2013, 01:41
Ответы с готовыми решениями:

После запуска exe-файла выдаёт "Прекращена работа программы"
#include &lt;iostream&gt; #include &lt;cstdlib&gt; using namespace std; int main(){ cout &lt;&lt; &quot;Vvedite...

Прекращена работа программы
Как можна переделать это код, чтобы испоьзовать классы, а не функции стандартного ввода-вывода? ...

Прекращена работа программы
#include &lt;iostream&gt; #include &lt;string&gt; #include &lt;conio.h&gt; #include &lt;windows.h&gt; #include...

Прекращена работа программы с++
Помогите . #include &lt;iostream&gt; #include &lt;vector&gt; #include &lt;iterator&gt; using namespace std;...

6
8 / 8 / 1
Регистрация: 05.06.2013
Сообщений: 60
03.09.2013, 01:44 2
Цитата Сообщение от TrickSonix Посмотреть сообщение
морского боя на консоле

Не по теме:

ты себя то слышал?

в консоли ничего не идет
0
150 / 137 / 35
Регистрация: 29.07.2012
Сообщений: 709
03.09.2013, 01:49 3
Цитата Сообщение от censor Посмотреть сообщение
в консоли ничего не идет
Если постараться то все выйдет.
Цитата Сообщение от TrickSonix Посмотреть сообщение
Player1.setCorabli(); //вот именно после добавления этой штуки и началась эта проблема
Скорей всего проблема в коде, в общем через отладку находите строку где у вас ошибка да и все дела.
0
5499 / 4894 / 831
Регистрация: 04.06.2011
Сообщений: 13,587
03.09.2013, 02:16 4
TrickSonix, бегло просмотрел некоторые места кода, у вас во многих местах выход за границу объявленных массивов. Например, setCorabl():
C++
1
2
3
4
if(numberOfShips[4] < 1)
{
       setCorabl(4);
}
Массив объялен так:
C++
1
int numberOfShips[4];
Значит последний допустимый индекс 3, у вас обращение по 4-му индексу.
В конструкторе тоже самое:
C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
Sea::Sea(string name)
{
    for (int i3 = 1; i3 <= 4; i3++)
        numberOfShips[i3] = 0;
    PlayerName = name;
    for (int i = 0; i <= 10; i++)
    {
        for (int j = 0; j <= 10; j++)
            pole[i][j] = 0;
    }
    for (int i1 = 0; i1 <= 10; i1++)
    {
        for (int j1 = 0; j1 <= 10; j1++)
            poleVistrelov[i1][j1] = 0;
    }
    for (int i2 = 0; i2 <= 20; i2++)
    {
        for (int j2 = 0; j2 <= 2; j2++)
            coordinatiCorablei[i2][j2] = 0;
    }
}
Использование, в параметрах for(), оператора <= (вместо <), приводит к выходу за пределы всех объявленных массивов.
Потом, по стандарту (который компилятор DevC++ с успехом обходит), нельзя статически создавать массив не константного размера:
C++
1
2
3
4
5
void Sea::setCorabl(int size)
{
    int num = 1;
    int xy[2*size];
...
1
1 / 1 / 0
Регистрация: 04.07.2013
Сообщений: 12
03.09.2013, 02:59  [ТС] 5
alsav22, спасибо. Эти ошибки вызваны тем что я массивы бегло пробежал в книге, и возможно то что там написано было несколько запутало меня . А по поводу массива неконстантного размера: как в общем случае правильно задать подобный массив, если вообще есть такая возможность?
0
Неэпический
18109 / 10696 / 2062
Регистрация: 27.09.2012
Сообщений: 26,933
Записей в блоге: 1
03.09.2013, 03:06 6
Цитата Сообщение от TrickSonix Посмотреть сообщение
если вообще есть такая возможность?
Можно выделить память в "куче".
0
5499 / 4894 / 831
Регистрация: 04.06.2011
Сообщений: 13,587
03.09.2013, 03:27 7
Цитата Сообщение от TrickSonix Посмотреть сообщение
А по поводу массива неконстантного размера: как в общем случае правильно задать подобный массив, если вообще есть такая возможность?
C++
1
2
3
4
5
void Sea::setCorabl(int size)
{
    int num = 1;
    int *xy = new int[2*size];
...
Только подумайте об утечках памяти. Я в логику кода не вникал, поэтому не знаю, как часто будет создаваться этот массив в программе: но принцип такой, когда массив становится не нужен - освободить память под ним (delete [] xy).

Добавлено через 5 минут
Потом, компилятор мне подсказывает:
proverka: значение возвращается не при всех путях выполнения
Не знаю, как в этом конкретном коде, но это также бывает причиной ошибок при работе. Допишите эту функцию так, чтобы возврат из неё был при всех вариантах.
0
03.09.2013, 03:27
IT_Exp
Эксперт
87844 / 49110 / 22898
Регистрация: 17.06.2006
Сообщений: 92,604
03.09.2013, 03:27
Помогаю со студенческими работами здесь

Прекращена работа программы
Написал программу. Суть в следующем - создаешь ее ярлык с заданными параметрами и при запуске с...

Прекращена работа программы
Всем доброго времени суток, и с Наступившим! Мне было дано задание составить блок-схему алгоритма...

Прекращена работа программы
#include &lt;iostream&gt; #include &lt;stdlib.h&gt; #include &lt;math.h&gt; #include &lt;stdio.h&gt; #include &lt;iomanip&gt;...

Прекращена работа программы
Компилирует без ошибок, но при работе вылетает #include &lt;stdio.h&gt; #include &lt;conio.h&gt;...


Искать еще темы с ответами

Или воспользуйтесь поиском по форуму:
7
Ответ Создать тему
КиберФорум - форум программистов, компьютерный форум, программирование
Powered by vBulletin
Copyright ©2000 - 2024, CyberForum.ru