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

Морской бой: Проверка на победителя никогда не завершается - C++

Восстановить пароль Регистрация
Другие темы раздела
C++ Собрать int из четырех байтов в файле http://www.cyberforum.ru/cpp-beginners/thread749915.html
Есть файл, данные сохранены последовательно, в основном чары иногда есть 4 байтовые числа, не могу прочитать эти числа в переменную, подскажите плиз!
C++ Работа с файлами (fseek) В чем проблема и как ее устранить ? #include <iostream> #include <conio.h> #include <string.h> #include <stdio.h> #include<io.h> using namespace std; void main() http://www.cyberforum.ru/cpp-beginners/thread749914.html
C++ Двумерные массивы (СЛАУ). Привести систему к треугольному виду
Коэффициенты системы линейных уравнений заданы в виде прямоугольной матрицы.С помощью допустимых преобразований привести систему к треугольному виду. Найти количество строк, среднее арифметическое элементов которых меньше данной величины. Незнаю как решить.
C++ иллюстрация полиморфизма
Доброго времени суток!написал примитив для иллюстрации полиморфизма,ориентировался по видеокурсам с ТыТрубы #include<iostream.h> #include<conio.h> class Shape { public : virtual float calc_area()=0; }; class Rectangle : public Shape
C++ Функция для записи в буфер из потока http://www.cyberforum.ru/cpp-beginners/thread749889.html
существует в с++ такая функция для загрузки потока данных в строку fgets(buf,10000, f); но недостаток этой функции в том, что при наличии в потоке символа конца строки, он прекращает запись из потока посему подскажите пожалуйста, существует ли функция которая будет записывать в строку из потока до тех пор пока не наступит конец файла, а не до попадания на символ конца строки.
C++ Представление кода в виде функции Столкнулся с проблемой, сделал код в виде функций, все запускается и вроде бы работает, но как введу значения жму Enter и программа просто виснет. В чем может быть проблема? Компилятор все норм компилит. Исходный: //--------------------------------------------------------------------------- #include <vcl.h> #pragma hdrstop #include <tchar.h> #include <iostream.h> #include <math.h> подробнее

Показать сообщение отдельно
Karmelo
0 / 0 / 0
Регистрация: 30.12.2012
Сообщений: 7
30.12.2012, 01:42     Морской бой: Проверка на победителя никогда не завершается
Здравствуйте дорогие форумчане. Есть у меня некий код морского боя, не полный. Проблема у меня следующая:
после расстановки кораблей, после первого выстрела программа самостоятельно добавляет еще один корабль в верхний левый угол массива. Хоть эта координата не отображается на "Поле боя", но проверка на победителя никогда не завершается. Учусь я не в России и мои коллеги, с которыми я делаю работу, не понимают русский. Таким образом хочу попросить Вашей помощи.

Все на С++.

Вот сам код:
все по файлам, как у меня в проекте.
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
284
285
286
287
288
289
290
291
292
293
294
295
296
297
298
299
300
301
302
303
304
305
306
307
308
309
310
311
312
313
314
315
316
317
318
319
320
321
322
323
324
325
326
327
328
329
330
331
332
333
334
335
336
337
338
339
340
341
342
343
344
345
346
347
348
349
350
351
352
353
354
355
356
357
358
359
360
361
362
363
364
365
366
367
368
369
370
371
372
373
374
375
376
377
378
379
380
381
382
383
384
385
386
387
388
389
390
391
392
393
394
395
396
397
398
399
Main.cpp___________________________________________________________________________________
 
#include <iostream>
#include <string>
#include<iomanip>
#include "Atacar.h"
#include"Coloca Navios.h"
#include"Coordenadas.h"
#include"DesenhaTabuleiro.h"
#include"Game Over.h"
#include"Jogador.h"
//#include"Jogo Comeзa.h"
#include"LoadBarcos.h"
#include"Navio.h"
#include"ResetTabuleiro.h"
#include"Tabuleiro.h"
 
