Requête de table multiple MySQLi

Puisque vous commandez par le nom de l'entreprise, vous devriez être en mesure de vérifier quand le nom de l'entreprise a changé afin de savoir quand créer un nouveau conteneur.

$currentCompany = "";

while($row = $joinresult->fetch_assoc()){
    $company = $row['name'];
    $fname = $row['fname'];
    $lname = $row['lname'];
    $name = $fname . " " . $lname;

    if($currentCompany != $company) {
        if($currentCompany != "")
            echo "</div>";

        $currentCompany = $company;

        echo "<div class='customer'>";
        echo "<div class='name'>";
        echo $company;
        echo "</div>";
    }

    echo "<div class='contacts'>";
    echo $name;
    echo "</div>";
}

Vous avez deux options, (A) utiliser une jointure ou (B) utiliser deux requêtes comme vous avez tenté de le faire. La jointure vous obligera à suivre la société pour un changement, mais les deux requêtes ne le seront pas. Il y a des avantages et des inconvénients aux deux options qui sont:

• A) Les jointures comme celle-ci sont rapides, mais vous retournerez les détails de la société pour chaque enregistrement, ce qui entraînera une charge de trafic supplémentaire dans les communications avec la base de données. Cela peut être un problème si le serveur de base de données se trouve sur un serveur séparé ou si vous avez affaire à un énorme ensemble de données. N'oubliez pas que même si les données supplémentaires semblent être une petite quantité, multipliez-les par le nombre de visiteurs simultanés attendus sur le site Web et vous pourriez saturer le lien de votre base de données.

  L'exemple fourni par @ abney317 est précis, mais au lieu de comparer le nom de l'entreprise, je comparerais le cid car non seulement il sera plus performant, mais il sera plus fiable.

• B) C'est la méthode que j'opterais, bien qu'elle soit un peu plus complexe à implémenter et nécessite deux requêtes au lieu d'une, c'est l'option la plus évolutive et maintenable dans le futur. Cela rendra également votre code un peu plus facile à suivre et à déboguer.

  Exemple:

  // Nested Query Test
  $rsCompanies = $mysqli->query("SELECT cid, name FROM companies ORDER BY name");
  while($company = $rsCompanies->fetch_assoc())
  {
    echo "<div class='customer'>";
    echo "<div class='name'>" . htmlspecialchars($company['name']) . "</div>";
    echo "<div class='contacts'>";
    echo "<ul>";
    $rsUsers = $mysqli->query("SELECT fname, lname FROM users WHERE cid = " . $company['id']);
    while($user = $rsUsers->fetch_assoc())
    {
      $name = htmlspecialchars($user['fname'] . " " . $user['lname']);
      echo "<li>$name</li>";
    }
    echo "</ul>";
    echo "</div>";
    echo "</div>";
  }
  

Quelques remarques supplémentaires:

• Utilisez toujours htmlspecialchars lors de la sortie de texte d'une base de données qui ne doit pas contenir de HTML, faute de quoi vous ouvrirez les vecteurs d'attaque XSS.

• Je n'ai pas échappé SQL à l'ID de l'entreprise, car selon le schéma de votre base de données, il s'agit d'un entier et comme il n'est pas fourni par l'utilisateur final, il est sûr de l'utiliser directement, évitant ainsi toute surcharge de traitement supplémentaire. p>

Si vous n'avez besoin d'afficher que quelques champs, essayez de récupérer les données en une seule requête: violon sql

SELECT c.name company_name,
group_concat(u.name) employees_name
FROM companies c JOIN users u on c.id = u.cid
group by c.name;