Variable Anzahl an Buttons per Schleife

[donmanolito]

Hi

Ich hab versucht ein Eingabe GUI zu machen wo ich eine variable Anzahl an Buttons per Schleife setze, über die ich einen Ordner per JFileChooser auswähle. Das soll dann nebenstehend in ein JFrame geschrieben werden und die Ordner sollen zusätzlich in einem File[] gespeichert werden um sie dann später zu verarbeiten.

So weit so gut. Das Problem ist jetzt das ich scheinbar keine richtige Zuordnung zu den Buttons hin kriege und daher immer nur die letzte Stelle im Array genutzt wird. Gibts dazu nen Trick? Ich kann mir kaum vorstellen das ich jeden Button einzeln aufführen mus.. :bahnhof:

import javax.swing.BorderFactory;
import javax.swing.JButton;
import javax.swing.JFileChooser;
import javax.swing.JFrame;
import javax.swing.JLabel;
import javax.swing.border.Border;
import java.awt.Color;
import java.awt.event.ActionEvent;
import java.awt.event.ActionListener;
import java.io.File;


@SuppressWarnings("serial")
public class Fenster extends JFrame{

	JFrame GUI = new JFrame();
	JButton B1 = new JButton("Erzeuge UPD File");
	JButton B2 = new JButton("Öffne UPD File");
	JFileChooser FC = new JFileChooser("Verzeichnis wählen");
	JButton IB = new JButton("Ausgabeordner");
	JLabel IL = new JLabel();
	
	public int ListLength = 10;
	public File[] dirList = new File[ListLength];
	public int JIval;
	


	public Fenster(){
		


	
	GUI.setSize(560, 30*ListLength+120);
	GUI.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
	GUI.setLocationRelativeTo(null);
	GUI.setTitle("ConfigMerger");
	GUI.setLayout(null);
	GUI.setResizable(false);
	
	GUI.add(B1);	
	GUI.add(B2);
	GUI.add(IB);
	GUI.add(IL);
	
	B1.setBounds(140, 50+30*ListLength, 130, 25);
	B1.setVisible(true);
	B1.setToolTipText("UPD File schreiben");
	B1.addActionListener(new Listener_merge(dirList));
	
	B2.setBounds(290, 50+30*ListLength, 130, 25);
	B2.setVisible(true);
	B2.setToolTipText("UPD File anzeigen");
//	B2.addActionListener(new Listener());		//TODO
	B2.setEnabled(false);
	
	IB.setBounds(30, 10, 130, 20);
	IB.setVisible(true);
	IB.setToolTipText("Ausgabeordner auswählen");
	IB.addActionListener(new Listener_Ordner(0, dirList));
	GUI.add(IB);
	
	
	IL.setVisible(true);
	IL.setBounds(170, 10, 350, 20);
	Border border = BorderFactory.createLineBorder(Color.black, 1);
	IL.setBorder(border);
	GUI.add(IL);
		
	
	for(int JIval = 1; JIval != ListLength + 1; JIval++ ){
		new JImport( JIval, dirList);		
	}
	
      
	GUI.setVisible(true);
	
	
	}
	
	public class JImport extends JFrame{
		int JIval;
		
		public JImport( int JIval, File[] dirList){
			JButton IB = new JButton("Seite " + JIval);
			JLabel IL = new JLabel();
			this.JIval = JIval;
									
			IB.setBounds(30, 10+30*JIval, 130, 20);
			IB.setVisible(true);
			IB.setToolTipText("Order Seite "+ JIval + " auswählen");
			IB.addActionListener(new Listener_Ordner(JIval, dirList));
			GUI.add(IB);
			
			
			IL.setVisible(true);
			IL.setBounds(170, 10+30*JIval, 350, 20);
			Border border = BorderFactory.createLineBorder(Color.black, 1);
			IL.setBorder(border);	
			GUI.add(IL);
			setJIval();
									
		}
		
		public int setJIval(){
			return JIval;
		}

	}
	
	
	public class Listener_merge implements ActionListener{
				
		Einlesen start;
		Ausgeben metupd;
		Ausgeben usrupd;
		int a = ListLength + 1;
		File[] dirList;
				
		public Listener_merge(File[] dirList) {
			this.dirList = dirList;
		}
		
		@Override
		public void actionPerformed(ActionEvent arg0) {
			Read();
			Write();
			B2.setEnabled(true);			
		}

		public void Read(){
			for(int i=1; i != a; i++){
				if(dirList[i].exists()){
			
					Einlesen start = new Einlesen(dirList[i]);
					this.start = start;
				}
			
			}
		}
		
