Swing Image anzeigen und probleme mit klassen

[XWing]

Hi, ich verstehe nicht so ganz wie ich es jetzt Schaffe ein image in Java swing anzeigen zu lassen. Und dazu habe ich probleme meine klassen aufzurufen. Ich habe fast alles in eine seperate klasse geschrieben. Kann jetzt aber nicht die klasse in der Main methode aufrufen. Und ich verstehe nicht ganz warum.

Main methode:

public class Main {

    public static void main(String[] args) {

        Startgame game = new Startgame();
        Startgame();
    }
}

Seperate Klasse:

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

public class Loadingscreen {
    public JFrame Frame;
    public JFrame Startgame(){
        JFrame Frame = new JFrame("Flagenspiel");
        Frame.setSize(700,700);
        Frame.setVisible(true);
        Frame.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
        return Startgame();
    }

    public JButton Startscreen() {
        JLabel Image = new JLabel();
        Frame.add(Image);
        ImageIcon icon = new ImageIcon("src/europa.png.");
        JPanel menu = new JPanel();
        Frame.add(menu);
        JButton button = new JButton("Start Game!");
        button.setBounds(200,300,150,60);
        menu.add(button);
        return Startscreen();
    }
}

}

Und bei denn beiden geschweiften Klammern in der Startgame und Startscreen methode wird es rot angezeigt.

 

[KonradN]

Was soll denn Startgame(); machen?

Wenn ich sehe: Startgame game = new Startgame(); dann ist die Erwartungshaltung, dass Du eine Klasse Startgame hast, aber das hast Du ja nicht. Du hast eine Klasse Loadingscreen. Von dieser Klasse kannst Du eine neue Instanz erzeugen und diese in einer Variablen speichern. Das wäre dann etwas wie:

'''Loadingscreen loadingscreen = new Loadingscreen();```

Dann hat die Klasse Loadingscreen eine Methode Startgame. Diese kann auf einer Instanz aufgerufen werden. Du hast ja in der Variablen loadingscreen eine Instanz der Klasse, also könntest Du die Methode aufrufen:
loadingscreen.Startgame();

Da ein JFrame zurück gegeben wird, könntest Du die Instanz auch in einer Variablen speichern. Dann würde der Befehl anders aussehen:
JFrame frame = loadingscreen.Startgame();

Dann fallen aber viele Dinge auf in der Klasse Loadingscreen, daher verändere ich die Klasse einmal und füge Kommentare ein:

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

public class LoadingScreen { // Upper CamelCase: "loading screen" -> LoadingScreen
    public JFrame frame; // Variablen werden im lower camelCase geschrieben, also kleines f

    public JFrame startGame(){ // Methoden werden im lower camelCase geschrieben: startGame
        frame = new JFrame("Flagenspiel"); // Kein JFrame - wir wollen die Instanzmethode frame nutzen und keine neue lokale Variable erzeugen!
        Frame.setSize(700,700);
        Frame.setVisible(true);
        Frame.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
        return frame; // Wir wollen doch das erzeugte JFrame zurück geben!
    }

    public JButton startScreen() { // lower camelCase bei Methoden
        JLabel image = new JLabel(); // lower camelCase bei lokale Variable: image
        Frame.add(image);
        ImageIcon icon = new ImageIcon("src/europa.png."); // Der . hinter dem png muss bestimmt weg, oder?
        JPanel menu = new JPanel();
        frame.add(menu);
        JButton button = new JButton("Start Game!");
        button.setBounds(200,300,150,60);
        menu.add(button);
        return button; // Du willst den erzeugten button zurück geben?
    }
}

// } Diese geschweifte Klammer war zu viel....

 

[XWing]

oh okay danke. Funktioniert jetzt größtenteils. Jetzt hab ich aber das problem das die ganze zeit das window gestartet wird.

public class Main {

    public static void main(String[] args) {

        Loadingscreen game = new Loadingscreen();
            game.Startgame();
            game.Startscreen();
    }
}

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

public class Loadingscreen {
    public JFrame Frame;
    public JFrame Startgame(){
        JFrame Frame = new JFrame("Flagenspiel");
        Frame.setSize(700,700);
        Frame.setVisible(true);
        Frame.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
        return Startgame();
    }

    public JButton Startscreen() {
        /*
        JLabel Image = new JLabel();
        Frame.add(Image);
        ImageIcon icon = new ImageIcon("src/europa.png");
        */
        JPanel menu = new JPanel();
        Frame.add(menu);
        JButton button = new JButton("Start Game!");
        button.setBounds(200,300,150,60);
        menu.add(button);
        return Startscreen();
    }
}

 

[KonradN]

Schau Dir an, wie der Return Befehl bei dem verbesserten Code von mir aussieht. Du rufst am Ende die Merhode erneut auf was zu dem beschriebenen führt.