Verschiedene Bilder per Knopfdruck anzeigen lassen

[Lanifee]

Hallo =)
Ich bin neu hier und habe noch nicht so viel Erfahrung im Programmieren. Allerdings habe ich die Aufgabe ein kleines Spiel zu programmieren.
Dafür lasse ich mir ein Bild im JFrame anzeigen. Ich hab vier Knöpfe und möchte, dass sich das Bild bei jedem Knopfdruck ändert. Ich bin jetzt soweit, dass ich das Programm starten kann und wenn ich einen Knopf drücke ändert sich auch das Bild, aber danach passiert nichts mehr. Wenn sich das Bild einmal geändert hat, bleibt es dabei und die Knöpfe reagieren nicht mehr.

Ich versuche jetzt einmal meinen Code einzufügen. Es wäre wirklich super, wenn mir jemand helfen könnte.

Ich bedanke mich im Voraus für jede Antwort!

Liebe Grüße!

package hundi;

import java.awt.event.ActionEvent;
import java.awt.event.ActionListener;

import javax.swing.ImageIcon;
import javax.swing.JButton;
import javax.swing.JFrame;
import javax.swing.JLabel;
import javax.swing.JOptionPane;

public class Tama {

public static void main(String[] args) {

JFrame tam = new JFrame("Mein Tamagotchi");
tam.setSize(1000, 800);
tam.setLocation(315, 6);

JButton eins = new JButton("füttern");
eins.setBounds(60, 500, 100, 20);
tam.add(eins);

JButton zwei = new JButton("spielen");
zwei.setBounds(60, 550, 100, 20);
tam.add(zwei);

JButton drei = new JButton("schlafen");
drei.setBounds(60, 600, 100, 20);
tam.add(drei);

JButton vier = new JButton("waschen");
vier.setBounds(60, 650, 100, 20);
tam.add(vier);

// JButton neu = new JButton("zurück");
// neu.setBounds(60, 400, 100, 20);
// tam.add(neu);

JLabel hintergrundbild = new JLabel(
new ImageIcon("C:/Users/Lana/Desktop/Programmieren/Tamagotchi/Sitzend.jpg"));
tam.getContentPane().add(hintergrundbild);
hintergrundbild.setVisible(true);

tam.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
tam.setVisible(true);

ActionListener schlafen = new ActionListener() {
public void actionPerformed(ActionEvent e) {
// JOptionPane.showMessageDialog(null, "Danke!");

hintergrundbild.setVisible(false);
JLabel bild = new JLabel(
new ImageIcon("C:/Users/Lana/Desktop/Programmieren/Tamagotchi/Liegend.jpg"));
tam.getContentPane().add(bild);
bild.setVisible(true);
}
};

ActionListener spielen = new ActionListener() {
public void actionPerformed(ActionEvent e) {

hintergrundbild.setVisible(false);
JLabel bildspiel = new JLabel(
new ImageIcon("C:/Users/Lana/Desktop/Programmieren/Tamagotchi/Spielend.jpg"));
tam.getContentPane().add(bildspiel);
bildspiel.setVisible(true);

}
};

// ActionListener back = new ActionListener() {
// public void actionPerformed(ActionEvent e) {

// hintergrundbild.setVisible(true);


drei.addActionListener(schlafen);
zwei.addActionListener(spielen);
// neu.addActionListener(back);
}
}

 

[Xyz1]

Hallo,

innerhalb von ActionListener bekommst du immer das gleiche Bild geschrieben, ändere das mal.

lg

 

[Lanifee]

Hallo,

erstmal Danke für die Antwort.
Leider ist mir nicht so ganz klar, was ich ändern muss.
Ich habe versuche das erste Bild mit dem ActionListener auf Knopfdruck unsichtbar zu machen. Damit ein anderes Bild angezeigt werden kann.
Ich hab auch das Gefühl, dass sich meine Bilder immer gegenseitig überschreiben und nur das letze definierte Bild angezeigt wird. Aber ich weiß nicht genau, wie ich das ändern kann.

