Warum funktioniert die x Wertänderung in diesem Programm?
-
03-07-2019 - |
Frage
Ich habe diesen Code erstellt, und wenn ich es laufen, bekommen keine Fehler, bis der Pfeil auf den Bildschirm verlässt (dh: (* I) -> x> maxx), wonach der O zufällig teleportieren (Well , ich bin es nicht zufällig erraten, aber ich versuche, ein Muster, um es zu finden).
EDIT: die zufällige Teleportation scheint nicht aufzutreten, wenn ich nach oben, und wenn ich nach unten zu verschieben, wird die O direkt auf den Boden teleportiert. Auch hat eine Panne aufgetreten, wo der O wird zu einem ‚>‘. (Ich versuche, herauszufinden, wie das passiert)
EDIT: die Transformation-in - '>' Glitch auftritt, wenn die O an der rechten unteren Ecke des Bildschirms ist (player.x = 9; player.y = 9), und die Sequenz "wqs" eingegeben
EDIT: Ich habe die Klassendeklarationen entfernt, weil ich ziemlich sicher bin, dass der Fehler innerhalb der _move ist () s und check ()
.EDIT: Die Transformations Glitch scheint auftreten, wenn ‚WQ‘ eingegeben wird, dann jedes andere Zeichen eingegeben wird (dh „Skiping“ der nächste Schritt)
EDIT: Die TRANFORM Glitch tritt auf, wenn player.x = 9; player.y = 8; und dann ‚q‘ gedrückt wird, wird der nächste Schritt der Spieler wndelt in ein ‚>‘
Dies ist der Code:
#include<vector>
#include<iostream>
#include<string>
using namespace std;
const int maxx = 10, maxy = 10; //two constants that show the size of the sector
char sector[maxx][maxy]; //array of characters used to display the sector
prgm player(0, 0, 'O'); //player definition at x0,y0,and displayed with 'O'
const int vsize = 1; //size of the enemy array (ie: how many enemies there will be
X1 a(9, 5, 'X', 10); //enemy "a", has a move function that moves it back and forth
virus * viral_data[vsize] = {&a}; //array of enemies used to set the sector
vector<antivirus*> antiviral_data; //vector of pointers to "antivirus" the weapon used
vector<antivirus*>::iterator I; //iterator for previous vector
void display() //function to display the sector
{
for(int i = 0; i < maxy; i++)
{
for(int j = 0; j < maxx; j++)
{
cout<<sector[j][i];
}
cout<<endl;
}
return;
}
void p_move() //function to get players input, then move the player or create "antivirus"
{
char dir;
cin>>dir;
switch(dir)
{
case 'w':
player.y--;
break;
case 'a':
player.x--;
break;
case 's':
player.y++;
break;
case 'd':
player.x++;
break;
case 'q':
antiviral_data.push_back(new aX1(player.x, player.y, '>')); //creates a new aX1 at the players position
break;
}
return;
}
void v_move() //uses the enemies move
{
for(int i = 0; i < vsize; i++)
{
viral_data[i]->move();
}
return;
}
void a_move() //uses the weapon (ie: moves the weapon forward)
{
for(I = antiviral_data.begin(); I < antiviral_data.end(); I++)
{
(*I)->move();
}
return;
}
void set() //sets the sector array (char)
{
for(int i = 0; i < maxy; i++)
{
for(int j = 0; j < maxx; j++)
{
sector[j][i] = ' '; makes the entire sector blank
}
}
sector[player.x][player.y]=player.sym; //sets the sector at the player's position to 'O'
for(int i = 0; i < vsize; i++)
{
sector[viral_data[i]->x][viral_data[i]->y] = viral_data[i]->sym; //sets the sector at each enemy's position to be 'X'
}
for(I = antiviral_data.begin(); I < antiviral_data.end(); I++)
{
sector[(*I)->x][(*I)->y] = (*I)->sym; //sets the sector at each weapon's position to be '>'
}
return;
}
void check() //prevents the player from moving off the screen, erases bullet if it moves of the screen (to prevent access to non-allocated memory)
{
if(player.x < 0)
{
player.x = 0;
}
if(player.y < 0)
{
player.y = 0;
}
if(player.x > (maxx-1))
{
player.x = (maxx-1);
}
if(player.y > (maxy-1))
{
player.y = (maxy-1);
}
//PROBLEM APPEARS TO OCCUR HERE
for(I = antiviral_data.begin(); I! = antiviral_data.end();)
{
if((*I)->x > maxx)
{
I = antiviral_data.erase(I);
}
else
{
I++;
}
}
//*-*-*-*-*-*-*-*-*-*-*-*-*-*-*
return;
}
int main()
{
while(true)
{
set(); //set sector
display(); //display sector
p_move(); //player's move
v_move(); //enemy's move
a_move(); //bullet's move
check();//check moves
}
return 0;
}
Lösung
In check (), der Test
((*I)->x > maxx)
sollte
((*I)->x >= maxx)
. Dies ist ein Off-by-one-Fehler, der lässt> ein Quadrat aus bekommen Sie den Bildschirm. Wenn die Anzeigeroutine versucht, um es anzuzeigen, es clobbers das Anzeigesymbol für die X.