using namespace std;
 
 
int main()
{
    LoadBarcos();
    ResetTabuleiro();
 
    //"PLACE SHIPS" phase of game
    //Loop through each jogador... 
    for (int ajgdr=1; ajgdr<3; ++ajgdr)
    {
        //Loop through each ship type to place
        for (int esteBarco=0; esteBarco<TIPOS_DE_NAVIOS; ++esteBarco)
        {
            //Display gameboard for jogador
            system("cls");
            DesenhaTabuleiro(ajgdr);
            //Give instructions
            cout << "\n";
            cout << "Instrucoes (Jogador " << ajgdr << ")\n\n";
            cout << "Можно ставить кораблики. \n";
            cout << "Пример 1 7 2  - поставит субмарину на X:7 Y:2\n\n";
            cout<< "1-Субмарина,              однопалубный \n"
                 
 
            int b;
            cin>>b;
            //Get input from user and loop until good data is returned
            COLOCA_NAVIOS umBarco;
            
            if(b==1){
            m1:
            umBarco.TipoBarco.noTabuleiro[0].X = -1;
            
            while (umBarco.TipoBarco.noTabuleiro[0].X == -1)
            {
                umBarco = Utilizador_poe_Submarino();
            }
            
            //Combine user data with "this ship" data
            //umBarco.TipoBarco.comprimento = ship[b].comprimento;
            //umBarco.TipoBarco.nome = ship[b].nome;
            for (int i=0; i<b;i++){
                if (jogador[ajgdr].tabuleiro[umBarco.TipoBarco.noTabuleiro[0].X][umBarco.TipoBarco.noTabuleiro[0].Y] != Agua){cout<<"coordenada invalida/n";goto m1;}
            for(int j=-1;j<2;j++){
                if(jogador[ajgdr].tabuleiro[umBarco.TipoBarco.noTabuleiro[i].X+j][umBarco.TipoBarco.noTabuleiro[i].Y] != Agua){cout<<"coordenada invalida1";goto m1;}
                if(jogador[ajgdr].tabuleiro[umBarco.TipoBarco.noTabuleiro[i].X][umBarco.TipoBarco.noTabuleiro[i].Y+j] != Agua){cout<<"coordenada invalida2";goto m1;}
                if(jogador[ajgdr].tabuleiro[umBarco.TipoBarco.noTabuleiro[i].X+j][umBarco.TipoBarco.noTabuleiro[i].Y+j] != Agua){cout<<"coordenada invalida3";goto m1;}
                if(jogador[ajgdr].tabuleiro[umBarco.TipoBarco.noTabuleiro[i].X+j][umBarco.TipoBarco.noTabuleiro[i].Y-+j] != Agua){cout<<"coordenada invalida4";goto m1;}
            
            }
            
            //Add the FIRST tabuleiro point to the current jogador's game board
                jogador[ajgdr].tabuleiro[umBarco.TipoBarco.noTabuleiro[0].X][umBarco.TipoBarco.noTabuleiro[0].Y] = Submarino;
            }
            
        }
            
            
        
        
}
        //Loop back through each jogador
    }
 
    //********* FINISHED WITH "PLACE SHIPS" PHASE *********************************
    //*****************************************************************************
 
    //Ready to play the game
    
    bool jogoComeзa = true;
    int esteJogador = 1;
    do {
        //Because we are ATTACKING now, the 
        //opposite jogador board is the display board
        int inimigo;
        if (esteJogador == 1) inimigo = 2;
        if (esteJogador == 2) inimigo = 1;
        system("cls");
        DesenhaTabuleiro(inimigo);
        
        //Get attack coords from this jogador
        bool bemIntroduzido = false;
        int x,y;
        while (bemIntroduzido == false) {
            bemIntroduzido = UtilizadorAtaca(x,y,esteJogador);
        }
 
        //Check board; if a ship is there, set as HIT.. otherwise MISS
        if (jogador[inimigo].tabuleiro[x][y] != Agua) {jogador[inimigo].tabuleiro[x][y] = Acertou;}
        else
        {jogador[inimigo].tabuleiro[x][y] = Falhou;}
 
        //Check to see if the game is over
        //If 0 is returned, nobody has won yet
        int Vitoria = GameOver(inimigo);
        if (Vitoria != 0) {
            jogoComeзa = false;
            break;
        }
        //Alternate between each jogador as we loop back around
        esteJogador = (esteJogador == 1) ? 2 : 1;
    } while (jogoComeзa);
 
    system("cls");
    cout << "\n\nCONGRATULATIONS!!!  JOGADOR " << esteJogador << " HAS WON THE GAME!\n\n\n\n";
 
    system("pause");
    return 0;
}
 
 
DesenhaTabuleiro.h_____________________________________________________________________________
 
