php e mysql, best practice

Question

Ho iniziato a lavorare con php e mysql oggi. Fondamentalmente, quello che ho, è una pagina vuota con dei pezzi che riempio dalla ricerca di un id in un database. Così sulla mia home page ho un URL che assomiglia a questo:

<a href="content/display.php?id=id1"> 

E poi nel mio display.php ho questo:

<?php 
    include '../includes/header.php'; 
    $id = $_GET['id']; 
    $mysqli = new mysqli('localhost','username','password','dbname'); 
    if($result = $mysqli->query("SELECT * FROM portfolio WHERE id='".$id."'")) 
    { 
     while($row = $result->fetch_object()) 
     { 
      $head = $row->head; 
      $img1 = $row->img1; 
      $img2 = $row->img2; 
      $img_url = $row->imgurl; 
      $img_thumb = $row->imgthumb; 
      $vid = $row->vid; 
      $swf = $row->swf; 
      $url = $row->url; 
      $url_text = $row->urltext; 
      $text = $row->text; 
     } 
    } 
    else echo $mysqli->error; 
?> 

E 'una tabella sparsa in che non tutti i campi avrà informazioni (molte potrebbero essere nulle). Fondamentalmente essi contiene i nomi di file e poi in html Ho codice che assomiglia a questo:

if(isset($img1)) 
        { 
         echo '<img src="images/'.$img1.'" />'; 
        } 

Un paio di domande,

1. È questo il modo migliore per farlo?
2. Ogni volta che visito display.php, sto riaprendo una connessione al database, giusto? Non può essere buono ...
3. Ho scelto di mettere i nomi dei file nel database, piuttosto che i nomi di interi percorsi, o anche i file veri e propri, immaginando che, se cambio il nome del file I può andare nel database e aggiornarlo per il file che voglio cambiare. Se cambio il percorso, posso semplicemente cambiarlo una volta in html. È la migliore idea?

Grazie!

Answer 1

1) No, anche se questo è il modo più semplice per iniziare. Dopo esserti sentito a tuo agio con le basi, dovresti dedicare del tempo a considerare diversi approcci alla struttura dell'applicazione. La regola più importante è separare le preoccupazioni. Non mescolare il codice del database con il codice della logica aziendale con il codice di presentazione. Ma come ho detto, non è qualcosa di cui dovresti preoccuparti il ​​primo giorno. Per ora basta imparare le basi.

2) Non c'è altro modo in realtà. Per un'applicazione Web, ciascuna richiesta del browser è come una singola esecuzione dell'applicazione. Esiste la possibilità di utilizzare le cosiddette connessioni persistenti del database, ma proprio come nel punto precedente, è qualcosa che non dovresti affrontare il primo giorno, poiché richiedono una configurazione specifica del tuo server web. Per il momento, usa solo le normali connessioni.

3) Questa è un'idea abbastanza sensata. Puoi anche definire il percorso dell'immagine come costante PHP, in modo che, nel caso in cui sia necessario un cambiamento, tu cambi solo questa costante.

4) Quello che dice sAc nella sua risposta è molto importante. Leggi le iniezioni SQL e come prevenirle.

Answer 2

si sono vulnerabili a SQL injection, digitare correttamente gettato le variabili:

$id = (int) $_GET['id']; 

utilizzare le funzioni quali mysql_real_escape_string o anche utilizzare meglio:

• Prepared Statements

Answer 3

SQL injection & le dichiarazioni preparate sono già menzionate.Un aggiunta a ciò potrebbe essere:

else echo $mysqli->error; 

Change che a:

else trigger_error($mysqli->error,E_USER_ERROR); 

Perché si chiede? Perché i visitatori non dovrebbero avere idea del tuo database e non possono correggere l'errore, quindi non dovrebbero vederlo. In questo modo, è possibile sviluppare in modo sicuro con display_errors attivo e sul sito attivo display_errors disattivato e in un registro errori.

Answer 4

Sembra che tu abbia una buona padronanza di ciò che vuoi fare. Non so quanti background di sviluppo hai, ma sarebbe una buona idea iniziare a conoscere MVC in php come CakePHP, Fuse, o anche Zend Framework (bleh !!!). Ti farò risparmiare tempo su applicazioni più robuste definendo tutta la tua interfaccia db di base, la gestione dei template, la gestione delle sessioni e ti preoccuperai di problemi di livello superiore, come quello che è per pranzo! :)