sql query in methode mit rückgabetyp

[dvdlly]

Hallo,
Ich möchte ein Kochrezept Empfehlungssystem programmieren. Ich habe eine Methode getRecommendation, die den ältesten Eintrag aus der Tabelle Recipes wählt, dann alle Ingredients auswählt, die in der Tabelle RecipesIngredients zu diesem Rezept gehören und dann ein Recipe objekt erzeugt, das Ingredient objekte enthält, die gemäß der Tabelle zu dem Rezept korrepsondieren.
Das Problem: ich möchte das Recipe objekt zurückgeben, aber die sql query muss ja im try block erfolgen. Gebe ich das Recipe objekt in dem try block zurück, dann kompiliert der code nicht. Gebe ich ein Recipe objekt am Ende der Methode getRecommendation zurück, dann enthält das objekt nicht die informationen aus dem try block.
[CODE lang="java" title="getRecommendation"]public Recipe getRecommendation() {
String url = "jdbc:mysql://localhost/Cooking_Recipes";
try {
Connection conn = DriverManager.getConnection(url,"root","");
Statement statement = conn.createStatement();
ResultSet ingredients = statement.executeQuery("select RI.Name,I.Name from ((select * from Recipes order by LastCooked asc limit 1) R right join RecipesIngredients RI on R.\n" +
"ID = RI.RecipeID) right join Ingredients I on RI.IngredientID = I.ID ;");
ingredients.next();
Recipe recipe = new Recipe(ingredients.getString(1));
LinkedList<Ingredient> all_ingredients = new LinkedList<>();
do {
Ingredient ingredient = new Ingredient(ingredients.getString(2));
all_ingredients.add(ingredient);
} while (ingredients.next());
recipe.addIngredients(all_ingredients);
return recipe;
}
catch (Exception e) {
e.printStackTrace();
}
// HERE NEEDS TO BE A RETURN STATEMENT
}[/CODE]

 

[mihe7]

Das Problem

Wenn das das einzige Problem wäre...

Aber zu Deiner Frage: das ist auch völlig richtig so. Wenn das Rezept zusammengestellt werden konnte, wir das Rezept zurückgegeben (try-Block). Tritt ein Fehler auf, musst Du Dir Überlegen, was Du tun willst:

a) ein Objekt zurückgeben? Wenn ja, welches? Ein Dummy-Rezept? Macht das Sinn? Evtl. ein Optional<Rezept>?
b) null zurückgeben, um anzuzeigen, dass kein (passendes) Rezept gefunden wurde?
c) eine Exception auslösen?

 

[dvdlly]

Was gibts denn noch für Probleme?

 

[Gelöschtes Mitglied 65838]

prepared statements...
try with ressources...


das letzte return musst du machen weil du theroetisch in die exception kommst und dann keinen return mehr hast
das kannst du auch umgehen indem du eine Runtime Exception wirfst oder eine eigene exception wirfst

 

[dvdlly]

Ich habe jetzt einfach ein Recipe objekt vor dem try block instanziiert, das ich danach ausgebe. Ich verstehe nicht so ganz, was du mit "prepared statements..." und "try with ressources..." sagen willst

 

[Gelöschtes Mitglied 65838]

deine Verbindung wird nicht geschlossen dh wenn du die methode zum zweiten mal aufrufst wird ein fehler kommen da die verbindung bereits offen ist bzw kann zu problemen führen
ein try with ressource schließt dir automatisch die verbindung


die query die du hast ist nicht sicher, kann ganz einfach mit "sql injection" ( findest du ganz viel im internet) verändert werden... deswegen ist der "mindest" standard für alles mit datenbanken dass man mindestens prepared statements benutzt



eine lösung ( ist bei weitem NICHT die beste ) wäre das

public Recipe getRecommendation() {
        String url = "jdbc:mysql://localhost/Cooking_Recipes";



    Recipe recipe = null;



        try {
            Connection conn = DriverManager.getConnection(url,"root","");
            Statement statement = conn.createStatement();
            ResultSet ingredients = statement.executeQuery("select RI.Name,I.Name from ((select * from Recipes order by LastCooked asc limit 1) R right join RecipesIngredients RI on R.\n" +
                    "ID = RI.RecipeID) right join Ingredients I on RI.IngredientID = I.ID ;");
            ingredients.next();
            recipe = new Recipe(ingredients.getString(1));
            LinkedList<Ingredient> all_ingredients = new LinkedList<>();
            do {
                Ingredient ingredient = new Ingredient(ingredients.getString(2));
                all_ingredients.add(ingredient);
            } while (ingredients.next());
            recipe.addIngredients(all_ingredients);

// nicht mehr hier returnen bie dieser lösung


        }
        catch (Exception e) {
            e.printStackTrace();
        }



        return recipe;



    }

ich weis halt nicht inwiefern dich design und clean code und sichcerheit interessiert... in letzter zeit wurde ich zu oft angenörgelt dass ich zu viel input geb und deswegen werd ich jetzt auch erstmal auf "wenig input" tendieren

 

[dvdlly]

