tabelle

[mermen]

ja, an split hab ich auch gedacht...aber wenn ich nun....

rang;name;richtung;....habe, soll er doch nicht nur einmal teilen....sondern die ganze zeile immer fortlaufend...und jeweils die "abschnitte" in variablen hinterlegen.....

wie du halt meintest mit der zuweisung des head!!

 

[The_S]

Wie schaut denn deine Datei aus?

 

[mermen]

also nun hab ich sie geändert :

rang;name;richtung;insgesamt;richtig;falsch;prozent
1;hans;deutsch->englisch;20;15;5;75,0
2;kurt;englisch->deutsch;40;20;20;50,0
.
.
.
.
usw...

 

[Leroy42]

sondern die ganze zeile immer fortlaufend...

Genau das macht split ja. Du kannst dann in einer Schleife einfach

daten[zeilennummer] = zeile.split(';') schreiben.

Das einzige Problem ist noch, vorher die Anzahl der Zeilen der Datei
zu bestimmen umd daten genügend groß zu machen.

Du kannst die von split erzeugten Stringarrays erst mal in einen Vector(ArrayList)
schreiben, und am Ende daraus einen Array erzeugen. Falls du dich mit Collections
allerdings überhaupt noch nicht auskennst, empfehle ich, bei deinen kleinen Datenmengen,
einfach die Datei zweimal einzulesen. Beim ersten Mal bestimmst du dann nur die
Anzahl der Zeilen, sodaß du daten richtig dimensionieren kannst.

String[][] daten = new String[anzahlZeilen][];

Beim zweiten Durchlaufs speicherst du dann die Daten wie Hobbit und ich beschrieben haben.

 

[The_S]

OK, Leroy42 hats dir schon beschrieben (@Leroy42, man sind wir ein Team ... :bae: ). Das einzige was mir noch auffällt ist eine Kleinigekeit an deinen Daten.

Du hast 7 Spalten im Kopf, aber 8 in den Daten (die letzte 0 ist zuviel). Wenn das absicht ist musste das natürlich dann später mitberücksichtigen. Ansonsten ausbessern .

 

[mermen]

da ich immer nur 10 zeilen der highscore anzeigen möchte, kann ich doch eine feste größe vorgeben....oder?

 

[mermen]

die letzte null gehört doch zu prozent...also 50,0 trennzeichen ist doch ein ";" ??

 

[The_S]

ja, dann brauchst du allerdings auch eine andere Schleife die nach 10 Zeilen (oder besser 11, da du den Kopf ja auch noch lesen musst) oder wenn das Ende der Datei erreicht ist aufhört zu lesen.

[edit] Achja richtig. Komma und Semikolon sehen sich so verdammt ähnlich

 

[Leroy42]

kann ich doch eine feste größe vorgeben....oder?

Eigentlich schon, aber das ist :bae:
(Außerdem sind ja nicht vom Spiel-Urknall an gleich 10 Highscores vorhanden)

Sei doch flexibel, und speichere die aktuelle Anzahl der Zeilen gleich mit in deiner
Datei. Dann ersparst du dir sowohl Collection-Gerödel als auch doppeltes Datei einlesen:

rang;name;richtung;insgesamt;richtig;falsch;prozent
10
1;hans;deutsch->englisch;20;15;5;75,0
2;kurt;englisch->deutsch;40;20;20;50,0
.
.
.
.
Edit: Und wieviele du dann anzeigst, bleibt deinem Programm überlassen. Evtl. auch dem User deines Programms.

 

[Leroy42]

Sorry, hab' noch ein paar Punkte vergessen, sind ja 10 highscores nach meiner Tabelle

....

Die Zeilenumbrüche poste ich mal nicht mit: Man muß ja nicht, ... nicht unbedingt

 

[mermen]

naja...ich werde die datei einfach mit 10 zeilen befüllen....also, muß nicht unbedingt flexibel sein...

