Witaj Gościu! ( Zaloguj | Rejestruj )

Forum PHP.pl

> PHP/PDO - Połączenie z bazą danych wyświetlenie listy prodyktów, prosba o propozycje ulepszenia, poprawienia kodu
wluczykuj
post
Post #1





Grupa: Zarejestrowani
Postów: 22
Pomógł: 0
Dołączył: 13.12.2016

Ostrzeżenie: (0%)
-----

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 
  1. <?php 
  2. session_start();
  3. include_once('dbcontroller.php');
  4.  
  5. ?>
  6. <html>
  7. <head>
  8. <title>Lista produktów z bazy danych</title>
  9. <link href="style.css" type="text/css" rel="stylesheet" />
  10. </head>
  11. <body>
  12.  
  13. <div class="product-grid">
  14.     <div class="txt-heading">Products</div>
  15.  
  16.  
  17. <?php
  18.     $product_array = $pdo->query("SELECT * FROM tblproduct ORDER BY id ASC");
  19.     $result = $product_array->fetchAll();    
  20.  
  21.     if(!empty($result)){
  22.             foreach($result as $key => $value){
  23.                 ?>
  24.                 <div class="product-item">
  25.                        <div class="product-image">
  26.                             <img src="<?php echo $result[$key]["image"];?>">
  27.                        </div>
  28.                        <div class="product-tile-footer">
  29.                        <div class="product-title">
  30.                             <?php echo $result[$key]["name"];?>
  31.                        </div>
  32.                         <div class="product-price">
  33.                             <?php echo $result[$key]["price"]." PLN";?> 
  34.                            </div>   
  35.                        </div>
  36.                 </div>
  37.  
  38.                 <?php
  39.             }
  40.         }
  41.     else{
  42.         echo "blad";
  43.     }
  44.     ?>
  45.  
  46. </div>
  47.  
  48. </body>
  49. </html>
  50.  
[PHP] pobierz, plaintext


dbcontroller.php
[PHP] pobierz, plaintext 
  1. <?php
  2. $host = "localhost";
  3. $username = "root";
  4. $password = "";
  5.  
  6. try {
  7.   $pdo = new PDO("mysql:host=$host;dbname=db_shop", $username, $password);
  8.         $pdo->setAttribute(PDO::ATTR_ERRMODE, PDO::ERRMODE_EXCEPTION);
  9.   echo "Connected successfully";
  10. } catch(PDOException $e) {
  11.   echo "Connection failed: " . $e->getMessage();
  12. }
  13. ?> 
[PHP] pobierz, plaintext
Go to the top of the page
+Quote Post
 
 Start new topic
Odpowiedzi
viking
post
Post #2





Grupa: Zarejestrowani
Postów: 6 381
Pomógł: 1116
Dołączył: 30.08.2006

Ostrzeżenie: (0%)
-----

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.
Go to the top of the page
+Quote Post

Posty w temacie
- wluczykuj   PHP/PDO - Połączenie z bazą danych wyświetlenie listy prodyktów   30.08.2020, 12:16:34
- - viking   Tylko co tu oceniać? Parę linijek kodu. Nie stosu...   30.08.2020, 15:01:24
- - wluczykuj   Dziękuję za odpowiedź. Uczę się. Nie znam się na...   30.08.2020, 15:41:48
- - viking   Poczytaj o standardach nazewnictwa zaczynając od P...   30.08.2020, 15:58:01

« Następny starszy · PHP · Następny nowszy »
 

Reply to this topicStart new topic
2 Użytkowników czyta ten temat (2 Gości i 0 Anonimowych użytkowników)
0 Zarejestrowanych:

 

Tryb wyświetlania: Przełącz na: Standardowy · Przełącz na: Linearny+ · Drzewo

Śledź ten temat · Wyślij temat na e-mail · Wydrukuj ten temat · Subskrybuj to forum

RSS Aktualny czas: 28.09.2025 - 03:50
Powered By IP.Board © 2025  IPS, Inc.
All changes by PHP.pl Administrators
Hosting zapewnia NQ.pl hosting, trac, svn