Collections RandomAccessfile & Linkedlist

[Bob79]

Hallo, ich hänge seit Tagen an einer Stelle fest und vielleicht hat einer von euch den umbau Tip für meinen code.

Was macht das Program?
Es liest eine CSV Datei "person.dat" ein und speichert die Werte einer Zeile in einen Objekt "person" in eine Linkedlist namens "Personen", welche aus serialisierten Objekten (pnr, pname, kst) besteht. Das funktioniert auch soweit.
Jetzt soll es in einer 2. Linkedlist ein Indexpaar aufbauen, welches den offset wert vom 0.byte (spalte 1 zeile 1) speichert und die pnr der ersten Zeile - und hier kommt der Knackpunkt - ich schaffe es nicht den Offset Zeiger richtig einzubauen so das es wie gewünscht funktioniert -> Siehe Soll/IST Ausgabe am ende des Posts.

import java.io.*;
import java.util.*;
import java.lang.Object;

public class Isam_v4 
{
	public static class Person implements Serializable
	{
		int pnr;
		String pname;
		int kst;
	}
	
	public static class Di implements Serializable
	{
		int pnr;
		long offset;
	}
	
	public static void einlesen()
	{
		List <Person> Personen = new LinkedList<Person>();
		List <Di> Isam = new LinkedList<Di>();
		
		System.out.println("");	
		System.out.println("CSV Datei person.dat wird eingelesen...");
		System.out.println("");
		
		try 
		{	
			RandomAccessFile file;
			file = new RandomAccessFile("person.dat","r");
			
			for(String line; (line = file.readLine()) != null;)
			{   
				
				String[] tmp	= line.split(";");	// CSV Werte person.dat trennen ";"
				Di idxin		= new Di();
				idxin.offset	= file.getFilePointer(); // DAS IST MEIN SORGENKIND
				idxin.pnr 		= Integer.parseInt(tmp[0]);
				Isam.add(idxin);
				Person eingabe 	= new Person();
				eingabe.pnr 	= Integer.parseInt(tmp[0]);
				eingabe.pname	= tmp[1];
				eingabe.kst		= Integer.parseInt(tmp[2]);
				/*
				System.out.println("Personal Nr.. "+eingabe.pnr);
				System.out.println("Name......... "+eingabe.pname);
				System.out.println("Kostenstelle.."+eingabe.kst);
				System.out.println(); */
				Personen.add(eingabe);
				
			}
				
			
			file.close();
			
			System.out.println("");		
			System.out.println("CSV Datei eingelesen, "+Personen.size()+" Eintraege gefunden" );
			
			System.out.println("");	
			System.out.println("Pnr ; Name ; Kostenstelle");
			System.out.println("-------------------------");	
			for (int i = 0; i < Personen.size(); i++)
			{
				Person csvaus	= new Person();
				csvaus 			= Personen.get(i);
				System.out.println(csvaus.pnr+";"+csvaus.pname+";"+csvaus.kst);
			}	
			System.out.println("");
			System.out.println(+Personen.size()+" Datensaetze ausgegeben");
			System.out.println("");
			
			System.out.println("");	
			System.out.println("Offset ; Pnr ");
			System.out.println("-------------------------");	
			for (int i = 0; i < Isam.size(); i++)
			{
				Di idxout		= new Di();
				idxout			= Isam.get(i);
				System.out.println(idxout.offset+";"+idxout.pnr);
			}	
			System.out.println("");
			System.out.println(+Isam.size()+" Datensaetze ausgegeben");
			System.out.println("");
				
		}
		catch(FileNotFoundException e)
		{
		  System.out.println("Es ist ein FileNotFoundFehler aufgetreten: "+e.getMessage());
		}
		catch(IOException f)
		{
		  System.out.println("Es ist ein IOExpeptionFehler aufgetreten: "+f.getMessage());

		}
	}
        public static void main(String[] args) throws IOException,FileNotFoundException 
	{
		BufferedReader in = new BufferedReader(
							new InputStreamReader(System.in));
		
		einlesen();
		
	}
}

Dateiname: "person.dat"

1;Max;2
2;Moritz;4
3;mry;5
4;mrx;12

IST-Ausgabe

Offset ; Pnr 
-------------------------
12;1
21;2
30;3
41;4

Soll-Ausgabe

Offset ; Pnr 
-------------------------
0;1
12;2
21;3
30;4

