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

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

Восстановить пароль Регистрация
 
TrickSonix
1 / 1 / 0
Регистрация: 04.07.2013
Сообщений: 12
03.09.2013, 01:41     Прекращена работа программы proj.exe #1
Доброго времени суток всем. У меня возникла неприятная проблемка. Я написал программу(даже не всю а только часть) которая, в последствии, будет что-то вроде морского боя на консоле. В процессе написания я ее запускал, чтобы проверить некоторые элементы, и она прекрасно работала. Но вот дойдя до определенного момента появилось вот что(см. скриншот). Идей почему это происходит и как с этим бороться у меня нет. Пользуюсь 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  
После регистрации реклама в сообщениях будет скрыта и будут доступны все возможности форума.
censor
8 / 8 / 0
Регистрация: 05.06.2013
Сообщений: 60
03.09.2013, 01:44     Прекращена работа программы proj.exe #2
Цитата Сообщение от TrickSonix Посмотреть сообщение
морского боя на консоле

Не по теме:

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

в консоли ничего не идет
Bend3r
 Аватар для Bend3r
142 / 129 / 17
Регистрация: 29.07.2012
Сообщений: 681
03.09.2013, 01:49     Прекращена работа программы proj.exe #3
Цитата Сообщение от censor Посмотреть сообщение
в консоли ничего не идет
Если постараться то все выйдет.
Цитата Сообщение от TrickSonix Посмотреть сообщение
Player1.setCorabli(); //вот именно после добавления этой штуки и началась эта проблема
Скорей всего проблема в коде, в общем через отладку находите строку где у вас ошибка да и все дела.
alsav22
5282 / 4801 / 442
Регистрация: 04.06.2011
Сообщений: 13,587
03.09.2013, 02:16     Прекращена работа программы proj.exe #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  [ТС]     Прекращена работа программы proj.exe #5
alsav22, спасибо. Эти ошибки вызваны тем что я массивы бегло пробежал в книге, и возможно то что там написано было несколько запутало меня . А по поводу массива неконстантного размера: как в общем случае правильно задать подобный массив, если вообще есть такая возможность?
Croessmah
Модератор
Эксперт С++
 Аватар для Croessmah
11828 / 6807 / 769
Регистрация: 27.09.2012
Сообщений: 16,878
Записей в блоге: 2
Завершенные тесты: 1
03.09.2013, 03:06     Прекращена работа программы proj.exe #6
Цитата Сообщение от TrickSonix Посмотреть сообщение
если вообще есть такая возможность?
Можно выделить память в "куче".
MoreAnswers
Эксперт
37091 / 29110 / 5898
Регистрация: 17.06.2006
Сообщений: 43,301
03.09.2013, 03:27     Прекращена работа программы proj.exe
Еще ссылки по теме:

C++ Прекращена работа программы
Прекращена работа программы с++ C++
C++ После запуска exe-файла выдаёт "Прекращена работа программы"

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

Или воспользуйтесь поиском по форуму:
alsav22
5282 / 4801 / 442
Регистрация: 04.06.2011
Сообщений: 13,587
03.09.2013, 03:27     Прекращена работа программы proj.exe #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: значение возвращается не при всех путях выполнения
Не знаю, как в этом конкретном коде, но это также бывает причиной ошибок при работе. Допишите эту функцию так, чтобы возврат из неё был при всех вариантах.
Yandex
Объявления
03.09.2013, 03:27     Прекращена работа программы proj.exe
Ответ Создать тему
Опции темы

Текущее время: 01:09. Часовой пояс GMT +3.
КиберФорум - форум программистов, компьютерный форум, программирование
Powered by vBulletin® Version 3.8.9
Copyright ©2000 - 2016, vBulletin Solutions, Inc.
Рейтинг@Mail.ru