Image wird nicht gezeichent?!

[FraxxeY]

Hallo, bin dabei ein kleines Glücksrad-Spiel zu entwerfen und hab dazu eine PNG mit einem Rad erstellt.

Das habe ich in meinem Java-Programm zeichnen lassen wollen, es wird aber nicht gezeichnet!

Der Code:

import javax.swing.*;
import java.awt.*;
import java.awt.geom.AffineTransform;
import java.awt.image.*;
import java.io.*;
import javax.imageio.*;

public class FENSTER extends JFrame
{
    private JButton roll;
    private JFrame frame;
    private JTextField chancebox;
    private JLabel scheibe, ping;
  
    public static void main(String[] args){
        new FENSTER();     
    }
 
    public FENSTER()
    {
        super("Gambling");
        setVisible(true);
        setResizable(false);
        //setSize(1280,720);
        setBounds(100, 100, 1280, 720);
        setDefaultCloseOperation(EXIT_ON_CLOSE);
        //setLayout(null);
        //getContentPane().setBackground(Color.darkGray);
     
        Icon x = new ImageIcon(getClass().getResource("ccc.png"));
    
        roll = new JButton("ROLL!", x);
        //roll.setVisible(true);
        roll.setBounds(390,500,500,80);
        //roll.setBackground(Color.orange);
        //roll.setOpaque(true);
        add(roll);
     
        scheibe = new WHEEL();
        scheibe.setBounds(10,10,5000,5000);
        add(scheibe); 
     
        //chancebox = new JTextField();
        //chancebox.setBounds(390, 450, 100, 25);
        //add(chancebox);
     
        repaint();     
    }
}
class WHEEL extends JLabel{

    public void paintComponent(Graphics g){
        AffineTransform at = AffineTransform.getTranslateInstance(100, 100);
        at.rotate(Math.toRadians(0));
        BufferedImage rad = LoadImage("rad.png");
        System.out.println("DDD");
 
        Graphics2D g2d = (Graphics2D) g;
        g2d.drawImage(rad, at, null);
    }

    BufferedImage LoadImage(String FileName){
        BufferedImage img = null;
        try{
            img = ImageIO.read(new File(FileName));
        }catch(IOException e){
        }
        return img;
    }
}

Wieso funktioniert das nicht? Wo hängts da? Ich verzweifle an einem bestimmt recht dummen Fehler

MFG

 

[Thallius]

Man erbt nicht von JFrame es sei denn man will es erweitern. Willst du aber nicht.

Klassennamen schreibt man zwar groß aber nicht komplett Groß.

Ist es wirklich dein Ernst das die Scheibe 5000x5000 Pixel groß sein soll?

Deine ganzen Bounds sind irgendwie komplett durcheinander. Dadurch müssten sich die Panel irgendwie überlappen. Das unterstützt Swing aber nicht. Es sei denn du würdest layered Panels benutzen.

Wenn du bei jedem mal neuzeichnen des Panels das Bild neu einlädst und erstellst, wirst du wahrscheinlich später eine Animation deines drehenden Rades haben, welche auf sagenhafte 1-2 Bilder pro Sekunde kommt.

 

[FraxxeY]

Schlimm wenn ich das trotzdem so lasse mit dem Frame?

Das mit den Klassennamen machen wir in der Schule so, Gewohnheit

Gehirnaussetzer, 500x500

Wie fix ich das?

Okay, das ist zu wenig Wie soll ich das dann ändern?

Gruß

 

[JStein52]

Wieso funktioniert das nicht?

Was passiert denn eigentlich genau ? Im LoadImage catchst du eien Exception machst aber gar nichts. Sicher dass er nicht da reinlaeuft ?
Und das mit den Namen und dem Ableiten von JFrame etc. ist zwar nicht schön aber gerade nicht dein Problem.

 

[FraxxeY]

Was passiert denn eigentlich genau ? Im LoadImage catchst du eien Exception machst aber gar nichts. Sicher dass er nicht da reinlaeuft ?
Und das mit den Namen und dem Ableiten von JFrame etc. ist zwar nicht schön aber gerade nicht dein Problem.

