si può pulire questo codice?

[amphioxus]

salve, mi chiedevo se c' è un modo più semplice per scrivere il codice seguente...funziona perfettamente...la mia è semplice curiosità e poi meno scrivi meno byte consumi

$var=$_GET['pagina'];
  

$query2 = "SELECT * FROM pillole WHERE ID < $var ORDER BY ID DESC LIMIT 1 ";
$query3 = "SELECT * FROM pillole WHERE ID > $var ORDER BY ID ASC LIMIT 1" ;



$risultato2 = mysql_query( $query2 );
$risultato3 = mysql_query( $query3 );

    
            if(mysql_num_rows($risultato2)>0){
                    

                while($scatola_temporanea=mysql_fetch_array($risultato2)){
                    $titolo2=$scatola_temporanea['titolo'];
                    $id2=$scatola_temporanea['ID'];
                    echo "<div class=\"left\"><a href=\"pillole.php?pagina=" . $id2 . "\">« $titolo2</a></div>";
                    }
                    }


    
            if(mysql_num_rows($risultato3)>0){
                    

                while($scatola_temporanea=mysql_fetch_array($risultato3)){
                    $titolo3=$scatola_temporanea['titolo'];
                    $id3=$scatola_temporanea['ID'];
                    echo "<div class=\"right\"><a href=\"pillole.php?pagina=" . $id3 . "\">$titolo3 »</a></div>";
                    }
                    } 

[las]

un inizio potrebbe essere questo:

$query2 = "SELECT * FROM pillole WHERE ID < ".$_GET['pagina']." ORDER BY ID DESC LIMIT 1 ";
$query3 = "SELECT * FROM pillole WHERE ID > ".$_GET['pagina']." ORDER BY ID ASC LIMIT 1" ;

$risultato2 = mysql_query( $query2 );
$risultato3 = mysql_query( $query3 );

             while($scatola_temporanea=mysql_fetch_array($risultato2)){
                    echo "<div class=\"left\"><a href=\"pillole.php?pagina=" . $scatola_temporanea['ID']. "\">« ".$scatola_temporanea['titolo']."</a></div>";
                    }
            

                while($scatola_temporanea=mysql_fetch_array($risultato3)){
                    echo "<div class=\"right\"><a href=\"pillole.php?pagina=" . $scatola_temporanea['ID']. "\">".$scatola_temporanea['titolo']." »</a></div>";
                    } 

poi se oltre al codice vuoi 'risparmiare anche risorse' eviterei di fare due query praticamente identiche, fai un unica query metti tutto dentro un array e poi ti cicli l'arrey in base alle tue esigenze.

[k.b]

Beh ad esempio potresti eliminare gli inutili cicli while visto che da ogni query estrai solo un elemento.

[amphioxus]

hai ragione dato che estraggo un solo elemento posso togliere il while

cmq grazie per l'aiuto

[amphioxus]

Originariamente inviato da las
un inizio potrebbe essere questo:

$query2 = "SELECT * FROM pillole WHERE ID < ".$_GET['pagina']." ORDER BY ID DESC LIMIT 1 ";
$query3 = "SELECT * FROM pillole WHERE ID > ".$_GET['pagina']." ORDER BY ID ASC LIMIT 1" ;

$risultato2 = mysql_query( $query2 );
$risultato3 = mysql_query( $query3 );

             while($scatola_temporanea=mysql_fetch_array($risultato2)){
                    echo "<div class=\"left\"><a href=\"pillole.php?pagina=" . $scatola_temporanea['ID']. "\">« ".$scatola_temporanea['titolo']."</a></div>";
                    }
            

                while($scatola_temporanea=mysql_fetch_array($risultato3)){
                    echo "<div class=\"right\"><a href=\"pillole.php?pagina=" . $scatola_temporanea['ID']. "\">".$scatola_temporanea['titolo']." »</a></div>";
                    } 

poi se oltre al codice vuoi 'risparmiare anche risorse' eviterei di fare due query praticamente identiche, fai un unica query metti tutto dentro un array e poi ti cicli l'arrey in base alle tue esigenze.

io ho messo $var perchè non vado molto d'accord con ". . " perchè se poi mi dimentico anche un particolare ci metto ore a capire che è li l'errore

[las]

Originariamente inviato da amphioxus
io ho messo $var perchè non vado molto d'accord con ". . " perchè se poi mi dimentico anche un particolare ci metto ore a capire che è li l'errore

se vuoi puoi usare gli array anche dentro le stringhe di testo, basta che ti ricordi di non mettere gli indici tra virgolette:

$query2 = "SELECT * FROM pillole WHERE ID < $_GET[pagina] ORDER BY ID DESC LIMIT 1 "; 

[k.b]

Il tutto ovviamente facendo finta che usare direttamente valori ottenuti via GET sia una buona idea.

[amphioxus]

Originariamente inviato da k.b
Il tutto ovviamente facendo finta che usare direttamente valori ottenuti via GET sia una buona idea.

in che senso scusami?

[oly1982]

I miei consigli:
- filtrare la variabile get per evitare sql injection non sarebbe male;
- limitandomi a leggere il codice postato le variabili utilizzate estratte dalla query sono ID e titolo... quindi perchè utilizzare * (che estrae tutto?);
- ciò che ti ha detto k.b: il ciclo while serve quando si è stratto più di un record per scorrere ognuno di essi, ma essendo che la query prevede un LIMIT 1 (oppure le query fatte su uguaglianza di ID) non ha se senso utilizzare un ciclo che si ha la certezza che avrà un solo ciclo... in ogni caso si tratta di un "cavilllo" cmq...

$var_get = isset($_GET['pagina']) ? intval($_GET['pagina']) : 0;

$query2 = "SELECT * FROM pillole WHERE ID < ".$var_get." ORDER BY ID DESC LIMIT 1 "; 
$query3 = "SELECT * FROM pillole WHERE ID > ".$var_get." ORDER BY ID ASC LIMIT 1" ; 

$risultato2 = mysql_query( $query2 ); 
$risultato3 = mysql_query( $query3 ); 

$scatola_temporanea2=mysql_fetch_array($risultato2);
if(!$scatola_temporanea2){
echo "<div class=\"left\"><a href=\"pillole.php?pagina=" . $scatola_temporanea['ID']. "\">« ".$scatola_temporanea['titolo']."</a></div>"; 
}

$scatola_temporanea3=mysql_fetch_array($risultato3);
if(!$scatola_temporanea3){
echo "<div class=\"right\"><a href=\"pillole.php?pagina=" . $scatola_temporanea['ID']. "\">".$scatola_temporanea['titolo']." »</a></div>";
} 

[k.b]

Originariamente inviato da amphioxus
in che senso scusami?

Cerca su google "SQL injection" e preparati all'apertura del vaso di Pandora