nun hab ich ein problem mir den eingelesenen zeilen...

hatte das array in die klasse geschrieben :

String [] zeile = new String [11];

dann lese ich in einer methode die zeilen einzeln aus und speicher sie im array...nun sagt er mir, wenn ich die zeile in einer anderen methode aufrufen möchte...

kopf[spaltenanzahl] = zeile[zeilen].split(";");

dass "Erstellen eines statischen (static) Verweises für das nicht-statische Feld zeile nicht möglich"....

wo liegt mein fehler??

 

[The_S]

Das du irgendwo static verwendet hast wo du es nicht sollst. Ohne Code ist es schwer zu sagen wo genau der Fehler liegt.

 

[mermen]

wenn ich die zeilen in das "klassenarray: zeile" einlese....in einer methode brauche ich einen rückgabewert in meiner methode???

ich habe da puplic void einlesenHighscore....muß ich dass array zurückgeben??? das geht doch gar nicht....????

static hab ich in der class "highscore" gar nicht verwendet!!

 

[Leroy42]

Sicher kannst du ein Array zurückgeben (ist schließlich nur eine Referenz).

Du kannst dein Kopf-Array und das Datenarray allerdings auch als Instanzvariable
deklarieren, sodaß einlesenHighscore dann direkt drauf zugreifen kann.

Trotzdem, wie Hobbit schon meinte, poste mal den Code von einlesenHighscore

 

[mermen]

der anfang:

import java.io.*;

public class Highscore 
{
	
	String [] zeile = new String [11];
	String name = "";
	float neuprozent;

dann:

public void einlesenHighscore ()
	{ 
	try 
	    { 
	    	String leseZeile = "";
	    	int zeilenzaehler = 0;	    
	    	BufferedReader Datei = new BufferedReader(new FileReader("Highscore.txt"));
	    	
	    	while ((leseZeile = Datei.readLine()) != null)  
			{ 
	            
	            	zeile [zeilenzaehler]= leseZeile;
	          
	            zeilenzaehler = zeilenzaehler + 1;
			
			} 
	       	Datei.close(); 
	       
	    }    
	     
	    catch (FileNotFoundException error)
	    {
	    	 System.out.println(" Datei nicht gefunden! Bitte stellen sie sicher, dass die Textdatei ´Highscore´im richtigen Ordner und vorhanden ist!");
	    } 
	    catch (IOException error) 
	     
	    { 
	    	 System.out.println(" Fehler beim Lesen der Datei!!"); 
	    }
	}

und nun bin ich hier....

public void stringZerlegung()
	{
		int zeilenzaehler = 0;
		
		while (zeilenzaehler<= 10)
		{
			if (zeilenzaehler == 0)
			{
				String[] kopf = new String[7]; 
				
				int zeilenanzahl = 0;
				while (zeilenanzahl <= 10)
				{
					String zeilen =zeile[zeilenanzahl];
					int spaltenanzahl = 0;
					while (spaltenanzahl <= 7)
					{
						kopf[spaltenanzahl] = zeilen.split(";");
						spaltenanzahl++;
			
					}
					zeilenanzahl++;
				}
			}
		
		}
		
	}

und...nun folgende fehlermeldung: "Typabweichung: Konvertierung von String[] auf String nicht möglich"

edit: in dieser zeile: kopf[spaltenanzahl] = zeilen.split(";");

 

[The_S]

Du splitest in einen String, musst aber in ein String-Array spliten.

kopf = zeilen.split(";");

btw: bin jetzt weg (weiß net ob ich überhaupt nochmal vor der Woche Urlaub morgen komm), leroy42, übernehmen sie

 

[Leroy42]

Wie wir dir bereits mehrfach gesagt haben: split liefert bereits einen Stringarray.

Du kannst dir das gesamte Schleifengerödel also sparen und einfach schreiben

String[] kopf = zeilen.split(";");

Außerdem definierst du kopf lokal in der Methode und gibst ihn nicht zurück ==> Inhalt geht verloren.

Ich schreibe gleich mal ein Skelett.

 

[Leroy42]

Hier mal das Grundgerüste ohne vollständige Fehlerbehandlung

public class Highscore {
	static final int SCORES = 10;

