ResultSet auf Einträge überprüfen

[needHelpInJava();]

Hallo, ich hole mir aus der Datenbank (Tabel Company / id,name) Daten raus und möchte es auf Einträge überprüfen. Leider muss ich mit dieser Überprüfung (siehe Code) ein zweites mal die Daten holen da (meine Vermutung) das erste "rs.next" eine Row überspringt. Wie bekomme ich eine richtige "if -überprüfung" hin?

public static void showCompanyList() throws SQLException {

        ResultSet rs = null;
        String sql = "SELECT * FROM company";

        try {
           rs = st.executeQuery(sql);
        }catch(SQLException e){
            System.out.println(e);
        }
        
        // Überprüfe ob es einen Eintrag gibt
        if(!rs.next()) {
            System.out.println("The company list is empty");
            System.out.println();
        }else {
            int counter = 1;
            
            try {
                rs = st.executeQuery(sql);
            }catch(SQLException e){
                System.out.println(e);
            }

            while (rs.next()) {
                System.out.println(counter + ". " + rs.getString("name"));
                counter++;
            }
            System.out.println();
        }
    }

 

[httpdigest]

Mach's doch so:

ResultSet rs = st.executeQuery(sql);
if (!rs.next()) {
  System.out.println("The company list is empty");
} else {
  int counter = 1;
  do {
    System.out.println(counter + ". " + rs.getString("name"));
  } while (rs.next());
}

 

[Jw456]

Setze mal das resultset nach der Abfrage der DB auf den Anfang.
first()

 

[needHelpInJava();]

Mach's doch so:

ResultSet rs = st.executeQuery(sql);
if (!rs.next()) {
  System.out.println("The company list is empty");
} else {
  int counter = 1;
  do {
    System.out.println(counter + ". " + rs.getString("name"));
  } while (rs.next());
}

Danke. Keine Ahnung wie ich da selber nicht darauf gekommen bin.

 

[KonradN]

Also ich würde von dem Standard Aufbau nicht abweichen - das führt in der Regel zu überkompliziertem Code.

Du machst die Abfrage und dann hast Du ein while (rs.next()). Das ist das, was man ja prinzipiell auch erwartet.

Dein Code kann dann so aussehen:

    public static void showCompanyList() throws SQLException {

        String sql = "SELECT * FROM company";

        try (ResultSet rs = st.executeQuery(sql)) {
            int counter = 0;

            while (rs.next()) {
                counter++;
                System.out.println(counter + ". " + rs.getString("name"));
            }

            if (counter == 0) {
                System.out.println("The company list is empty");
            }

            System.out.println();
        } catch(SQLException e){
            System.out.println(e);
        }
    }

Da ResultSet AutoClosable erbt, sollte man da prinzipiell sowas auch schließen - daher das try with resources.

 

[Neumi5694]

Also ich würde von dem Standard Aufbau nicht abweichen - das führt in der Regel zu überkompliziertem Code.

Du machst die Abfrage und dann hast Du ein while (rs.next()). Das ist das, was man ja prinzipiell auch erwartet.
[...]

@needHelpInJava(); Das wäre auch meine Standardlösung, allerdings würde ich gleich in eine Liste einlesen und am Ende die Länge der Liste prüfen, ohne Countervariable. Die Ausgabe auf den Bildschirm folgt dann später, nachdem der DB Zugriff beendet wurde.