Buenas,
Tengo un problema al intentar colisionar dos figuras en sfml, la cosa es que colisiona pero en vez de chocar y estarse quieto sin poder avanzar se coloca en la otra parte de la otra figura y así consecutivamente.
Aquí un vídeo previo del error:
El vídeo es capturado por mi, dura 30 seg. Para mostrar el error.
[youtube=640,360]https://www.youtube.com/watch?v=i8TFZbfo7EE[/youtube]
sf::FloatRect rect = rec.getGlobalBounds();
sf::FloatRect rect2 = rec2.getGlobalBounds();
if(rect.intersects(rect2)){
if(rec.getPosition().x<400){
std::cout << rec.getPosition().x << std::endl;
rec.setPosition(350, rec.getPosition().y);
}
if(rec.getPosition().x>400){
rec.setPosition(450, rec.getPosition().y);
}
if(rec.getPosition().y<400){
rec.setPosition(rec.getPosition().x, 350);
}
if(rec.getPosition().y>400){
rec.setPosition(rec.getPosition().x, 450);
}
}
rec -> es el rectángulo que manejo con el keyboard.
rec2 -> es el rectángulo que está situado fijamente en el centro de la ventana.
Si supiesen a que debe deberse dicho error agradecería alguna respuesta.
Gracias.
Saludos!
A que se debe es algo obvio, pusiste los setPosition().
Para empezar, y dado que tienes la rect2, en vez de poner 400, 350 y 450, deberías poner los campos de la variable rect2.
No es que haya un error, simplemente el concepto es incorrecto.
Intenté hacer lo que me indicaste pero no daba resultado al igual también intenté esto:
if(rec.getPosition().x<400){
std::cout << rec.getPosition().x << std::endl;
rec.setPosition(rec.getPosition().x, rec.getPosition().y);
}
Que no pueda avanzar más si colisiona sin ser posicionado, pero al igual sigue con el mismo error.
Gracias por la respuesta.
Saludos!
rec.setPosition(rec.getPosition().x, rec.getPosition().y);
Mira atentamente esa línea.
A parte:
Ese ejemplo que pones en el vídeo no explica nada si no indicas cuales son las coordenadas de los objetos. ¿Qué es 400?
Lo que indiqué era cambiar esos valores por valores de las variables. Poner un 400 para quien ve ese código es como poner un 7568. No dice nada. Cambia cada valor por el campo de la variable que ha de ser. No va a corregir el problema ni mucho menos, pero haré cl programa más legible y más fácil de corregir (y más fácil de rehacer en caso de que quieras cambiar la posición del cuadrado ese)
Así está repartido:
sf::RenderWindow window(sf::VideoMode(800, 600), "Colision");
//Rectangualo
sf::RectangleShape rec(sf::Vector2f(50, 50));
rec.setFillColor(sf::Color::Cyan);
//Otro rectangulo
sf::RectangleShape rec2(sf::Vector2f(50, 50));
rec2.setFillColor(sf::Color::Magenta);
rec2.setPosition(400, 300);
En el centro de la ventana (800x600).
Código completo:
#include <SFML/Graphics.hpp>
#include <iostream>
int main(){
//Creamos la ventana
sf::RenderWindow window(sf::VideoMode(800, 600), "Colision");
//Rectangualo
sf::RectangleShape rec(sf::Vector2f(50, 50));
rec.setFillColor(sf::Color::Cyan);
//Otro rectangulo
sf::RectangleShape rec2(sf::Vector2f(50, 50));
rec2.setFillColor(sf::Color::Magenta);
rec2.setPosition(400, 300);
window.setFramerateLimit(200);
//Si la ventana está abierta
while(window.isOpen()){
//Creamos un evento
sf::Event ventana;
//Llamamos a los eventos
while(window.pollEvent(ventana)){
//Cerramos la ventana cuando se solicite
if(ventana.type == sf::Event::Closed){
window.close();
}
}
//Movimiento rectangulo
if(sf::Keyboard::isKeyPressed(sf::Keyboard::Up)){
rec.move(0, -1);
}
if(sf::Keyboard::isKeyPressed(sf::Keyboard::Down)){
rec.move(0, 1);
}
if(sf::Keyboard::isKeyPressed(sf::Keyboard::Left)){
rec.move(-1, 0);
}
if(sf::Keyboard::isKeyPressed(sf::Keyboard::Right)){
rec.move(1, 0);
}
//Colisiones
if(rec.getPosition().x<0){
rec.setPosition(0, rec.getPosition().y);
}
if(rec.getPosition().x>750){
rec.setPosition(750, rec.getPosition().y);
}
if(rec.getPosition().y<0){
rec.setPosition(rec.getPosition().x, 0);
}
if(rec.getPosition().y>550){
rec.setPosition(rec.getPosition().x, 550);
}
sf::FloatRect rect = rec.getGlobalBounds();
sf::FloatRect rect2 = rec2.getGlobalBounds();
if(rect.intersects(rect2)){
if(rec.getPosition().x<400){
std::cout << rec.getPosition().x << std::endl;
rec.setPosition(350, rec.getPosition().y);
}
if(rec.getPosition().x>400){
rec.setPosition(450, rec.getPosition().y);
}
if(rec.getPosition().y<400){
rec.setPosition(rec.getPosition().x, 350);
}
if(rec.getPosition().y>400){
rec.setPosition(rec.getPosition().x, 450);
}
}
//Limpiamos ventana
window.clear();
//Dibujamos el rectangulo
window.draw(rec);
window.draw(rec2);
//Actualizamos ventana
window.display();
}
}
Saludos!
Pues cambia el .x<400 por rec2.left, y así con cada uno. Los valores, en variables, especialmente en este tipo de programa.
He probado tu forma pero no funcionaba y al igual con tu idea intente hacer algo similar haber si funcionaba al principio parecía que iba a funcionar pero al final no.
Probé con algo así:
sf::FloatRect rect = rec.getGlobalBounds();
sf::FloatRect rect2 = rec2.getGlobalBounds();
if(rect.intersects(rect2)){
if(rec.getPosition().x<rect2.left){
rec.setPosition(rec2.getPosition().x-50, rec2.getPosition().y);
}
if(rec.getPosition().x>rect2.left){
rec.setPosition(rec2.getPosition().x+50, rec2.getPosition().y);
}
if(rec.getPosition().y<rect2.left){
rec.setPosition(rec2.getPosition().x, rec2.getPosition().y-50);
}
if(rec.getPosition().y>rect2.left){
rec.setPosition(rec2.getPosition().x, rec2.getPosition().y+50);
}
}
Puse rect2.left ya que tenia que declararla y no valía rec2.left
Disculpa que te esté dando trabajo, te agradezco la mano que intentas echarme.
He estado buscando información y tal y encontré alguna otra forma también posible de hacerlo:
#include <SFML/Graphics.hpp>
#include <iostream>
void comprobarIntersecta();
int main(){
//Creamos la ventana
sf::RenderWindow window(sf::VideoMode(800, 600), "Colision");
//Rectangualo
sf::RectangleShape rec(sf::Vector2f(50, 50));
rec.setFillColor(sf::Color::Cyan);
//Otro rectangulo
sf::RectangleShape rec2(sf::Vector2f(50, 50));
rec2.setFillColor(sf::Color::Magenta);
rec2.setPosition(400, 300);
window.setFramerateLimit(200);
//Si la ventana está abierta
while(window.isOpen()){
//Creamos un evento
sf::Event ventana;
//Llamamos a los eventos
while(window.pollEvent(ventana)){
//Cerramos la ventana cuando se solicite
if(ventana.type == sf::Event::Closed){
window.close();
}
}
//Colisiones ventana bordes
if(rec.getPosition().x<0){
rec.setPosition(0, rec.getPosition().y);
}
if(rec.getPosition().x>750){
rec.setPosition(750, rec.getPosition().y);
}
if(rec.getPosition().y<0){
rec.setPosition(rec.getPosition().x, 0);
}
if(rec.getPosition().y>550){
rec.setPosition(rec.getPosition().x, 550);
}
sf::FloatRect rect = rec.getGlobalBounds();
sf::FloatRect rect2 = rec2.getGlobalBounds();
bool chocar [4] = {false,false,false,false}; //Norte, sur, este y oeste
if(rect.intersects(rect2)){
comprobarIntersecta();
}
if(sf::Keyboard::isKeyPressed(sf::Keyboard::Up)){
if(!chocar[0])
rec.move(0, -1);
}
else if(sf::Keyboard::isKeyPressed(sf::Keyboard::Down)){
if(!chocar[1])
rec.move(0, 1);
}
else if(sf::Keyboard::isKeyPressed(sf::Keyboard::Left)){
if(!chocar[2])
rec.move(-1, 0);
}
else if(sf::Keyboard::isKeyPressed(sf::Keyboard::Right)){
if(!chocar[3])
rec.move(1, 0);
}
//Limpiamos ventana
window.clear();
//Dibujamos el rectangulo
window.draw(rec);
window.draw(rec2);
//Actualizamos ventana
window.display();
}
}
void comprobarIntersecta(){
sf::RectangleShape rec(sf::Vector2f(50, 50));
rec.setFillColor(sf::Color::Cyan);
if(rec.getPosition().x<400){
chocar[0] = true;
}
}
Pero de igual modo supuestamente hay que modificar el bool por true cuando colisione o tal así.
Gracias.
Saludos!
Repasa el algoritmo. Piensa en qué pasa en cada momento, y en qué es incorrecto.
if(rec.getPosition().x<rect2.left)
if(rec.getPosition().x>rect2.left)
¿Qué ocurre si x==rect.left? En vez de > pon >=, o <= en vez de <, como veas.
Si repasas el algoritmo y sacas valores por pantalla, haciendo las operaciones acabarás viendo por qué ocurre lo que ocurre. Sinó, puedes hacerlo en papel con valores pequeños (1-5) para encontrar el error, o mejor aun, para encontrar la forma correcta.
Mirarlo en internet no te va a ayudar a entenderlo, y menos copiando código sin más.
Probé como me dijiste y por una parte responde bien entre sí...
De nuevo un corto vídeo del funcionamiento.
Solo funciona si voy de izquierda a derecha. <-->
[youtube=640,360]https://www.youtube.com/watch?v=DHS5weUasYU[/youtube]
#include <SFML/Graphics.hpp>
#include <iostream>
int main(){
//Creamos la ventana
sf::RenderWindow window(sf::VideoMode(800, 600), "Colision");
//Rectangualo
sf::RectangleShape rec(sf::Vector2f(50, 50));
rec.setFillColor(sf::Color::Cyan);
//Otro rectangulo
sf::RectangleShape rec2(sf::Vector2f(50, 50));
rec2.setFillColor(sf::Color::Magenta);
rec2.setPosition(400, 300);
window.setFramerateLimit(200);
//Si la ventana está abierta
while(window.isOpen()){
//Creamos un evento
sf::Event ventana;
//Llamamos a los eventos
while(window.pollEvent(ventana)){
//Cerramos la ventana cuando se solicite
if(ventana.type == sf::Event::Closed){
window.close();
}
}
//Movimiento rectangulo
if(sf::Keyboard::isKeyPressed(sf::Keyboard::Up)){
rec.move(0, -1);
}
if(sf::Keyboard::isKeyPressed(sf::Keyboard::Down)){
rec.move(0, 1);
}
if(sf::Keyboard::isKeyPressed(sf::Keyboard::Left)){
rec.move(-1, 0);
}
if(sf::Keyboard::isKeyPressed(sf::Keyboard::Right)){
rec.move(1, 0);
}
//Colisiones
if(rec.getPosition().x<0){
rec.setPosition(0, rec.getPosition().y);
}
if(rec.getPosition().x>750){
rec.setPosition(750, rec.getPosition().y);
}
if(rec.getPosition().y<0){
rec.setPosition(rec.getPosition().x, 0);
}
if(rec.getPosition().y>550){
rec.setPosition(rec.getPosition().x, 550);
}
sf::FloatRect rect = rec.getGlobalBounds();
sf::FloatRect rect2 = rec2.getGlobalBounds();
if(rect.intersects(rect2)){
if(rec.getPosition().x==rect2.left){
rec.setPosition(rec.getPosition().x-50, rec.getPosition().y);
}
if(rec.getPosition().x==rect2.left){
rec.setPosition(rec.getPosition().x+50, rec.getPosition().y);
}
if(rec.getPosition().y==rect2.left){
rec.setPosition(rec.getPosition().x, rec.getPosition().y-50);
}
if(rec.getPosition().y==rect2.left){
rec.setPosition(rec.getPosition().x, rec.getPosition().y+50);
}
}
//Limpiamos ventana
window.clear();
//Dibujamos el rectangulo
window.draw(rec);
window.draw(rec2);
//Actualizamos ventana
window.display();
}
}
Parte que modifiqué:
if(rect.intersects(rect2)){
if(rec.getPosition().x==rect2.left){
rec.setPosition(rec.getPosition().x-50, rec.getPosition().y);
}
if(rec.getPosition().x==rect2.left){
rec.setPosition(rec.getPosition().x+50, rec.getPosition().y);
}
if(rec.getPosition().y==rect2.left){
rec.setPosition(rec.getPosition().x, rec.getPosition().y-50);
}
if(rec.getPosition().y==rect2.left){
rec.setPosition(rec.getPosition().x, rec.getPosition().y+50);
}
}
Gracias.
Saludos!
#ACTUALIZACIÓN
Acabo de probar otra forma que da resultado por una parte, pero por la otra no.
Si voy de izquierda a derecha colisiona y no puede avanzar "bien", si voy de derecha a izquierda colisiona y no puede avanzar "bien", eso si quitamos la colisión de "y". Y si añado la colisión de "y" se desplaza el rectángulo a los lados como se ve al principio del siguiente vídeo:
[youtube=640,360]https://www.youtube.com/watch?v=KOVwKT6rf54[/youtube]
Código al que hago referencia en el vídeo:
sf::FloatRect rect = rec.getGlobalBounds();
sf::FloatRect rect2 = rec2.getGlobalBounds();
if(rect.intersects(rect2)){
if(rec.getPosition().x<=rect2.left){
rec.setPosition(rec.getPosition().x-1, rec.getPosition().y);
}
if(rec.getPosition().x>=rect2.left){
rec.setPosition(rec.getPosition().x+1, rec.getPosition().y);
}
if(rec.getPosition().y<=rect2.left){
rec.setPosition(rec.getPosition().x, rec.getPosition().y-1);
}
if(rec.getPosition().y>=rect2.left){
rec.setPosition(rec.getPosition().x, rec.getPosition().y+1);
}
}
Por una parte parece que funciona:
if(rec.getPosition().x<=rect2.left){
rec.setPosition(rec.getPosition().x-1, rec.getPosition().y);
}
if(rec.getPosition().x>=rect2.left){
rec.setPosition(rec.getPosition().x+1, rec.getPosition().y);
}
Ya que colisionan y no pueden avanzar bien, eso si están solo estas funciones.
Si añado las demás funciones que hace referencia a "y" ya al colisionar se desplaza solo.
f(rec.getPosition().x<=rect2.left){
rec.setPosition(rec.getPosition().x-1, rec.getPosition().y);
}
if(rec.getPosition().x>=rect2.left){
rec.setPosition(rec.getPosition().x+1, rec.getPosition().y);
}
if(rec.getPosition().y<=rect2.left){
rec.setPosition(rec.getPosition().x, rec.getPosition().y-1);
}
if(rec.getPosition().y>=rect2.left){
rec.setPosition(rec.getPosition().x, rec.getPosition().y+1);
}
Saludos!
if(rec.getPosition().y>=rect2.left)
Eso será rect2.top, no .left.
De cualquier forma, es emétodo lo que hace es corregir progresivamente la superposición de los rectángulos. Una colisión es una colisión.
Una opción, es, en el código de movimiento, tras moverse, comprobar con intersecs(). En caso verdadero, deshaces el movimiento. Es una forma muy simple, pero efectiva en casos también simples.
De igual modo al colocar (.top):
if(rec.getPosition().x<=rect2.left){
rec.setPosition(rec.getPosition().x-1, rec.getPosition().y);
}
if(rec.getPosition().x>=rect2.left){
rec.setPosition(rec.getPosition().x+1, rec.getPosition().y);
}
if(rec.getPosition().y<=rect2.top){
rec.setPosition(rec.getPosition().x, rec.getPosition().y-1);
}
if(rec.getPosition().y>=rect2.top){
rec.setPosition(rec.getPosition().x, rec.getPosition().y+1);
}
Me da el mismo error que muestra al comenzar el vídeo, intento chocar con la figura pero al chocar se desplaza sola cuando se debería estar quieta sin poder avanzar más hasta que yo cambie de movimiento.
if(rec.getPosition().y<=rect2.top){
rec.setPosition(rec.getPosition().x, rec.getPosition().y-1);
}
if(rec.getPosition().y>=rect2.top){
rec.setPosition(rec.getPosition().x, rec.getPosition().y+1);
}
Referente a:
Una opción, es, en el código de movimiento, tras moverse, comprobar con intersecs(). En caso verdadero, deshaces el movimiento. Es una forma muy simple, pero efectiva en casos también simples.
¿Te refieres a crear una función?
Saludos!
Si yo no digo que eso lo corrigiera, pero era técnicamente incorrecto. Si funcionó era porque X e Y son iguales.
Lo que digo es que, tras hacer el .move() al captar las teclas, en caso de que esté en una zona incorrecta, que haga un .move inverso.
Hola, podrias utilizar otro rectangulo que se mueve, detectas si colisiona con rec2
//Movimiento otro rectangulo
if(sf::Keyboard::isKeyPressed(sf::Keyboard::Up)){
otroRec.move(0, -1);
}
// ... detectando teclas para otroRec
if( !otroRec.intersects(rect2) ) { // Si no hay interseccion, actualizas el move
rec.move(otroRec.getPosition().x, otroRec.getPosition().y);
}
// otroRec no lo tienes que pintar
Si te interesa averiguar sobre programación de juegos, vi por ahi un manual para juegos en C++ con Allegro, usa clases, excelentes conceptos me dio una idea sobre los videojuegos. Incluso hay un capitulo sobre colisiones.
El link : CURSO DE PROGRAMACIÓN DE VIDEJUEGOS CON C++ Y ALLEGRO (http://www.cin.ufpe.br/~yrms/curso_programacion.pdf)
Saludos.