Rechteck Koolision

[test54124]

Hallo,

habe folgendes Problem und zwar habe ich mir eine Klasse geschrieben bei der man durch die Navi. Taste ein Rechteck bewegen kann. Es gibt auch noch ein zweites Rechteck und sobald das erste auf das 2 zweite trifft soll eine Meldung ausgegeben werden. Dieses funktioniert jedoch nicht.

import java.applet.*;
import java.awt.*;
import java.awt.event.KeyEvent;
import java.awt.event.KeyListener;
 
  public class Viereck extends Applet implements KeyListener, Runnable {
   int x1=50,y1=55,x2=100,y2=100;
   boolean col=false;
   boolean richtung=true;
   Rectangle rectOne = new Rectangle(x1, y1,50,50);
   Rectangle rectTwo =new Rectangle(x2, y2,50,50);
 
   public void init()  {
	   this.addKeyListener(this);
		this.setFocusable(true);
    setBackground (Color.white);
    Thread myThread = new Thread(this);
	myThread.start();
  
   }
 
  
 
   public void paint (Graphics g) {
    g.setColor(Color.black);
    g.fillRect(x1,y1,50,50);
    g.setColor(Color.black);
    g.fillRect(x2,y2,50,50);
   }



@Override
public void keyPressed(KeyEvent e) {

	if (e.getKeyCode() == KeyEvent.VK_RIGHT) {
		x1++;
		
	
	}
	if (e.getKeyCode() == KeyEvent.VK_LEFT) {
		x1--;
		
	
	}
	if (e.getKeyCode() == KeyEvent.VK_DOWN) {
		y1++;
		
	
	}
	if (e.getKeyCode() == KeyEvent.VK_UP) {
		y1--;
		
	
	}
	repaint(); 
	
	}

public void überprüf(){
	col=rectOne.intersects(rectTwo);

	  if(col ==true){
	        System.out.println("überrr");
	        }
	        else{
	        	System.out.println("not");
	        	
	        }
	  
}
	
public void keyReleased(KeyEvent arg0) {
	// TODO Auto-generated method stub
	
}

public void keyTyped(KeyEvent arg0) {
	// TODO Auto-generated method stub
	
}

public void run() {
	while(true){
	überprüf();
    
	}
	
}
  }

 

[Marco13]

Java ist keine "symbolische" Programmiersprache. Das x1, y1 usw. wird anfangs einmal im Rectangle gespeichert, und wenn man x1 danach ändert, bekommt das rectangle.x davon nichts mit. Statt der Zeilen wie
x1++;
sowas schreiben wie
rectOne.x++;
würde vermutlich erstmal helfen...

 

[test54124]

danke das wars

 

[HimBromBeere]

Noch ein Zusatz: eine Funktion sollte man eher nicht überprüf() nennen, wegen der Umlaute. Entweder du entscheidest dich fuer ueberpruef, was aber mehr als haesslich aussieht, oder du nimmst einfach check().

 

[Marco13]

JApplet wäre besser als Applet
Fields sollten private sein
Endlosschleife ohne Thread.sleep
...
Man könnte die Liste lange fortsetzen, deswegen hatte ich gesagt, dass das erstmal hilft - weitere Fragen werden wohl noch kommen