Ups, dass die connection nicht geschlossen wird war ein versehen. Hast du nicht gesagt, es müsste noch ein prepared statement benutzt werden?

 

[Gelöschtes Mitglied 65838]

ja das prepared statement ist die art und weise wie die db mit deinem statement umgeht

im moment gibst du ein "query" an deine db die ist aber anfällig... ich hab jetzt nur das "return" problem behandelt in dem code

 

[mihe7]

Man könnte das in etwa so angehen:

class RecipeRepository {
    private final ConnectionSupplier connections;

    private static final int LAST_RECIPE_NAME = 1;
    private static final int LAST_RECIPE_INGREDIENT = 2;

    private static final String QUERY_LAST_RECIPE = 
            "select RI.Name,I.Name " +
            "from (select * from Recipes order by LastCooked asc limit 1) R" +
            "inner join RecipesIngredients RI on R.ID = RI.RecipeID) " +
            "inner join Ingredients I on RI.IngredientID = I.ID"

    public RecipeRepository(ConnectionSupplier connections) {
        this.connections = connections;
    }

    public Optional<Recipe> findRecommendation() {
        try(Connection conn = connections.get();
            PreparedStatement statement = conn.prepareStatement(QUERY_LAST_RECIPE);
            ResultSet queryResult = statement.executeQuery()) {

            if (queryResult.next()) {
                return Optional.of(loadRecipe(ingredients));
            }
        } catch (SQLException ex) {
            ex.printStackTrace();
            return Optional.empty();
        }
    }

    private Recipe loadRecipe(ResultSet queryResult) throws SQLException {
        Recipe recipe = new Recipe(queryResult.getString(LAST_RECIPE_NAME));
        recipe.addIngredients(loadIngredients(queryResult));
        return recipe;
    }

    private List<Ingredient> loadIngredients(ResultSet queryResult) throws SQLException {
        List<Ingredient> ingredients = new LinkedList<>();
        do {
            ingredients.add(new Ingredient(ingredients.getString(LAST_RECIPE_INGREDIENT)));
        } while (ingredients.next());

        return ingredients;
    }
}

ConnectionSupplier ist dabei einfach ein Interface

@FunctionalInterface
public interface ConnectionSupplier {
    Connection get() throws SQLException;
}

um unabhängig vom verwendeten "Verbindungsmechanismus" zu sein, wobei ich einen Connection-Pool empfehlen würde.

Das SQL habe ich etwas umgeschrieben, weil ich nicht glaube, dass Du alle Zutaten haben willst und dann das ggf. passende Rezept dazu. Falls doch, nimm einfach wieder Dein SQL.

Was das PreparedStatement betrifft: das wäre in diesem Fall aus Sicherheitsgründen nicht erforderlich, da Du kein SQL zusammenbaust.

Ach so: der Spaß ist nur runtergeschrieben, nicht getestet und kann Fehler enthalten.

 

[dvdlly]

Danke 🙂

 

[Gelöschtes Mitglied 65838]

eine "Exception e" zu werfen ist mager... wirf gezielte exceptions mit "gutem" inhalt... beste biespiel ist wenn du in javafx css lädsts da kommt die meldung

css couldnt be loaded

ja DANKE... und WARUM... WO hast du gesucht.... ach ich könnte den ganzen tag pulvern 😀

 

[dvdlly]

Edit: Hab ne blöde frage gestellt, aber weiß nicht wie man posts löscht.

 

[fhoffmann]

Deine Methode wird so nicht fuktionieren. Das ResultSet enthält möglicherweise noch nicht alle Ergebnisse; ein Teil der Ergebnisse wird möglicherweise erst von der Datenbank abgefragt, wenn du result.next() aufrufst; das ist aber nicht mehr möglich, wenn du die Verbindung zur Datenbank bereits geschlossen hast.

 

[dvdlly]

Wie würdet ihr eine SQL query schreiben, die für ein gegebenes Recipe objekt prüft, ob Zutaten und das Rezept selbst in der DB enthalten sind und falls das nicht zutrifft (für einzelne Zutaten oder das Rezept selbst) diese in die Datenbank einfügt?

 

[mihe7]

Wenn ich Dich richtig verstehe, willst Du ein Rezept komplett speichern, wobei die Zutaten ggf. ergänzt werden sollen. Du könntest die ID von Rezept und Zutat in die Klassen aufnehmen, dann kannst Du anhand dieser feststellen, was schon eine ID bekommen hat.

Allerdings: die Abbildung von Objekten auf RDBMS wird sehr schnell ein sehr komplexes Thema. Die Queries sind ja nur ein kleiner Teil, dazu kommt das Transaktionsmanagement oder der Wunsch, Änderungen an den Objekten automatisch nachzuverfolgen (Unit of Work) und das ganze auch noch mit etwas Effizienz (Cache). Daher würde ich Dir bei solchen Anforderungen zu einem ORM wie Hibernate oder EclipseLink raten. Da ist zwar auch einiges zu lernen/berücksichtigen aber das Gros der Arbeit nimmt Dir der ORM ab.