Probleme mit zweidimensionalem Array

[megachucky]

Hallo.


Ich möchte aus einer Datenbank daten auslesen und sie in einem zweidimensionalem Array speichern, damit ich damit nachher einfach eine JTable daraus machen kann.

Ich habe dazu die folgende Methode geschrieben.

Die Daten aus der DB werden korrekt eingelesen.

Nun bin ich mir aber nicht sicher, ob mein Konstrukt zum speichern in Array[][] richtig ist, und wie ich eigentlich einen zweidimensionalen Array (zum Testen) auf der Konsole ausgeben kann?

Muss ja irgendwie mit ner for - Schleife gehen, oder? In meine Büchern steht dazu leider nix, und hier hab ich bei der Suche auf die Schnelle auch nix gefunden.


Hier die Methode:

public String[][] getAllCustomer()
	{
		connection = new DatabaseConnection().connectToDatabase();
		String[][] allCustomer;
		
		try
		{
			// Die Parameter sind nötig, dass die ResultSet scrollable ist und so die Anzahl der Reihen 
			// berechnet werden kann mit last() und beforeFirst()
			Statement st = connection.createStatement(ResultSet.TYPE_SCROLL_INSENSITIVE, 
					ResultSet.CONCUR_UPDATABLE);
			String getAllCustomerQuery = "SELECT * FROM kunde";
			ResultSet rs = st.executeQuery(getAllCustomerQuery);
			
			// Anzahl Reihen in 'rs' herausfinden
			rs.last();
			int rowCount = rs.getRow();
			rs.beforeFirst();
			allCustomer = new String[rowCount][6];
			
			
			
			while ( rs.next() )
			{
				// Ausgabe nur zum testen
				System.out.println(rs.getString(1) + " - " + rs.getString(2) + " - " + rs.getString(3) + " - " +
						rs.getString(4) + " - " + rs.getString(5) + " - " + rs.getString(6));
				System.out.println("RowCount: " + rowCount);
				
				
				// Array beginnt bei 0, nicht bei 1 !!!
				int i=0;
				
				
				String s1 = rs.getString(1);
				allCustomer[i][0] = s1;
				
				String s2 = rs.getString(2);
				allCustomer[i][1] = s2;
				
				String s3 = rs.getString(3);
				allCustomer[i][2] = s3;
				
				String s4 = rs.getString(4);
				allCustomer[i][3] = s4;
				
				String s5 = rs.getString(5);
				allCustomer[i][4] = s5;
				
				String s6 = rs.getString(6);
				allCustomer[i][5] = s6;
				
				
				i++;
				
				
				
			}
			
			
			
			
			connection.close();
		}
		catch (SQLException e)
		{
			allCustomer = null;
			
			System.err.println("Error while executing the getAllCustomer query.");
			e.printStackTrace();
		}
		
		
		return allCustomer;
	}

 

[SlaterB]

Die Ausgabe ist doch simpel:

