lydiavangammeren / educom-webshop-database

0 stars 0 forks source link

Verdeel grote functies in kleinere functies #25

Open JeroenHeemskerk opened 1 year ago

JeroenHeemskerk commented 1 year ago

De functie om een order aan te maken is vrij groot, misschien kan je de volgende delen in functies plaatsen.

function insertOrder($conn, $userId)` {

https://github.com/lydiavangammeren/educom-webshop-database/blob/613af425ab6e3e591fadc348271d5f5de7a2bdc5/db_repository.php#L154-L163

}
function getMaxOrderNr($conn) {

https://github.com/lydiavangammeren/educom-webshop-database/blob/613af425ab6e3e591fadc348271d5f5de7a2bdc5/db_repository.php#L165-L172

}
function updateOrderNr($conn, $orderId, $updatedOrderNr) {

https://github.com/lydiavangammeren/educom-webshop-database/blob/613af425ab6e3e591fadc348271d5f5de7a2bdc5/db_repository.php#L175-L180

}
function insertOrderLines($conn, $orderId, $shoppingcartproducts) {

https://github.com/lydiavangammeren/educom-webshop-database/blob/613af425ab6e3e591fadc348271d5f5de7a2bdc5/db_repository.php#L182-L190

}

Hierdoor wordt je saveOrder functie iets als:

function saveOrder($user_id, $shoppingcartproducts)
{
    $conn = connectWithDB();
    try {
       $orderId= insertOrder($conn, $user_id);
       $maxOrderNr = getMaxOrderNr($conn);
       updateOrderNr($conn, $orderId, $maxOrderNr + 1);
       insertOrderLines($conn, $orderId, $shoppingcartproducts);
    } finally {
        mysqli_close($conn);
    }
}
JeroenHeemskerk commented 1 year ago

saveOrder gebruit $user_id dit is in snakeCase, terwijl de andere id's zoals $orderId in camelCase zijn. probeer hier consistent in te zijn.