JTextPane.getText() funktioniert nicht

[Extremefall]

Hallo,
ich habe eine ArrayList, die aus den Zeilen einer Textdatei besteht. Das Auslesen und alles klappt auch super, nur wenn ich das ganze in einem JTextPane ausgeben will, wird nur der letzte Wert ausgeben. Ich habe dann versucht, mit getText den vorherigen Text abzurufen und so immer zu erweitern, was nicht funktionierte.

try 
		    { 
		      RandomAccessFile stunden; 
		      RandomAccessFile fach;
		      stunden = new RandomAccessFile( "stunden.txt", "r" ); 
		      fach = new RandomAccessFile( "fach.txt", "r");
		      for ( String line; (line=stunden.readLine()) != null; ) {
		    ArrayList<String> fachlist = new ArrayList<String>();
		    fachlist.add(fach.readLine());
		      ArrayList<String> stundenlist = new ArrayList<String>();
		      stundenlist.add(line);
		      int counter =0; 
		      counter++;
		      for(int i = 0; i < stundenlist.size(); i++) {
				    	jTextPane1 = new JTextPane();
				    	if (i==0){
				    	jTextPane1.setText(stundenlist.get(i));
				    	}
				    	if (i!=0){
				    	jTextPane1.setText(jTextPane1.getText()+ stundenlist.get(i));
				    	}
		    	}
		      }

Es wird also nur der letzte Werte ausgeben. Wie kann man das Problem beheben?

 

[Suinos]

jTextPane1 = new JTextPane();

Du erzeugst in der Schleife bei jedem Durchgang ein neues Objekt, welches natürlich keinen Text hat.

 

[Extremefall]

Stimmt. Danke, ist mir gar nicht aufgefallen. Da gehört es nun mal wirklich nicht hin. Danke

Edit: Leider ist der Fehler noch nicht behoben. Worin könnte das Problem sonst wohl bestehen?

 

[jgh]

Ihr macht es allen leicher, wenn ihr ein KSKB postet!!!

Bei dir sollte es schon langen, die Zeile 15 vor Beginn der For-Schleife aufzurufen.

edit: to late

 

[Extremefall]

Leider funktioniert es immer noch nicht. Ich habe es nun am Anfang des Programms stehen, aber daran scheint es nicht zu liegen.

Hier noch mein Code dafür:

try 
		    { 
		      RandomAccessFile stunden; 
		      RandomAccessFile fach;
		      stunden = new RandomAccessFile( "stunden.txt", "r" ); 
		      fach = new RandomAccessFile( "fach.txt", "r");
		      for ( String line; (line=stunden.readLine()) != null; ) {
		    ArrayList<String> fachlist = new ArrayList<String>();
		    fachlist.add(fach.readLine());
		      ArrayList<String> stundenlist = new ArrayList<String>();
		      stundenlist.add(line);
		      int counter =0; 
		      counter++;
		      for(int i = 0; i < stundenlist.size(); i++) {
				    	if (i==0){
				    	jTextPane1.setText(stundenlist.get(i));
				    	}
				    	if (i!=0){
				    	jTextPane1.setText(jTextPane1.getText()+ stundenlist.get(i));
				    	}
		    	}
		      
		      }
		    }
		    catch ( FileNotFoundException e ) {
		      System.err.println( "Datei gibt’s nicht!" ); 
		    }
		    catch ( IOException e ) {
		      System.err.println( "Schreib-/Leseprobleme!" ); 
		    }

		   
			{
				jTextPane2 = new JTextPane();
				jTextPane2.setText("kkkkkkkkkkkkkkkkkkkkkkkkkkkkkkkkkkkkkkkkkkkkkkkkkkkkkkkkkkk");
			}

 

[Fr33z3m4n]

Hi,

das liegt daran, dass du in der schleife nach i=0 abfragst, bei jeder neuen Zeile ist dieses aber der fall, da du die schleife immer von neu beginnst.

Bsp.:

TextDatei enthält 2 Zeilen:
For(1 Zeile) {
for(StundenlisteAusZeile1) {
wenn erster Durchlauf, nimm Textpane und setzeText auf xx:
wenn nicht erster Durchlauf, nimm Textpane und füge den Text hinzu.
}
}

For(2 Zeile) {
for(StundenlisteAusZeile2) {
wenn erster Durchlauf, nimm Textpane und setzeText auf xx:
wenn nicht erster Durchlauf, nimm Textpane und füge den Text hinzu.
}
}

In der Textpane hast du nun also nur den Durchlauf von StundenlisteAusZeile2
Hoffe du hast verstanden, was ich damit meinte.

 

[jgh]

hört sich sinnig an.

Mit einem StringBuilder kannst du dir die Abfrage(ob i=0 ) schenken:

StringBuilder sb = new StringBuilder();
for(int i = 0; i < stundenlist.size(); i++) {
                      sb.append(stundenlist.get(i));
                }
jTextPane1.setText(sb.toString());

 

[Extremefall]

Hi,

das liegt daran, dass du in der schleife nach i=0 abfragst, bei jeder neuen Zeile ist dieses aber der fall, da du die schleife immer von neu beginnst.

Bsp.:

TextDatei enthält 2 Zeilen:
For(1 Zeile) {
for(StundenlisteAusZeile1) {
wenn erster Durchlauf, nimm Textpane und setzeText auf xx:
wenn nicht erster Durchlauf, nimm Textpane und füge den Text hinzu.
}
}

For(2 Zeile) {
for(StundenlisteAusZeile2) {
wenn erster Durchlauf, nimm Textpane und setzeText auf xx:
wenn nicht erster Durchlauf, nimm Textpane und füge den Text hinzu.
}
}

In der Textpane hast du nun also nur den Durchlauf von StundenlisteAusZeile2
Hoffe du hast verstanden, was ich damit meinte.

Ich kann dir gerade nicht folgen: Meinst du mit der Abfrage den Teil der For-Schleife. Wenn ja, dort wird ja nur i mit dem Startwert 0 festgelegt und bei jedem weiteren Durchlauf hochgezählt. Oder meinst du i==0? Ich weiß leider gerade nicht, was du damit genau meinst. Könntest du mir weiterhelfen? MFG

Edit: Der Tipp mit dem StringBuilder scheint nicht zu funktionieren, es bleibt beim letzten Wert.

 

[Suinos]

Anders gesagt:

Deine

stundenlist

enthält immer nur die aktuelle Linie.
Am Besten entfernst du die ganze zweite Schleife und setzt den Text einfach auf

alten Text + gelesene Linie

.

--

Deine Formatierung ist übrigens schrecklich.
Mit einer guten Darstellung hättest du den Logikfehler schnell gefunden.

 

[Fr33z3m4n]

OK, mal ein wenig ausholen ^^

Vor dieser schleife, hast du deine Textpane initialisiert, sie ist sogar noch leer.

for ( String line; (line=stunden.readLine()) != null; ) {
            ArrayList<String> fachlist = new ArrayList<String>();
            fachlist.add(fach.readLine());
              ArrayList<String> stundenlist = new ArrayList<String>();
              stundenlist.add(line);
              int counter =0; 
              counter++;
              for(int i = 0; i < stundenlist.size(); i++) {
                        if (i==0){
                        jTextPane1.setText(stundenlist.get(i));
                        }
                        if (i!=0){
                        jTextPane1.setText(jTextPane1.getText()+ stundenlist.get(i));
                        }
                }
              
              }

for ( String line; (line=stunden.readLine()) != null; ) {

wird so oft durchlaufen, wie Zeilen in deiner TXT vorhanden sind.

 for(int i = 0; i < stundenlist.size(); i++) {
                        if (i==0){
                        jTextPane1.setText(stundenlist.get(i));
                        }
                        if (i!=0){
                        jTextPane1.setText(jTextPane1.getText()+ stundenlist.get(i));
                        }
                }

Wir bleiben jetzt mal bei Zeile 1 deiner TXT.


HMMM MOM, ich hab mir deinen Code gerade nochmal genauer angeschaut, verschieb mal eine EndSchleife

try 
            { 
              RandomAccessFile stunden; 
              RandomAccessFile fach;
              stunden = new RandomAccessFile( "stunden.txt", "r" ); 
              fach = new RandomAccessFile( "fach.txt", "r");
              for ( String line; (line=stunden.readLine()) != null; ) {
            ArrayList<String> fachlist = new ArrayList<String>();
            fachlist.add(fach.readLine());
              ArrayList<String> stundenlist = new ArrayList<String>();
              stundenlist.add(line);
              int counter =0; 
              counter++;
}
              for(int i = 0; i < stundenlist.size(); i++) {
                        if (i==0){
                        jTextPane1.setText(stundenlist.get(i));
                        }
                        if (i!=0){
                        jTextPane1.setText(jTextPane1.getText()+ stundenlist.get(i));
                        }
                }
            }
            catch ( FileNotFoundException e ) {
              System.err.println( "Datei gibt’s nicht!" ); 
            }
            catch ( IOException e ) {
              System.err.println( "Schreib-/Leseprobleme!" ); 
            }
 
           
            {
                jTextPane2 = new JTextPane();
                jTextPane2.setText("kkkkkkkkkkkkkkkkkkkkkkkkkkkkkkkkkkkkkkkkkkkkkkkkkkkkkkkkkkk");
            }

 

[Fr33z3m4n]

Soo,

bevor hier noch alles undeutlicher wird ^^
Hatte ej gerade Eclipse auf

try {
			RandomAccessFile stunden;
			RandomAccessFile fach;
			stunden = new RandomAccessFile("stunden.txt", "r");
			fach = new RandomAccessFile("fach.txt", "r");
			ArrayList<String> fachlist = new ArrayList<String>();
			ArrayList<String> stundenlist = new ArrayList<String>();
			for (String line; (line = stunden.readLine()) != null;) {
				fachlist.add(fach.readLine());
				stundenlist.add(line);
				int counter = 0;
				counter++;
			}
			StringBuilder sb = new StringBuilder();
			for (int i = 0; i < stundenlist.size(); i++) {
				sb.append(stundenlist.get(i));
			}
			jTextPane1.setText(sb.toString());
		} catch (FileNotFoundException e) {
			System.err.println("Datei gibt’s nicht!");
		} catch (IOException e) {
			System.err.println("Schreib-/Leseprobleme!");
		}

		{
			jTextPane2 = new JTextPane();
			jTextPane2.setText("kkkkkkkkkkkkkkkkkkkkkkkkkkkkkkkkkkkkkkkkkkkkkkkkkkkkkkkkkkk");
		}

 

[Extremefall]

Bin gerade selbst auf den Logikfehler gekommen. Ich stand gerade echt auf dem Schlauch.

Habe es jetzt mit solch einer For Schleife gelöst:

for(int i = 0; i < 1; i++) {
jTextPane1.setText(jTextPane1.getText()+" "+ stundenlist.get(i));
}

 

[Fr33z3m4n]

ich würde dennoch das Initialisieren der ArrayLists außerhalb der Schleife machen, oder willst du immer eine neue Liste erstellen ?

 

[Extremefall]

Es ist doch außerhalb der Schleife oder wie meinst du es jetzt? Das deklarieren wird in der Schleife vorgenommen.

 

[Fr33z3m4n]

for ( String line; (line=stunden.readLine()) != null; ) {
ArrayList<String> fachlist = new ArrayList<String>();
fachlist.add(fach.readLine());
ArrayList<String> stundenlist = new ArrayList<String>();

Bei deinem Code ist das innerhalb dieser, und wird jedes mal neu erstellt.
In meinem Bsp. hab ich das außerhalb dieser verlagert, damit du auch später auf diese zurückgreifen kannst, ohne nur den letzten Eintrag zu haben.

 

[Extremefall]

Schon gut. Hat sich erledigt.