Aiuto, non trovo l'errore

[mademimo]

Ciao,
sono nuova e ho bisogno del vostro aiuto. Sto seguendo una guida su youtube dove c'è un corso su come creare una ricerca all'interno del proprio sito con php e mysql e al 4° video mi blocco perchè mi da il seguente errore:

Hai cercato la parola google search
Notice: Undefined variable: x in C:\xampp\htdocs\searchengine\search.php on line 29

Notice: Undefined variable: construct in C:\xampp\htdocs\searchengine\search.php on line 31
1 risultati trovati!

Mentre il codice php è il seguente:

<?php

//get data
$button = $_GET ['submit'];
$search = $_GET ['search'];

if (!$button)
    echo "Non hai inserito la parola";
else
{
    if (strlen ($search) <=2)
        echo " Termine troppo corto";    
    else
    {
        echo "Hai cercato la parola [b]$search[/b]<hr size='1'>";
        
        //connect to our database
        mysql_connect("localhost","root","root");
        mysql_select_db("phpmyadmin");
        
        
            //explode our search term
            $search_exploded = explode(" ",$search);
            
            foreach($search_exploded as $search_each)
            {

                //construct query
                $x++;
                if ($x==1)
                    $construct .= "keywords LIKE '%$search_each%'";
                else
                    $construct .= "OR keywords LIKE '%$search_each%'";

            }

    //echo out construct
    
    $construct = "SELECT * FROM searchengine WHERE $construct";
    $run = mysql_query($construct);
    
    $foundnum = mysql_num_rows($run);
    
    if ($foundnum==0)
        echo "Nessun risultato trovato.";
    else
    {
        echo "$foundnum risultati trovati!

";
    
        
    }
    
    }
}
?>

Mi potete aiutare a capire come mai mi da quest'errore?

Vi ringrazio anticipatamente,
ciao
Debora

[m4rko80]

Usi $construct .= che serve ad aggiugere qualcosa a $construct ma non l'hai mai creata prima.

Dovresti avere a priori $construct = ''; o semplicemente la variabile senza valore prima.

[luca200]

Sono errori che non pregiudicano il funzionamento, ma effettivamente il codice così è piuttosto brutto. Basta aggiungere

$x=0;
$construct = ''; 

prima del foreach

[bstefano79]

gia che ci sei farei anche

$search=trim($search); 

prima di controllare la lunghezza di $search

[mademimo]

Grazieeee

ho perso una giornata a capire dove sbagliavo. E' bastato aprire una discussione e tempo 5 minuti me l'avete già risolta... siete grandi!

Veramente tante grazie, ciao
Debora

[mademimo]

Vorrei che quando premo enter o clicco sul bottone search, mi invia il form, questo il codice html:

<form action="search.php" method="get">
<font face="Tahoma, Geneva, sans-serif" size="5">
    <center>
    	Ricerca nel sito

        <input type="text" size="50" name="search">
        <input type="submit" name="submit" value="search"/>
        </center>
        </font>
</form>

questo il codice php

<?php
<?php

//get data
$submit = $_GET ['submit'];
$search = $_GET ['search'];

if (!$submit)
    echo "Non hai inserito la parola";
else
{
    $search=trim($search);
    if (strlen ($search) <=2)
        echo " Termine troppo corto";    
    else
    {
        echo "Hai cercato la parola [b]$search[/b]<hr size='1'>";
        
        //connect to our database
        mysql_connect("localhost","root","root");
        mysql_select_db("Sql2446hi");
        
        
            //explode our search term
            $search_exploded = explode(" ",$search);
            
            $x=0; 
            $construct = '';

            foreach($search_exploded as $search_each)
            {

                //construct query
                $x++;
                if ($x==1)
                    $construct .= "keywords LIKE '%$search_each%'";
                else
                    $construct .= "OR keywords LIKE '%$search_each%'";

            }

    //echo out construct
    
    $construct = "SELECT * FROM searchengine WHERE $construct";
    $run = mysql_query($construct);
    
    $foundnum = mysql_num_rows($run);
    
    if ($foundnum==0)
        echo "Nessun risultato trovato.";
    else
    {
        echo "$foundnum risultati trovati!

";
        
        while ($runrows = mysql_fetch_assoc ($run))
        {
            //get data
            $title = $runrows ['title'];
            $desc = $runrows ['description'];
            $url = $runrows ['url'];
            
            echo "
            [b]$title[/b]

            $desc

            [url='$url']$url[/url]


            ";
        }
    
        
    }
    
    }
}
?>

Vi ringrazio nuovamente, ciao
Debora

[bstefano79]

if (!$submit) 
    echo "Non hai inserito la parola"; 

questo non va bene, sostituisci la prima parte come segue

<?php 
if (!isset($_GET ['search']) || $_GET ['search']=="") 
    echo "Non hai inserito la parola"; 
else 
{
    $search = trim($_GET ['search']); 
     if (strlen ($search) <=2) 
        echo " Termine troppo corto";
    //etc etc

[mademimo]

Grazie Stefano

Debora

[mademimo]

Ciao a tutti!

Chiedo nuovamente il vostro aiuto...

Sempre per il codice sopra indicato, come faccio a far sì che la ricerca la faccia anche su title e description?

La mia query é
SELECT * FROM `searchengine` WHERE 1

Grazie,
Debora