for (array-Länge) {
  Array teilArray = array[i];
  
  ausgabe(jetzt kommt teilArray "+i);
  for (teilArray-Länge) {
      ausgabe(jetzt kommt Element "+j+": "+teilArray[j]);
  }
}

Pass aber auf, dass du unterschiedliche Laufvariablen nimmst,
i außen, j innen.

------------------

Tipp: Die Ausgabe und generell das Handling von Arrays hat NICHTS mit der Datenbankanfrage zu tun.
Teste das lieber erstmal in eigenen Testklassen, statt verschiedene unklare Programmteile zu vermischen..



-----------
getRow() liefert nicht die Anzahl der Zeilen im ResultSet

http://java.sun.com/j2se/1.4.2/docs/api/java/sql/ResultSet.html#getRow()

Wahrscheinlich ist dies auch gar nicht vor dem Durchlauf des Sets zu ermitteln.
Benutze daher am besten zunächst eine ArrayList, der du beliebig viele Elemente hinzufügen kannst.
Danach dann ein Array draus machen.

--------------

In jeder while-Schleife definierst du i=0 neu, das i++ am Ende ist ohne Funktion.
Alle Werte werden also immer ins erste Teilarray geschrieben.

--------------

            String s1 = rs.getString(1); 
            allCustomer[i][0] = s1; 
             
            String s2 = rs.getString(2); 
            allCustomer[i][1] = s2; 
             
            String s3 = rs.getString(3); 
            allCustomer[i][2] = s3; 
             
            String s4 = rs.getString(4); 
            allCustomer[i][3] = s4; 
             
            String s5 = rs.getString(5); 
            allCustomer[i][4] = s5; 
             
            String s6 = rs.getString(6); 
            allCustomer[i][5] = s6;

sorgt höchstens für Ärger durch Tippfehler,
besser:

for (int k=0; k<5; k++) {
    String s = rs.getString(k+1); 
    allCustomer[i][k] = s; 
}

Ebenso eine Schleife für das System.out.println davor, oder gleich kombinieren:

String info = "Zeile "+i+": ";
for (int k=0; k<5; k++) {
    String s = rs.getString(k+1); 
    allCustomer[i][k] = s; 
    info += s+", ";
}
System.out.println(info);

 

[megachucky]

Hallo.

Danke für die ausführliche Antwort.


Also die DB Query habe ich ja schon getestet, die passt:

// Ausgabe nur zum testen
System.out.println(rs.getString(1) + " - " + rs.getString(2) + " - " + rs.getString(3) + " - " +
rs.getString(4) + " - " + rs.getString(5) + " - " + rs.getString(6));
System.out.println("RowCount: " + rowCount);


Das mit dem getRow "Trick" habe ich von JAVA Insel, das funktionert auch. Sind ja drei Zeilen, nicht nur die eine mit getRow().


Die Schleifen für die Strings und println sind eine gute Idee, die werde ich gleich einbauen.




Stimmt, das mit dem zweidim. Array ist wirklich einfach, ist wusste eben nur nicht genau, dass ich mit Array dann automatisch auf den "äußeren" Array zugreife. Das war mein Problem.

 

[megachucky]

Hab es jetzt mal so probiert, das will aber auch nicht ganz klappen.

Ich glaub nach der ersten for schleife muss noch was rein, steh aber grad irgendwie aufm Schlauch

public class TestGetAllCustomerQuery 
{
	public static void main(String[] args)
	{
		String [][] s = new Server().getAllCustomer();
		
		for (int i=0; i<s.length; i++)
		{
			for (int j=0; j<s.length; j++)
			{
				System.out.println(s[i].toString());
			}
		}
		
		
		
	}
}

 

[Lord BenneX]

Wie wärs mit:

public class TestGetAllCustomerQuery
{
public static void main(String[] args)
{
String [][] s = new Server().getAllCustomer();

for (int i=0; i<s.length; i++)
{
for (int j=0; j<s.length; j++)
{
System.out.println(s[j].toString());
}
}



}
}

Oder sehe ich das falsch ;D

Gruß

Lordy

 

[megachucky]

Ok, danke. Das geht nun.


Hab nun auch mal die Schleife für die Strings probiert. Da kommt bei mir leider bei der letzten Spalte jeweils 'null' statt dem String.

Ohne die Schleife funtioniert es aber. Finde leider den Logikfehler in der Schleife nicht.

int i=0;
			
			while ( rs.next() )
			{
				/*// Ausgabe nur zum Testen, ob die Daten aus der DB angekommen sind.
				System.out.println(rs.getString(1) + " - " + rs.getString(2) + " - " + rs.getString(3) + " - " +
						rs.getString(4) + " - " + rs.getString(5) + " - " + rs.getString(6));
				System.out.println("RowCount: " + rowCount);*/
				
				
				for (int k=0; k<5; k++) 
				{
				    String s = rs.getString(k+1);  // k+1 da Array bei 0 anfängt und ResultSet bei 1
				    allCustomer[i][k] = s;
				} 
				
				
				/*String s1 = rs.getString(1);
				allCustomer[i][0] = s1;
				
				String s2 = rs.getString(2);
				allCustomer[i][1] = s2;
				
				String s3 = rs.getString(3);
				allCustomer[i][2] = s3;
				
				String s4 = rs.getString(4);
				allCustomer[i][3] = s4;
				
				String s5 = rs.getString(5);
				allCustomer[i][4] = s5;
				
				String s6 = rs.getString(6);
				allCustomer[i][5] = s6;*/
				
				
				i++;
				
				
				
			}

 

[SlaterB]

k<5 falsch,
k<6 richtig

solche festen Werte wie die 6 kannst du auch in eigene Variablen tun,
falls sie mehrmals verwendet werden,

noch besser:
k<allCustomer.length()