Maak in een generieke database laag geen functies die voor 1 specifiek recept zijn

[JeroenHeemskerk]

De functie hieronder is voor een burger, maar dit is niet de bedoeling, je wilt de ingrediënten voor een Willekeurig recept kunnen opvragen. https://github.com/KimLamers/educom-verrukkulluk/blob/b45cbae6545dadb740f2296fe75e3b87d7fcfca4/lib/ingredient.php#L37-L49

Beter is het om als parameter een recept id mee te geven en daarvoor de ingrediënten ophalen:

 public function getIngredientsByRecipeId($recipe_id) { 

     $recipe_id = mysqli_real_escape_string($this->connection, $recipe_id); // See issue #17 
     $sql = "select * from ingredients where recipe_id = $recipe_id"; 

     $result = mysqli_query($this->connection, $sql); 
     //...
}

[JeroenHeemskerk]

Ik zou dit soort functies ook namen geven als:

[get|read|search|create|update|delete](All)?{{onderwerpnaam}}(By{{zoekcriterium}})? in plaats van select{{onderwerpnaam}} of insert{{onderwerpnaam}}

dus bijv.

function getAllArticles() { .. }
function createIngredient(Ingredient $new_ingredient) { ... }
function deleteUserById($user_id) { ... }
function getAllIngredientsForRecipeId($recipe_id) { ... }