[Java/BlueJ] Pfeiltastensteuerung -need help!

[Tim0n_]

Hallo liebe com,

Und zwar habe ich ein kleines problem, ich hab von meinem Lehrer in der schule eine extra Aufgabe bekommen wobei es um das steuern eines objektes (in dem fall einen Wurm/Raupe) per tasteneingabe..
Er selbst weiss nicht ob es möglich ist aber ich denke schon

Zum verständnis: Wir haben von ihm ein sogesehen vorgefertigten code bekommen mit Klassen für Rechtecke, Kreise, Quadrate und Dreiecke insklusive einer ausgabemöglichkeit in Form einer Leinwand.
Unsere aufgabe war es bisher einen Baum zu "zeichen/programmieren"
sprich stamm, krone, äpfel, Vogelhaus und einen Wurm.

mein Ergebnis sieht folgendermaßen aus:

 Spoiler

so wie gesagt habe ich nun Die Aufgabe bekommen diesen kleinen aüßerst hübschen wurm mit pfeiltasten (vorerst nur nach rechts und links) zu bewegen.

ich hab mir dazu mehrere codes im Internet angesehen (stichwort Keylistener) aber irgendwie nur bahnhof verstanden
Außerdem müsste ich den wurm irgendwie gruppieren, da er mittlerweile noch aus 5 verschiedenen Objekten besteht und das denke ich zuviel schreibarbeit ist das alles einzelnd zu verschieben lassen..

meine Idee war folgende: Ich wollte die 5 Objekte zu einem Instanzieren und diese instanz dann per keylistener zu verschieben und dabei brüchte ich ein wenig hilfe weil ich nicht so ganz weiss wie ich das angehen soll ..

Ich hoffe das mir jemand helfen kann Mfg Timon

Ps: hier ist mein jetziger code den ich mir zusammengeastelt habe aber ich hab den mittlerweile wieder rausgenommen weil der 0 funktioniert hat..

Code:

import java.awt.*;
import java.awt.event.*;
import javax.swing.*;
public class Steuerung extends JApplet implements KeyListener
{   
    private Raupe r1;
    public void init()
    {
        addKeyListener(this);
        r1 = new Raupe();
    }
    public void keyTyped(KeyEvent event) 
    {
    }
    public void keyPressed(KeyEvent event) 
    {
        if (event.getKeyCode() == KeyEvent.VK_LEFT) r1.horizontalBewegen();
        if (event.getKeyCode() == KeyEvent.VK_RIGHT) r1.horizontalBewegen();
       
        repaint();
    }
    public void keyReleased(KeyEvent event) 
    {
    }
    
}

wie schon zusehen ich würde ich gerne die einzelnden Objekte zu einer Klasse Raupe mit der variable r1 zusammenfügen aber da scheint mein konstructor oder wie das teil heisst nicht funktioniert zu haben (hab den code leider nicht mehr)

rechtschreibfehler behaltet bitte für euch, hatte diese nacht keinen schlaf ich darf das

[Mostey]

Die Raupe ist wirklich sehr hübsch. =D

Zur Gruppierung: Bau dir doch ne Klasse die das Skelett (also die Grundform) abbildet. Diese Klasse könnte entweder ne Liste oder Collection (weiß nicht wie das bei Java heißt, ich habe noch nie so wirklich in Java programmiert) haben in der alle Objekte enthalten sind, die gezeichnet werden müssen. Das setzt natürlich eine Schnittstelle (oder zumindest eine Klasse) voraus, die generisch gestaltet ist.

Ich denke nicht, das diese Voraussetzung erfüllt wird. Deshalb würde notfalls auch eine Klasse ausreichen, die alle Objekte enthält die zu zeichnen sind. Dort definierst du einfach mal ne Methode Move(Point destination) und lässt die Objekte verschieben. Somit trennst du den Part auch von der Interaktion, denn wenn du irgendwann mal die Raupe mit deinen Maustasten bewegen willst, würdest du hier entweder redundanten Code haben oder du müsstest alles in den neuen Eventhandler schieben.