#include<iostream>
#include"Tabuleiro.h"
#include "Jogador.h"
using namespace std;
bool jogoComeзa;
void DesenhaTabuleiro(int esteJogador)
{
    //Draws the board for a jogador (esteJogador)
    cout << "JOGADOR " << esteJogador << "'s GAME BOARD\n";
    cout << "----------------------\n";
 
    //Loop through top row (board_width) and number columns
    cout << "   ";
    for (int c=1; c<COMPRIMENTO_TABULEIRO; c++) {
        if (c < 10)
            //Numbers only 1 character long, add two spaces after
            cout << "0"<< c << " ";
        else if (c >= 10)
            //Numbers 2 characters long, add only 1 space after
            cout << c<< " ";
    }
    cout << endl;//"\n";
 
    //Loop through each tabuleiro point and display to console
    for (int a=1; a<ALTURA_TABULEIRO; a++){
        cout << endl;
        if (a<10) cout <<"0"<< a << " ";
        else cout << a << " ";
        //cout << endl;
        for (int c=1; c<COMPRIMENTO_TABULEIRO; c++){
            
            //If this is the FIRST (left) tabuleiro point, number the tabuleiro first
            
                //if (c==1) cout << a << " ";
            //If a was 1 character long, add an extra space to keep numbers lined up
            if (c<10 && c==1) cout << " ";
        
            //Display contents of this tabuleiro (if game isn't running yet, we are placing ships
            //so display the ships
            
            if (jogoComeзa == false) cout << jogador[esteJogador].tabuleiro[c][a] << "  ";
            //Don't show ships, BUT show damage if it's hit
            
            if (jogoComeзa == true && jogador[esteJogador].tabuleiro[c][a] != Submarino) 
                
                {cout << jogador[esteJogador].tabuleiro[c][a] << "  ";}
            
            else if (jogoComeзa == true && jogador[esteJogador].tabuleiro[c][a] == Submarino)
            {cout << Agua << "  " <<endl;}  
            //If we have reached the border.. line feed
 
            if (c == COMPRIMENTO_TABULEIRO) cout <<"\n";
        }
        cout << endl;
    }
    cout << endl;
}
 
 
Tabuleiro.h__________________________________________________________________________________
 
#pragma once
#ifndef TABULEIRO_H
#define TABULEIRO_H
 
#include <iostream>
#include <string>
 
using namespace std;
 
const int COMPRIMENTO_TABULEIRO = 11;//Высота 
const int ALTURA_TABULEIRO = 11;//Ширина 
const int TIPOS_DE_NAVIOS = 2;//Кораблей в общим  
 
const char Agua = 222; //ASCII Character Code
const char Acertou = 'X';
const char Submarino = 'U';
const char Falhou = '0';
 
 
#endif
 
ResetTabuleiro.h______________________________________________________________________________
 
#include "Jogador.h"
 
void ResetTabuleiro()
{
    //Loop through each jogador
    for (int jgdr=1; jgdr<3; ++jgdr)
    {
        //For each tabuleiro point, set contents to 'water'
        for (int c=0; c<COMPRIMENTO_TABULEIRO; ++c){
            for (int a=0; a<ALTURA_TABULEIRO; ++a){
                jogador[jgdr].tabuleiro[c][a] = Agua;
        }}
        //Loop back to next jogador
    }
}
 
 
LoadBarcos.h___________________________________________________________________________________
 
#pragma once
#ifndef LOADBARCOS_H
#define LOADBARCOS_H
 
 
#include "Coloca Navios.h"
#include "Navio.h"
void LoadBarcos()
{
    //Sets the default data for the ships
    //we plan to include in the game
    //IMPORTANT!! > MUST MATCH TIPOS_DE_NAVIOS -Default=5 (0-4)
    navio[1].nome = "Submarina"; navio[1].comprimento= 1;
    
}
#endif
 
 
Atacar.h______________________________________________________________________________________
 
#include <iostream>
#include <string>
#include "Tabuleiro.h"
using namespace std;
 
 
bool UtilizadorAtaca(int & x, int & y, int theplayer)
{
    cout << "\nJOGADOR " << theplayer << ", ENTER COORDINATES TO ATTACK: ";
    bool bemIntroduzido = false;
    cin >> x >> y;
    if (x<1 || x>=COMPRIMENTO_TABULEIRO) return bemIntroduzido;
    if (y<1 || y>=ALTURA_TABULEIRO) return bemIntroduzido;
    bemIntroduzido = true; 
    return bemIntroduzido;
}
 
 
Game Over.h_________________________________________________________________________________
 
