[Java/BlueJ] Pfeiltastensteuerung -need help!

11/16/2014 11:27 Tim0n_#1

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 :p 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 :D
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:

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 ;)

11/16/2014 12:43 Mostey#2

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:

Code:

    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.

11/16/2014 13:28 Tim0n_#3

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 ;)

11/16/2014 13:38 Mostey#4

Quote:

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.. )

[Only registered and activated users can see links. Click Here To Register...]

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

Zu der Pfeiltastengeschichte: [Only registered and activated users can see links. Click Here To Register...]

11/16/2014 13:54 Tim0n_#5

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

Spoiler

Code:

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

Code:

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 ..

11/16/2014 19:49 Mostey#6

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.

11/16/2014 22:35 Tim0n_#7

sooooooooooo ich hab mir mal was zusammengeschustert :)

Spoiler

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 ! :D

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

Code:

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!

11/17/2014 17:53 Mostey#8

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