C# Code der demonstriert was ich meine:

    public interface IMoveableShapeGroup
    {
        void Move(Point point);
    }

    public class Shape
    {
        public Point Position { get; set; }
        public string Description { get; set; }

        public Shape(string description, Point position)
        {
            Position = position;
            Description = description;
        }
    }

    public class Raupe : IMoveableShapeGroup
    {
        public List<Shape> ShapeComponents { get; set; }

        public Raupe(List<Shape> shapeComponents)
        {
            ShapeComponents = shapeComponents;
        }

        public void Move(Point point)
        {
            foreach (var shapeComponent in ShapeComponents)
                shapeComponent.Position = point;
        }
    }

PS: Dein Lehrer ist eine Banane, wenn er sich nicht sicher ist ob das umzusetzen ist.

[Tim0n_]

Erstmal vielen dank für die Umfangreiche Antwort

Also klar, das war auch mein gedankengang aber irgendwie scheitert es dabei weil ich nicht genau weiss wie ich die objekte gruppieren kann :s

Ich kann die Gegenstände per code verscheiben lassen das bekome ich hin, jedoch weiss ich nicht wie ich die überleitung zu den pfeiltasten bekomme bzw wie ich diese dummen objekte in kleine arrays stecken kann ;D

also das mit den listen habe ich kapiert, aber im internet finde ich nur Beispiele für String listen aber ich will ja sogesehen keine random daten zusammenfügen sondern meine schon existierenden objekte

(tut mir leid ich bin wie gesagt noch in den anfängen.. )

ja er ist ne banane aber ne coole banane, er schaffts problemlos schüler in seinen bann zu ziehen da fällt die fachkompetenz kaum auf und man bekommt immer super nioten

[Mostey]

Originally Posted by Tim0n_

also das mit den listen habe ich kapiert, aber im internet finde ich nur Beispiele für String listen aber ich will ja sogesehen keine random daten zusammenfügen sondern meine schon existierenden objekte

(tut mir leid ich bin wie gesagt noch in den anfängen.. )

Der von dir gelieferte Code ist nicht ausreichend um dir diesbezüglich zu helfen, da musst du konkreter werden.

Zu der Pfeiltastengeschichte:

[Tim0n_]

das hier ist mein bisheriger code überhaupt nichts besonderes ganz im gegenteil:

 Spoiler

public class Baum
{

    Rechteck Stamm = new Rechteck();
    Kreis Krone = new Kreis();
    Kreis Apfel1 = new Kreis();
    Kreis Apfel2 = new Kreis();
    Kreis Apfel3 = new Kreis();
    Quadrat VHaus = new Quadrat();
    Dreieck VDach = new Dreieck();
    Kreis VLoch = new Kreis(); 
    Rechteck VStange = new Rechteck();
    Kreis RKopf = new Kreis();
    Rechteck RKoerper = new Rechteck();
    Kreis RRAuge = new Kreis();
    Kreis RLAuge = new Kreis();
    Rechteck RMund = new Rechteck();
    Kreis Sonne = new Kreis();
    
    
    
