Problem mit meinen Klassen

[Butterkeks01]

Hey,
Ich habe ein Problem, un zwar gibt der getter ImageSelection.getName() immer nur "null" zurück und ich weiß nicht warum.
Obwohl ich in der unteren Klasse den String "name" zugewiesen habe.

Vielleicht kann mir einer helfen
PS: Bin noch ein relativer Neuling, deswegen sorry sollte ich irgendwas nicht sofort verstehen ^^

package test;


import java.awt.Font;
import java.awt.event.ActionEvent;
import java.awt.event.ActionListener;
import java.io.File;
import java.util.Scanner;

import javax.swing.Icon;
import javax.swing.ImageIcon;
import javax.swing.JButton;
import javax.swing.JFrame;
import javax.swing.JLabel;
import javax.swing.SwingConstants;
import javax.swing.border.Border;
import javax.swing.border.LineBorder;


public class DragImage {
    public static void main(String args[]) {
        
        Border border = LineBorder.createGrayLineBorder();
        Font labelFont = new Font("Arial", Font.CENTER_BASELINE, 130);
        
        JFrame frame = new JFrame();
        JLabel jLabel = new JLabel("+", SwingConstants.CENTER);
        JButton jButton = new JButton("test");
        
        
        
        
        jLabel.setTransferHandler(new ImageSelection());
        
        String name = ImageSelection.getName();
        
        frame.setVisible(true);
        frame.setSize(900, 700);
        frame.setTitle("Automatisierung");
        frame.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
        frame.setLocationRelativeTo(null);
        frame.setResizable(false);
        frame.setLayout(null);
        
        
        String adress = "";
        Icon icon = new ImageIcon(adress);
        jLabel.setBounds(10, 8, 134, 188);
        jLabel.setFont(labelFont);
        jLabel.setVisible(true);
        jLabel.setBorder(border);
        jLabel.setIcon(icon);
        
        jButton.setBounds(300, 8, 134, 188);
        jButton.setVisible(true);
        jButton.addActionListener(new ActionListener() {
            
            @Override
            public void actionPerformed(ActionEvent e) {
                System.out.println("test " + name);
                
            }
        });
        
        frame.add(jButton);
        frame.add(jLabel);

        
        
        
        
    }


}

package test;

import java.awt.datatransfer.DataFlavor;
import java.awt.datatransfer.Transferable;
import java.awt.datatransfer.UnsupportedFlavorException;
import java.io.IOException;
import javax.swing.AbstractButton;
import javax.swing.JComponent;
import javax.swing.JLabel;
import javax.swing.TransferHandler;

class ImageSelection extends TransferHandler implements Transferable {
    
    static String name;
    private static final long serialVersionUID = 3181571469332838353L;
    private static final DataFlavor flavors[] = { DataFlavor.javaFileListFlavor };

    @Override
    public boolean canImport(JComponent comp, DataFlavor flavor[]) {
        if (!(comp instanceof JLabel) && !(comp instanceof AbstractButton)) {
            return false;
        }
        for (int i = 0, n = flavor.length; i < n; i++) {
            for (int j = 0, m = flavors.length; j < m; j++) {
                if (flavor[i].equals(flavors[j])) {
                    return true;
                }
            }
        }
        return false;
    }

    

    @Override
    /*
     * This will set the name.
     */
    public boolean importData(JComponent comp, Transferable t) {
                    try {

                        name = String.valueOf(t.getTransferData(flavors[0]));
                        
                        System.out.println(name);
                        
                    } catch (UnsupportedFlavorException | IOException e) {
                        e.printStackTrace();
                    }   
        return false;
        
    }
    
    
    public static String getName() {
        return name;
        
    }
    

    

    @Override
    public Object getTransferData(DataFlavor flavor) throws UnsupportedFlavorException, IOException {
        // TODO Auto-generated method stub
        return false;
    }



    @Override
    public DataFlavor[] getTransferDataFlavors() {
        // TODO Auto-generated method stub
        return null;
    }



