Ausgabe überschreiben

[eiligerwind]

Hallo Zusammen,
ich habe folgende Methode: Diese gibt mir aus, welche Wörter ich zuvor in die Datei "woerter.bin" geschrieben habe. Diese Methode ist Teil in einem Editor in dem ich weitere Wörter hinzufügen kann. Die Anzeige klappt auch soweit.
Allerdings wird beim nochmaligen Aufruf der Methode (erfolgt im Editor über einen Button) die Anzeige einfach fortgeschrieben. Ich würde gerne die alte Ausgabe (hier über Labels) bei erneuten Aufruf überschreiben. Ich habe irgendwie keinen Ansatzpunkt...mit Remove (am Anfang der Methode komme ich nicht zum gewünschten Ziel... und auch meine Versuche den Filepointer beim erneuten Aufruf einfach weiterzuschieben...klappt nicht)

hoffe auf einen Denkanstoß

[CODE lang="java" title="Methode zum Anzeigen"]private void datenLesen() {

try {
//eine neue Instanz für den Lesezugriff
datei = new RandomAccessFile("woerter.bin","r");

//den Zeiger auf das 4. Byte setzen, da erst am dem 5. Byte gelesen
//werden soll, weil in den Bytes 1 - 4 die Anzahl der Wörter im Typen Int
//abgespeichert wird
datei.seek(4);

//Mit der While-Schleife wird die Ausgabe gesteuert
//solange die aktuelle Position kleiner ist als die
//länge der Datei wird der Index ausgelesen
//und das Wort ausgegeben
while(datei.getFilePointer() < datei.length()) {

add(new JLabel(datei.readInt() + ". Wort: "));
add(new JLabel(datei.readUTF()));

//das Fenster neu packen
pack();

}
datei.close();
}

catch (IOException e) {

JOptionPane.showMessageDialog(this,"Beim Laden ist ein Problem aufgetreten");
}
[/CODE]

 

[kneitzel]

Du kannst natürlich alle JLabel entfernen. Dazu kannst Du diese entweder aus dem übergeordneten Container heraus suchen (getComponents(), dann für jedes Element prüfen, ob es ein JLabel ist um dann remove() aufzurufen.) oder die JLabel, die Du erzeugt hast, auch noch separat merken und dann die Liste durchgehen und löschen.

Das lässt dich natürlich noch weiter ausbauen. Man könnte prüfen, welche Label noch vorhanden sein müssen, ggf. paar Label löschen und neue hinzufügen ...

 

[eiligerwind]

Hallo kneitzel,

vielen Dank für die Ideen. Ich werde es mal mit getComponents versuchen und ein If Schleife zur Prüfung auf JLabel nutzen. Wäre der
Syntax:

if (Components==Jlabel) {
do...
}

in etwa richtig, oder muss ich das anders angehen?

vielen Dank und VG
eiligerwind

 

[kneitzel]

Nein, Du bekommst ja ein Array von Component zurück. Jedes Element musst Du dann prüfen mittels instanceof um zu schauen, ob es vom Typ JLabel ist.

 

[eiligerwind]

also

if(Components instanceof JLabel) {
do somthing...
}

 

[kneitzel]

Component[] components = getComponents();
for (Component component: components) {
    id (component instanceof JLabel) {
        JLabel label = (JLabel) component;
        // So something with label
    }
}

 

[eiligerwind]

Hallo Kneitzel,

vielen Dank für den Codeschnipsel. So ganz verstehe ich den nicht...der schleifenparameter for was drückt der aus? Und eine weitere Frage, bezieht sich die Methode remove dann auf die Variable label, oder auf Components?

merke gerade dass noch einiges an Grundlagen bei mir fehlt *hüst*

 

[kneitzel]

Also die Erklärung des Codes:
getComponents gibt ein Array mit allen Component aus, die in dem Container enthalten sind.
Component ist die awt Klasse, auf die alle Controls zurück gehen. Container ist die awt Klasse, auf die alle Controls zurück gehen, die andere Controls beinhalten können.
Das sind somit Super-Klassen von den anderen Klassen wie z.B. JFrame, JButton, JLink, ....

Die for Schleife ist eine sogenannte "for each" Schleife. Das, was ich das geschrieben habe, könnte man auch als traditionelle for Scheife schreiben:

for (int index = 0; index < components.length; index++) {
    Component component = components[i];
    // ...
}

Dann kommt ein Tippfehler - das sollte heißen:
if (component instanceof JLabel) {
Hier wird geprüft, ob in component ein JLabel steckt. Es kann irgend ein Component drin stecken. Also evtl. hast Du ja neben JLabel noch paar JButtons und so hinzugefügt. Die sind dann im Array auch enthalten. Da wäre aber dann diese Überprüfung negativ.

Wenn es ein JLabel ist, dann können wir es auch in einer JLabel Variable speichern. Dies ist optional. Das kann hilfreich sein, je nachdem, was Du machen willst. Wenn Dir die Möglichkeiten ausreichen, die Component bietet, dann brauchst du das nicht. Also nur zum Entfernen brauchst Du diese Variable nicht.

 

[eiligerwind]

vielen Dank für die Ausführung. Die traditionelle "for Schleife" kommt mir bekannt vor ...

wenn die Speicherung nicht notwendig ist, gehe ich davon aus, dass remove(JLabel) dann ausreichend wäre?... das hatte ich allerdings versucht ohne Erfolg. Der Code ist zwar weiterhin lauffähig (keine exceptions) aber überschrieben wird die erste Ausgabe der Labels mit den zugehörigen Wörtern dennoch nicht. Möglicherweise habe ich es aber auch falsch eingebaut...

private void datenLesen() {
        
        try {
            
            Component[] components = getComponents();
            for (Component component: components) {
                if (component instanceof JLabel) {
                    JLabel label = (JLabel) component;
                    // So something with label
                    remove(label);
            }
            
            
            
            
            //eine neue Instanz für den Lesezugriff
            datei = new RandomAccessFile("woerter.bin","r");
            
            //den Zeiger auf das 4. Byte setzen, da erst am dem 5. Byte gelesen
            //werden soll, weil in den Bytes 1 - 4 die Anzahl der Wörter im Typen Int
            //abgespeichert wird
            datei.seek(4);
            
        
            //Mit der While-Schleife wird die Ausgabe gesteuert
            //solange die aktuelle Position kleiner ist als die
            //länge der Datei wird der Index ausgelesen
            //und das Wort ausgegeben
            while(datei.getFilePointer() < datei.length()) {
                
                
                add(new JLabel(datei.readInt() + ". Wort: "));
                add(new JLabel(datei.readUTF()));
                
                
                //das Fenster neu packen
                pack();
                
    
                
            }
            datei.close();
        }
        }
        catch (IOException e) {
            
            JOptionPane.showMessageDialog(this,"Beim Laden ist ein Problem aufgetreten");
        }

 

[kneitzel]

Also vom Code her sieht das eigentlich so aus, wie ich mir das gedacht habe. Nur für das Entfernen hätte es die Variable label nicht gebraucht, aber genau so sollte das Label vom übergeordneten Fenster entfernt werden.

Die remove Methode hat in der Dokumentation noch " If the container has already been displayed, the hierarchy must be validated thereafter in order to reflect the changes."

Du rufst aber pack() auf im Rahmen des Hinzufügens. Das sollte es eigentlich bewerkstelligen.

Was man alles testen könnte:
a) Wurden Elemente entfernt? Dazu einfach vor und nach der Schleife einmal getComponentCount() aufrufen / ausgeben.
b) Nach der Schleife einmal ein revalidate() aufrufen.

 

[eiligerwind]

Hmmm... Die Methode getComponentCount() gibt sowohl vor als auch nach der Schleife den Wert 1 aus.

auch das explizite revalidate() nach der Schleife führt nicht zu einer Änderung

 

[eiligerwind]

[CODE lang="java" title="geänderter Code"]private void datenLesen() {

try {

Component[] components = getComponents();
getComponentCount();
System.out.println("Anzahl der Labels:" + getComponentCount());
for (Component component: components) {
if (component instanceof JLabel) {
JLabel label = (JLabel) component;
// So something with label
remove(label);
revalidate();
}
getComponentCount();
System.out.println("Anzahl der Labels:" + getComponentCount());



//eine neue Instanz für den Lesezugriff
datei = new RandomAccessFile("woerter.bin","r");

//den Zeiger auf das 4. Byte setzen, da erst am dem 5. Byte gelesen
//werden soll, weil in den Bytes 1 - 4 die Anzahl der Wörter im Typen Int
//abgespeichert wird
datei.seek(4);


//Mit der While-Schleife wird die Ausgabe gesteuert
//solange die aktuelle Position kleiner ist als die
//länge der Datei wird der Index ausgelesen
//und das Wort ausgegeben
while(datei.getFilePointer() < datei.length()) {


add(new JLabel(datei.readInt() + ". Wort: "));
add(new JLabel(datei.readUTF()));


//das Fenster neu packen
pack();



}
datei.close();
}
}
catch (IOException e) {

JOptionPane.showMessageDialog(this,"Beim Laden ist ein Problem aufgetreten");
}

}[/CODE]

 

[kneitzel]

Hmm, das ist nach dem du schon einmal die Datei eingelesen hast und du schon viele Label hinzugefügt hast? Das add ist das add von JFrame oder was rufst du da auf? Zeig ggf. mal die ganze Klasse ...

Frage mich gerade wo ich meinen Denkfehler habe - muss sonst mal eine kleine Test-Anwendung schreiben, die es nachstellt ...

 

[eiligerwind]

JA das ist nachdem ich schon 17 Wörter eingelesen habe...

und ja die Klasse ist aus JFrame. Wobei noch eine unterklasse des Listeners für die Button existiert. Der gesamte Code ist folgender:


public class WortListe extends JFrame {

/**
*
*/
private static final long serialVersionUID = 1L;
private static final JLabel Components = null;
//für die Anzahl der Wörter
private int anzahlWoerter;
//für die Eingabe neuer Wörter
private JTextField eingabeWoerter;
//für den Button einlesen und die Wortliste anzeigen
private JButton einlesen,anzeigen;
//eine Instanzvariable vom Typ RandomAccessFile
private RandomAccessFile datei;
//eine Instanzvariable vom Typ MeineWoerter
private MeineWoerter woerterTemp;


//das JLabel für die Anzahl der Wörter
private JLabel woerterAnzahlZeigen;

//eine innereKlasse für die Wörter
class MeineWoerter {
//die Wörter des Hang-Man Spiels
private String woerter;

MeineWoerter(String wort) {

//das neue Wort setzen
woerter = wort;
//den Index des neuen Wort setzen
datenSchreiben();
}

//die Methode für das Schreiben des Wortes in die Datei
private void datenSchreiben() {

try {

//eine neue Instanz für das Schreiben der Datei
datei = new RandomAccessFile("woerter.bin","rw");

//die Anzahl der Wörter erhöhen
//die Anzahl der Wörter steht zugleich für den
//Index des einzelnen Wortes
anzahlWoerter++;

//die Anzahl der Wörter wird in den ersten 4 Bytes
//als Int - Wert gespeichert
datei.writeInt(anzahlWoerter);

//den Zeiger ans Ende der Datei setzen
datei.seek(datei.length());

//jedes Wort erhält zuerst einen Index
//anschließend wird das Wort angehängt.

//den Index am Ende an die Datei anhängen
datei.writeInt(anzahlWoerter);
//das Wort nach dem Index an die Datei Anhängen
datei.writeUTF(woerter);
//datei wieder schließen.
datei.close();

}

catch(IOException e) {
/////////WIE REFERENZ zum ERZEUGEN??
JOptionPane.showMessageDialog(null,"Beim Schreiben ist ein Problem aufgetreten");
}
}
}


//eine innere Klasse für den ActionListener
class MeinListener implements ActionListener{

@Override
public void actionPerformed(ActionEvent e) {
// TODO Auto-generated method stub

//das erstellt Wort speichern
if (e.getActionCommand().equals("lesen"))
werteEinlesen();

//die Liste der Wörter anzeigen
if (e.getActionCommand().equals("anzeigen"))
datenLesen();

}


}


public WortListe(String titel) {

//den Titel erstellen
super(titel);

//das Layout setzen
this.setLayout(new GridLayout(0,2));

//die Schaltfläche zum Einlesen der Dateien erstellen
einlesen = new JButton("Einlesen");
einlesen.setActionCommand("lesen");

//die Schaltfläche zum anzeigen der Wortliste erstellen
anzeigen = new JButton("Wörter anzeigen");
anzeigen.setActionCommand("anzeigen");

//die Schaltflächen mit dem Listener verbinden
MeinListener listener = new MeinListener();
einlesen.addActionListener(listener);
anzeigen.addActionListener(listener);


//ein Label für die Wörter erstellen
add(new JLabel("Wort eingeben: "));

//das Text Feld für die Wörter erstellen
eingabeWoerter = new JTextField();
//und einfügen
add(eingabeWoerter);

//ein Label für die eingegeben Wörter erstellen
add(new JLabel("Anzahl vorhanden Wörter: "));

//die Anzahl der Wörter welche bereits gespeichert wurden,
//ermitteln
try {
//eine neue Instanz für das Lesen der Datei
datei = new RandomAccessFile("woerter.bin","r");
//die Anzahl der Wörter ist in den ersten 4 Bytes gespeichert
anzahlWoerter = datei.readInt();

datei.close();

}

catch (IOException e) {
//falls die Datei noch nicht erstellt wurde, die Anzahl
//der Wörter auf 0 setzen.
anzahlWoerter = 0;
}

//die Anzahl der Wörter in einem JLabel anzeigen
add(woerterAnzahlZeigen = new JLabel(Integer.toString(anzahlWoerter)));
//den Button einlesen erstellen
add(einlesen);
//den Button anzeigen erstellen
add(anzeigen);

//das Fenster packen
pack();
//das Standardverhalten festlegen
setDefaultCloseOperation(JFrame.DISPOSE_ON_CLOSE);
//das Fenster anzeigen
setVisible(true);
}


private void werteEinlesen() {
//wenn kein Wort eingeben wurde, dann die Methode verlassen
if(eingabeWoerter.getText().isEmpty())
return;

//ein neues Wort erstellen
new MeineWoerter(eingabeWoerter.getText());


System.out.println("Die Anzahl der Wörter:" + anzahlWoerter);
//den Text des JLabels neu setzen
woerterAnzahlZeigen.setText(Integer.toString(anzahlWoerter));


}





private void datenLesen() {

try {

Component[] components = getComponents();
getComponentCount();
System.out.println("Anzahl der Labels:" + getComponentCount());
for (Component component: components) {
if (component instanceof JLabel) {
JLabel label = (JLabel) component;
// So something with label
remove(label);
revalidate();
}
getComponentCount();
System.out.println("Anzahl der Labels:" + getComponentCount());



//eine neue Instanz für den Lesezugriff
datei = new RandomAccessFile("woerter.bin","r");

//den Zeiger auf das 4. Byte setzen, da erst am dem 5. Byte gelesen
//werden soll, weil in den Bytes 1 - 4 die Anzahl der Wörter im Typen Int
//abgespeichert wird
datei.seek(4);


//Mit der While-Schleife wird die Ausgabe gesteuert
//solange die aktuelle Position kleiner ist als die
//länge der Datei wird der Index ausgelesen
//und das Wort ausgegeben
while(datei.getFilePointer() < datei.length()) {


add(new JLabel(datei.readInt() + ". Wort: "));
add(new JLabel(datei.readUTF()));


//das Fenster neu packen
pack();



}
datei.close();
}
}
catch (IOException e) {

JOptionPane.showMessageDialog(this,"Beim Laden ist ein Problem aufgetreten");
}

}

 

[eiligerwind]

hm...nachdem das irgendwie nicht klappt mit dem kompletten überschreiben, hätte ich noch folgende Idee...

mithilfe einer If Schleife zu ermitteln, ob der Aufruf der Methode zum ersten mal passiert, oder ein erneuter Aufruf erfolgt ist. Dafür würde ich einfach eine zusätzliche Variable nutzen, die ich beim Aufruf der Methode hochzaehle...

wenn nun ein erneuter Aufruf erfolgt, soll dann nur die zusätzlich eingegeben Wörter dazu gelesen werde. Hierfür müsste ich nach dem ersten Aufruf der Methode den FilePointer an das Ende der Datei setzen und mir diese Position merken (weitere Variable)...diese würde ich dann immer an die Datei übergeben.

Allerdings funktioniert dass auch nicht wie gewollt... hier mal mein Versuch

[CODE lang="java" title="Daten Lesen"] private long datenLesen() {

anzahlMethodenaufrufe++;
long tempDateiEnde = 0;

try {

//eine neue Instanz für den Lesezugriff
datei = new RandomAccessFile("woerter.bin","r");

//den Zeiger auf das 4. Byte setzen, da erst am dem 5. Byte gelesen
//werden soll, weil in den Bytes 1 - 4 die Anzahl der Wörter im Typen Int
//abgespeichert wird
if (anzahlMethodenaufrufe==1)
datei.seek(4);
else datei.seek(tempDateiEnde);



//Mit der While-Schleife wird die Ausgabe gesteuert
//solange die aktuelle Position kleiner ist als die
//länge der Datei wird der Index ausgelesen
//und das Wort ausgegeben
while(datei.getFilePointer() < datei.length()) {


add(new JLabel(datei.readInt() + ". Wort: "));
add(new JLabel(datei.readUTF()));

tempDateiEnde= datei.length();

//das Fenster neu packen
pack();


}
datei.close();
}

catch (IOException e) {

JOptionPane.showMessageDialog(this,"Beim Laden ist ein Problem aufgetreten");
}
return tempDateiEnde;

}[/CODE]

 

[eiligerwind]

Habe den Fehler gefunden...

durch das Initialisieren der Variable tempDateiEnde beim Aufruf der Methode wird nicht das gewünschte Ergebnis erreicht. Nun klappt es wie es sein soll. Das Anzeigen der Wortliste wird einfach fortgeschrieben, anstatt es nochmals komplett einzulesen.

danke auch an Kneitzel, auch wenn ich den Ansatz mit dem Löschen der Labels nicht zum Laufen gebracht habe ...

VG eiligerwind