    public void maleBaum()
    {
        Stamm.sichtbarMachen();
        Stamm.breiteAendern(80);
        Stamm.hoeheAendern(120);
        Stamm.farbeAendern("braun");
        Stamm.horizontalBewegen(40);
        Stamm.vertikalBewegen(140);
        
        
        Krone.sichtbarMachen();
        Krone.groesseAendern(190);
        Krone.farbeAendern("gruen");
        Krone.horizontalBewegen(25);
        Krone.vertikalBewegen(-50);

        
        Apfel1.sichtbarMachen();
        Apfel1.groesseAendern(25);
        Apfel1.farbeAendern("rot");
        Apfel1.horizontalBewegen(80);
        Apfel1.vertikalBewegen(100);
        
        Apfel2.sichtbarMachen();
        Apfel2.groesseAendern(20);
        Apfel2.farbeAendern("rot");
        Apfel2.horizontalBewegen(120);
        Apfel2.vertikalBewegen(60);
        
        Apfel3.sichtbarMachen();
        Apfel3.groesseAendern(35);
        Apfel3.farbeAendern("rot");
        Apfel3.horizontalBewegen(80);
        Apfel3.vertikalBewegen(20);
        
        VHaus.sichtbarMachen();
        VHaus.groesseAendern(40);
        VHaus.farbeAendern("blau");
        VHaus.horizontalBewegen(180);
        VHaus.vertikalBewegen(130);
        
        VDach.sichtbarMachen();
        VDach.farbeAendern("blau");
        VDach.horizontalBewegen(200);
        VDach.vertikalBewegen(100);
        
        VLoch.sichtbarMachen();
        VLoch.groesseAendern(15);
        VLoch.farbeAendern("schwarz");
        VLoch.horizontalBewegen(172);
        VLoch.vertikalBewegen(80);
        
        VStange.sichtbarMachen();
        VStange.breiteAendern(17);
        VStange.hoeheAendern(5);
        VStange.farbeAendern("schwarz");
        VStange.horizontalBewegen(140);
        VStange.vertikalBewegen(110);
        
        RKopf.sichtbarMachen();
        RKopf.groesseAendern(20);
        RKopf.farbeAendern("gruen");
        RKopf.horizontalBewegen(20);
        RKopf.vertikalBewegen(220);
        
        RKoerper.sichtbarMachen();
        RKoerper.hoeheAendern(10);
        RKoerper.breiteAendern(30);
        RKoerper.farbeAendern("gruen");
        RKoerper.horizontalBewegen(-42);
        RKoerper.vertikalBewegen(240);
        
        RRAuge.sichtbarMachen();
        RRAuge.groesseAendern(5);
        RRAuge.farbeAendern("schwarz");
        RRAuge.horizontalBewegen(30);
        RRAuge.vertikalBewegen(225);
        
        RLAuge.sichtbarMachen();
        RLAuge.groesseAendern(5);
        RLAuge.farbeAendern("schwarz");
        RLAuge.horizontalBewegen(22);
        RLAuge.vertikalBewegen(225);
        
        RMund.sichtbarMachen();
        RMund.hoeheAendern(3);
        RMund.breiteAendern(10);
        RMund.farbeAendern("schwarz");
        RMund.horizontalBewegen(-15);
        RMund.vertikalBewegen(242);
        
        Sonne.sichtbarMachen();
        Sonne.groesseAendern(100);
        Sonne.farbeAendern("gelb");
        Sonne.horizontalBewegen(230);
        Sonne.vertikalBewegen(-100);
    }
  
   
}

wobei meine gruppierung sich nur auf "RKopf", "RKoerper", "RRAuge", "RLAuge" und "RMund" beziehen (das sind die einzelteile der Raupe)

zum verständnis wie ich zu diesen ganzen variablen und so komme packe ich hier nochmal den code der kreis klasse rein, der rest ist ähnlich aufgebaut wennn überhaupt nur mit leichten Abweichungen:

 Spoiler

import java.awt.geom.Ellipse2D;

/**
 * Ein Kreis, der manipuliert werden kann und sich selbst auf einer Leinwand
 * zeichnet.
 * 
 * @author Michael Kölling und David J. Barnes
 * @version 2006.03.30
 */

public class Kreis {
	private int durchmesser;

	private int xPosition;

	private int yPosition;

	private String farbe;

	private boolean istSichtbar;

	/**
	 * Erzeuge einen neuen Kreis an einer Standardposition mit einer
	 * Standardfarbe.
	 */
	public Kreis() {
		durchmesser = 30;
		xPosition = 20;
		yPosition = 60;
		farbe = "blau";
		istSichtbar = false;
	}

	/**
	 * Mache diesen Kreis sichtbar. Wenn es bereits sichtbar ist, tue nichts.
	 */
	public void sichtbarMachen() {
		istSichtbar = true;
		zeichnen();
	}

	/**
	 * Mache diesen Kreis unsichtbar. Wenn es bereits unsichtbar ist, tue
	 * nichts.
	 */
	public void unsichtbarMachen() {
		loeschen();
		istSichtbar = false;
	}

