Input/Output Datenbank

[KingOfExceptions]

Hey leute,

ich hab angefangen eine datenbank zu schreiben , bei der man eine nummer, ein stichwort und ein bildverzeichnis angeben kann. dann werden diese drei sachen in einem vector abgespeichert und dann kann man danach entweder mithilfe der nummer oder mithilfe des stichworts nach dem objekt suchen und dann wird mithilfe des verzeichnisses das bild angezeigt. soweit kein problem. ich möchte aber dass man auch nach schließen und neuöffnen die alten nummern, stichwörter und verzeichnisse der bilder hat. dazu hab ich einen buffered writer gemacht, der die einzelnen sachen mit einem / abspeichert.

public void abspeichern()
	{
		int ö=0;
		try
		{
			BufferedWriter writer=new BufferedWriter(new FileWriter("Speicherung"));
			
			if(zdhs!=0)
			{
				for(;ö<(zdhs*3);ö++)
				{
					writer.write(heft.elementAt(ö));
					writer.write("/");
					
				}
			}
			
			writer.close();
		}
		
		catch(IOException exception)
		{
			JOptionPane.showMessageDialog(null, "Es ist ein Fehler beim Speichern aufgetreten." );
		}
	}

dann möchte ich dass ich beim öffnen diese sachen in einen vector stecken kann:

public void laden()
	{
		
		try
		{
			BufferedReader reader=new BufferedReader(new FileReader("Speicherung"));
			
			String zeile=reader.readLine();
			
			while(zeile!=null)
			{
				 String[] values=zeile.split("/");
				
				for(int i=0;i<values.length;i++)
				{
					heft.add(Integer.parseInt(values[i]), zeile);
				}
			}
			reader.close();
		}
		
		catch(IOException exceptio)
		{
			JOptionPane.showMessageDialog(null, "Es ist ein Fehler beim Laden aufgetreten." );
		}
	}

jedoch bekomme ich folgende komische fehlermeldung:

java.lang.ExceptionInInitializerError
Caused by: java.lang.ArrayIndexOutOfBoundsException: 123 > 0
	at java.util.Vector.insertElementAt(Unknown Source)
	at java.util.Vector.add(Unknown Source)
	at OldDaten.laden(OldDaten.java:131)
	at OldDaten.<init>(OldDaten.java:55)
	at OldDaten.<clinit>(OldDaten.java:15)
Exception in thread "main"

damit kann ich nichts anfangen . hier (bei bedürfnis) die zeilen 131 55 und 15

heft.add(Integer.parseInt(values[i]), zeile);

zeile 131

laden();

zeile 55

static OldDaten frm=new OldDaten("Die Oldtimer Datenbank");

zeile 15

kann mir da jemand helfen? ???:L

 

[ARadauer]

for(;ö<(zdhs*3);ö++) ... interessant ;-)

 

[Sunchezz]

Zeige doch mal bitte wie du den Vector "heft" initialisiert hast.

 

[KingOfExceptions]

@ ARadauer: ich hatte so viele variablen, da wollte ich mir einfach sicher sein, dass es die noch nicht gibt

Vector <String> heft=new Vector <String>(50,5);

heft.addElement(shgseriennummer);
heft.addElement(shgschlagwörter);
heft.addElement(geingabe);
zdhs++;
JOptionPane.showMessageDialog(null,"Das Heft wurde zur Datenbank hinzugefügt.");

 

[KingOfExceptions]

ach hier ist die main wenn sie jemand braucht:

public static void main(String[]args)
	{
		
		frm.setDefaultCloseOperation(JFrame.DO_NOTHING_ON_CLOSE);
		frm.setSize( 680, 680);
		frm.setResizable(false);
		frm.setLocation(100,10);
		frm.setVisible( true );
		
		
	}

 

[nrg]

@ ARadauer: ich hatte so viele variablen, da wollte ich mir einfach sicher sein, dass es die noch nicht gibt

über den namespace von variablen und dem initialisierungsteil einer for-schleife bist du dir aber im klaren, oder?

 

[nillehammer]

Du gibtst bei der add-Methode einen Index mit, den es noch garnicht gibt. Bei einem leeren Vector gibt es nur den Index 0. Ich glaube, Vector ist hier auch nicht die richtige Klasse. Für Deinen Anwendungsfall wäre Map wohl besser geeignet.

Ach ja, und benutze niemals Umlaute für Variablennamen.

 

[KingOfExceptions]

ja ich wollts aber nicht drauf ankommen lassen und was ist mit dem intialisierungsteil der for schleife? dass da ein ; am anfang stehen muss kommt mir immer komisch vor aber eclipse will es so

 

[KingOfExceptions]

@ nillehammer: wo meinst du ?

 

[nrg]

zwischen der offenen Klammer und dem Semikolon kommt die Variable, die du in Zeile 3 deklarierst . dadurch ist sie auch nur im header und body der for-schleife gültig.

                for(int i = 0; i < zdhs*3; i++)
                {
                    writer.write(heft.elementAt(i));
                    writer.write("/");
                    
                }

 

[KingOfExceptions]

Ich glaube, Vector ist hier auch nicht die richtige Klasse. Für Deinen Anwendungsfall wäre Map wohl besser geeignet.

ja nur 1. hab ich noch nie mit map gearbeitet und 2. ich muss dann 500 zeilen ändern :/ ich hoffe auf eine alternative

 

[nillehammer]

ja ich wollts aber nicht drauf ankommen lassen und was ist mit dem intialisierungsteil der for schleife? dass da ein ; am anfang stehen muss kommt mir immer komisch vor aber eclipse will es so

