MySQL 10 höchsten Werte aus einer Tabelle ermitteln

Dimax

Top Contributor
Hallo Leute,
wie kann man 10 höchsten Werte aus einer Tabelle ermitteln?
Ich habe eine Tabelle "Programm". Das sieht ungefähr so aus
projekt_id | programm_id | komplexität |

ich brauche 10 programm_id's mit dem größten Komplexitätswert.

Vielen dank im voraus .
 
Hab grade getestet,dass war die Lösung die ich gesucht habe. Danke Robat nochmals.Weist du vielleicht auch wie man die Ergebnisse in Variablen speichert.
ich hab bis jetzt ResultSet benutzt. ResultSet.getString(columnindex), aber wenn ich im column 1 10 rows hab wie kriege ich column 1 row 2 zmb.?
 
Java:
ResultSet resultSet = stmt.execteQuery(query);
while(resultSet.next()) { // für jede row im ResultSet
     String x = resultSet.getString("firstCol");
     int y = resultSet.getInt("secondCol");
}
 
aber wenn ich im column 1 10 rows hab wie kriege ich column 1 row 2 zmb.?
Du hast ein ResultSet, z. B. rs, dann
Java:
List<Long> ergebnis = new ArrayList<>();
while (rs.next()) {
    ergebnis.add(rs.getLong(1));
}
return ergebnis;
Mit der Liste kannst Du dann machen, was Du willst, z. B. liste.get(1); liefert die Programm-ID des Programms mit der zweitgrößten Komplexität.
 
Danke Jungs für die Hilfe.
Hab probiert grade.
Java:
public static String [][] analysieren()  {
        List<String> topTenKomplex=new ArrayList<>();
        String topTen=new String("SELECT programm_id FROM Programm"+
                                                "ORDER BY komplexität  DESC LIMIT 10");
try{
Connection cn=null;          
            Class.forName(sDbDrv);
            cn = DriverManager.getConnection(sDbUrl, sUsr, sPwd);
Statement st1=null;
ResultSet rs1=null;
st1=cn.createStatement();
rs1=st1.executeQuery(topTen);
while(rs1.next()){
                topTenKomplex.add(rs.getString(1));
            }
System.out.println(topTenKomplex.get(1));
}catch(Exception e){e.printStackTrace();}
}
Folgende Fehelermeldung:
Java:
java.sql.SQLException: After end of result set
    at com.mysql.cj.jdbc.exceptions.SQLError.createSQLException(SQLError.java:127)
    at com.mysql.cj.jdbc.exceptions.SQLError.createSQLException(SQLError.java:95)
    at com.mysql.cj.jdbc.exceptions.SQLError.createSQLException(SQLError.java:87)
    at com.mysql.cj.jdbc.exceptions.SQLError.createSQLException(SQLError.java:61)
    at com.mysql.cj.jdbc.result.ResultSetImpl.checkRowPos(ResultSetImpl.java:506)
    at com.mysql.cj.jdbc.result.ResultSetImpl.getString(ResultSetImpl.java:869)
    at daten.einlesen.ProgrammAnalyse.analysieren(ProgrammAnalyse.java:56)
    at daten.einlesen.ProgrammAnalyse.main(ProgrammAnalyse.java:19)
woran kann es liegen?
 
Noch ein paar Anmerkungen: die Connection erzeugt man für gewöhnlich nicht jedesmal neu und Class.forName ist bei einem aktuellen JDBC-Treiber nicht mehr notwendig. new String() ist überflüssig. Die Initialisierungen mit null kannst Du Dir auch sparen. Außerdem kannst Du try-with-resources verwenden. Sollte in etwa so funktionieren:

Java:
public static void analysieren()  {
    String topTen = "SELECT programm_id FROM Programm"+
            " ORDER BY komplexität  DESC LIMIT 10";
    try(Connection cn = DriverManager.getConnection(sDbUrl, sUsr, sPwd);
          Statement st = cn.createStatement();
          ResultSet rs = st.executeQuery(topTen)) {
        List<String> topTenKomplex=new ArrayList<>();
        while (rs.next()) {
            topTenKomplex.add(rs.getString(1));
        }
        System.out.println(topTenKomplex.get(1));
    } catch (SQLException ex) {
        ex.printStackTrace();
    }
}
 
Ich erzeuge Connection in jeder Methode wo ich sie brauch und zum Schluss schlisse. Ist das Falsch?
Falsch wäre es, wenn es nicht funktioniert 🙂

Warum man das normalerweise nicht macht, hat wenigstens zwei Gründe:

1. es ist relativ teuer, eine Verbindung aufzubauen
2. Kopplung und Kohäsion

Grundsätzlich muss man immer abwägen. Wenn ich in meinem Programm an einer Stelle alle Stunde mal eine DB-Abfrage machen müsste, dann würde ich da auch nicht rumeiern.

Hat man z. B. ein UI, wo man ständig die DB bemühen, ggf. für eine "Maske" mehrere Abfragen ausführen muss, dann sieht die Sache bzgl. 1. schon anders aus.

Gleiches gilt für die Programmstruktur: Verbindungsaufbau ist eine Sache, Verbindung benutzen, um Abfragen auszuführen, eine andere.

