Problem mit Tabulatoren bei Zeilen zählen einer Textdatei

[freakdran]

Hallo,
ich habe ein Problem beim zählen von Zeilen aus einer (Text-)Datei.
In der Aufgabenstellung wird gesagt, dass eine (Text-)Datei eingelesen werden soll und darauf die Zeilen gezählt und ausgegeben werden sollen. Dabei gilt:

Eine Zeile ausschließlich aus Leerzeichen und Tabulatorzeichen gilt nicht als Zeile.

Wie weit ich bisher bin:

String dateiname = readLine();
		int zaehler = 0;


		if(isFilePresent(dateiname)) {

			Object datei = openInputFile(dateiname);

			while(!isEndOfInputFile(datei)) {
				String zeile = readLine(datei);
				if(strLen(zeile) != 0 && !strEqual(zeile, "\t")) {
					zaehler++;
					println(zeile);
				}
			}
			closeInputFile(datei);
		} else
			throw new GDIException("Fehler: Datei nicht vorhanden.");

		println("Anzahl der Zeilen mit Text: " + zaehler);

Ich bin bisher so weit, dass ich leere Zeilen durch die Überprüfung der Länge und ein Tabulatorzeichen ignoriere. Einzelne Leerzeichen könnte man ja auch einfach mit

!strEqual(zeile, " ");

ignorieren. Genauso wie man sowohl Tabulatorzeichen und Leerzeichen einfach mit

String.replace(Zeichen, "")

ersetzen könnte, was wir jedoch (da es bisher weder in der Vorlesung behandelt wurde, noch in unsrer Hilfsbibliothek steht) nicht benutzen dürfen.

Deshalb bin ich ziemlich ratlos wie ich nun mehrere Leer-/Tabulatorzeichen in einer Zeile ignorieren kann.

Wenn ich die Testdatei (Anhang) einlese gibt mir die Konsole

1 (1)
2 (2)
3 (3)
5 (4)
7 (5)
10 (6)

12 (7)

14 (9)
Anzahl der Zeilen mit Text: 10

aus, da Zeile 4, 8, 9 leere Zeilen sind, Zeile 6 ein Tabulator ist, Zeile 11 zwei Tabulatoren sind und Zeile 13 ein Leerzeichen ist.

 

[arilou]

Sieh' dir mal java.lang.String.trim() an...

 

[freakdran]

Hatte ich auch überlegt an Tabs bzw " " zu trennen und dann einfach wieder auf Länge zu prüfen, aber da gab es das letzte mal schon Probleme, da es (bisher) noch nicht behandelt wurde und wir es mit einem viel umständlicheren Weg machen sollten.. das heißt ich müsste den String so lange auf das gesuchte Char, also Leerzeichen oder Tabulator, untersuchen bis es kommt und ihn dann trennen..

 

[Flown]

Was hat deine Hilfsbibliothek für Methoden?

 

[Joose]

Was machen denn deine Methode "strLen" bzw. "strEqual"?
Warum gehst du nicht einfach jede Zeile Zeichen für Zeichen durch?

Pseudocode:

while(!isEndOfInputFile(datei)) {
	zeile einlesen
	if(zeilenlänge > 0){
		for( i = 0 ; wenn i < zeilenlänge; i++) {
			if( zeichen kein leerzeichen oder tabulator) {
				// dann ist zeile nicht leer
				zaehler++;
				// zeile ausgeben
				sysout(zeile)
				break; // schleife beenden
			}
		}
	}
}

 

[freakdran]

Ist im Anhang, es geht ja aber auch nicht drum ob es in der Hilfsbibliothek ist oder nicht, sondern eher drum ob es in der Vorlesung bisher behandelt wurde.

Was machen denn deine Methode "strLen" bzw. "strEqual"?
Warum gehst du nicht einfach jede Zeile Zeichen für Zeichen durch?

Pseudocode:

while(!isEndOfInputFile(datei)) {
	zeile einlesen
	if(zeilenlänge > 0){
		for( i = 0 ; wenn i < zeilenlänge; i++) {
			if( zeichen kein leerzeichen oder tabulator) {
				// dann ist zeile nicht leer
				zaehler++;
				// zeile ausgeben
				sysout(zeile)
				break; // schleife beenden
			}
		}
	}
}

