Форум программистов, компьютерный форум CyberForum.ru

С++ для начинающих

Войти
Регистрация
Восстановить пароль
 
TrickSonix
1 / 1 / 0
Регистрация: 04.07.2013
Сообщений: 12
#1

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

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

Доброго времени суток всем. У меня возникла неприятная проблемка. Я написал программу(даже не всю а только часть) которая, в последствии, будет что-то вроде морского боя на консоле. В процессе написания я ее запускал, чтобы проверить некоторые элементы, и она прекрасно работала. Но вот дойдя до определенного момента появилось вот что(см. скриншот). Идей почему это происходит и как с этим бороться у меня нет. Пользуюсь 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  
Similar
Эксперт
41792 / 34177 / 6122
Регистрация: 12.04.2006
Сообщений: 57,940
03.09.2013, 01:41
Здравствуйте! Я подобрал для вас темы с ответами на вопрос Прекращена работа программы proj.exe (C++):

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

Прекращена работа программы - C++
Написал следующую программу на visual c++ express 2010: #include &lt;iostream&gt; using namespace std; void main() { const int rows=3,...

Прекращена работа программы - C++
#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;conio.h&gt; #include...

Прекращена работа программы - C++
При запуске консольного приложения выскакивает окно, где говорится о том, что прекращена работа программы. Сообщите, пожалуйста, в каких...

Прекращена работа программы с++ - C++
Помогите . #include &lt;iostream&gt; #include &lt;vector&gt; #include &lt;iterator&gt; using namespace std; bool sorted(int a,int razmer) { ...

Прекращена работа программы - C++
#include &lt;iostream&gt; #include &lt;string&gt; #include &lt;conio.h&gt; #include &lt;windows.h&gt; #include &lt;stdio.h&gt; /* run this program using the...

Надоела реклама? Зарегистрируйтесь и она исчезнет полностью.
censor
8 / 8 / 0
Регистрация: 05.06.2013
Сообщений: 60
03.09.2013, 01:44 #2
Цитата Сообщение от TrickSonix Посмотреть сообщение
морского боя на консоле

Не по теме:

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

в консоли ничего не идет
Bend3r
149 / 136 / 18
Регистрация: 29.07.2012
Сообщений: 709
03.09.2013, 01:49 #3
Цитата Сообщение от censor Посмотреть сообщение
в консоли ничего не идет
Если постараться то все выйдет.
Цитата Сообщение от TrickSonix Посмотреть сообщение
Player1.setCorabli(); //вот именно после добавления этой штуки и началась эта проблема
Скорей всего проблема в коде, в общем через отладку находите строку где у вас ошибка да и все дела.
alsav22
5417 / 4813 / 442
Регистрация: 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];
...
TrickSonix
1 / 1 / 0
Регистрация: 04.07.2013
Сообщений: 12
03.09.2013, 02:59  [ТС] #5
alsav22, спасибо. Эти ошибки вызваны тем что я массивы бегло пробежал в книге, и возможно то что там написано было несколько запутало меня . А по поводу массива неконстантного размера: как в общем случае правильно задать подобный массив, если вообще есть такая возможность?
Croessmah
Модератор
Эксперт CЭксперт С++
13155 / 7418 / 831
Регистрация: 27.09.2012
Сообщений: 18,256
Записей в блоге: 3
Завершенные тесты: 1
03.09.2013, 03:06 #6
Цитата Сообщение от TrickSonix Посмотреть сообщение
если вообще есть такая возможность?
Можно выделить память в "куче".
alsav22
5417 / 4813 / 442
Регистрация: 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: значение возвращается не при всех путях выполнения
Не знаю, как в этом конкретном коде, но это также бывает причиной ошибок при работе. Допишите эту функцию так, чтобы возврат из неё был при всех вариантах.
MoreAnswers
Эксперт
37091 / 29110 / 5898
Регистрация: 17.06.2006
Сообщений: 43,301
03.09.2013, 03:27
Привет! Вот еще темы с ответами:

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

Прекращена работа программы - C++
Здравствуйте)) У меня возникла такая вот проблема: мне задали написать программу,которая определяет количество дней от текущей даты до...

Массив. Прекращена работа программы - C++
Доброго времени суток! Я так понял, что неправильно задаю массив. #include &quot;stdafx.h&quot; #include &quot;iostream&quot; int _tmain(int argc,...

Прекращена работа программы, неполадки с конструктором - C++
При запуске выбывает &quot;Прекращена работа программы&quot;. Скорее всего проблемы с конструктором для Note mas; . Может намудрил с указателем на...


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

Или воспользуйтесь поиском по форуму:
Yandex
Объявления
03.09.2013, 03:27
Ответ Создать тему
Опции темы

КиберФорум - форум программистов, компьютерный форум, программирование
Powered by vBulletin® Version 3.8.9
Copyright ©2000 - 2017, vBulletin Solutions, Inc.
Рейтинг@Mail.ru