#include "Tabuleiro.h"
#include "Jogador.h"
 
bool GameOver(int INIMIGO)
{
    bool vencedor = true;
    //Loop through enemy board
    for (int c=0; c<COMPRIMENTO_TABULEIRO; c++){
        for (int a=0; a<ALTURA_TABULEIRO; a++){
                //If any ships remain, game is NOT over
                if (jogador[INIMIGO].tabuleiro[a][c] = Submarino)
                    {
                        vencedor = false;}
                else {
                    return vencedor;}
                    
        }}
    //If we get here, somebody won, game over!
    return vencedor;
}
 
 
Coordenadas.h____________________________________________________________________________________
 
 
#ifndef COORDENADAS_H
#define COORDENADAS_H
 
// here is your header file code
 
 
 
#include <iostream>
 
using namespace std;
 
typedef struct COORDENADAS {
 
    //A location on the tabuleiro defined
    
    int X;
    int Y;
}COORDENADAS;
#endif
 
 
ColocaNavios.h_____________________________________________________________________________________
 
#pragma once
#ifndef COLOCA_NAVIOS_H
#define COLOCA_NAVIOS_H
/*#include<iostream>
#include<string>
#include <stdlib.h>
#include "Coloca Navios.h"*/
#include"Navio.h"
 
enum BARCOS {SUBMARINO, CRL};
typedef struct COLOCA_NAVIOS {
    BARCOS nome;
    NAVIO TipoBarco;
}COLOCA_NAVIOS;
 
COLOCA_NAVIOS Utilizador_poe_Submarino()
    {
    int x, y;
    COLOCA_NAVIOS rtn;
    //Using this as a bad return
    rtn.TipoBarco.noTabuleiro[0].X = -1;
    //Get 2 integers from user
 
    cin >> x >> y;
    
    //int *buscar =&b;
    //p= *buscar;
    if (x<1 || x>COMPRIMENTO_TABULEIRO) return rtn;
    if (y<1 || y>ALTURA_TABULEIRO)      return rtn;
 
    //Good data
    rtn.TipoBarco.noTabuleiro[0].X= x;
    rtn.TipoBarco.noTabuleiro[0].Y = y;
    return rtn;
};
 
#endif
 
 
Navio.h__________________________________________________________________________________________
 
#ifndef NAVIO_H
#define NAVIO_H
 
#include<string>
#include "Coordenadas.h"
#include"Tabuleiro.h"
 
using namespace std;
 
 
struct NAVIO {
    //Ship name
    string nome;
    //Total points on the tabuleiro
    int comprimento;
    //Coordinates of those points
    COORDENADAS noTabuleiro[5]; //0-4 max comprimento of biggest ship
    //Whether or not those points are a "hit"
    bool acertos[5];
}navio[TIPOS_DE_NAVIOS];
 
#endif
 
Jogador.h________________________________________________________________________________________
 
#pragma once
#ifndef JOGADOR_H
#define JOGADOR_H
#include"Tabuleiro.h"
struct JOGADOR {
    char tabuleiro[COMPRIMENTO_TABULEIRO][ALTURA_TABULEIRO];
}jogador[3]; //Ignore jogador 0, just using jogador's 1 & 2
 
 
#endif

Подозреваю вот эту часть, так как после выстрела идет проверка, и вот после нее появится кораблик. Чтоб увидеть его нужно поменять границы проверки. Чтоб проверял не от 0, а от 1. int с=1 и int a=1.

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
bool GameOver(int INIMIGO)
{
    bool vencedor = true;
    //Loop through enemy board
    for (int c=0; c<COMPRIMENTO_TABULEIRO; c++){
        for (int a=0; a<ALTURA_TABULEIRO; a++){
                //If any ships remain, game is NOT over
                if (jogador[INIMIGO].tabuleiro[a][c] = Submarino)
                    {
                        vencedor = false;}
                else {
                    return vencedor;}
                    
        }}
    //If we get here, somebody won, game over!
    return vencedor;
}


P.S. Всех с наступающим НОВЫМ ГОДОМ!
После регистрации реклама в сообщениях будет скрыта и будут доступны все возможности форума.
 
Текущее время: 14:21. Часовой пояс GMT +3.
КиберФорум - форум программистов, компьютерный форум, программирование
Powered by vBulletin® Version 3.8.9
Copyright ©2000 - 2016, vBulletin Solutions, Inc.
Рейтинг@Mail.ru