Array wird überschrieben

[Assaultv2.0]

hi wollte mal fragen ob mir jemand helfen kann bei folgendem code part...

	    for (String line; (line = f.readLine()) != null;) {

		i++;
		if (i == 1) {
		    Host = line;
		} else if (i == 2) {
		    Sid = line;
		}
		;

		if (i >= 3) {
		    i = 0;
		    DBA[anrX] = new DBS(Sid, Host);
		    anrX = anrX + 1;
		}
	    }

erstmal zur erklärung, er soll immer 2 zeilen einlesen host und sid und dann die 3 zeile überspringen, die eingelesenen zeilen sollen dann als object in dem array gespeichert werden.

das problem ist jetzt, ich lese das erste object ein und habe für dba[0] host1 und sid1 das klappt ja noch, dann liest er mir die nächsten 2 zeilen ein zb. host2 und sid2 speichert dann in dba[1], soweit auch noch ok aber er überschreibt dann auch dba[0] und die haben dann auch die werte host2 und sid2....

woran könnte das liegen? wahrscheinlich dürfte der codeschnipsel nicht viel bringen deshalb hier noch ein paar^^

hier lese ich die einzelnen Positionen wieder aus

	for (anrX = 0; anrX >= 0; anrX = anrX + anrX) {

	    hostname = DBA[anrX].hostname;
	    sid = DBA[anrX].sid;

das array erstelle ich mit

DBS[] DBA = new DBS[5];

&

public class DBS {

	static String hostname;
	static String sid; 
		
	public DBS(String Sidü, String Hostnameü){//Konstruktor
		hostname = Hostnameü;
		sid = Sidü;
	}
}

 

[xysawq]

kurz mal so als Frage nebenbei:

bei

for (anrX = 0; anrX >= 0; anrX = anrX + anrX) {

       hostname = DBA[anrX].hostname;
       sid = DBA[anrX].sid;

quält mich dieses "anrX = anrX + anrX", es wurde doch mit 0 initialisiert, d.h. es bleibt immer 0 und da die Bedingung "anrX >= 0" ist, also die Schleife solange weiter läuft wie anrX größer-gleich 0 ist, müsste die ewig laufen.


dann noch:

for (String line; (line = f.readLine()) != null;) {

      i++;
      if (i == 1) {
          Host = line;
      } else if (i == 2) {
          Sid = line;
      }
      ; <--- was soll das da???

      if (i >= 3) {
          i = 0;
          DBA[anrX] = new DBS(Sid, Host);
          anrX = anrX + 1;
      }
       }

es kann gut sein, dass dir das Semikolon da etwas verhaut.

Aber mal so als Tipp am Rande, du solltest, wenn Fehler auftreten, ein paar System.out.println(...) machen, immer wenn du dein Array beschrieben oder ausgelesen hast, oder noch besser während dessen, damit du genau siehst in welchem Codeabschnitt der Fehler passiert und wir nicht versuchen müssen jeden Codeschnipsel zu verstehen.

Aber versuche erstmal mit meinem bisherigen "Lösungsansatz" was zu machen, wenns nicht geht bitte wie beschrieben vorgehen und die exakte Fehlerposition finden.

 

[Guest]

Hallo!

Zunächst würde ich ArrayLists verwenden. Dann brauchst du nicht solche Zählvariablen
wie anrX mit dir rumschleppen. Mit add(...) kannst du dann neue Elemente zur Laufzeit hinzufügen.
Das vereinfacht hier das Erstellen von DBS-Objekten.

Weiterhin springt f in die nächste Zeile, wenn readLine() aufgerufen wurde. Dadurch ist es nicht
erforderlich, sich mittels einem Zähler zu merken, wo du gerade bist (hier war es die Variable i).
Vorausgesetzt, du kennst das Format der Datei (ich nehme mal an, du liest aus einer Datei).

Hier mein (ungetesteter) Code.

ArrayList<DBS> DBA = new ArrayList<DBS>(); // ArrayLists sind besser als normale Arrays

String s = null;
while( ( s = f.readLine() ) != null ) {  // Sind wir am Ende der Datei?
      DBA.add( new DBS( s,                  // sind wir hier, so wurde in s der Host eingelesen
                                   f.readLine() )); // sid einlesen
      f.readLine();  // Zeile überspringen
}

So, ich sehe gerade:

Deine for-Schleife in deinem zweiten Schnipsel macht nicht viel Sinn:

Zunächst setzt du zu Beginn der Schleife anrX auf 0.
Bei jedem Schleifendurchlauf führst du dann anrX = anrX + anrX aus.
Hier wäre das jetzt anrX = 0 + 0 = 0. Also wäre anrX wieder 0.
Wenn ich jetzt nichts übersehen habe, ist das eine Unendlichschleife.

So würde ich die Daten wieder auslesen (ich gebe sie mal in der Konsole aus):

// size liefert dir die Anzahl an Einträgen im ArrayList
// Die Zählung der Elemente im ArrayList beginnt bei 0 !
for( int i = 0; i < DBA.size(); i++ ) { // besser wären generische for-Schleifen, aber es soll verständlich bleiben
    System.out.println( DBA.get(i).hostname ); // DBA.get(i) liefert mir den i-ten Eintrag in der ArrayList
    System.out.println( DBA.get(i).sid ); // analog
}

Letzte Frage: Warum machst du hostname und sid static? Dadurch gibt es nur ein hostname und sid, egal wieviele
DBS-Objekte du hast. Sollte es nicht lieber public heißen? Vielleicht übersehe ich da was

Mal hoffen, ich habe dein Problem richtig verstanden und konnte helfen
(ist mein erster Post hier im Forum :-D )

 

[Assaultv2.0]

das mit dem anrx ist mir auch aufgefallen hab einfach in anrX++ geändert

das simikolon hab ich jetzt weggemacht aber er überschreibt den array platz DBA[0] immer noch ...

hat das mit dem system.out.println iergendweleche vorteile gegen über dem eclipse debugger ich meine dort sehe ich doch auch wie sich die variablen ändern ?

 

[BigBoomer]

Der überschriebt das, weil hostname und sid in deiner Klasse static sind.
Sie müssen public sein.

System.out.println() ist nur ne andere Lösung zu debuggen. Ob sie besser
ist, will ich hier nicht diskutieren.

 

[assault]

cool der tipp mit dem public hat geklapt thx war mir garnicht aufgefallen hätte ich mir aber auch denken können ^^