irritierende Flag-Werte

[PunktPunktPunkt]

Hallo,

Vorab: sorry für den ebenso irritierenden Titel; mir ist einfach nichts treffenderes eingefallen.
Also erstmal der Code:

public void actionPerformed(ActionEvent arg0) {
		grafikenMalen.move();
		if ((grafikenMalen.x > 0) && (grafikenMalen.x < 10) && (grafikenMalen.inGebiet1 = true)) {
			grafikenMalen.inGebiet1 = false;
			grafikenMalen.inGebiet2 = true;
			grafikenMalen.hintergrund = "IMG2.JPG";
			grafikenMalen.ii = new ImageIcon(this.getClass().getResource(grafikenMalen.hintergrund));
			grafikenMalen.image2 = grafikenMalen.ii.getImage();
			grafikenMalen.x = 200;
		}
		else if ((grafikenMalen.x > 0) && (grafikenMalen.x < 10) && (grafikenMalen.inGebiet2 = true)) {
			grafikenMalen.hintergrund = "IMG1.gif";
			grafikenMalen.x = 150;
			grafikenMalen.ii = new ImageIcon(this.getClass().getResource(grafikenMalen.hintergrund));
			grafikenMalen.image2 = grafikenMalen.ii.getImage();
			grafikenMalen.inGebiet1 = true;
			grafikenMalen.inGebiet2 = false;
		}	
		repaint();
	}

Und die move()-Methode sieht so aus:

public void move() {
			        if (zielx > x)
					x++;
				else if (zielx < x)
					x--;
				if (ziely > y)
					y++;
				else if (ziely < y)
					y--;
}

Also meine Intention ist glaube ich recht klar zu erkennen: wenn an den linken Rand des Bildschirms gelaufen wird, dann soll das Hintergrundbild verändert werden; das klappt beim ersten Bild auch. Nur beim zweiten springt der immer wieder auf Position x=200 wenn man am Rand geht ohne dass das Hintergrundbild sich ändert, was ja darauf schließen lässt, dass das inGebiet1-Flag immer noch gesetzt ist (er scheint ja die erste, nicht wie ich es will die zweite, der if-Clauses zu durchlaufen). Ich versteh nicht ganz warum, weil in der ersten if-Clause wird doch, sobald in sie eingetreten wird, inGebiet1 auf false gesetzt, er müsste doch eigentlich dann die zweite if-Clause machen wenn man an den Rand geht (weil ja eben die Bedingung der ersten, dass inGebiet1 true ist, nicht erfüllt sein dürfte, die Bedingung der zweiten, inGebiet2 = true, aber doch)?
Wahrscheinlich ein sehr offensichtlicher Fehler, aber ich raffs grad nicht. Weiß vielleicht jemand wo mein Denkfehler ist?

Danke!

Grüße,
PPP

 

[njans]

grafikenMalen.inGebiet1 = true

Das machst du 2 mal. Das ist keine Abfrage, sondern eine Zuweisung! Keine Ahnung ob du das als Ursache nehmen kannst, du arbeitest da mit Klassen die meine Kristallkugel nicht beleuchten kann.

 

[PunktPunktPunkt]

Oh, mit "==" gehts. Anfängerfehler, aber ich bin ein ziemlicher Anfänger also passt das wohl ganz gut. 😀 Danke dir!

 

[PunktPunktPunkt]

Ein weiteres Problem: vielleicht kann ich das eben hier stellen, dann brauch dafür nicht extra ein neuer Thread aufgemacht werden.
Weiß vielleicht jemand, ob es einen einfachen Trick gibt, JLabels in den Vordergrund zu setzen? Ich will, dass sie auf dem Bild erscheinen, leider überdeckt das Bild diese aber.

Wahrscheinlich ein recht verbreitetes Problem, aber seltsamerweise konnt ich über Google keine Antwort darauf finden. Wäre super wenn jemand nen Rat dazu hätte!

 