	String[] kopf;
	String[][] daten = new String[SCORES][];

	public boolean liesHighscore() {
		try {
			BufferedReader highscore = new BufferedReader(new FileReader("Highscore.txt"));
			kopf = highscore.readLine().split(';');
			for (int i=0; i<SCORES; ++i) {
				String zeile = highscore.readLine();
				if (zeile == null) {
					System.err.println("Datei zu klein");
					return false;
				} else
					daten[i] = zeile.split(';');
			}
		}
		return true;
	}
}

 

[mermen]

warum gebe ich einen boolean wert zurück, hat das einen hintergrund...wofür bräuchte ich den???

 

[Leroy42]

Du mußt es nicht unbedingt.

Aber da du in dieser Methode IO-Operationen machst, die mißlingen könnten (Scanner fällt
auf USB-Memory-Stick auf der die Highscore-Datei gespeichert ist und bricht ab)
mußt du entsprechend reagieren. Und da deine Methode keine Exception wirft, weiß
die aufrufende Methode ja gar nicht, ob und wie die Datei eingelesen wurde. Sie würde
bein einem IO-Fehler einfach mit einem evtl. halb-kaputten Daten weitermachen.

Der boolean soll dem Aufrufer zeigen, ob das Einlesen geklappt hat.

 

[norman]

na in leroy42s beispiel liefert die methode true zurück, wenn das lesen der highscrore geklappt hat. im fehlerfalle false..

die daten sind ja in den instanzvariablen (klassenvariablen) kopf und daten gespeichert..

[edit]..ein paar sekunden..hm
btw: deine signatur is so geil, leroy42 :lol: :lol:

 

[mermen]

aber das selbe kann ich ja auch bewirken, indem ich eine catch-anweisung einfüge, die im fehlerfall dann anspringt!?...also füge ich eine catch-anweisung ein und spare mir das boolean....

(denn wie oft fällt mein scanner auf meinen usb-stick wo ich dann auch noch gleichzeitig die highscore einlese!!?? ;-) )

 

[Leroy42]

btw: deine signatur is so geil, leroy42 :lol: :lol:

Finde ich auch

Darum habe ich fantasygirl, eine Teilnehmerin eines anderen Forums bei
der ich sie gefunden habe, auch gleich gefragt ob ich sie in diesem Forum benutzen
darf

Edit: Mit dem letzten sie meinte ich natürlich die Signatur :shock:

 

[norman]

Edit: Mit dem letzten sie meinte ich natürlich die Signatur :shock:

gut, dass du es dazu sagst :wink:

 

[Leroy42]

aber das selbe kann ich ja auch bewirken, indem ich eine catch-anweisung einfüge, die im fehlerfall dann anspringt!?...also füge ich eine catch-anweisung ein und spare mir das boolean....

Das ist richtig. Aber was machst du dann in dem catch-Block? Da der Aufrufer der Methode ja nicht
mitbekommt, daß etwas schiefging und einfach weitermachen würde, bleibt dir ja nur übrig, nach
Ausgabe einer Fehlermeldung, mit System.exit(irgendwas) das Programm zu beenden.

Naja, vielleicht initialisierst du kopf und daten mit leeren Werten und interpretierst den Einlesefehler
als noch nicht vorhande Highscore-Tabelle; dann könnte das Programm ja auch weiterlaufen und
müßte nicht beendet werden.

 

[norman]

aber das selbe kann ich ja auch bewirken, indem ich eine catch-anweisung einfüge, die im fehlerfall dann anspringt!?...also füge ich eine catch-anweisung ein und spare mir das boolean....