    @Override
    public boolean isDataFlavorSupported(DataFlavor flavor) {
        // TODO Auto-generated method stub
        return false;
    }
    
}

 

[kneitzel]

Also ich verstehe derzeit nicht, wo Du glaubst, dass name gesetzt wird. Die Methode main wird aufgerufen, Du erzeugst dann eine neue Instanz der Klasse, aber das setzt den namen nicht. Und dann speicherst Du den name in der lokalen variable name. An der Stelle wird name also null sein.

Und diese lokale Variable wird dann ausgegeben, wenn ich es richtig gesehen habe.

Und generell solltest Du das Design überdenken. Diese static Elemente in ImageSelection scheinen sehr dubios zu sein - vor allem da es auch noch Instanzen der Klasse geben soll ...

 

[httpdigest]

Wo/wann rufst du denn `ImageSelection.importData(JComponent, Transferable)` auf, so dass der Name tatsächlich gesetzt wird?

 

[Butterkeks01]

Okay das hört sich plausibel an, könnt ihr mir vielleicht schreiben was ich genau machen muss, also zum Beispiel wenn ich ImageSelection.importData(JComponent, Transferable) aufrufe was ic hanstelle von JComponent und Transferable nehmen muss.
Wäre Super

 

[Butterkeks01]

Also ich verstehe derzeit nicht, wo Du glaubst, dass name gesetzt wird. Die Methode main wird aufgerufen, Du erzeugst dann eine neue Instanz der Klasse, aber das setzt den namen nicht. Und dann speicherst Du den name in der lokalen variable name. An der Stelle wird name also null sein.

Und diese lokale Variable wird dann ausgegeben, wenn ich es richtig gesehen habe.

Und generell solltest Du das Design überdenken. Diese static Elemente in ImageSelection scheinen sehr dubios zu sein - vor allem da es auch noch Instanzen der Klasse geben soll ...

Kannst du mir vielleicht zeigen was ich machen muss damit es funktioniert?

 

[kneitzel]

Also ich verstehe derzeit nicht, was Dein Code überhaupt machen soll und das macht es relativ schwer bis unmöglich, konkrete Hinweise zu geben.

- Am Anfang wäre mein Tipp immer, auf static ganz zu verzichten (bis auf die main Methode natürlich).
- Dann wäre der nächste Tipp, die Funktionalität in der main sehr gering zu halten. In der main Methode sollten nur existenzielle Objekte für die Applikation erzeugt werden. Dies ist aber minimal.
- Komplexe Dinge werden immer in Klassen gekapselt. Als Beispiel nehmen wir einfach einmal das, was Du in der main aufbaust. Da wird ja eine Frame erstellt und gefüllt. Das ist prädestiniert für eine Klasse und im Konstruktor werden dann alle Element erzeugt. Also sowas wie:

public class MainFrame {
    private Border border;
    private Font labelFont;
        
    private JFrame frame;
    private JLabel jLabel;
    private JButton jButton;
    