	/**
	 * Bewege diesen Kreis einige Bildschirmpunkte nach rechts.
	 */
	public void nachRechtsBewegen() {
		horizontalBewegen(20);
	}

	/**
	 * Bewege diesen Kreis einige Bildschirmpunkte nach links.
	 */
	public void nachLinksBewegen() {
		horizontalBewegen(-20);
	}

	/**
	 * Bewege diesen Kreis einige Bildschirmpunkte nach oben.
	 */
	public void nachObenBewegen() {
		vertikalBewegen(-20);
	}

	/**
	 * Bewege diesen Kreis einige Bildschirmpunkte nach unten.
	 */
	public void nachUntenBewegen() {
		vertikalBewegen(20);
	}

	/**
	 * Bewege diesen Kreis horizontal um 'entfernung' Bildschirmpunkte.
	 */
	public void horizontalBewegen(int entfernung) {
		loeschen();
		xPosition += entfernung;
		zeichnen();
	}

	/**
	 * Bewege diesen Kreis vertikal um 'entfernung' Bildschirmpunkte.
	 */
	public void vertikalBewegen(int entfernung) {
		loeschen();
		yPosition += entfernung;
		zeichnen();
	}

	/**
	 * Bewege diesen Kreis langsam horizontal um 'entfernung' Bildschirmpunkte.
	 */
	public void langsamHorizontalBewegen(int entfernung) {
		int delta;

		if (entfernung < 0) {
			delta = -1;
			entfernung = -entfernung;
		} else {
			delta = 1;
		}

		for (int i = 0; i < entfernung; i++) {
			xPosition += delta;
			zeichnen();
		}
	}

	/**
	 * Bewege diesen Kreis langsam vertikal um 'entfernung' Bildschirmpunkte.
	 */
	public void langsamVertikalBewegen(int entfernung) {
		int delta;

		if (entfernung < 0) {
			delta = -1;
			entfernung = -entfernung;
		} else {
			delta = 1;
		}

		for (int i = 0; i < entfernung; i++) {
			yPosition += delta;
			zeichnen();
		}
	}

	/**
	 * Ändere den Durchmesser dieses Kreises in 'neuerDurchmesser' (Angabe in
	 * Bildschirmpunkten). 'neuerDurchmesser' muss größer gleich Null sein.
	 */
	public void groesseAendern(int neuerDurchmesser) {
		loeschen();
		durchmesser = neuerDurchmesser;
		zeichnen();
	}

	/**
	 * Ändere die Farbe dieses Kreises in 'neueFarbe'. Gültige Angaben sind
	 * "rot", "gelb", "blau", "gruen", "lila" und "schwarz".
	 */
	public void farbeAendern(String neueFarbe) {
		farbe = neueFarbe;
		zeichnen();
	}

	/**
	 * Zeichne diesen Kreis mit seinen aktuellen Werten auf den Bildschirm.
	 */
	private void zeichnen() {
		if (istSichtbar) {
			Leinwand leinwand = Leinwand.gibLeinwand();
			leinwand.zeichne(this, farbe, new Ellipse2D.Double(xPosition,
					yPosition, durchmesser, durchmesser));
			leinwand.warte(10);
		}
	}

	/**
	 * Lösche diesen Kreis vom Bildschirm.
	 */
	private void loeschen() {
		if (istSichtbar) {
			Leinwand leinwand = Leinwand.gibLeinwand();
			leinwand.entferne(this);
		}
	}

}

wie ihr schon sehen könnt ein ziemlich einfacher selbsterklärender code :P
und vielleicht gerade weil so simpel ist denke ich vielleicht einfach nur zu kompliziert ..

[Mostey]

Fasse doch einfach mal die Raupe zusammen wie ich dir zuvor schon gedeutet habe. Du hast dann ein Objekt der Klasse Raupe die alle Körperteile enthält und kannst damit dann interagieren. Du definierst dort die Move Methode und verschiebst jedes Körperteil einfach um X Stellen nach rechts (oder links, je nach Pfeiltaste) indem du die X Koordinate änderst. Versuch's einfach mal und liefere Ansätze, dann bessert das hier bestimmt nochmal jemand aus wenn es nicht ganz so funktioniert aber du solltest wirklich mal selbst versuchen hier weiterzukommen.