strLen und strEqual sind in der Hilfsbib, strLen gibt die Länge des Strings udn strEqual prüft ob der String == String ist.

Aber danke für den Code, er ginge bestimmt, aber gäbe es nicht noch was kürzeres?

 

[Flown]

Anders gefragt, was habt ihr bis jetzt in der Vorlesung benutzt/gelernt?

 

[freakdran]

Bisher was trennen und Zeichen weglassen angeht keine Funktion die das übernimmt, also kein trim, split, replace etc.. (damit wäre ich schon fertig)
Und sonst brauche ich hier denke ich nichts außer if, for und while. Oder liege ich da falsch?

 

[Flown]

Aber aus String an der Stelle i einen Character auslesen und vergleichen schon?

 

[freakdran]

Ja, deshalb sag ich ja, dass das von Joose funktioniert, jedoch hänge ich jetzt dran was die Char bei einem Tabulator ist, bzw mit was ich sie vergleiche.. ' ' denke ich ja nicht und '\t' funktioniert nicht, 'HT' will er mir nicht nehmen also bin ich wieder ratlos..

Edit: Ok.. auf einmal will doch ' ' und ' ' nehmen... ich verstehs manchmal nicht..

 

[Joose]

Aber danke für den Code, er ginge bestimmt, aber gäbe es nicht noch was kürzeres?

Wie du selbst schon gesagt hast: Du darfst das eine oder andere nicht verwenden.
Daher solltest du die Frage "Geht das nicht kürzer" hinten anstellen und einfach mal die Aufgabe lösen.
Also Zusatz kannst du eben noch die optimierte Version von dir dazugeben.

 

[freakdran]

Ja, es funktioniert jetzt soweit also bedank ich mich herzlich bei allen die geholfen haben.
Und es sieht doch nicht so schlimm aus wie ich befürchtet hatte.

 

[Joose]

Kein Problem!
Wie oben schon gesagt: Zuerst einfach mal die Aufgabe lösen, Optimierungen am Code kann man dann immer noch einfügen

 

[Flown]

Poste doch mal deinen Code, dass andere auch etwas von deiner Lösung haben!

 

[freakdran]

int anzahl = 0;		

		if(isFilePresent(filename)) {	
			Object datei = openInputFile(filename);
			while(!isEndOfInputFile(datei)) {	
				String zeile = readLine(datei);		
				if(strLen(zeile) != 0) {	//wenn die Länge der Zeile == 0 wäre, wäre sie leer, also würde nichts passieren
					for(int zeichen = 0; zeichen < strLen(zeile); zeichen++) {		
						if(strCharAt(zeile, zeichen) != ' ' && strCharAt(zeile, zeichen) != '	') {	
							if(strCharAt(zeile, zeichen) != '/') { 
								anzahl++;
								break;		
							} else
								break;
						}
					}
				}
			}
		} else
			throw new GDIException("Der eingegebene Dateipfad ist ungültig!");
		return anzahl;
	}

Hab noch gesehen, dass Zeilen die nur Kommentare enthalten auch nicht gezählt werden, deshalb noch das andere if. Und es ist eben mit Hilfsbibliothek geschrieben weshalb nicht alles zu 100% ohne sie funktioniert.

 

[Joose]

							if(strCharAt(zeile, zeichen) != '/') { 
								anzahl++;
								break;		
							} else
								break;

[ot]
Mach das "break" nach dem inneren if
Weil du es so oder so machen musst -> spart eine Zeile Code

Und speicher dir am Anfang deiner Schleife 1x dein Character ab, dann musst man nicht 3x den Character an Position X suchen.
[/ot]

 

[freakdran]

[ot]
Mach das "break" nach dem inneren if
Weil du es so oder so machen musst -> spart eine Zeile Code

Und speicher dir am Anfang deiner Schleife 1x dein Character ab, dann musst man nicht 3x den Character an Position X suchen.
[/ot]

Zu dem char abspeichern, stimmt, hab ich jetzt auch gemacht, aber meinst du es so?

if(zeichen != ' ' && zeichen != '	') {
     if(zeichen != '/')  //Zeilen die mit / anfangen sind nur Kommentar, werden also nicht berücksichtigt
          anzahl++;
     break;

Also die JUnit die wir zum prüfen bekommen haben läuft so, aber das spart dann sogar 2 Zeilen

 

[Joose]

Genau so