(denn wie oft fällt mein scanner auf meinen usb-stick wo ich dann auch noch gleichzeitig die highscore einlese!!?? ;-) )

wenn du den fehler in der methode liesHighscore() mit einem catch auffängst, ist das ja erstmal nicht schlecht. aber davon kriegt die aufrufende methode (zb die main()) nicht zwangsläufig etwas mit.

du rufst beispielsweise irgendwo auf:

anweisung1();
liesHighscore();
anweisung2();

..anweisung2 brauch evtl die daten, die in *kopf* und *daten* von liesHighscore() reingeschrieben wurden. wenn das aber nicht geklappt hat, siehts schlecht aus für anweisung2
=> darum:

anweisung1();
boolean geklappt = liesHighscore();
if (geklappt) {
   anweisung2();
} else {
   // evtl. nochmal liesHighscore() versuchen oder sowas..
}

 

[mermen]

ich habe gerade folgende fehlermeldung beim split:

Die Methode split(String, int) im Typ String ist für die Argumente nicht anwendbar (char)

 

[Leroy42]

 // evtl. nochmal liesHighscore() versuchen

Der war gut!
[schild=8 fontcolor=000000 shadowcolor=C0C0C0 shieldshadow=1]Ich gebe nicht auf! Verdammich, es muß doch mal klappen![/schild]

 

[norman]

[edit] das ist bockmist, einfach überspringen.. [/edit]

split(deinString, ';');
nicht: split(deinString, ";");
- glaube ich

 

[Leroy42]

Du benutzt auch die falsche Methode:

String[] split(String regExpr)

Ich sehe gerade, daß du einen regulären Ausdruck, keinen char, übergeben mußt.
Daher doppelte Anführungszeichen um das Semikolon
kopf = zeile.split(";")

 

[mermen]

split(deinString, ';');
nicht: split(deinString, ";");
- glaube ich

ist schon geändert....hatte ich vorhin nur mal probiert...aber egal...stet nun so!

kopf = highscore.readLine().split(';');

 

[mermen]

ok...fehler beseitigt!!!! ;-)

waren die ";" statt ';'

 

[mermen]

also...ich aber nun echt überfragt, wie ich nun weitermache....nun kann ich doch die klasse von hobbit verwenden...oder???

 

[mermen]

bist du noch da leroy42??

 

[Leroy42]

nööh

 

[mermen]

kannst du mir sagen, wann die methoden

getFormattedHead(String[] head, String[][] matrix)
getFormattedmatrix(String[] head, String[][] matrix)
verwendet werden???

rufe ich die separat auf??

 

[Leroy42]

Prinzipiell sollte das jetzt mit Hobbits Klassen funktionieren.

An manchen Stellen kannst du dir die Übergabe deiner Daten (head, data) ersparen, da
diese jetzt ja als Instanzvariablen vorliegen und dadurch von jeder Methode direkt
benutzt werden können.

 

[The_S]

kannst du mir sagen, wann die methoden

getFormattedHead(String[] head, String[][] matrix)
getFormattedmatrix(String[] head, String[][] matrix)
verwendet werden???

rufe ich die separat auf??

Die verwendest du wenn du noch was anderes mit der Tabelle machen möchtest, sie dafür aber zur Ausgabe bereit formatiert sein muss (kA für was du die gebrauchen kannst, ich habs halt ma eingebaut ... )

 

[mermen]

ok....dann werd ich sie wohl einfach weglassen!!
es sei denn jemand nennt mir einen guten grund sie doch noch zu benutzen!??...

 

[Leroy42]

Sie sind zumindest mit Vorsicht zu genießen.
Die Namen getFormatted... suggerieren, daß es Methoden seien, die
nur etwas zurückliefern.

In Wahrheit verändert aber getFormattedMatrix auch das übergebene Array.

Deshalb sollte es besser heißen formatMatrix. Die jetzt formatierte Matrix
kann natürlich immer noch zurückgegeben werden.

 