		public void Write(){
			Ausgeben metupd = new Ausgeben(start.getDcstfmet().upd, "dcstfmet.upd");
			this.metupd = metupd;
			Ausgeben usrupd = new Ausgeben(start.getDcstfusr().upd, "dcstfusr.upd");
			this.usrupd = usrupd;
		}
		
	}
	
	
	public class Listener_Ordner implements ActionListener{
		int JIval;
		File[] dirList;
		
		public Listener_Ordner(int JIval, File[] dirList){
			this.JIval = JIval;
			this.dirList = dirList;
		}
		
		public Listener_Ordner(){
			
		}
		
		@Override
		public void actionPerformed(ActionEvent arg0) {
				
			FC.setDialogType(JFileChooser.OPEN_DIALOG); 
			FC.setFileSelectionMode(JFileChooser.DIRECTORIES_ONLY);
			int x = FC.showOpenDialog(null);
			
			if(x == JFileChooser.APPROVE_OPTION){
				dirList[JIval]=FC.getSelectedFile();
				for(int z = 0; z != ListLength + 1; z++ ){
					IL.setBounds(175, 10+30*z, 350, 25);
					if(dirList[z].exists()){
				
					IL.setText(String.valueOf(dirList[z]));
				}
			}			
			GUI.setVisible(true);
			}
			
		}
	}
	
}
	
[/Java]

 

[Harry Kane]

Ds sieht ziemlich chaotisch aus.
1. Java Namenskonventionen beachten: Variablennamen klein schreiben, Klassennamen in UpperCamelCase.
2. Warum erweitert JImport JFrame? Die im Konstruktor von JImport erzeugten JButtons werden doch der GUI Instanz hinzugefügt. Die JImport-Instanzen werden offenbar nirgends sichtbar gemacht.
3. Deine setJIval-Methode ist ein Knaller: eine Methode, die mit set beginnt, in der aber nix gesetzt wird, sondern stattdessen etws zurückgegeben wird, und die zwar aufgrufen wird, aber ohne ihren Rückgabewert zu verwenden.
Woran merkst du denn genau, dass du "scheinbar keine richtige Zuordnung zu den Buttons [hinkriegst] und daher immer nur die letzte Stelle im Array genutzt wird"?

 

[donmanolito]

Ähmm ja... Ich versuche mich erst seit ca einer Woche an Java... Also bitte nicht zu streng sein ;-) Ich hab mir das so aus Youtube Tutorials und Foren "erarbeitet" und probiere bisher aus und versuche aus meinen Fehlern zu lernen :-D

Woran ich das erkenne? Nun ja es ist ziemlich egal welchen der Buttons ich drücke, der Pfadname wird immer nur im untersten JFrame ausgegeben...

 

[Harry Kane]

Und was ist der "unterste JFrame"? Ich kann nur eine Instanz von JFrame sehen, die sichtbar ist.
Ich hätte ja versucht, deinen Code laufen zu lassen, aber ohne die Klassen "Einlesen" und "Ausgeben" geht das nicht.

 

[donmanolito]

BIn grad auf Arbeit und hab keinen Zugriff auf den Code. Falls du JImport meinst dann sollten das die Klasse sein die über die Schleife eine variable Anzahl an Buttons her stellt.

Einlesen macht nichts anderes als die Daten im übergebenen Verzeichnis auflisten und dann nacheinander einer Verarbeiten Klasse zuzuschieben. Dort wird nach Duplikaten, Kommentaren, ect gesucht und das dann alles in eine ArrayList geschrieben.

Ausgeben nimmt das dann aus der entsprechenden ArrayList und schreibt es in eine neue Datei.

Im Prinzip soll das Programm dazu da sein verschiedene Files aus verschiedenen Ordnern zusammen zu fügen und daraus eine neue File zu schreiben.

 

[Harry Kane]

Die Frage ist nicht, was ich meine, sondern was du meinst.
Ich sehe in deinem ganzen Code nur einen JFrame, der sichtbar ist, nämlich die Variable GUI aus der Klasse Fenster (was natürlich die Frage aufwirft, warum Fenster JFrame erweitert. Ich halte das für überflüssig).
Dann redest du vom "untersten JFrame", was impliziert, dass es mehrere gibt.
Dass ich deinen Code nicht kompilieren kann, weil er nicht vollständig ist, habe ich oben schon gesagt (und sollte eigentlich selbstverständlich sein).

 

[donmanolito]

GUI erzeugt das einzige Fenster, das ist richtig. Um im GUI die Buttons und Labels zu platzieren gibt es diese Schleife

 	for(int JIval = 1; JIval != ListLength + 1; JIval++ ){
		new JImport( JIval, dirList);		
	}

Hier wird auf die Klasse JImport zugegriffen die dann "JIVal" x (Buttons & Labels) im GUI Frame erzeugt.

 

[donmanolito]

Hat sich erledigt... Kaum packt man die Klasse in eine ArrayList schon klappts ;-)