Ich weiß das du dafür nichts kannst aber das Design ist miserabel und dein Lehrer sollte wirklich mal bedenken, ob er sich nicht besser mal weiterbildet bevor er weiter lehrt. Vor allem lässt das eine Kommentar darauf schließen, dass er den Code schon seit mehr als 8 Jahren verwendet und das ist echt traurig.

[Tim0n_]

sooooooooooo ich hab mir mal was zusammengeschustert

 Spoiler

import java.awt.*;
import javax.swing.*;
import java.awt.event.*;


 
public class Steuerung extends java.applet.Applet implements KeyListener {
 
    int xPosition = 0;
    int yPosition = 480;
    int breiteRaupe = 40;
    int hoeheRaupe = 20;
 
    @Override
    public void init() {
	super.init();
	addKeyListener(this);
    }
 
    public void paint(java.awt.Graphics Raupe) {
	Raupe.setColor(new Color(0, 255, 0));
	Raupe.fillRect(xPosition, yPosition, breiteRaupe, hoeheRaupe);
    }
 
    @Override
    public void keyReleased(KeyEvent e) {
 
    }
 
    @Override
    public void keyPressed(KeyEvent e) {
	if (e.getKeyCode() == KeyEvent.VK_RIGHT) {
	    xPosition += 10;
	    repaint();
	}
       
	if (e.getKeyCode() == KeyEvent.VK_LEFT) {
	    xPosition += -10;
	    repaint();
	}
    }
    
    @Override
    public void keyTyped(KeyEvent e) {
    }
}

und siehe da ich habe einen kleinen grünen kasten den ich nun am unteren "bildschirmrand" vom applet entlangwandern lassen kann mithilfe der Pfeiltasten LINKS und RECHTS !

jedoch verstehe ich nicht so ganz warum das system in der form funktioniert und nicht mit meinen anderen gezeichneten objekten :/
hier der alte code, überarbeitet mit dem funktionierenden code von oben:

 Spoiler

import java.awt.*;
import javax.swing.*;
import java.awt.event.*;


    
 /**
 * Baumszenerie aus Stamm (Rechteck), Krone (Kreis), Äpfeln (Kreis), Vogelhaus(Dreieck+Quader)
 */

    
public class Baum extends java.applet.Applet implements KeyListener
{

    
    
    Rechteck Stamm = new Rechteck();
    Kreis Krone = new Kreis();
    Kreis Apfel1 = new Kreis();
    Kreis Apfel2 = new Kreis();
    Kreis Apfel3 = new Kreis();
    Quadrat VHaus = new Quadrat();
    Dreieck VDach = new Dreieck();
    Kreis VLoch = new Kreis(); 
    Rechteck VStange = new Rechteck();
    Kreis Sonne = new Kreis();
    Kreis RKopf = new Kreis();
    Rechteck RKoerper = new Rechteck();
    Kreis RRAuge = new Kreis();
    Kreis RLAuge = new Kreis();
    Rechteck RMund = new Rechteck();
    
     @Override
    public void init() {
	super.init();
	addKeyListener(this);
    }
    