In der paintComponent steht ein System.out.println...das wird bereits nicht mehr erreicht. In der exception wird nichts ausgegeben wenn ich println von e mach

 

[JStein52]

Und welche Elemente siehst du am Bildschirm ? Siehst du den roll-Button mitsamt richtigem Icon ?
Und du bist sicher dass die paintComponent nicht aufgerufen wird ? Dann wäre die Frage warum ? Weil dieses JLabel verdeckt ist und gar nichts zu painten ist ?

 

[JStein52]

Und überhaupt: müsste das nicht paintComponents(...) heissen ?

 

[JCODA]

Und überhaupt: müsste das nicht paintComponents(...) heissen ?

Nein, aber es kann zu komischen Verhalten kommen, falls nicht super.paintComponent(g) aufgerufen wird.

 

[FraxxeY]

Also, der roll button ist da:
http://prntscr.com/bo2j4z

Das ist der Code, so wie er oben steht ausgeführt.

Alles da außer das Rad

 

[JStein52]

Der Tip von @JCODA ist sicher auch noch zu beachten aber auch nicht das Problem da ja deine paintComponent gar nicht aufgerufen wird. Hast du mal versucht einen Konstruktor für WHEEL einzubauen der mindestens mal super() aufruft ?

 

[X5-599]

@FraxxeY
Sicher, dass das dein aktueller Code ist? Denn so wie es aussieht, müsste das Wheel das einzigste sein, was angezeigt wird. Da die Zeile mit setLayout(null) auskommentiert ist, wird von JFrame der Standard Layoutmanager verwendet (BorderLayout). Wenn dabei zwei Komponenten an dieselbe Stelle gesetzt werden, wird das jeweils erste durch das jeweils letzte ersetzt. Die setBounds() der Komponenten werden dabei übrigens auch ignoriert.

Wie führst du den Code eigentlich aus? Als jar gepackt oder aus einer IDE?

 

[FraxxeY]

@FraxxeY
Sicher, dass das dein aktueller Code ist? Denn so wie es aussieht, müsste das Wheel das einzigste sein, was angezeigt wird. Da die Zeile mit setLayout(null) auskommentiert ist, wird von JFrame der Standard Layoutmanager verwendet (BorderLayout). Wenn dabei zwei Komponenten an dieselbe Stelle gesetzt werden, wird das jeweils erste durch das jeweils letzte ersetzt. Die setBounds() der Komponenten werden dabei übrigens auch ignoriert.

Wie führst du den Code eigentlich aus? Als jar gepackt oder aus einer IDE?

Ich weiß auch nicht weiter, die Zeile ist auskommentiert so weit ich weiß, hab gerade keinen Zugriff auf den Code. Ich starte direkt aus Eclipse heraus.

 

[JStein52]

Hast du mal versucht einen Konstruktor für WHEEL einzubauen der mindestens mal super() aufruft ?

Hast du das schon probiert ?

 

[X5-599]

Was genau soll das bringen?

Der Code von @FraxxeY funktioniert jedenfalls so bei mir. Natürlich muss sichergestellt sein, dass das Bild auch geladen werden kann.

 

[JStein52]

Natürlich muss sichergestellt sein, dass das Bild auch geladen werden kann.

Ja klar. Nach seiner Aussage wird aber paintComponent gar nicht erst aufgerufen und Exceptions kommen auch keine ? Mhmmm ...

 

[X5-599]

Ich habe es so gemacht:

public class TestGui
{
	class Inhalt extends JLabel
	{
		public void paintComponent(Graphics g)
		{
			try
			{
				AffineTransform at = AffineTransform.getTranslateInstance(100, 100);
				at.rotate(Math.toRadians(45));
				
				Image img = ImageIO.read(Inhalt.class.getResource("/input/icon.png"));
				Graphics2D g2 = (Graphics2D)g;
				g2.drawImage(img, at, null);
			}
			catch(IOException e)
			{
				e.printStackTrace();
			}
			
		}
	}
	
