2D-Grafik Bilder in GridLayout einfügen

[Karuka]

Hallo Leute!

Ich hab mich jetz schon längere zeit allein daran Versucht mein Problem zu lösen (mit hilfe von google, sufu hier im Forum, etc...) aber ohne erfolg..

Jedenfalls kommen wir zum eigentichen Problem.. mein Ziel war es mehrere Bilder einzulesen und angeordnet auszugeben. Meine einzigen Ergebnisse waren, dass ich die Bilder nur übereinanderliegend ausgebe.
Um dieses Problem zu lösen bin ich nach etwas längerer Suche auf das GridLayout gestoßen, welches vielversprechend klang.. Jedoch scheitere ich daran, die Bilder in das GridLayout einzufügen und weiß auch nichtmehr weiter.

Hoffentlich kann mir jemand helfen, den die brauche ich dringend...

Hier der Quellcode:

import java.awt.Dimension;
import java.awt.Graphics;
import java.awt.Image;

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

public class ImageTest {

  public static void main(String[] args) {
    ImagePanel bild = new ImagePanel(new ImageIcon("bild.png").getImage());
    ImagePanel bild2 = new ImagePanel(new ImageIcon("bild2.png").getImage());
    
    JFrame frame = new JFrame();
    frame.getContentPane().add(bild);
    frame.getContentPane().add(bild2);
    frame.pack();
    frame.setVisible(true);
    

  }
}

class ImagePanel extends JPanel {

  private Image img;

  public ImagePanel(String img) {
    this(new ImageIcon(img).getImage());
  }

  public ImagePanel(Image img) {
    this.img = img;
    Dimension size = new Dimension(img.getWidth(null), img.getHeight(null));
    setPreferredSize(size);
    setMinimumSize(size);
    setMaximumSize(size);
    setSize(size);
    setLayout(new GridLayout(0,2));
         add(bild);
         add(bild2);

  }

  public void paintComponent(Graphics g) {
    g.drawImage(img, 0, 0, null);
  }
}

 

[Gast2]

Dein Code ist leicht unstrukturiert...
Was soll dein Klasse ImagePanel machen? Genau ein Bild darstellen oder mehrere Bilder darstellen?
Innerhalb der Klasse ImagePanel gibts keine Variablen bild1 oder bild2, was genau willst du da machen?

    frame.getContentPane().add(bild);
    frame.getContentPane().add(bild2);

Dein Frame hat von Haus aus erstmal nen BorderLayout. Durch die beiden add's wird die erste Komponente überschrieben.

 

[Fab1]

Was spricht gegen ein JLabel mit ImageIcon?

public class Test {
 
  public static void main(String[] args) {
    JLabel bild = new JLabel(new ImageIcon("C:/Bilder/Apfel.jpg"));
    JLabel bild2 = new JLabel(new ImageIcon("C:/Bilder/Apfel.jpg"));
    
    JFrame frame = new JFrame();
    JPanel panel = new JPanel();
    frame.add(panel);
    
    panel.setLayout(new GridLayout(0,2));
    panel.add(bild);
    panel.add(bild2);
    
    frame.setVisible(true);
    frame.pack();
  }
}

So in der Art, wolltest du es ja haben nehm ich mal an oder?

[EDIT]Was mir gerade noch aufgefallen ist, folgende Methode dient zum schließen der Anwendung wenn du auf das "x" drückst.

frame.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);

Ansonsten wunderst du dich nämlich warum dein PC immer langsamer wird, oder du im Debugmodus 100 Instanzen deines Programms am laufen hat, obwohl du sie ja geschlossen hast. Außer natürlich du beendest die Anwendung immer mit dem roten Terminate Symbol in Eclipse.[/EDIT]

 

[Karuka]

Wie EikeB schon sagte, war mein Quellcode etwas sehr unstrukturiert und durch Fab1 bin ich auf eine neue Idee gekommen und habe auch das ganze nocheinmal neu angefangen, diesesmal auch Strukturierter ^^"
Vielen dank euch beiden.

Jedenfalls dadurch das ich neu angefangen habe bin ich auch einen guten Schritt weiter.
Jetzt gibt es aber ein neues Problem, nämlich die Bilder werden zwar anscheinend eingefügt aber, zusammen mit den beiden Buttons die ich als Test dort eingebaut habe, mir nur als "leere" Spalte des Layouts angezeigt anstelle der beiden Bilder die dort sein sollten.

Hier der neue Quellcode:

 import java.awt.*;
 import java.io.*;
 import javax.imageio.ImageIO;
 import javax.swing.*;

 public class Layout extends JFrame {
  
     public Layout() {
       setLayout(new GridLayout(2,2));

       JButton button1 = new JButton("button 1");
       add(button1);
         
       JButton button2 = new JButton("button 2");
       add(button2);
     }
     
     public static void main(String[] args){
       Layout derp = new Layout();
       JFrame frame = new JFrame();
       JPanel panel1 = new JPanel();
       JPanel panel2 = new JPanel();
       JLabel bild1 = new JLabel(new ImageIcon("bilder/remisaku.png"));
       JLabel bild2 = new JLabel(new ImageIcon("bilder/patchy.png"));

       panel1.add(bild1);
       panel2.add(bild2);
       
       derp.add(panel1);
       derp.add(panel2);
       derp.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
       derp.setVisible(true);
       derp.pack();
       derp.setTitle("herp");
     }
 }

 

[FaxXer]

Ich versteh dein Problem jetzt nicht ganz.
Bei mir zeigt es alles an wie es sein soll. Oben zwei Buttons und unten zwei Bilder?

Mach doch mal ein Screenshot von deinem Fenster.

 

[Karuka]

Das Fenster sieht wie folgt aus:

Dort wo der rote Rahmen ist sollten eigentlich die beiden Bilder sein aber ich bekomme nur die graue Standartfläche vom Fenster.

 

[Fab1]

Wo liegen denn die Bilder genau? Ansonsten kannst du ja mal versuchen den absoluten Pfad anzugeben.

 JLabel bild = new JLabel(new ImageIcon("Apfel.jpg"));// wenn das Bild direkt im Ordner "src" liegt
    JLabel bild2 = new JLabel(new ImageIcon("/bilder/Apfel.jpg")); // wenn das bild im Ordner "src\Bilder" liegt

 

[Karuka]

Danke! der Dateipfad war nicht ganz richtig ( ein / vergessen). Es sind dann doch immer die kleinen Unachtsamkeitsfehler die einem das Leben schwer machen

Mein Problem hat sich gelöst und ich kann Bilder im GridLayout einfügen, danke an allen teilhabenden!