    public void maleBaum()
    {
        Stamm.sichtbarMachen();
        Stamm.breiteAendern(80);
        Stamm.hoeheAendern(120);
        Stamm.farbeAendern("braun");
        Stamm.horizontalBewegen(40);
        Stamm.vertikalBewegen(140);
        
        
        Krone.sichtbarMachen();
        Krone.groesseAendern(190);
        Krone.farbeAendern("gruen");
        Krone.horizontalBewegen(25);
        Krone.vertikalBewegen(-50);

        
        Apfel1.sichtbarMachen();
        Apfel1.groesseAendern(25);
        Apfel1.farbeAendern("rot");
        Apfel1.horizontalBewegen(80);
        Apfel1.vertikalBewegen(100);
        
        Apfel2.sichtbarMachen();
        Apfel2.groesseAendern(20);
        Apfel2.farbeAendern("rot");
        Apfel2.horizontalBewegen(120);
        Apfel2.vertikalBewegen(60);
        
        Apfel3.sichtbarMachen();
        Apfel3.groesseAendern(35);
        Apfel3.farbeAendern("rot");
        Apfel3.horizontalBewegen(80);
        Apfel3.vertikalBewegen(20);
        
        VHaus.sichtbarMachen();
        VHaus.groesseAendern(40);
        VHaus.farbeAendern("blau");
        VHaus.horizontalBewegen(180);
        VHaus.vertikalBewegen(130);
        
        VDach.sichtbarMachen();
        VDach.farbeAendern("blau");
        VDach.horizontalBewegen(200);
        VDach.vertikalBewegen(100);
        
        VLoch.sichtbarMachen();
        VLoch.groesseAendern(15);
        VLoch.farbeAendern("schwarz");
        VLoch.horizontalBewegen(172);
        VLoch.vertikalBewegen(80);
        
        VStange.sichtbarMachen();
        VStange.breiteAendern(17);
        VStange.hoeheAendern(5);
        VStange.farbeAendern("schwarz");
        VStange.horizontalBewegen(140);
        VStange.vertikalBewegen(110);
                       
        Sonne.sichtbarMachen();
        Sonne.groesseAendern(100);
        Sonne.farbeAendern("gelb");
        Sonne.horizontalBewegen(230);
        Sonne.vertikalBewegen(-100);
        
        
    }
  
   public void maleRaupe()
    {
        RKopf.sichtbarMachen();
        RKopf.groesseAendern(20);
        RKopf.farbeAendern("gruen");
        RKopf.horizontalBewegen(20);
        RKopf.vertikalBewegen(220);
        
        RKoerper.sichtbarMachen();
        RKoerper.hoeheAendern(10);
        RKoerper.breiteAendern(30);
        RKoerper.farbeAendern("gruen");
        RKoerper.horizontalBewegen(-42);
        RKoerper.vertikalBewegen(240);
        
        RRAuge.sichtbarMachen();
        RRAuge.groesseAendern(5);
        RRAuge.farbeAendern("schwarz");
        RRAuge.horizontalBewegen(30);
        RRAuge.vertikalBewegen(225);
        
        RLAuge.sichtbarMachen();
        RLAuge.groesseAendern(5);
        RLAuge.farbeAendern("schwarz");
        RLAuge.horizontalBewegen(22);
        RLAuge.vertikalBewegen(225);
        
        RMund.sichtbarMachen();
        RMund.hoeheAendern(3);
        RMund.breiteAendern(10);
        RMund.farbeAendern("schwarz");
        RMund.horizontalBewegen(-15);
        RMund.vertikalBewegen(242);
    }   
    
    
    @Override
    public void keyReleased(KeyEvent e) {
 
    }
 
    @Override
    public void keyPressed(KeyEvent e) {
	if (e.getKeyCode() == KeyEvent.VK_RIGHT) {
	    RKopf.horizontalBewegen(5);
	    repaint();
			}
    }
 
    @Override
    public void keyTyped(KeyEvent e) {
    }
}

dazu zu sagen das er mir keine errors auspuckt, compilen tut er alles problemlos, es passiert nur nix wenn ich das so im appletviewer öffne :/

PS: Edit: Ich habe den wurm nun komplett nach dem anderen muster "programmiert" und er bewegt sich ohne probleme und das auch ausgesprochen simpel, wenn jedoch jemand noch eine idee hat wie ich das applet nun auf mein baum layout bekomme wäre ich ihm sehr verbunden wenn er mir noch einen Lösungsansatz geben würde, ansonsten Danke ich dir Mostey!

[Mostey]

Wieso nennst du eine Klasse Steuerung und implementierst aber in der Klasse Baum genauso die Steuerungsfunktion? Das macht keinen Sinn und ist zu trennen.

Außerdem verstehe ich nicht wo genau das Problem ist. Du bewegst in dem Code nur RKopf (was vermutlich den Kopf der Raupe repräsentiert), kein Wunder das der Baum sich dann nicht bewegt ;O