[mermen]

irgentwie verstehe ich nicht was die methode "getBreite(...)" auf seite 3 dieses threads macht...kann mir das einer erklären???
schritt für schritt....

ich weiß zwar was ich rauskriege...aber ich kann den ablauf irgentwie nicht nachvollziehen!??...

 

[mermen]

hier nochmal der code den ich nicht verstehe....

class KonsolenTabelle { 
	
		 
	// gibt die benötigte Breite für jede Spalte in einem Array zurück 
	 
	private int[] getBreite(String[] head, String[][] matrix) { 
		 
		String[] curSpalte = new String[matrix.length + 1]; 
		int[] breiten  = new int[matrix[0].length]; 
		for (int i = 0; i < matrix[0].length; i++) { 
			curSpalte[0] = head[i]; 
			for (int j = 0; j < matrix.length; j++) { 
				curSpalte[j + 1] = matrix[j][i]; 
			} 
			breiten[i] = maxBreite(curSpalte); 
		} 
		return breiten; 
	}

 

[Leroy42]

  // gibt die benötigte Breite für jede Spalte in einem Array zurück 
    
   private int[] getBreite(String[] head, String[][] matrix) { 

      // curSpalte betrachtet jeweils eine Spalte der Matrix
      String[] curSpalte = new String[matrix.length + 1]; 

      // breiten beinhaltet für jede Spalte die maximale Anzahl der Zeichen in dieser Spalte
      int[] breiten  = new int[matrix[0].length]; 

      // i durchläuft die Spalten
      for (int i = 0; i < matrix[0].length; i++) { 

         // der Tabellenkopf ist auch Teil der Spalte
         curSpalte[0] = head[i]; 

         // j durchläuft jede Zeile der Matrix der i-ten Spalte
         for (int j = 0; j < matrix.length; j++) { 

            // Der Inhalt der Zelle wird in das Zielarray kopiert
            curSpalte[j + 1] = matrix[j][i]; 
         } 

         // Die Methode maxBreite bestimmt jetzt die maximale Zeichenanzahl von curSpalte
         // diese wird in das breiten-Array gespeichert.
         breiten[i] = maxBreite(curSpalte); 
      } 

      // Rückgabe aller Maximalspaltenbreiten
      return breiten; 
   }

Das funktioniert zwar so, allerdings werden die Texte selbst unnötigerweise
kopiert. Man könnte das Kopieren vermeiden, indem man die Bestimmung
des jeweiligen Maximums in diese Methode integriert. Bei diesen kleinen
Datenmengen ist das aber vollkommen schnurz; eben Geschmackssache.

 

[mermen]

also du meinst, dass ich die spaltengröße vorgeben kann....so dass nicht jede spalte auf ihre länge überprüft wird....hab ich das richtig verstanden!??....

dann bräuchte ich doch die gesamte mehtode nicht....oder???

 

[mermen]

also, so wie es zur zeit steht, ist das ok für mich....ich werde das wohl so lassen, da ich eh keine aufwendigen daten bearbeiten muß, sondern nur diese 10 highscoreeinträge!!!

also, das mit der ausgabe klappt nun, aber ich muß natürlich meinen aktuell, erspielten eintrag überprüfen, ob er nun besser ist als die anderen und ihn dann auch in der highscore speichern...

hat da jemand einen guten ansatz??...
ich krieg das ja nun hin, dass ich die prozente miteinander vergleiche...aber ich müßte doch die zeilen wieder erneut einlesen (aus der datei) und (wenn nötig) die reihenfolge verändern und wieder abspeichern.....

 

[The_S]

Sortier nach der Prozentzahl und schmeiß den letzten Eintrag raus.

@ Leroy42 immer diese Code-Fuchserei ... das war doch nur schnell hingeklatscht, lasst mich halt in ruhe von wegen Methoden-Namen und Performance ... :wink: