Witaj Gościu! ( Zaloguj | Rejestruj )

Forum PHP.pl

 
 Reply to this topicStart new topic
> PHP/PDO - Połączenie z bazą danych wyświetlenie listy prodyktów, prosba o propozycje ulepszenia, poprawienia kodu
wluczykuj
post 30.08.2020, 12:16:34
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
viking
post 30.08.2020, 15:01:24
Post #2





Grupa: Zarejestrowani
Postów: 6 365
Pomógł: 1114
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.


--------------------
Odpowiedzi na często zadawane pytania:
Konfiguracja serwera Apache + PHP 7 pod Windows | Kodowanie znaków na stronach www | PHPTAL w Zend Framework | Programowanie obiektowe w PHP | PDO uniwersalnym sposobem na obsługę baz danych | Kurs PHP (część 1): Podstawowy opis języka
Go to the top of the page
+Quote Post
wluczykuj
post 30.08.2020, 15:41:48
Post #3





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

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

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 
  1.     $product_array = $pdo->query("SELECT * FROM tblproduct ORDER BY id ASC");
  2.     $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
Go to the top of the page
+Quote Post
viking
post 30.08.2020, 15:58:01
Post #4





Grupa: Zarejestrowani
Postów: 6 365
Pomógł: 1114
Dołączył: 30.08.2006

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

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.


--------------------
Odpowiedzi na często zadawane pytania:
Konfiguracja serwera Apache + PHP 7 pod Windows | Kodowanie znaków na stronach www | PHPTAL w Zend Framework | Programowanie obiektowe w PHP | PDO uniwersalnym sposobem na obsługę baz danych | Kurs PHP (część 1): Podstawowy opis języka
Go to the top of the page
+Quote Post
« Następny starszy · PHP · Następny nowszy »
 

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

 

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

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

RSS Wersja Lo-Fi Aktualny czas: 19.04.2024 - 16:30
Powered By IP.Board © 2024  IPS, Inc.
All changes by PHP.pl Administrators
Hosting zapewnia NQ.pl hosting, trac, svn