[Schandro]

googel mal nach "java swing z-index"

 

[njans]

Das klingt nach einem Job für JLayeredPane.

 

[PunktPunktPunkt]

Hm, neues Problem.
Also ich hab ein bisschen mit dem JLayeredPane rumgespielt, da hat der aber nie das Hintergrundbild angezeigt. Ich bin mir nun nicht mehr sicher, ob ich davor irgendwas verändert hab, was dafür gesorgt hat dass das Bild allgemein nicht mehr angezeigt wird. Natürlich meine eigene Blödheit, dass ich keine Sicherungskopie von dem funktionierenden gemacht hab, aber sieht vielleicht jemand, warum das Hintergrundbild nicht mehr angezeigt wird?

import javax.swing.JFrame;
import javax.swing.JLabel;


public class Test extends JFrame {
	
	public Test() {
		add(new Board2());
		
		setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
        setSize(800, 600);
        setLocationRelativeTo(null);
        setTitle("Star");
        setResizable(false);
        setVisible(true);
	}
	
	public static void main(String[] args) {
		new Test();
	}

}

import java.awt.Color;
import java.awt.Graphics;
import java.awt.Graphics2D;
import java.awt.Toolkit;
import java.awt.event.ActionEvent;
import java.awt.event.ActionListener;
import java.awt.event.KeyAdapter;
import java.awt.event.KeyEvent;
import java.awt.event.MouseAdapter;
import java.awt.event.MouseEvent;

import javax.swing.ImageIcon;
import javax.swing.JLabel;
import javax.swing.JPanel;
import javax.swing.Timer;


public class Board2 extends JPanel implements ActionListener {

	private Timer timer;
	private GrafikenMalen grafikenMalen;
	
	public Board2() {
		addMouseListener(new BewegungenAdapter());		
		setFocusable(true);
		setBackground(Color.BLACK);
		setDoubleBuffered(true);
		
		grafikenMalen = new GrafikenMalen();
		timer = new Timer (5, this);
		timer.start(); // alle 5ms ActionPerformed ausführen
	}
	
	public void paint(Graphics g) {
		super.paint(g);
		
		Graphics2D g2d = (Graphics2D) g;
		g2d.drawImage(grafikenMalen.getHintergrund(), 0, 0, this);
		g2d.drawImage(grafikenMalen.getImage(), grafikenMalen.getX(), grafikenMalen.getY(), this);
		
		Toolkit.getDefaultToolkit().sync();
		g.dispose();
	}
	
	class BewegungenAdapter extends MouseAdapter {
		public void mousePressed (MouseEvent e) {
			grafikenMalen.zielx = e.getX();
			grafikenMalen.ziely = e.getY();
		}
	}
	
	@Override
	public void actionPerformed(ActionEvent arg0) {
		grafikenMalen.move();
		if ((grafikenMalen.x > 0) && (grafikenMalen.x < 10) && (grafikenMalen.inGebiet1 == true)) {
			grafikenMalen.inGebiet1 = false;
			grafikenMalen.inGebiet2 = true;
			grafikenMalen.hintergrund = "IMG2.JPG";
			grafikenMalen.ii = new ImageIcon(this.getClass().getResource(grafikenMalen.hintergrund));
			grafikenMalen.image2 = grafikenMalen.ii.getImage();
			grafikenMalen.x = 200;
		}
		else if ((grafikenMalen.x > 0) && (grafikenMalen.x < 10) && (grafikenMalen.inGebiet2 == true)) {
			grafikenMalen.hintergrund = "IMG1.gif";
			grafikenMalen.x = 150;
			grafikenMalen.ii = new ImageIcon(this.getClass().getResource(grafikenMalen.hintergrund));
			grafikenMalen.image2 = grafikenMalen.ii.getImage();
			grafikenMalen.inGebiet1 = true;
			grafikenMalen.inGebiet2 = false;
		}
		
		if ((grafikenMalen.x % 2) == 0) {
			grafikenMalen.star = "missile.png";
			grafikenMalen.ii = new ImageIcon(this.getClass().getResource(grafikenMalen.star));
			grafikenMalen.image2 = grafikenMalen.ii.getImage();
		}
		else if ((grafikenMalen.x % 2) != 0) {
			grafikenMalen.star = "star.png";
			grafikenMalen.ii = new ImageIcon(this.getClass().getResource(grafikenMalen.star));
			grafikenMalen.image2 = grafikenMalen.ii.getImage();
		}
		
		repaint();
	}
}

