Textdatei einlesen und JFrame füllen

[stehohrhase]

Hallo zusammen und ein Frohes neues Jahr,

ich möchte auch mal um Eure Hilfe bitten:

Ich habe unter anderem ein Parameter-Textfile, welches so aufgebaut ist:

[Label01]
Parameter 1 = Text
Parameter 2 = Text
Parameter 3 = Text

[Label02]
Parameter 1 = Text

[Label03]
Parameter 1 = Text
Parameter 20 = Text
Parameter 30 = Text

Die Label und Parameter sind jetzt nur exemplarisch, sind aber eindeutig.
Die Parameter sind teils eindeutig, aber einige wiederholen sich auch unterhalb der verschiedenen Labels.

Jetzt möchte ich diese Textzeile zeilenweise einlesen und die Werte in einem JFrame anzeigen.
Das JFrame habe ich in der Hauptklasse erzeugt und wollte nun "ordentlich" Klassen für die einzelnen Funktionen erstellen.

In der Hauptklasse habe ich folgende Methode:

private void jButton1ActionPerformed(ActionEvent evt) {
	    Parameter p = new Parameter();                          //Klasse Parameter aufrufen
	    jTextField1.setText(p.identity[0]);                       //Werte aus Klasse Parameter in
	    jTextField2.setText(p.identity[1]);                       //JFrame eintragen
	    jTextField3.setText(p.identity[2]);

In der aufgerufenen Klasse habe ich folgendes gemacht:

public class Parameter {
	String activeParameter = "N/A";
	String strLine;
	String [] identity = new String[4];
        String [] wparameter = new String[4];
	{
        
        //Datei öffnen

	try {
		FileInputStream fstream = new FileInputStream("c:/test/Chip.ini");
		DataInputStream in = new DataInputStream(fstream);
                BufferedReader br = new BufferedReader(new InputStreamReader(in));
		
               //Datei zeilenweise einlesen

		while ((strLine = br.readLine()) != null) 	{
			if (strLine.startsWith ("[")) {
				activeParameter = strLine;
			}

            //Datei interpretieren, bei Übereinstimmung von Label und Parameter -> Methode aufrufen

			if (activeParameter.contains("[Label01]") & strLine.contains("Parameter 1")) {
				parseidentity(0);
			}
			if (activeParameter.contains("[Label01]") & strLine.contains("Parameter 2")) {
				parseidentity(1);
			}
			if (activeParameter.contains("[Label01]") & strLine.contains("Parameter 3")) {
				parseidentity(2);
			}
			if (activeParameter.contains("[Label02]") & strLine.contains("Parameter 1")) {
				parseweitereParameter(1);
			}
		}
		
		//Datei schliessen
		in.close();
		
		} catch (Exception e){//Catch exception if any
			System.err.println("Error: " + e.getMessage());
		}
	}
		
        // Hier trenne splitte ich am "=" Zeichen, verwerfe Leerzeichen und schreibe in die Variable
	void parseidentity(int i) {
		String[] split = strLine.split("=");
		identity[i] = split[1];
		identity[i] = identity[i].trim(); 		
	}
	void parseweitereParameter(int i) {
		String[] split = strLine.split("=");
		wparameter[i] = split[1];
		wparameter[i] = wparameter[i].trim(); 		
	}

}

Ist das so sauber programmiert ?
Was da eingelesen werden soll, sind so 250 Parameter.

- schreibt mal alle Werte in Variablen und die Hautklasse liest diese aus und überträgt sie ins JFrame
- oder sollte die aufgerufene Klasse das JFrame direkt füllen ? Brauche ich dann überhaupt so viele Variablen ?

- Wenn das so gemacht wird, ist das ok, das man direkt auf die Variablen zugreift oder erstellt man für diese Menge an Variablen Gettermethoden ?

LG, stehohrhase

 

[Gucky]

Es hat sich durchgesetzt, dass man für alle Variablen in einer Klasse Gettermethoden und die Vafiablen als private deklariert.

Das man unterschiedliche Aufgaben in unterschiedlichen Klassen unterbringt ist eine der Ideen der OOP.

Beim Öffnen der Datei sehe ich keine Methode. Ist das ein Tippfehler? Denn eigentlich müsste der Compiler dann meckern.

Da du weißt, wie viele Labels es werden, hätte ich es so gemacht das du ein Array mit ArrayLists hast und diese befüllst. Das wäre übersichtlicher. Hinterher kannst du es ja wieder in ein Array umwandeln.

 

[knilch]

Hi,
Ein paar Anmerkungen zu deinem Code:
Erst mal das hier:

    {
       
        //Datei öffnen
 
    try {
        FileInputStream fstream = new FileInputStream("c:/test/Chip.ini");
        DataInputStream in = new DataInputStream(fstream);
                BufferedReader br = new BufferedReader(new InputStreamReader(in));
       
               //Datei zeilenweise einlesen
 
        while ((strLine = br.readLine()) != null)   {
            if (strLine.startsWith ("[")) {
                activeParameter = strLine;
            }
 
            //Datei interpretieren, bei Übereinstimmung von Label und Parameter -> Methode aufrufen
 
            if (activeParameter.contains("[Label01]") & strLine.contains("Parameter 1")) {
                parseidentity(0);
            }
            if (activeParameter.contains("[Label01]") & strLine.contains("Parameter 2")) {
                parseidentity(1);
            }
            if (activeParameter.contains("[Label01]") & strLine.contains("Parameter 3")) {
                parseidentity(2);
            }
            if (activeParameter.contains("[Label02]") & strLine.contains("Parameter 1")) {
                parseweitereParameter(1);
            }
        }
       
        //Datei schliessen
        in.close();
       
        } catch (Exception e){//Catch exception if any
            System.err.println("Error: " + e.getMessage());
        }
    }

- Das sollte von einer Methode ausgeführt werden..
- Die ganzen if- Abfragen innerhalb dieser Struktur kann mit einer Switch- Anweisung eleganter gelöst werden.
- Du benuntzt immer ein String- Array der grösse 4, was ist, wenn nun mehr als 4 Parameter vorhanden sind (vielleicht nicht zur Zeit, aber später mal ein 5. oder 6. Parameter dazukommt?
Mit ArrayList<String> kannst du beliebig viele Elemente einfügen (naja es gibt schon speichergrenzen, aber ich denke nicht, dass du diese erreichen wirst)
Nachfolgend mal ein Beispiel:

public HashMap<String, ArrayList<String>> getParameter(String fileName) {
	HashMap<String, ArrayList<String>> labelMap = new HashMap<String, ArrayList<String>>();
	String activeParameter = "";
	String strLine;
	
	//Datei öffnen
	try {
		FileInputStream fstream = new FileInputStream(fileName);
		DataInputStream in = new DataInputStream(fstream);
		BufferedReader br = new BufferedReader(new InputStreamReader(in));
		String key = "";
		//Datei zeilenweise einlesen

		while ((strLine = br.readLine()) != null)   {
			if (strLine.startsWith ("[")) {
				activeParameter = strLine;
				switch (activeParameter) {
				case "[Label01]":
					key = "[Label01]";
					labelMap.put(key, new ArrayList<String>());
					break;
				case "[Label02]":
					key = "[Label02]";
					labelMap.put(key, new ArrayList<String>());
					break;		
				case "[Label03]":
					key = "[Label03]";
					labelMap.put(key, new ArrayList<String>());
					break;
				}
			}
			else {
				labelMap.get(key).add(strLine);  // Einfügen der Parameter = ... in die jeweilige ArrayListe welche dem Label zugeordnet ist
			}

		}
		//Datei schliessen
		in.close();

	} catch (Exception e){//Catch exception if any
		System.err.println("Error: " + e.getMessage());
	}
	return labelMap;
}

Dieses Beispiel gibt dir eine HashMap zurück. Die HashMap besteht aus einem Key z.B. "Label01" und der Dazugehörigen ArrayListe<String> in welcher alle Parameter vorhanden sind.
Damit du nun für jedes Label die Parameter aus der ArrayListe erhälst kannst du so vorgehen:

public static void main(String[] args) {
	Parameter p = new Parameter();
	String fileName = "c:/test/Chip.ini";
	HashMap<String, ArrayList<String>> map = p.getParameter(fileName);
	
	for (Map.Entry<String, ArrayList<String>> entry : map.entrySet()) {
	    System.out.println(entry.getKey());
	    for (String str: entry.getValue()) {
			System.out.println(str);
		}
	}	
}

 

[stehohrhase]

Danke für die Info.

@Gucky: ich hatte den Text etwas gekürzt, um es übersichtlich zu halten.

@knilch: Das hat soweit funktioniert.

Ich hatte mit meinen if-schleifen den Parameter überprüft und den Wert in das Array geschrieben.
Daher wusste ich, unter welchem Index, welcher Wert war.
Die Parameter stehen nicht immer in der gleichen Reihenfolge, da die Dateien bis jetzt von Hand editiert werden... das heißt es könnte stehen:

[COMMUNICATION]
BAUDRATE=9600
IP = 127.0.0.1
DHCP =0

oder

[COMMUNICATION]
IP = 127.0.0.1
DHCP =0
BAUDRATE=9600

Die Werte sind jetzt im ArrayList, komplett mit Leerzeilen. Wie würdet Ihr nun die einzelnen Parameter den Labels im JFrame zu weisen? z.B. das Feld mit der Baudrate finden und die 9600 zuweisen.
jTextField_Baudrate.setText(wert); - Wie die List durcharbeiten ? Oder nenne ich die Label so, das ich mit den gespeicherten Werten mir die Zuweisung zusammenbauen kann oder wie würdet Ihr durch eine
große Anzahl Werte durchgehen und Felder zuweisen?

z.B.
[COMMUNICATION]
BAUDRATE=9600

jTextField_COMMUNICATIONBAUDRATE.setText("9600");

 

[knilch]

Hi,
Vielleicht hilft dir das weiter:

package test;

import java.awt.Container;
import java.awt.Dimension;
import java.awt.Toolkit;
import java.io.BufferedReader;
import java.io.DataInputStream;
import java.io.FileInputStream;
import java.io.InputStreamReader;
import java.util.ArrayList;
import java.util.HashMap;
import java.util.Map;

import javax.swing.JFrame;
import javax.swing.JLabel;

public class Gui extends JFrame{

	private static final long serialVersionUID = 7137499759912767461L;
	private String fileName = "C:/test/Chip.ini";

	public Gui() {
		initGui();
	}

	public void initGui() {
		Container contentPane = this.getContentPane();
		contentPane.setLayout(null);
		this.setSize( 500, 250 );

		Toolkit toolkit = Toolkit.getDefaultToolkit();
		Dimension screenSize = toolkit.getScreenSize();
		this.setLocation((screenSize.width - this.getWidth()) / 2, (screenSize.height - this.getHeight()) / 2);


		this.setSize(800, 400);
		setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
		HashMap<String, ArrayList<String>> map = getParameter();
		int x = 10;
		int y = 10;


		StringBuilder sb = new StringBuilder();
		for (Map.Entry<String, ArrayList<String>> entry : map.entrySet()) {
			String key = entry.getKey();
			JLabel label = new JLabel(key);
			label.setBounds(x, y, 700, 25);
			contentPane.add(label);
			for (String string : entry.getValue()) {
				String [] splitt = string.split("=");
				sb.append(splitt[0] + ": " + splitt[1] + "   ");
			}
			label.setText(key + "   "+ sb.toString());
			sb.setLength(0);
			y += 20;
		}
		this.setVisible(true);
	}



	public static void main(String[] args) {
		new Gui();
	}

	public HashMap<String, ArrayList<String>> getParameter() {
		HashMap<String, ArrayList<String>> labelMap = new HashMap<String, ArrayList<String>>();
		String activeParameter = "";
		String strLine;

		//Datei öffnen
		try {
			FileInputStream fstream = new FileInputStream(fileName);
			DataInputStream in = new DataInputStream(fstream);
			BufferedReader br = new BufferedReader(new InputStreamReader(in));
			String key = "";
			//Datei zeilenweise einlesen

			while ((strLine = br.readLine()) != null)   {
				if (strLine.startsWith ("[")) {
					activeParameter = strLine;
					switch (activeParameter) {
					case "[Label01]":
						key = "[Label01]";
						labelMap.put(key, new ArrayList<String>());
						break;
					case "[Label02]":
						key = "[Label02]";
						labelMap.put(key, new ArrayList<String>());
						break;		
					case "[Label03]":
						key = "[Label03]";
						labelMap.put(key, new ArrayList<String>());
						break;	
					case "[COMMUNICATION]":
						key = "[COMMUNICATION]";
						labelMap.put(key, new ArrayList<String>());
						break;
					}
				}
				else {
					if(strLine.length() > 0)
						labelMap.get(key).add(strLine);
				}

			}
			//Datei schliessen
			in.close();

		} catch (Exception e){//Catch exception if any
			System.err.println("Error: " + e.getMessage());
		}
		return labelMap;
	}



}

 

[stehohrhase]

Soweit sieht das schon mal gut aus.
Ich hänge an einer Stelle:

- Leerstellen sind raus
- (key + splitt[0]) ergibt "Label01_BAUDRATE"
- (splitt[1]) ergibt 9600

Meinen vorhanden JLabel habe ich "Label01_BAUDRATE" genannt.

Wie adressiere ich den am einfachsten.

(key + splitt[0]).setText(splitt[1])

sollte das ergeben:

Label01_BAUDRATE.setText("9600")

Kann man sowas zusammen bauen ?, oder muss ich für jede möglichkeit eine case machen?

for (String string : entry.getValue()) {
                String [] splitt = string.split("=");
                splitt[0] = splitt[0].trim();
                splitt[1] = splitt[1].trim();
             
                System.out.println(key + splitt[0]);
                System.out.println(splitt[1]);
            }

 

[Gucky]

Ich verstehe deinen Ansatz nicht. Du willst zwei Werte addieren und dann mit dem Punktoperator auf eine Methode des Ergebnisses zugreifen?

 

[knilch]

Hi,
zu deinen Fragen:

Wie adressiere ich den am einfachsten.

(key + splitt[0]).setText(splitt[1])

sollte das ergeben:

Label01_BAUDRATE.setText("9600")

Ein label aus key+splitt[0] kannst du nicht erstellen,
soetwas:

 JLabel key+splitt[0] = new JLabel();

geht nicht....

aber du kannst jedem Label einen unique Name zuweisen. So kannst du später auf das Label, via label.getName() zugreifen.
so zum Beispiel:

public HashMap<String, ArrayList<String>> getParameter() {
....
for (Map.Entry<String, ArrayList<String>> entry : map.entrySet()) {
    String key = entry.getKey();
    for (String string : entry.getValue()) {
        String [] splitt = string.split("=");
        JLabel label = new JLabel(key);
        label.setName(key+splitt[0]);
        label.setBounds(x, y, 100, 25);
        label.setText(splitt[1]);
        contentPane.add(label);
        x += 120;
    }
    y += 60;
    x = 0;
   }
....
}

Ausserhalb der Methode

 initGui() {
...
...
}

kannst du dann auf die Labels zugreifen, indem du alle Elemente, die im ContentPane registiert sind, in einen Array (Typ Component) eingefügt werden und dann über alle Elemente iteriert wird z.B. so:

public JLabel getLabel(String name) {
	Component[] components = contentPane.getComponents();
	
	if(components.length <= 0 || components == null)
		return null;
	
	for (Component component : components) {
		if(component.getName().equals(name))
			return (JLabel)component;
	}
	return null;
}

Diese Methode gibt dir dann das JLabel zurück, das du möchtest: z.B.

public static void main(String[] args) {
	Gui g = new Gui();
	JLabel label = g.getLabel("[COMMUNICATION]BAUDRATE");
	if(label != null) {
		String text = label.getText();
		label.setText(text + " neu");
	}
}

Nun wird dem Label "[COMMUNICATION]BAUDRATE" der Wert: 9600 neu zugewiesen.

Das kannst du nun mit allen anderen Labels genau gleich machen.

Ich hoffe das hilft dir weiter.

 

[stehohrhase]

In der "public void initGui()" gibt es den Container "contentPane".
Leider kann ich in der Methode "public JLabel getLabel(String name)" nicht drauf zugreifen.
Bei:
"Component[] components = contentPane.getComponents();" habe ich den Fehler:
"contentPane cannot be resolved"


@Gucky: Ich suche nach der besten Möglichkeit, 250 Parameter aus einer Textdatei zu lesen, und diese geordnet in einem JFrame anzeigen zu lassen.

 

[knilch]

Hi,
Hier mal der ganze Code:

package test;

import java.awt.Component;
import java.awt.Container;
import java.awt.Dimension;
import java.awt.Toolkit;
import java.io.BufferedReader;
import java.io.DataInputStream;
import java.io.FileInputStream;
import java.io.InputStreamReader;
import java.util.ArrayList;
import java.util.HashMap;
import java.util.Map;
import java.util.TreeMap;

import javax.swing.JFrame;
import javax.swing.JLabel;

public class Gui extends JFrame{

	private static final long serialVersionUID = 7137499759912767461L;
	private String fileName = "C:/test/Chip.ini";
	private Container contentPane;

	public Gui() {
		initGui();
	}

	public void initGui() {
		contentPane = this.getContentPane();
		contentPane.setLayout(null);
		this.setSize( 500, 250 );

		Toolkit toolkit = Toolkit.getDefaultToolkit();
		Dimension screenSize = toolkit.getScreenSize();
		this.setLocation((screenSize.width - this.getWidth()) / 2, (screenSize.height - this.getHeight()) / 2);

		this.setSize(800, 400);
		setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
		Map<String, ArrayList<String>> map = getParameter();
		int x = 10;
		int y = 10;

		for (Map.Entry<String, ArrayList<String>> entry : map.entrySet()) {
		    String key = entry.getKey();
		    for (String string : entry.getValue()) {
		        String [] splitt = string.split("=");
		        JLabel label = new JLabel(key);
		        label.setName(key+splitt[0]);
		        label.setBounds(x, y, 100, 25);
		        label.setText(splitt[1]);
		        contentPane.add(label);
		        x += 120;
		    }
		    y += 60;
		    x = 0;
		   }
		
		this.setVisible(true);
	}

	public static void main(String[] args) {
		Gui gui = new Gui();
		
	    JLabel label = gui.getLabel("[COMMUNICATION]BAUDRATE");
	    if(label != null) {
	        String text = label.getText();
	        label.setText(text + " neu");
	    }
	}
	
    public JLabel getLabel(String name) {
        Component[] components = contentPane.getComponents();
       
        if(components.length <= 0 || components == null)
            return null;
       
        for (Component component : components) {
            if(component.getName().equals(name))
                return (JLabel)component;
        }
        return null;
    }

	public Map<String, ArrayList<String>> getParameter() {
		Map<String, ArrayList<String>> labelMap = new HashMap<String, ArrayList<String>>();
		String activeParameter = "";
		String strLine;

		//Datei öffnen
		try {
			FileInputStream fstream = new FileInputStream(fileName);
			DataInputStream in = new DataInputStream(fstream);
			BufferedReader br = new BufferedReader(new InputStreamReader(in));
			String key = "";
			//Datei zeilenweise einlesen

			while ((strLine = br.readLine()) != null)   {
				if (strLine.startsWith ("[")) {
					activeParameter = strLine;
					switch (activeParameter) {
					case "[Label01]":
						key = "[LABEL 1]";
						labelMap.put(key, new ArrayList<String>());
						break;
					case "[Label02]":
						key = "[LABEL 2]";
						labelMap.put(key, new ArrayList<String>());
						break;		
					case "[Label03]":
						key = "[LABEL 3]";
						labelMap.put(key, new ArrayList<String>());
						break;	
					case "[COMMUNICATION]":
						key = "[COMMUNICATION]";
						labelMap.put(key, new ArrayList<String>());
						break;
					}
				}
				else {
					if(strLine.length() > 0) {
						labelMap.get(key).add(strLine);
					}
				}
			}
			//Datei schliessen
			in.close();

		} catch (Exception e){//Catch exception if any
			System.err.println("Error: " + e.getMessage());
		}
		return labelMap;
	}

}

 

[stehohrhase]

Danke. Funktioniert 1A !

 

[Gucky]

Was bei Knilchs letztem Code anders ist, ist dass das contentPane nun in der gesamten Klasse verfügbar ist, da es außerhalb von Methoden und ganz am Anfang deklariert wurde.

 

[stehohrhase]

Habe doch noch ein Anliegen.

Diese Methode rufe ich nun über eine Schleife auf, um alle Labels zu beschriften.
Da ich nicht alle möglichen Labels auf der Oberfläche habe, die laut Arraylist möglich wären,
kommt es durch die Schleife auch vor, das ich nach einem Label suche, welches es nicht gibt.

public JLabel getLabel(String name) {
        Component[] components = contentPane.getComponents();
       
        if(components.length <= 0 || components == null)
            return null;
       
        for (Component component : components) {
            if(component.getName().equals(name))
                return (JLabel)component;
        }
        return null;
    }

Hier stürzt die Methode dann mit der Meldung "Exception in thread "main" java.lang.NullPointerException" ab:

if(component.getName().equals(name))

Sollte nicht ein "null" zurückkommen, wenn nichts passt ?

 

[stehohrhase]

Habe es selber gefunden !

Sobald ein Label dabei ist, wo ich "label.setName()" nicht gesetzt habe, kommt es zu dem verhalten.

 

[stehohrhase]

Dank Eurer Hilfe hat das ganz gut geklappt.
Da ich die Oberfläche etwas individueller machen möchte, habe ich ein JTabbedPane und JPanel auf meinem Contentpane.

Aufgebaut in etwa so:

Der entsprechend Code:

public void initGui() {
        contentPane = this.getContentPane();
        contentPane.setLayout(null);
        contentPane.setLocation(new java.awt.Point(100, 100));
        this.setSize( 0, 0 );
        Toolkit toolkit = Toolkit.getDefaultToolkit();
        Dimension screenSize = toolkit.getScreenSize();
        this.setLocation((screenSize.width - this.getWidth()) / 2, (screenSize.height - this.getHeight()) / 2);
        this.setSize(581, 321);
        setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);        
        {
        JLabel label = new JLabel();
        label.setName("Name1");
    	label.setText("Name1");
    	label.setBounds(37, 190, 100, 25);
        contentPane.add(label);
        } 
        {
        jTabbedPane1 = new JTabbedPane();
        jTabbedPane1.setName("jTabbedPane1");
        contentPane.add(jTabbedPane1);
        jTabbedPane1.setBounds(222, 86, 291, 151);
        	{
        	jPanel1 = new JPanel();
        	GroupLayout jPanel1Layout = new GroupLayout((JComponent)jPanel1);
        	jPanel1.setLayout(jPanel1Layout);
        	jTabbedPane1.addTab("jPanel1", null, jPanel1, null);
        		{
        		JLabel label = new JLabel();
        		jPanel1.add(label);
        		label.setName("Name2");
        		label.setText("Name2");
        		label.setBounds(42, 47, 84, 59);
        		//contentPane.add(label);      // <<-- Das ist falsch !?!
        		}	
        	}
        	{
        		jPanel2 = new JPanel();
        		jTabbedPane1.addTab("jPanel2", null, jPanel2, null);
        	}
        	{
        		jPanel3 = new JPanel();
        		jTabbedPane1.addTab("jPanel3", null, jPanel3, null);
        	}
        }

        this.setVisible(true);
    }

Wenn ich in Zeile 34 das label adde, dann ist es nicht mehr auf dem JPanel zu sehen, sondern auf dem untersten Pane und wird auch nicht mehr beschriftet, obwohl es im contentpane vorhanden ist.
Wo ist mein Denkfehler ?