LG

 

[Meniskusschaden]

innerhalb von ActionListener bekommst du immer das gleiche Bild geschrieben, ändere das mal.

Das kann ich nicht nachvollziehen. Zumindest für die Buttons zwei und drei sind es doch andere Bilder als das Startbild.

Ich denke das Hauptproblem ist, dass du bei jedem Button-Klick ein neues Bild erzeugst und hinzufügst. Du machst aber nur das ursprüngliche Startbild unsichtbar, aber nicht die Bilder, die zu den Buttons zwei und drei gehören.

 

[Xyz1]

Hab es noch nicht Begucken können, bitte mal in Java-Tags oder als PN.

 

[Meniskusschaden]

Hab es noch nicht Begucken können

Oh, wie konntest du denn dann antworten.

bitte mal in Java-Tags oder als PN.

Es kann ja jeder machen, was er will, aber ich finde es nicht so toll, Themen aus dem öffentlichen Forum auf die persönliche Ebene umzuleiten. Wenn man sich mit dem Thema beschäftigt hat, möchte man schon gerne wissen, wie es weiter geht.

 

[Xyz1]

Und ich finde es nicht so toll, wenn meine Signatur nicht beachtet wird...

 

[Lanifee]

@Meniskusschaden
ich hab schon versucht mit dem Button-Klick die anderen Bilder auch unsichtbar zu machen, aber dann funktioniert das Programm nicht mehr.
Ich hab auch über legt, die Bilder alle vor dem Button-Klick zu erzeugen. Aber dann funktioniert nur noch das, was ich als letztes erzeugt hab. Ich weiß nicht, wie ich das anders machen kann.


Ich versuche nochmal den Code in Tags zu packen, aber leider weiß ich nicht genau, wie das geht.

package hundi;

import java.awt.event.ActionEvent;
import java.awt.event.ActionListener;

import javax.swing.ImageIcon;
import javax.swing.JButton;
import javax.swing.JFrame;
import javax.swing.JLabel;
import javax.swing.JOptionPane;

public class Tama {

    public static void main(String[] args) {

        JFrame tam = new JFrame("Mein Tamagotchi");
        tam.setSize(1000, 800);
        tam.setLocation(315, 6);

        JButton eins = new JButton("füttern");
        eins.setBounds(60, 500, 100, 20);
        tam.add(eins);

        JButton zwei = new JButton("spielen");
        zwei.setBounds(60, 550, 100, 20);
        tam.add(zwei);

        JButton drei = new JButton("schlafen");
        drei.setBounds(60, 600, 100, 20);
        tam.add(drei);

        JButton vier = new JButton("waschen");
        vier.setBounds(60, 650, 100, 20);
        tam.add(vier);

        // JButton neu = new JButton("zurück");
        // neu.setBounds(60, 400, 100, 20);
        // tam.add(neu);

        JLabel hintergrundbild = new JLabel(
                new ImageIcon("C:/Users/Lana/Desktop/Programmieren/Tamagotchi/Sitzend.jpg"));
        tam.getContentPane().add(hintergrundbild);
        hintergrundbild.setVisible(true);

        tam.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
        tam.setVisible(true);

        ActionListener schlafen = new ActionListener() {
            public void actionPerformed(ActionEvent e) {
                // JOptionPane.showMessageDialog(null, "Danke!");

                hintergrundbild.setVisible(false);
                JLabel bild = new JLabel(
                        new ImageIcon("C:/Users/Lana/Desktop/Programmieren/Tamagotchi/Liegend.jpg"));
                tam.getContentPane().add(bild);
                bild.setVisible(true);
            }
        };

        ActionListener spielen = new ActionListener() {
            public void actionPerformed(ActionEvent e) {

                hintergrundbild.setVisible(false);
                JLabel bildspiel = new JLabel(
                        new ImageIcon("C:/Users/Lana/Desktop/Programmieren/Tamagotchi/Spielend.jpg"));
                tam.getContentPane().add(bildspiel);
                bildspiel.setVisible(true);

            }
        };

        // ActionListener back = new ActionListener() {
            // public void actionPerformed(ActionEvent e) {

                // hintergrundbild.setVisible(true);

      
        drei.addActionListener(schlafen);
        zwei.addActionListener(spielen);
        // neu.addActionListener(back);
    }
}

 