import java.awt.Image;
import java.awt.event.KeyEvent;
import java.awt.event.MouseEvent;

import javax.swing.ImageIcon;


public class GrafikenMalen {
	public String star = "star.png";
	public String hintergrund = "IMG1.gif";
	
	// x und y (wie auch getX() und getY()) betreffen nur den Star, nicht den Hintergrund!!
	public int zielx;
	public int ziely;
	public int x;
	public int y;
	public Image image;
	public Image image2;
	public ImageIcon ii;
	public boolean inGebiet1 = true;
	public boolean inGebiet2 = false;
	
	public GrafikenMalen() {
		ii = new ImageIcon(this.getClass().getResource(star));
		image = ii.getImage();
		ii = new ImageIcon(this.getClass().getResource(hintergrund));
		image2 = ii.getImage();
		
		//Startwerte?:
		x = 40;
		y = 60;
		zielx = 40;
		ziely = 60;
	}
	
	public void move() {
		// if (inGebiet1 = true) {
			// if (! ( (y > 0) && (y < 10) )) {
				if (zielx > x)
					x++;
				else if (zielx < x)
					x--;
				if (ziely > y)
					y++;
				else if (ziely < y)
					y--;
				
			// } else y = 11;
		// }
		/* else if (inGebiet2 = true) {
			if (! ( (y > 0) && (y < 80) )) {
				if (zielx > x)
					x++;
				else if (zielx < x)
					x--;
				if (ziely > y)
					y++;
				else if (ziely < y)
					y--;
			} else y = 61;
		} */
	}
	
	public int getX() {
		return x;
	}
	
	public int getY() {
		return y;
	}
	
	public Image getImage() {
		return image;
	}
	
	public Image getHintergrund() {
		return image2;
	}
}

 

[PunktPunktPunkt]

Ich kanns mir echt nicht erklären, ich könnt schwören alles ist ganz genauso wie es war, als das Hintergrundbild noch problemlos angezeigt wurde. Spätestens wenn man sich zum linken Rand bewegt müsste doch ein Bild geladen werden... sieht nicht vielleicht jemand den Fehler?

 

[PunktPunktPunkt]

Oh, und dann doch sofort den Fehler gefunden.

if ((grafikenMalen.x % 2) == 0) {
            grafikenMalen.star = "missile.png";
            grafikenMalen.ii = new ImageIcon(this.getClass().getResource(grafikenMalen.star));
            grafikenMalen.image2 = grafikenMalen.ii.getImage();
        }
        else if ((grafikenMalen.x % 2) != 0) {
            grafikenMalen.star = "star.png";
            grafikenMalen.ii = new ImageIcon(this.getClass().getResource(grafikenMalen.star));
            grafikenMalen.image2 = grafikenMalen.ii.getImage();
        }

auskommentiert und nun gehts. Was ist daran einzuwenden, er soll doch nur das star-Bild ändern, nicht aber den Hintergrund?

 

[SlaterB]

aber image2 ist doch der Hintergrund? ändere zu image bzw. in der actionPerformed lieber gar nix,
nur setStar(String) aufrufen und in der Klasse GrafikenMalen richtig laden

 

[PunktPunktPunkt]

Ach, ich Idiot. Danke!
Nun hat der Thread seinen Soll sicherlich mehr als erfüllt und kann endlich als erledigt gesetzt werden. 😀