Mit Wartbarkeit auf diesem MySQLi/PHP-Code-Segment zu kämpfen

Question

Ich habe ein Stück Code, der funktioniert, aber es ist ziemlich chaotisch. Ziel ist es, alle relevanten Zeilen aus einer Benutzersuche abzurufen. Zum Beispiel:

Der Code, der funktioniert:

if($_SERVER["REQUEST_METHOD"] == "POST") { 
    $search = trim(filter_input(INPUT_POST,"user_search",FILTER_SANITIZE_SPECIAL_CHARS)); 
    $search = preg_replace("#[^0-9a-z]#i", "", $search); 

    if(!isset($error_message)) { 
    $query = $db->query("SELECT * FROM User WHERE U_Forename LIKE '%$search%' 
        OR U_Surname LIKE '%$search%' OR U_Team LIKE '%$search' 
        OR U_City LIKE '%$search%'"); 
    } 
} 

    <?php 
      if ($query->num_rows == 0) { 
      $output = "No results found!"; 
      } else { 
      echo "<h2>Search results for: " . $search . "</h2>"; 
      while($row = $query->fetch_assoc()) { 
       $id = $row['U_ID']; 
       $firstname = $row['U_Forename']; 
       $lastname = $row['U_Surname']; 
       $team = $row['U_Team']; 
       $city = $row['U_City']; 
       echo "<div class='searchresults'>"; 
       echo "<p> Name: " . ucfirst($firstname) . " " . ucfirst($lastname) . "</p>"; 
       echo "<p> Favourite Team: " . ucfirst($team) . "</p>"; 
       echo "<p> Location: " . ucfirst($city) . "</p>"; 
       echo "<a class='lift' href='profile.php?=[$id]'>View Profile</a>"; 
       echo "<a class='lift' href='#'>Follow Driver</a>"; 
       echo "</div>"; 
      } 
      } 
      ?> 

Was ich versucht habe: Wenn der Nutzer nach 'Jo', wird es John und Josh abrufen

function getSearchResults($search) { 
    global $db; 
    $searchArray = array(); 
    $query = $db->query("SELECT * FROM User WHERE U_Forename LIKE '%$search%' 
        OR U_Surname LIKE '%$search%' OR U_Team LIKE '%$search' 
        OR U_City LIKE '%$search%'"); 
    if ($query->num_rows == 0) { 
    echo "No results found!"; 
    } else { 
    while($row = $query->fetch_assoc()) { 
     $searchArray['U_ID'] = $row['U_ID']; 
     $searchArray['U_Forename'] = $row['U_Forename']; 
     $searchArray['U_Surname'] = $row['U_Surname']; 
     $searchArray['U_Team'] = $row['U_Team']; 
     $searchArray['U_City'] = $row['U_City']; 
    } 
    } 
    return $searchArray; 
} 

Allerdings wird dies nicht arbeiten als nur das erste Ergebnis von der fetch_assoc() ist in der $ searchArray und nichts anderes gespeichert! Hat jemand irgendwelche Vorschläge? Ich bin mir nicht sicher, wo ich als nächstes anfangen soll.

James.

Answer 1

Sie überschreiben das Array jedes Mal, wenn Sie die Schleife ausführen. Sie müssen ein Array von Arrays erstellen, damit Sie ein Array zurückgeben können. Im Folgenden habe ich Ihren Code vereinfacht, um die aktuelle Zeile an das Array anzufügen, das Sie zurückgeben. Sie können dann eine foreach auf die zurückgegebenen Daten tun und Ihren Code

while($row = $query->fetch_assoc()) { 
    $searchArray[] = $row; 
} 

auch angezeigt werden, eine protip: Ihr DB-Anschluss als Parameter übergeben und vermeiden Sie es als eine globale Verwendung (die andere Best Practice ist)

function getSearchResults($search) { 
    global $db; 

Wird

function getSearchResults($search, $db) { 

Und last but not least, nicht über Ihre Echo Dinge Funktion (haben die Ebene, die diese Funktion tun genannt). Stattdessen nur ein leeres Array zurück

if ($query->num_rows == 0) { 
    return $searchArray; 
}