Wenn die Klasse (oder gar die Methode), die die Abfrage durchführt, auch die Verbindung aufbaut, sind diese Dinge stark gekoppelt (bilden also eine Einheit), obwohl sie logisch nicht zusammengehören (schwache Kohäsion). Das erklärte Ziel ist das genaue Gegenteil davon.
 
Falsch wäre es, wenn es nicht funktioniert 🙂

Wenn die Klasse (oder gar die Methode), die die Abfrage durchführt, auch die Verbindung aufbaut, sind diese Dinge stark gekoppelt (bilden also eine Einheit), obwohl sie logisch nicht zusammengehören (schwache Kohäsion). Das erklärte Ziel ist das genaue Gegenteil davon.
Wenn ich dich richtig verstanden habe ,wäre dann für mich besser eine Methode zu basteln die mir Verbindung bereitstellt.So was wie
Java:
    public static Connection eureVerbindungMeinHerr(){      
        String sDbUrl = MYSQLparameter.getURL();
        String sUsr = MYSQLparameter.getUser();
        String sPwd = MYSQLparameter.getPassword();
       
        try{
            Connection cn=DriverManager.getConnection(sDbUrl, sUsr, sPwd);
        }catch(SQLException e){e.printStackTrace();}
        return cn;       
    }
Und dan überall wo ich es brauch die Methode aufrufen?
Java:
public static irgendeineMethode(){
Connection cn=eureVerbindungMeinHerr();
//hier mache ich etwas
cn.close()
}
 
So passiert genau das gleiche wie jetzt, in jeder Methode wird die Verbindung neu hergestellt und auch geschlossen und die Methode ist selbst dafür verantwortlich.

Besser wäre es, der Methode die Connection zu übergeben - Erstellen und schließen ist dann nicht mehr Verantwortung der einzelnen Methoden und du kannst, wenn nötig, die Verbindung mehrmals verwenden.
 
Jein. Ein Teil von dem, was Du jetzt gemacht hast, ist DRY (don't repeat yourself). Dieses Prinzip solltest Du grundsätzlich möglichst überall durchsetzen.

Du rufst in irgendeineMethode jetzt eine globale Methode auf. D. h. das Verhältnis zwischen irgendeineMethode und eureVerbindungMeinHerr ist immer noch fix im Code; Du kannst Die Verbindung nicht tauschen.

Was ich meinte, ist etwas wie:

Java:
class Users {
    private static final String SQL_LOGIN =
        "SELECT 1 FROM users WHERE username=? AND password=?";
    private static final String SQL_COUNT =
        "SELECT count(*) FROM users";

    private Connection conn;

    public Users(Connection conn) { this.conn = conn; }

    public boolean login(String username, String password) throws SQLException {
        try(PreparedStatement stmt = conn.prepareStatement(SQL_LOGIN)) {
            stmt.setString(1, username);
            stmt.setString(2, hash(password));
            try(ResultSet rs = stmt.executeQuery()) {
                return rs.next();
            }
        }
    }

    public int count() throws SQLException {
        try(Statement stmt = conn.createStatement();
             ResultSet rs = stmt.executeQuery(SQL_COUNT);
             int result = 0;
             if (rs.next()) { result = rs.getInt(1); }
             return result;
        }
    }
}
 
Danke Jungs für die Bemühungen mir etwas beizubringen.
Leider muss ich feststellen ,dass ich mich noch auf einem tiefen Wissensstand befinde und
die letzten Vorschläge nicht umsetzen kann.
Aber auch weil mein Programm ,ein internes Projekt mit nur 1 User ,pwd usw. ist.
Also so was wie class Users brauch ich nicht.Ist alles bei mir in MYSQLparameter gespeichert.
Und für den Prinzip des Verbindungsaufbau sowie auch für die Schließung hab leider kein Plan(.
Den code von mihe7 kann ich verstehen aber für mein Programm ist es nicht nötig die Verbindungen
an mehrere users zu koppeln .
 
Darum:
Wenn ich in meinem Programm an einer Stelle alle Stunde mal eine DB-Abfrage machen müsste, dann würde ich da auch nicht rumeiern.
🙂

Probleme bekommst Du vermutlich erst, wenn Dein Code komplexer wird. So lange es für Dich in Ordnung ist, ist doch alles gut.

für mein Programm ist es nicht nötig die Verbindungen
an mehrere users zu koppeln .
?!?

Nochmal zur Klarstellung: es geht darum, eine Verbindung an verschiedenen Stellen zu verwenden. Die Klasse Users war dabei nur ein Beispiel. Ich hätte auch, wie von @mrBrown vorgeschlagen, den Parameter einfach an die Methode hängen können:
Java:
public void analyze() {
    Connection conn = neueVerbindungAufbauen();
    ...
    conn.close();
}

// vs

public void analyze(Connection conn) {
    ...
}
 
Das ist meine Angewohnheit von C. Da wird der Speicherplatz für neue Variable nur erteilt nicht geleert und deswegen kann neue Variable alles mögliche enthalten.Ich glaube in Java ist es anders aber bin mir nicht sicher.Wie ist es in Java?
In lokalen Variablen steht auch in Java „nichts“ drin, man kann deshalb nicht lesend drauf zugreifen, bis sie zugewiesen sind.

Initialisieren mit null ist aber deshalb Unsinn, weil du direkt danach einen richtigen Wert zuweißt. In C wären solche Zuweisungen mit Default-Werten genauso überflüssig 😉
 

Zurück
Oben