	public TestGui()
	{
		JFrame frame = new JFrame();
		frame.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
		
		JLabel inhalt = new Inhalt();
		frame.add(inhalt, BorderLayout.CENTER);
		
		frame.pack();
		frame.setSize(200, 200);
		frame.setLocationRelativeTo(null);
		frame.setVisible(true);
	}
	
	public static void main(String[] args)
	{
		SwingUtilities.invokeLater(new Runnable() {
			
			@Override
			public void run()
			{
				new TestGui();
			}
		});
	}
}

Ist ähnlich dem Code von @FraxxeY. Ich leite lediglich nicht von JFrame ab, sondern erzeuge eines. Und mein JLabel ist eine innere Klasse.

 

[JStein52]

funktioniert jedenfalls so bei mir

Ist ja nun nicht ganz der Code des TE. Hat der bei dir doch nicht funktioniert ?

 

[X5-599]

Habe ich nicht ausprobiert, weil wie gesagt das für mich so aussieht, dass in seinem geposteten Code lediglich das WHEEL angezeigt werden müsste. Er sagt aber, das gerade dieses nicht angezeigt wird. Da muss irgendwas anderes nicht stimmen. Ich wollte nur zeigen, dass es nicht an einem fehlenden Konstruktor des abgeleiteten JLabels liegen kann.

Meine Vermutung ist, dass die Reihenfolge vo JButton und WHEEL im obersten Code vertauscht ist. Also, dass der JButton zum Schluss dem JFrame ge'added wurde. Das würde nämlich dazu führen, dass das paintComponent() des JLabels nie ausgeführt wird, weil das WHEEL durch den JButton ersetzt wurde.
Ist natürlich alles reine Spekulation.

 

[X5-599]

Ich habe gerade den original Code getestet. Im Grunde funktioniert der. Die setBounds() der Komponenten wirken sich auch bei gesetzten BorderLayout (Standard für JFrame) aus. Wusste ich zb nicht. Habe angenommen die werden von LayoutManagern ignoriert...
Also es werden der JButton und das JLabel gezeichnet.

Die paintComponent() Methode wird demnach also auch ausgeführt. Sieht man auch am System.out.println("DDD"). Wie gesagt, wenn das Bild "rad.png" nicht gefunden wird, sieht man natürlich auch nichts mehr vom JLabel. Dessen paintComponent() Methode wird aber dennoch ausgeführt.
Sollte das System.out.println("DDD") nicht in der Konsole erscheinen, weiss ich auch nicht mehr weiter.

Mein Rat wäre das ImageIO.read(new File("...")) durch:

ImageIO.read(getClass().getResource("rad.png"));

zu ersetzen. Denn ich nehme an, das Bild (ccc.png) des JButtons wird angezeigt? Dieses wird ja auf gleiche Weise geladen.

 

[JStein52]

Sollte das System.out.println("DDD") nicht in der Konsole erscheinen

Hat er oben gesagt, dass das nicht rauskommt. Aber wenn der gleiche Code bei dir funktioniert scheint bei ihm doch irgendwas anders zu sein was wir gerade nicht wissen. Und die Sache mit dem Konstruktor war auch nur ein herumstochern weil ich ja auch keinen prinzipiellen Fehler erkennen kann. Vielleicht sagt ja unser TE noch was dazu

 

[JStein52]

ImageIO.read(getClass().getResource("rad.png"));

Und das war mir auch nicht klar warum er das anders macht als beim Button wo es ja funktioniert.

 

[JStein52]

Und wenn in WHEEL das Bild nicht gefunden wird kommt natürlich auch eine Exception ! @FraxxeY hat zwar gesagt wenn er ein println im catch einbaut kommt da auch nichts aber das kann alles so nicht sein.
Also: Auf jeden Fall im catch eine Ausgabe machen denn einfach nichts machen ist total sinnlos ! Und vielleicht ist ihm auch nicht bewusst dass mit seinen beiden Methoden die Bilder einzulesen die in verschiedenen Ordnern stehen müssen damit sie auch gefunden werden ....