String zerteilen und in Datei schreiben

[Spezi]

Hi,

ich habe folgendes problem: mein programm schreib strings in eine textdatei. zur übersichtlichkeit mache ich einrückungen mit tabstops \t. leider sind manche strings viel zu groß, sodass es in meiner datei dann etwa so aussieht

z.B.

Text
......Kapitel
................wörterwörterwörterwörterwörterwörter
wörter
wörter
......Kapitel

ich habe versucht das zu umgehen und teile deswegen den string mit der funktion substring(). leider funktioniert das nicht so ganz. momentan bekomme ich die fehlermeldung "String index out of range: xx"

if (meinString>=53){
        out.write("\t\t\t--"+meinString.substring(0, 53));
        for (lengthcount = 53; lengthcount < meinString; lengthcount+=53 ){
                out.write("\t\t\t"+meinString.substring(lengthcount, lengthcount+53));
                out.newLine();
        }
} else {
         out.write("\t\t\t--"+meinString);
         out.newLine();
}

kann mir jemand einen tipp geben was falsch ist oder wie ich das problem eleganter lösen könnte?


grüße
spezi

 

[Lim_Dul]

if (meinString>=53){

Das kann nicht funktionieren. Wenn, dann:

if (meinString.length()>=53) {

 

[Spezi]

ok mein fehler, hab ich so im quellcode stehen, aber habs schlecht reinkopiert.

hier nochmal der korrekte code

if (meinString.length()>=53){
        out.write("\t\t\t--"+meinString.substring(0, 53));
        for (lengthcount = 53; lengthcount < meinString.length; lengthcount+=53 ){
                out.write("\t\t\t"+meinString.substring(lengthcount, lengthcount+53));
                out.newLine();
        }
} else {
         out.write("\t\t\t--"+meinString);
         out.newLine();
}

 

[Mario_H]

Wenn dein String z.B. 54 Zeichen lang ist,
versucht dein Programm substring(53, 106) zu machen,

das geht natürlich schief.

Du musst überprüfen, ob wirklich noch 53 Zeichen übrig sind, sonst meinString.lenght für das Ende einsetzten

--> substring(53, 54)

 

[Spezi]

danke der tipp war echt gut...

leider habe ich immer noch ein problem, und zwar dass alles richtig eingerückt wird bis auf die zweite zeile:

text
.....kapitel
..............wörterwörterwörterwörter
wörter
..............wörterwörterwörterwörter
..............wörterwörterwörterwörter
.....kapitel

hier nochmal der neue code

if (meinString.length()>=53){
     out.write("\t\t\t--"+meinString.substring(0, 53));
     for (lengthcount = 53; lengthcount < meinString.length(); lengthcount+=53 ){
          if ((lengthcount+53)<=meinString.length()){
               out.write("\t\t\t"+meinString.substring(lengthcount, lengthcount+53));
               out.newLine();
          }else{
          out.write("\t\t\t"+meinString.substring(lengthcount, meinString.length()));
          out.newLine();
}

 

[André Uhres]

//package arraydemo; 
/* 
* StringTeilen.java 
*/ 
import java.util.*; 
public class StringTeilen { 
    /*Diese Methode teilt einen String in Blöcke und gibt eine Liste der Blöcke zurück. 
     (Strukturierte Programmierung: ist am übersichtlichsten!)*/ 
    public static List<String> teilen(final String str, final int blocklength) { 
        if(blocklength <= 0) 
            throw new IllegalArgumentException(); 
        char[] array = str.toCharArray();               //String als char[] 
        List<String> teile = new ArrayList<String>();   //in diese Liste kommen die Teile 
        StringBuilder block = new StringBuilder();      //ein Teilblock 
        for (int i = 0; i < array.length; i=i+blocklength) { 
            block.delete(0, block.length());            //clear block 
            for (int j = i; j < array.length && j < i+blocklength; j++) 
                block.append(array[j]);                 //Teilblock füllen 
            teile.add(block.toString());                //die Teile in der Liste sammeln 
        } 
        return teile; 
    } 
    public static void main(String[] args) { 
        for(String block:StringTeilen.teilen("1234567890123", 5))System.out.println(block); 
    } 
}

 

[Mario_H]

Na, würd sagen, dass das praktisch der gleiche Code ist, nur dass die eine Variante substring und die andere Variante ein Char-Array verwendet.

Aber aus dem Problem mit der 2. Zeile werd ich nicht schlau.
Hast du andere Fehlerquellen schon ausgeschlossen?

 

[Spezi]

also ich habe mal beim notepad zeilenumbruch deaktiviert...
das hat insofern geholfen, dass die meisten 2. zeilen eingerückt sind, aber dennoch gibt es zeilen bei denen es nicht passiert ist ???:L sehr seltsam...

wie funktioniert denn dann die ausgabe der teile? lässt sich dass über eine for schleife machen?

 

[Mario_H]

p.s. Lass doch mal im Einzelschritt laufen und schau, was die 2. von den anderen Zeilen unterscheidet.
Führt meistens zum Erfolg...

 

[Spezi]

die lösung mit der methode wäre für mich aber geeigneter.
da ich das mit mehreren strings machen muss habe ich das auch mehrmals im code stehen, wollte erstmal testen wie man das machen kann... jetzt würde mich interessieren wie sich mit der methode die teile (

return teile;

ausgeben lassen. hab leider kaum mit arrays gearbeitet...

 

[André Uhres]

d.. jetzt würde mich interessieren wie sich mit der methode die teile (

return teile;

ausgeben lassen. hab leider kaum mit arrays gearbeitet...

hab den Code oben vervollständigt

 

[Spezi]

danke.

zum punkt mir der 2.zeile habe ich herausgefunden woran es lag:

if (meinString.length()>=53){
     out.write("\t\t\t--"+meinString.substring(0, 53));
     out.newLine(); //hier hat ein newLine gefehlt...
     for (lengthcount = 53; lengthcount < meinString.length(); lengthcount+=53 ){
          if ((lengthcount+53)<=meinString.length()){
               out.write("\t\t\t"+meinString.substring(lengthcount, lengthcount+53));
               out.newLine();
          }else{
          out.write("\t\t\t"+meinString.substring(lengthcount, meinString.length()));
          out.newLine();

 

[Mario_H]

Ja, im nachhinein offensichtlich...

So, jetzt musst du dich für eine Variante entscheiden.
Du kannst deinen Code natürlich auch in eine Methode packen, falls das, wie oben genannt, das ausschlagebende Kriterium ist.

 

[Spezi]

naja, ich würde mir eine menge zeilen sparen und außerdem alles übersichtlicher machen, deswegen werd ich das wohl in eine methode packen

 

[Gast]

Ja auf jeden Fall.
Man sollte sich in seinem Code nie wiederholen!

Wollte nur sagen, dass du deinen Code natülich auch in eine Methode packen kannst, und nicht den anderen Code verwenden musst, nur weil der schon in ner Methode drin ist...


p.s. Wenn substring nicht genau so arbeitet wie das mit dem Char-Array, vermute ich dass substring doch bisschen schneller ist...

 

[Mario_H]

Oh, der Gast bin ich.

[Bin eigentlich auch registriert, aber hab mein Passwort grad nicht zur Hand, muss zu Hause mal nachschauen...]

 

[Gast]

Da du Länge des Textes nicht vorhersagen kannst eignet sich Rekursion auch ganz gut. Am besten speicherst du dann die Substrings in einer Liste. Übergib deine Textzeile der Methode cutText und du erhälst eine Liste mit Teilstrings zurück. Noch variabler bist zu, wenn du die Anzahl der Tabstopps als zusätzlichen Parameter übergibst. Dann kannst du die Methode für alle Zeilen verwenden:

private List cutText(String text) {
        cutText(text, new ArrayList());
}

private List cutText(String text, List values) {
       if (text.length() >= 53) {
	values.add(text.substring(0, 53));
                // statt \t\t\t auch variable Substring durch zus.Param.
                String newText = "\t\t\t"+text.substring(54);
 	return cutText(newText, values);
       }
       else {
	values.add(text);
	return values;
       }
}

 

[Spezi]

habe nur ein problem... ich packe meinen code eine methode

private void StringTeilen() {
if (meinString.length()>=53){
     out.write("\t\t\t--"+meinString.substring(0, 53));
     out.newLine(); //hier hat ein newLine gefehlt...
     for (lengthcount = 53; lengthcount < meinString.length(); lengthcount+=53 ){
          if ((lengthcount+53)<=meinString.length()){
               out.write("\t\t\t"+meinString.substring(lengthcount, lengthcount+53));
               out.newLine();
          }else{
          out.write("\t\t\t"+meinString.substring(lengthcount, meinString.length()));
          out.newLine();

}

allerdings verwende ich darin out.write was allerdings in einer anderen methode steht, genauso wie meinString... somit kann er in der neuen methode "out" und "meinString" nicht auflösen...

 

[Mario_H]

Übergib den String und das out-Objekt als Parameter:

private void stringTeilen(String meinString, <Typ> out)
{
...
}

p.s. Der andere Gast bin ich nicht..
Ist zwar ne gute die er da vorschlägt, aber du bist ja noch am lernen und willst dass dein Code funktioniert und nicht einfach was abschreibben, oder?

Aber mal genau anschauen schadet sicher nicht.

 

[Spezi]

natürlich will ich lernen, deswegen lag mir auch soviel daran, dass mein eigener code funktioniert.

für mich ist der code des anderen gasts auch sehr einleuchtend werde trotzdem meinen code für das programm verwenden

 

[Mario_H]

Das ist die richtige Einstellung.

Klappt's jetzt?

 

[Spezi]

also ich habe das mal versucht, aber leider hat es nicht so geklappt.

hat doch geklappt, ich weiß nur nicht warum, dann ich habe einfach mal "public StringTeilen();" gelöscht und seither klappts...

public StringTeilen(); //multiple markers?

private void StringTeilen(Ergebnis e, BufferedWriter out){} 

StringTeilen(e, out);

 

[Mario_H]

class WieAuchImmer
{
 void SoNeMethode()
 {
   /* ganz viel Code (vermutlich) */

  //--> Hier stand mal der Code den du ganz am anfang gepostet hast.
  stringTeilen(meinString, out);

   /* noch mehr Code (vermutlich) */
 }

 private void stringTeilen(String meinString, BufferedWriter out)
 {
  //hier steht dein Code, soweit wie du ihn weiterentwickelt hast, dass er funktioniert
 } 
}

 

[Spezi]

warum muss ich

public StringTeilen();

nicht angeben??

 

[Mario_H]

Was soll das sein?

Der Methodenaufruf?
Das ist stringTeilen(meinString, out);

oder die Methodendeklaration?
Das ist private void stringTeilen(String meinString, BufferedWriter out)

 

[Mario_H]

Denk dir bitte für:

class WieAuchImmer 
{ 
 //meinetwegen noch andere Methoden...

void SoNeMethode()

genau das, was da stehen muss, dass weiter unten
//--> Hier stand mal der Code den du ganz am anfang gepostet hast.

passt.

 

[Spezi]

ich habe das programm bekommen und soll es um einige funktionen erweitern. und so hat derjenige das mit den methoden gemacht:

public class irgendwie extends JFrame {
    
    private int bla = 0;
    ..
    ..

    public irgendwie() {
       methode();
       nocheine();
}

private void methode() {};

methodenaufruf();

 

[Guest]

ich habe das programm bekommen und soll es um einige funktionen erweitern. und so hat derjenige das mit den methoden gemacht:

public class irgendwie extends JFrame {
    
    private int bla = 0;
    ..
    ..

    public irgendwie() {
       methode(); //<-- das ist der Aufruf für [*]
       nocheine();
}

private void methode() {};  //<-- [*] hier.

methodenaufruf(); //<-- was das ist, ist hier nicht zu erkennen.

Wenn du Wörter in "irgendwie" änderst, dann bitte so, dass der Code trotzdem so funltionieren würde.
--> Es ist nirgens eine Methode methodenaufruf() deklariert,
bsp.: private void methodenaufruf() { //bla bla bla }