[Meniskusschaden]

Ich glaube am einfachsten wäre es, ein einziges JLabel zu verwenden und es immer sichtbar zu lassen. Anstatt neue JLabel zu erzeugen und dem Frame hinzuzufügen, kannst du dem Label in den ActionListeners einfach mittels setIcon() das jeweils gewünschte Bild zuweisen.

 

[Lanifee]

Die ist Idee ist schon mal Super!
Da bin ich noch gar nicht drauf gekommen.
Leider funktioniert ist das nicht ganz.

 hintergrundbild.setIcon("C:/Users/Lana /Desktop/Programmieren/Tamagotchi/Liegend.jpg");

Ich hab versucht das in den ActionListener reinzuschreiben. Aber dann kommt die Fehlermeldung: The method setIcon (Icon)in the type JLabel is not applicable for the arguments (string).

Muss ich das Bild erst mit new IconImage hinzufügen?

 

[Meniskusschaden]

setIcon erwartet keinen String, sondern ein ImageIcon. Das musst du also noch mit new ImageIcon("...")erzeugen.

 

[Lanifee]

Oh Danke! Das hatte ich versucht, aber es hat nicht funktioniert (mal wieder...)
Aber ich glaube ich habe jetzt eine Idee wieso nicht. Das versuche ich nochmal.

 

[Lanifee]

Okay, ich hab doch noch eine Frage.
Wenn ich das mit mit new ImageIcon erzeuge, woher bekommt das dann einen Namen, bzw wie greife ich darauf zu?
Ich weiß nämlich nicht, was ich dann bei setIcon() in die Klammern schreiben muss.
Tut mir leid, ich bin wirklich totale Anfängerin.
Aber ich finde deine Hilfe super!

 

[Lanifee]

Yay, ich hab's geschafft! Ich kann dem neuen ImageIcon ja einfach einen Namen geben. =)
Danke für die Hilfe.
Stern für dich, Meniskusschaden! =)

 

[Lanifee]

=)

 

[Viktim]

Ich geb dir mal noch ein paar Tipps was deinen Code angeht:
1.) Du solltest die Buttons nicht eins, zwei, ... nennen sondern wirklich schlafen, waschen etc. (Das hilft dir sehr, wenn du später nochmal auf deinen Code gucken musst.)
2.)Es wäre mir ein Rätsel, wenn dein setBounds() funktioniert, weil man dafür eigentlich das Layout auf null setzten muss, sprich:

tam.getContentPane().setLayout(null);

3.) Klassen und Methode werden dir später auch helfen
Du könntest z.B. deine ActionListener in eigene Klassen auslagern...

 

[Lanifee]

Oh, das mit dem Layout wusste ich nicht. Ich dachte eigentlich, dass das funktioniert.^^

Wäre es sinnvoll jeden ActionListener in eine eigene Klasse zu stecken?
Wie würde man das dann im Code schreiben? Kann man einfach auf die andere Klassen zugreifen, wenn sie im selben Paket sind?

 

[Joose]

Wäre es sinnvoll jeden ActionListener in eine eigene Klasse zu stecken?

Kommt ganz darauf an wie umfangreich der Listener ist bzw. was er machen soll.

Wie würde man das dann im Code schreiben? Kann man einfach auf die andere Klassen zugreifen, wenn sie im selben Paket sind?

Wie du einen normalen ActionListener hinzufügst. Ein Objekt des Listeners erstellen und per add Methode hinzufügen.

myLabel.addActionListener(new MyActionListener1());
myLabel.addActionListener(new MyActionListener2());