Forum PHP.pl > PHP/PDO - Połączenie z bazą danych wyświetlenie listy prodyktów

Pomoc - Szukaj - Użytkownicy - Kalendarz

Pełna wersja: PHP/PDO - Połączenie z bazą danych wyświetlenie listy prodyktów

wluczykuj

30.08.2020, 12:16:34

Cześć,

poniżej wklejam kody index.php i dbcontroller.php, prosiłbym o propozycje / wskazanie / podpowiedź co należałoby poprawić, zmienić, ulepszyć tak by było to jak najbardziej zbliżone do obecnych standardów, lepszych praktyk?

struktura plików w tym przypadku jest bardzo prosta:

index.php

[PHP] pobierz, plaintext 
<?php 
session_start();
include_once('dbcontroller.php');
 
?>
<html>
<head>
<title>Lista produktów z bazy danych</title>
<link href="style.css" type="text/css" rel="stylesheet" />
</head>
<body>
 
<div class="product-grid">
    <div class="txt-heading">Products</div>
 
 
<?php
    $product_array = $pdo->query("SELECT * FROM tblproduct ORDER BY id ASC");
    $result = $product_array->fetchAll();    
 
    if(!empty($result)){
            foreach($result as $key => $value){
                ?>
                <div class="product-item">
                       <div class="product-image">
                            <img src="<?php echo $result[$key]["image"];?>">
                       </div>
                       <div class="product-tile-footer">
                       <div class="product-title">
                            <?php echo $result[$key]["name"];?>
                       </div>
                        <div class="product-price">
                            <?php echo $result[$key]["price"]." PLN";?> 
                           </div>   
                       </div>
                </div>
 
                <?php
            }
        }
    else{
        echo "blad";
    }
    ?>
 
</div>
 
</body>
</html>
 
[PHP] pobierz, plaintext

dbcontroller.php

[PHP] pobierz, plaintext 
<?php
$host = "localhost";
$username = "root";
$password = "";
 
try {
  $pdo = new PDO("mysql:host=$host;dbname=db_shop", $username, $password);
        $pdo->setAttribute(PDO::ATTR_ERRMODE, PDO::ERRMODE_EXCEPTION);
  echo "Connected successfully";
} catch(PDOException $e) {
  echo "Connection failed: " . $e->getMessage();
}
?> 
[PHP] pobierz, plaintext

viking

30.08.2020, 15:01:24

Tylko co tu oceniać? Parę linijek kodu. Nie stosujesz composer co tu nie ma znaczenia ale w większym projekcie już robi różnicę. Nazewnictwo plików, mieszanie kodu php z html, jakieś zbędne zmienne z danymi połączenia, brak sprawdzania co zwróciło query, brak deklaracji doctype, prawie same divy w kodzie.

wluczykuj

30.08.2020, 15:41:48

Dziękuję za odpowiedź.

Uczę się. Nie znam się na tym. Próbuję. Wydaje mi się zatem, że najlepiej zacząć od czegoś prostego. Z pewnością ten przykład, który podałem, dla wyjadacza jest absurdalnie prosty.. Może nawet śmieszny. Ale na 100% jest tu wiele elementów które trzeba zrobić lepiej i dlatego prosiłem o parę wskazówek co poprawić. A nie gotowca na zasadzie: "Zróbcie za mnie, zapisze se na dysku. Zapomnę."

Więc, chce drążyć temat dalej:

1. "Nazewnictwo plików" - Co z nazewnictwem jest nie tak?

2. "mieszanie kodu php z html" -jak więc to ogarnąć najlepiej?"

3 ."jakieś zbędne zmienne z danymi połączenia" - masz tu na myśli:

[PHP] pobierz, plaintext 
    $product_array = $pdo->query("SELECT * FROM tblproduct ORDER BY id ASC");
    $result = $product_array->fetchAll();    
[PHP] pobierz, plaintext

Co tu można zaradzić?

4. brak sprawdzania co zwróciło query - o co tu chodzi?

5. brak deklaracji doctype - to rozumiem, dzieki.

6. prawie same divy w kodzie. - ok rozumiem, bardziej chciałem skupić się na nauce z php, ale dziekuje za zwrócenie uwagi.

I jeszcze moje pytania:
Ten kod w pliku dbcontroller.php jest ok, czy zupełnie nie tak się to powinno robić?

Pozdrawiam

viking

30.08.2020, 15:58:01

Poczytaj o standardach nazewnictwa zaczynając od PSR-1. Przejrzyj najlepiej wszystkie ogólnie żeby mieć jako takie pojęcie za co odpowiadają. Możesz sobie użyć w edytorze/konsoli narzędzi typu https://github.com/FriendsOfPHP/PHP-CS-Fixer. Dane połączenia miałem na myśli host, user, pass. Kompletnie tu zbędne a odpowiedź sobie na pytanie co by się stało gdybyś ten kod miał udostępnić wszystkim na githubie. Query może zwrócić false i dochodzisz do sytuacji false->fetchAll. W tym kodzie jest wszystko razem: nawiązanie połączenia, sesja, zapytania, błędy.

To jest wersja lo-fi głównej zawartości. Aby zobaczyć pełną wersję z większą zawartością, obrazkami i formatowaniem proszę kliknij tutaj.