Was bei der For-Schleife in den Klammern steht:
1. Initialisierung einer Variablen (also initial einen Wert zuweisen), ggf. sogar inkl. Deklaration (also die Variable überhaupt erst definieren)
2. Bedingung
3. Aktion
Da Du die Variable außerhalb deklarierst und initialisierst, lässt Du den Teil 1. leer. Der leere Teil 1. wird durch die leere Anweisung ";" gekennzeichnet. Das int ö=3 schreibst Du am besten in die Klammern nach dem for rein.

 

[KingOfExceptions]

zwischen der offenen Klammer und dem Semikolon kommt die Variable, die du in Zeile 3 deklarierst . dadurch ist sie auch nur im header und body der for-schleife gültig.

                for(int i = 0; i < zdhs*3; i++)
                {
                    writer.write(heft.elementAt(i));
                    writer.write("/");
                    
                }

ok hab ich geändert.

 

[KingOfExceptions]

Was bei der For-Schleife in den Klammern steht:
1. Initialisierung einer Variablen (also initial einen Wert zuweisen), ggf. sogar inkl. Deklaration (also die Variable überhaupt erst definieren)
2. Bedingung
3. Aktion
Da Du die Variable außerhalb deklarierst und initialisierst, lässt Du den Teil 1. leer. Der leere Teil 1. wird durch die leere Anweisung ";" gekennzeichnet. Das int ö=3 schreibst Du am besten in die Klammern nach dem for rein.

hab ich geändert

 

[nillehammer]

@ nillehammer: wo meinst du ?

Hmm? Was meinst Du mit "wo meinst du?"

 

[KingOfExceptions]

Du gibtst bei der add-Methode einen Index mit, den es noch garnicht gibt. Bei einem leeren Vector gibt es nur den Index 0.

wo meinst du das?

 

[nillehammer]

Ach und was ich auch grad noch sehe, Du rufst ja nur ein einziges Mal readLine auf. Hat Deine Datei nur eine einzige Zeile? Falls nicht, auch das gehört dann in eine Schleife.

 

[KingOfExceptions]

schon aber wie sage ich ihm welche zeile er lesen soll?

 

[nillehammer]

[Java]
heft.add(Integer.parseInt(values), zeile);
[/Java]
Hier parst Du offensichtilch mehrmals Werte zu int. Damit rufst Du dann die Methode add(int, String) auf. Das int bedeutet den Index (also die Position) im Vector, die Du gerne setzen möchtest. Bei einem leeren Vector gibt es z.b. die Position 123 nicht. Die gibt es erst bei einem Vector, der mindestens 124 Elemente enthält.

 

[KingOfExceptions]

ach so ok! und wie kann ich das so ändern dass er einfach nur jede eingelesene datei einfach im vector abspeichert?

 

[nillehammer]

ach so ok! und wie kann ich das so ändern dass er einfach nur jede eingelesene datei einfach im vector abspeichert?

Hä? Wieso jetzt jede einzelne Datei? Hast Du etwa mehrere? In Deinem Code liest Du jedenfalls nur eine Datei ein und davon auch nur die erste Zeile.

 

[KingOfExceptions]

nein nicht datei sondern nur jedes objekt halt. sorry

 

[KingOfExceptions]

sagen wir mal so: ich hab eine datei in der viele sachen mit einem / getrennt drin stehen. wie kann ich sagen dass der computer JEDE zeile auslesen soll und dann die sachen zwischen den / dem vector heft hinzufügt? einfach nacheinander? zb:

0123/JAVA/toll/
-> heft.elementAt(0) = 0123
-> heft.elementAt(1)= JAVA
-> heft.elementAt(2)=toll
-> usw.

 

[nillehammer]

Ich habe mir Deinen Code nochmal angesehen und versuche mal zu beschreiben, was er macht. Ob es das ist, was Du willst?

Methode Abspeichern:
- Du hast eine Menge Strings (was da drinen steht kann man nicht erkennen)
- Diese gehst Du durch und speicherst sie in einer einzigen Zeile einer Datei. Dabei trennst Du mit "/"

Methode Laden:
- Deine Datei hat nur eine Zeile. Insofern ist der einmalige Aufruf von readLine() in Ordnung.
- Das Ergebnis von readLine() ist ein ziemlich langer String.
- Den splittest Du mit "/" in ein Array von Einzelstrings auf.

Was im Code überhaupt nicht ersichtlich ist:
Wo kommen da jetzt Nummer, Beschreibung und Bildverzeichnis her?
Was ist mit Objekt gemeint? Und wo kommt das her?

 

[KingOfExceptions]

ok für das laden:

ich habe drei strings: einer steht für eine seriennummer(soll in string sein), einer für ein stichwort, und einer für ein bildverzeichnis. was ich damit mache ist eig. egal.

diese sachen sollen beim schließen des programms in eine datei gemacht werden und dann mit / abgetrennt werden. (klappt auch)

beim öffnen sollen diese dateien getrennt werden(die sachen zwischen den / ) und sollen einem vector (heft) hinzugefügt werden.

ich hoffe das ist einigermaßen verständlich

 

[KingOfExceptions]

aber wie genau ich das machen soll wenn nicht so wie ich es versucht habe, ist mir ein rätsel

 

[KingOfExceptions]

ok ich habs geregelt sind zwar noch ein paar kleine bugs drin müsste ich aber alleine schaffen sonst melde ich mich

 

[Sunchezz]

while ((String s = r.readLine()) != null) {
  vector.add(s);
}

damit hast du alle ZEILEN in einem Vector.

Der Reader liest bei jedem Aufruf von "readLine()" immer automatisch die nächste Zeile in der Datei.
Bedingt durch die Abfragedefinition hört er aber auf, wenn in dem gelesenen String "nichts" mehr enthalten ist.


Edit:
mal wieder die Seitenanzahl des Threads ignoriert -.-
Post einfach ignorieren bitte!