Hier noch infos zu meinen System:
BS: OS X 10.6.6
IDE: Jedit + shell

javac 1.6.0_22
java version "1.6.0_22"
Java(TM) SE Runtime Environment (build 1.6.0_22-b04-307-10M3261)
Java HotSpot(TM) 64-Bit Server VM (build 17.1-b03-307, mixed mode)

Wenn ich den folgendes Stück code vor die "for-schleife" schiebe wird die ISAM-LinkedList nicht richtig aufgebaut...

Di idxin		= new Di();
idxin.offset	= file.getFilePointer();

Habe auch schon versucht durch andere Schleifen das passend hinzubekommen, aber ich komm irgendwie nicht drauf

Hier noch der API Link zur RAF




EDIT 1: War noch n copy & paste fehler drin, jetzt lässt dich code auch wieder kompilieren ;-)
EDIT 2: Anbei jetzt noch inhalt für eine Testdatei
EDIT 3: Den Java Code in den richtigen BBcode gepackt. Ist mir bei Tagedieb`s posting erst aufgefallen das ich da was falsch formatiere bzw. nicht farbig war.

 

[tagedieb]

Du setzt den Offset am Ende der Zeile und nicht am Anfang.
Wenn du den Offset bei 0 startest und am Ende der Schleife den neuen Offset setzt muesste es klappen.

	RandomAccessFile file;
	file = new RandomAccessFile("person.dat","r");
	long offset = 0l;
	for(String line; (line = file.readLine()) != null; offset = file.getFilePointer())
	{   
		String[] tmp	= line.split(";");	// CSV Werte person.dat trennen ";"
		Di idxin		= new Di();
		idxin.offset	= offset; // DAS IST MEIN SORGENKIND

 

[Bob79]

Hallo tagedieb und schonmal danke für Deine Antwort, bin für jeden Lösungsansatz echt dankbar =)

Du hast es richtig erkannt, ich setze den offset Wert nach dem ersten "readLine()" welcher sich in der Schleifenbedingung befindet -> Dadurch ist der Offset immer != 0 wenn er nacher durch die Funktion "file.getFilePointer()" aufgerufen wird.

In Deinen Beispiel bleibt das Problem bestehen bzw. hab ich Deinen Lösungsansatz falsch verstanden. Hab es so gebaut wie du beschrieben hast aber immer noch nicht das gewünschte Ergebnis

Ich hab jetzt den code mal auf das notwendige verkürzt und so umgebaut das er den Offset richtig ausgibt, jedoch wird dann die Linkedlist nicht korrekt befüllt weil ich das Objekt "Di" nicht in der Schleife neu erzeuge (Ich glaube zumindest das es daran liegt). Hab das gefühl das ich den Wald vor lauter Bäumen nicht sehe

import java.io.*;
import java.util.*;
import java.lang.Object;

public class Isam_v6
{
	public static class Di implements Serializable
	{
		int pnr;
		long offset;
	}
	
	
	public static void einlesen()
	{	
		List <Di> Isam = new LinkedList<Di>();
		
		System.out.println("");	
		System.out.println("CSV Datei person.dat wird eingelesen...");
		System.out.println("");
		
		try 
		{	
			RandomAccessFile file;
			file = new RandomAccessFile("person.dat","r");
			Di idxin		= new Di();
			idxin.offset	= file.getFilePointer(); // Hiert ist der offset 0, keine zeile wurde gelesen
			System.out.println(file.getFilePointer());	
			
			for(String line; (line = file.readLine()) != null;) // file.readline speichert hier schon die erste Zeile
			{   
				idxin.offset	= file.getFilePointer(); // Hiert ist der offset 12, eine zeile wurde gelesen
				System.out.println(file.getFilePointer());	
				
				idxin.offset	= file.getFilePointer(); // DAS IST MEIN SORGENKIND
				Isam.add(idxin);
			}	
			file.close();
			System.out.println("");	
			System.out.println("Offset ; Pnr ");
			System.out.println("-------------------------");	
			for (int i = 0; i < Isam.size(); i++)
			{
				Di idxout		= new Di();
				idxout			= Isam.get(i);
				System.out.println(idxout.offset+";"+idxout.pnr);
			}	
			System.out.println("");
			System.out.println(+Isam.size()+" Datensaetze ausgegeben");
			System.out.println("");
		
				
		}
		catch(FileNotFoundException e)
		{
		  System.out.println("Es ist ein FileNotFoundFehler aufgetreten: "+e.getMessage());
		}
		catch(IOException f)
		{
		  System.out.println("Es ist ein IOExpeptionFehler aufgetreten: "+f.getMessage());

		}
	}
    
	public static void main(String[] args) throws IOException,FileNotFoundException 
	{
		einlesen();
	}
}

Ausgabe beim einlesen vermeidlich korrekt, jedoch wird falsch in die Linkedlist gespeichert, er merk sich nur den letzten offset (wiel Di nicht in der Schleife neu erzeugt wird und nur ein Objekt existiert?!) Achja das PNR im mom 0 ist leigt dadran, das ich den Teil entfernt habe um das offset Problem in den griff zu bekommen ;-)

CSV Datei person.dat wird eingelesen...

0
12
21
30
41

Offset ; Pnr 
-------------------------
41;0
41;0
41;0
41;0

4 Datensaetze ausgegeben

Meine 3. Lösungsanstätze wären
1. Noch ne Schleife irgendwie drumherumbauen, das es erledigt.
2. Die Schleifenbedingung ändern, so das ich da kein readline brauche
3. Fußgesteuerte Schleife mit passender Bedingung

Das sind so versiche die ich bisher gestartet habe aber da bin ich bisher auch auf keinen grünen Zweig gekommen.

 

[tagedieb]

In Deinen Beispiel bleibt das Problem bestehen

Wenn ich meinen Code aus obigen Post verwende erhalte ich:

0
9
21
30

Der zweite Index muss 9 sein und nicht 12.


Du musst natuerlich immer ein neues DI Objekt erstellen ansonsten fuegst du immer wieder dasselbe DI der Liste hinzu. Wenn du dann den Offset setzt aenderst du alle Eintraege, da es sich ja um dasselbe Objekt handelt.

 

[Bob79]

Doh! Du hattest natürlich vollkommen recht!

Ich hab es irgendwie geschafft deinen Vorschlag so umzustetzen das es nicht geklappt hat und das trotz gepostet code, auh man. Hier so hier die fertige Komplettlösung falls jemand auf das gleiche Problem stossen sollte:

import java.io.*;
import java.util.*;
import java.lang.Object;

public class Isam_v7
{
	public static class Di implements Serializable
	{
		int pnr;
		long offset;
	}
	
	
	public static void einlesen()
	{	
		List <Di> Isam = new LinkedList<Di>();
		
		System.out.println("");	
		System.out.println("CSV Datei person.dat wird eingelesen...");
		System.out.println("");
		
		try 
		{	
			 RandomAccessFile file;
			 file = new RandomAccessFile("person.dat","r");
			 
			 long offset = 0l;
			 for(String line; (line = file.readLine()) != null; offset = file.getFilePointer())
			 {   
			 	 Di idxin        = new Di();
			 	 String[] tmp    = line.split(";");  // CSV Werte person.dat trennen ";"
			 	 idxin.offset	= offset; 
			 	 System.out.println(idxin.offset);
			 	 Isam.add(idxin);

			}	
			file.close();
			System.out.println("");	
			System.out.println("Offset ; Pnr ");
			System.out.println("-------------------------");	
			for (int i = 0; i < Isam.size(); i++)
			{
				Di idxout		= new Di();
				idxout			= Isam.get(i);
				System.out.println(idxout.offset+";"+idxout.pnr);
			}	
			System.out.println("");
			System.out.println(+Isam.size()+" Datensaetze ausgegeben");
			System.out.println("");
		
				
		}
		catch(FileNotFoundException e)
		{
		  System.out.println("Es ist ein FileNotFoundFehler aufgetreten: "+e.getMessage());
		}
		catch(IOException f)
		{
		  System.out.println("Es ist ein IOExpeptionFehler aufgetreten: "+f.getMessage());

		}
	}
    
	public static void main(String[] args) throws IOException,FileNotFoundException 
	{
		einlesen();
	}
}

Ausgabe:

CSV Datei person.dat wird eingelesen...

0
8
19
27

Offset ; Pnr 
-------------------------
0;0
8;0
19;0
27;0

4 Datensaetze ausgegeben

Dickes Danke an den Tagedieb, you`ve made my day - hab da echt schon viel Zeit an den paar Zeilen code verloren und es wären wahrscheinlich noch mehr geworden.

Die Abweichunge des Offset-Wers kommen durch Leerzeichen in meiner lokalen person.dat zustande.