Verbesserungen des Codes

Diese Seite verwendet Cookies. Durch die Nutzung unserer Seite erklären Sie sich damit einverstanden, dass wir Cookies setzen. Weitere Informationen

  • Verbesserungen des Codes

    Hey.

    Ich habe nur indirekt eine Frage,
    nämlich ob man den folgenden Code (er ist nicht von mir, aber open source)
    noch verbessern kann da er produktiv eingesetzt werden soll.

    Und vielleicht kann mir jemand noch verraten wie ich neben "name" auch noch "isbn" durchsuchen lassen kann?
    Isbn ist die Spalte neben Name, arbeitet man da weiterhin mit JOIN?

    Würde mich über Antworten freuen,
    Greetz

    Quellcode

    1. <p id="searchresults">
    2. <?php
    3. $db = new mysqli('localhost', 'xxx', 'xxx', 'xxx');
    5. if(!$db) {
    6. echo 'Kann mich nicht mit der Datenbank verbinden.';
    7. } else {
    8. if(isset($_POST['queryString'])) {
    9. $queryString = $db->real_escape_string($_POST['queryString']);
    11. if(strlen($queryString) >0) {
    12. $query = $db->query("SELECT * FROM search s INNER JOIN categories c ON s.cat_id = c.cid WHERE name LIKE '%". $queryString . "%' ORDER BY cat_id LIMIT 8");
    14. if($query) {
    16. $catid = 0;
    17. while ($result = $query ->fetch_object()) {
    18. if($result->cat_id != $catid) {
    19. echo '<span class="category">'.$result->cat_name.'</span>';
    20. $catid = $result->cat_id;
    21. }
    22. echo '<a href="'.$result->url.'">';
    23. echo '<img src="bilder/'.$result->img.'" alt="" />';
    25. $name = $result->name;
    26. if(strlen($name) > 35) {
    27. $name = substr($name, 0, 35) . "...";
    28. }
    29. echo '<span class="searchheading">'.$name.'</span>';
    31. $description = $result->desc;
    32. if(strlen($description) > 80) {
    33. $description = substr($description, 0, 80) . "...";
    34. }
    36. echo '<span>'.$description.'</span></a>';
    37. }
    38. echo '<span class="seperator"><a href="xxx" title="Kontakt">Nix gefunden?</a></span><br class="break" />';
    39. } else {
    40. echo 'Es gibt ein Problem mit dem query.';
    41. }
    42. } else {
    43. }
    44. } else {
    45. echo 'Denkst du wirklich es wäre so leicht? ;)';
    46. }
    47. }
    48. ?>
    49. </p>
    Alles anzeigen

    Dieser Beitrag wurde bereits 1 mal editiert, zuletzt von schreient ()

  • Naja, programmiertechnisch ist der Code nicht gut. Man sollte sich besser am Model-View-Controller Pattern orientieren.

    Was die Datenbank betrifft:
    * LIKE Abfragen mit % vorne und hinten sind nicht optimal, da der Index nicht genutzt werden kann.
    * es ist besser wenn du mit einem Volltextindex arbeitest (MATCH .. AGAINST)
    * befindet sich "name" in der categories Tabelle? Dann solltest du lieber von categories nach search joinen und nicht umgekehrt.
    * Im Fall von name solltest du einen Tabellen Prefix verwenden, damit jeder Programmierer weiß, welche Tabelle du abfragst
    * Wennmöglich solltest du nur die Spalten selektieren die du benötigst

    Lg

  • Ok, vielen Dank erneut für die Info(´s) und Antwort, ich hab werd mich damit näher beschaffen.
    War ich also doch in der richtigen Annahme das es mehr als nur eine Verbesserung geben kann und ich hier erneut richtig bin :)

    In der "categories" Tabelle befindet sich eine ID und cat_name, welches in meinem Falle nur für die Genre steht.
    In der "search" befinden sich alle anderen Infos, wie: ID, Name, Vorschaubild, Beschreibung und Ziel-URL.

    Ich hätte das ganze generell gerne später umgestrickt; da ich befürchte wenn in der search xxx Millionen Einträge sind, die Datenbank das nicht mitmacht,
    vom aufrufen in phpmyadmin oder dem auslesen, ganz zu schweigen.