Was ist falsch(Kolision)

[StrikeTom]

Halllo Leute,
weiß jemand was falsch ist(ich find den fehler einfach nicht)?

	public boolean collide()
	{
			if ((y+10 >= startY) && !(x >= endX) && (x+10 >= startX) && !(y >= startY) && (y <= endX)) 
    	{
			player.setMoveable(false);
    		player.moveable = false;
    		player.setSpeed(0);
  			return true;
    	}
  		else
  		{
			player.setMoveable(true);
			player.moveable = true;
			player.setSpeed(5);
  			return false;
  		}
	}

Und noch die bewegung:

      player.y -= (keyPressed[UP] ? 15 : 0);
      player.x += (keyPressed[RIGHT] ? 7 : 0);
      player.y += (player.moveable() ? 5 : 0);
      player.x -= (keyPressed[LEFT] ? 7 : 0);

(ist natürlich in einem thread+while-schleife)
player.x/x sind die position des spielers
this.x/y die positionen der linie mit der der Spieler colidieren soll
player.moveable() gibt einen booleanwert zurück.
Danke im voraus

 

[knoppers]

Sorry, aber was soll man dir da jetzt genauer helfen. Ein bisschen Beschreibung wäre auch nicht schlecht. Nur Code nutzt auch nichts. Ein fehler von was genau? Compilierungsfehler (Sehe ich keinen)? Spielfehler? Laufzeitfehler? Was macht er beim Debuggen?

 

[agentone]

weiß jemand was falsch ist

Nö! Aber ich weiß, was schon auf den ersten Blick falsch, im Sinne von "hässlicher Code", ist:

player.setMoveable(false); //Ein Setter...
player.moveable = false; //... und eine öffentliche Variable, die direkt hintereinander auf denselben Wert gesetzt werden?!

und

player.moveable() //Warum nicht isMoveable() ?

Und nun zum eigentlichen, aber unbekannten Problem:
-Welchen Typs ist der Fehler (Siehe knoppers)
-Was wird falsch ausgegeben?
-Welche geometrischen Objekte (Rechteck, Kreis, Strecke) sollen auf Kollision überprüft werden?
-Was haben start/endX/Y für eine Bedeutung bei der ganzen Sache?

 

[StrikeTom]

Nö! Aber ich weiß, was schon auf den ersten Blick falsch, im Sinne von "hässlicher Code", ist:

Ja, ich weiß. Ich war nunmal schon am verzweifeln

player.moveable() //Warum nicht isMoveable() ?

Das ist nun wirklich egal

-Welchen Typs ist der Fehler

Das objekt Kolidiert nicht

-Welche geometrischen Objekte (Rechteck, Kreis, Strecke) sollen auf Kollision überprüft werden?

Kreis

-Was haben start/endX/Y für eine Bedeutung bei der ganzen Sache?

Die koordinaten der linie

Ich hoffe ich habe genug informationen gepostet.
Braucht ihr den ganzen code?

 

[knoppers]

if ((y+10 >= startY) && !(x >= endX) && (x+10 >= startX) && !(y >= startY) && (y <= endX))

Mit dieser Abfrage (unabhängig davon von x und was endX usw. ist, kannst du rein mathematisch gar nicht diese Objekte (Rechteck, Kreis, Strecke) gleichzeitig auf true abfragen, da brauchst du schon mathematisch etwas mehr.

Oder du posstest für diese Sache ein bisschen mehr Code mit deiner, oder einer Logic, weil hieraus kann kein Programmierer und auch kein Mathematiker so einfach irgendetwas sagen! (Ausnahme bestätigen die Regeln natürlich)

du musst ja irgendwo anders in deinem Code wenn ich das jetzt richtig verstehe noch etwas anderes mit startY usw. machen. Da ein Kreis, Rechteck, Viereck wenig, bis fast gar keine gemeinsamen geometrischen Vergleichsdaten liefern.

 

[StrikeTom]

Ich wollte eigendlich die variablen der linie benutzen:
x,y: position der linie
l: länge der linie
Das sollt mir anfangs auch genügen
Da hab ich die Kollosion so gemacht:

if (player.y + player.a >= this.y && player.x + player.a > this.x && player.x < this.x + this.l)

Als das nicht funktioniert hat hab ich erst das mit dem startX usw... gemacht
So sa dann die Klasse aus:

package SpielVersuch;

import java.awt.Color;
import java.awt.Graphics;

public class SimpleLine
{
	int startX;
	int endX;
	int startY;
	int endY;
	Figur player;
	int orientation = 10;
	public static final int HORIZONTAL = 0;
	public static final int VERTICAL = 1;
	int x, y, l;

	public SimpleLine(int orientation, int x, int y, int l, Figur f)
	{
		this.x = x;
		this.y = y;
		this.l = l;
		this.player = f;
		this.orientation = orientation;
	}
	public SimpleLine(int xs, int ys, int xe, int ye, Figur f, int a)
	{
		this.startX = xs;
		this.startY = ys;
		this.endX = xe;
		this.endY = ye;
		this.player = f;
	}
	public void paint(Graphics g)
	{
		//Will ich noch für vertikal machen
		if(orientation == HORIZONTAL)
		{
			 g.drawLine(x, y, x + l, y);
		}
//		else
//		{
//			 g.drawLine(startX, startY, endX, endY);
//		}
	}
	public boolean collide()
	{
		//hier ist das  mit x,y,l
		if (player.y + player.a >= this.y && player.x + player.a > this.x && player.x < this.x + this.l)
		{
			player.setMoveable(false);
    		player.setSpeed(0);
  			return true;
        }
		else
		{
			player.setMoveable(true);
  			player.setSpeed(5);
  			return false;
		}
//		if ((player.y+10 >= startY) && !(player.x >= endX) && (player.x+10 >= startX) && !(player.y >= startY) && (player.y <= endX)) 
//    	{
//			player.setMoveable(false);
//    		player.setSpeed(0);
//  			return true;
//    	}
//  		else
//  		{
//			player.setMoveable(true);
//			player.setSpeed(5);
//  			return false;
//  		}
	}
}

Ich hoffe es hilft weiter(Es ist etws unübersichtlich mit den vielen Komentaren)
Danke

 

[knoppers]

Sorry, aber ich glaube bei deim Programm wird dir hier so schnell keiner wirklich Hilfe geben können. Weil keiner so richtig weiss, was könnte im Moment x oder y, usw. sein.

Ich glaube du musst hier mal selber etwas mehr debuggen. Dann kommst du von alleine auf deinen Logic Fehler. Oder immer wieder dein x y und die jeweiligen Vergleichsposition für x und y ausgeben lassen. Dann wirst du sehen wo er nach deiner Meinung nach kollisieren müsste aber es nicht tut.