    public MainFrame() {
        border = LineBorder.createGrayLineBorder();
        labelFont = new Font("Arial", Font.CENTER_BASELINE, 130);
        
        frame = new JFrame();
        jLabel = new JLabel("+", SwingConstants.CENTER);
        jButton = new JButton("test");
        jLabel.setTransferHandler(new ImageSelection());
        String name = ImageSelection.getName();
        frame.setVisible(true);
        frame.setSize(900, 700);
        frame.setTitle("Automatisierung");
        frame.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
        frame.setLocationRelativeTo(null);
        frame.setResizable(false);
        frame.setLayout(null);
        
        String adress = "";
        Icon icon = new ImageIcon(adress);
        jLabel.setBounds(10, 8, 134, 188);
        jLabel.setFont(labelFont);
        jLabel.setVisible(true);
        jLabel.setBorder(border);
        jLabel.setIcon(icon);
        
        jButton.setBounds(300, 8, 134, 188);
        jButton.setVisible(true);
        jButton.addActionListener(new ActionListener() {
}

==> Das ist aber nur ein Beispielcode!

- Bezüglich das TransferHandlers:
Du hast da ja ganz offensichtlich eine Instanz der Klasse, die Du benutzt:
jLabel.setTransferHandler(new ImageSelection());
==> Die Instanz kannst Du ja auch in einer Instanzvariable speichern. Dann könnte name auch nicht mehr statisch sein. Und in dem Action Listener könntest Du auf die Instanz von ImageSelection zugreifen und getName() aufrufen. (Wenn da also der Name später gesetzt wird, dann kannst Du ihn abfragen!)

Das wären einfach nur ein paar Ideen, die mir so durch den Kopf gehen. Aber ohne genaue Details zu kennen, ist es sehr schwer, zu einem Design etwas zu sagen.

 

[Butterkeks01]

Also ich verstehe derzeit nicht, was Dein Code überhaupt machen soll und das macht es relativ schwer bis unmöglich, konkrete Hinweise zu geben.

- Am Anfang wäre mein Tipp immer, auf static ganz zu verzichten (bis auf die main Methode natürlich).
- Dann wäre der nächste Tipp, die Funktionalität in der main sehr gering zu halten. In der main Methode sollten nur existenzielle Objekte für die Applikation erzeugt werden. Dies ist aber minimal.
- Komplexe Dinge werden immer in Klassen gekapselt. Als Beispiel nehmen wir einfach einmal das, was Du in der main aufbaust. Da wird ja eine Frame erstellt und gefüllt. Das ist prädestiniert für eine Klasse und im Konstruktor werden dann alle Element erzeugt. Also sowas wie:

public class MainFrame {
    private Border border;
    private Font labelFont;
       
    private JFrame frame;
    private JLabel jLabel;
    private JButton jButton;
   
    public MainFrame() {
        border = LineBorder.createGrayLineBorder();
        labelFont = new Font("Arial", Font.CENTER_BASELINE, 130);
       
        frame = new JFrame();
        jLabel = new JLabel("+", SwingConstants.CENTER);
        jButton = new JButton("test");
        jLabel.setTransferHandler(new ImageSelection());
        String name = ImageSelection.getName();
        frame.setVisible(true);
        frame.setSize(900, 700);
        frame.setTitle("Automatisierung");
        frame.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
        frame.setLocationRelativeTo(null);
        frame.setResizable(false);
        frame.setLayout(null);
       
        String adress = "";
        Icon icon = new ImageIcon(adress);
        jLabel.setBounds(10, 8, 134, 188);
        jLabel.setFont(labelFont);
        jLabel.setVisible(true);
        jLabel.setBorder(border);
        jLabel.setIcon(icon);
       
        jButton.setBounds(300, 8, 134, 188);
        jButton.setVisible(true);
        jButton.addActionListener(new ActionListener() {
}

==> Das ist aber nur ein Beispielcode!

- Bezüglich das TransferHandlers:
Du hast da ja ganz offensichtlich eine Instanz der Klasse, die Du benutzt:
jLabel.setTransferHandler(new ImageSelection());
==> Die Instanz kannst Du ja auch in einer Instanzvariable speichern. Dann könnte name auch nicht mehr statisch sein. Und in dem Action Listener könntest Du auf die Instanz von ImageSelection zugreifen und getName() aufrufen. (Wenn da also der Name später gesetzt wird, dann kannst Du ihn abfragen!)

Das wären einfach nur ein paar Ideen, die mir so durch den Kopf gehen. Aber ohne genaue Details zu kennen, ist es sehr schwer, zu einem Design etwas zu sagen.

Ja Du hast recht, mein Code ist wirklich nicht schön. Ich habe außerdem noch ein bisschen rum probiert und habe es erneuert und tatsächlich ich habe mir mit den static methoden ein Eigentor geschossen ^^

Ich habe also die Static Methoden zu Instanz methoden geändert und dann über this.name den richtigen Wert bekommen.

Und nochmal zu meiner Struktur, ich sollte wirklich lernen mehr in Klassen zu machen, da ich momentan so ziemlich alles versuche in der "main" methode zu machen.

Nochmals Danke 🙂
Mit freundlichen Grüßen
Butterkeks