Czy powinno się drukować kod html w php ?

Question

Chcę zrobić profil głowny użytkownika na portalu i mam problem. Otóż głowna jest podzielona na trzy kolumny, a  w każdej kolumnie znajdują się trzy informacje o użytkowniku. Zrobiłem tą tablicę w php drukując kod html. I tutaj mam pytanie czy takie podejście jest poprawne, a jeżeli nie to jak mógłbym wyświetlic te informacje z bazy i nie drukowac kodu html. Z góry dzięki.

class profil 
{
  private $database;
    
  public function __construct($database)
  {
   $this->database = $database;
  }
    public function display()
    {
    $id = $_GET['id'];
    $sql = $this->database->connect()->prepare("SELECT id, first_name, last_name, age, avatar, birth_date, gender FROM users where id = :id");
     $sql->bindParam(':id',$id, PDO::PARAM_INT);
            $sql->execute();

          if($sql->rowCount())
          {
         while ($row = $sql->fetch())
         {
                    echo "<div class='bg-dark col-xl-4'>";
                        echo "<div class='row'>";
                            echo "<div>";
                                echo "<div class='mb-4'>Country:"."</div>";
                            echo "</div>";
                        echo "</div>";
                        echo "<div class='row'>";
                            echo "<div>";
                                echo "<div class='mb-4'>City:"."</div>";
                            echo "</div>";
                        echo "</div>";
                        echo "<div class='row'>";
                            echo "<div>";
                                echo "<div class='mb-4'>Age: ".$row['age']. "</div>";
                            echo "</div>";
                        echo "</div>";
                    echo "</div>";
                    echo "<div class='bg-info col-xl-4'>";
                        echo "<div class='row'>";
                            echo "<div>";
                                echo "<div><img src=".$row['avatar']." height='250px'  class='img-thumbnail' width='200px'></div>";
                            echo "</div>";
                        echo "</div>";
                        echo "<div class='row'>";
                            echo "<div>";
                                echo "<div class='gp' id='name'>".$row['first_name']. " " .$row['last_name']."</div>";
                            echo "</div>";
                        echo "</div>";
                    echo "</div>";
                    echo "<div class='bg-danger col-xl-4'>";
                        echo "<div class='row'>";
                            echo "<div>";
                                echo "<div class='mb-4'Birth-date: ".$row['birth_date']. "</div>";
                            echo "</div>";
                        echo "</div>";
                        echo "<div class='row'>";
                            echo "<div>";
                                echo "<div class='mb-4'>Marital status: "."</div>";
                            echo "</div>";
                        echo "</div>";
                        echo "<div class='row'>";
                            echo "<div>";
                                if($row['gender']=='M')
                                {
                                    echo "<div class='mb-4'>Gender: Male</div>";
                                }else
                                {
                                    echo "<div class='mb-4'>Gender: Female</div>";
                                }
                            echo "</div>";
                        echo "</div>";
                    echo "</div>";
             }
      }
    }
 }

<header>
<div class=" container-fluid">
    <div class=" row">
    <?php
$profil = new profil();
$show = $profil->display();

?>
  
  </div>
  </div>
</header>

 

Answer 1

Ogólnie należy oddzielać widok strony od jej logiki. Można to robić na kilka sposobów, ale jeżeli jesteś na początku zabawy z PHP, możesz do tego zrobić oddzielną klasę, która renderuje jakiś szablon.

Przykładowa klasa:

class View
{
  public function render(array $params): void
  {
    // Główna templatka twojej strony
    require_once("templates/layout.php");
  }
}

W $params przekazujesz dane, która mają zostać wyświetlone na twojej stronie. W pliku, który w moim przypadku nazywa się layout.php, mogę wyświetlić te dane, załóżmy że wywołałem w jakimś miejscu metodę render, $view->render(['message' => 'Hello World!']), aby to wrzucić do templatki należy w niej zrobić coś takiego:

<!DOCTYPE html>
<html lang="en">
<head>
  <meta charset="UTF-8">
  <meta http-equiv="X-UA-Compatible" content="IE=edge">
  <meta name="viewport" content="width=device-width, initial-scale=1.0">
  <title>Document</title>
</head>
<body>
  <p><?php echo $params['message'] ?></p>
  // ALBO
  <p><?= $params['message'] ?></p>
</body>
</html>

Możesz tutaj dorzucać różne zagnieżdżenia plików, poprzez require/include, ale na przyszłość jeżeli chcesz iść w tym kierunku poczytaj sobie o MVC i o silnikach szablonów. Sam PHP może być też używany do wyświetlania danych w strukturze HTML, ale nie jest to najlepsza opcja.

Answer 2

How to make a simple HTML template engine in PHP -  Smarty

A template engine is used to separate the presentation from the business logic. A good developer knows this is very important - not only it allows for delegating responsibilities (the designer works on the presentation layer while the programmer works on the business logic), but it also provides a more easier maintenance.

 

There are a lot (and I mean A LOT) of template engines for PHP. A very popular example is Smarty. Most of these template engines have a lot of advanced options and require the user to learn a new syntax for building the template files.

• Twig ( The flexible, fast, and secure template engine for PHP )
• Latte ( Latte is a template engine for PHP which eases your work and ensures the output is protected against vulnerabilities, such as XSS. )
• Mustache