GDRCD 5 - Anagrafe con elenco

27/01/2011 - Molto utile come modifica.

Solo una cosa, togli tutti quelle strutture di controllo.

Per i pulsanti fai un Array che contiene le lettere dell'alfabeto e richiamale con un ciclo foreach

e $_REQUEST['op'] controllalo con uno switch

esempio:
$pulsante = $_REQUEST['op'];
switch ($pulsante) {

case "A":
if ($pulsante != "") {
$query = "SELECT nome, cognome FROM personaggio WHERE nome LIKE 'A%' ORDER BY nome ASC";
$result = mysql_query($query);
}
break;
case "B":
[etc...]
}
così invece di avere uno script che è lungo 945 righe e pesa 38kb, hai un script di 240 righe circa che pesa solo 7kb.

Grazie ^^

29/01/2011 - e invece una cosa del genere:
if ($pulsante != "") {
$pulsante = $_REQUEST['op'];
$query = "SELECT nome, cognome FROM personaggio WHERE nome LIKE '" .pulsante."%' ORDER BY nome ASC";
}
non l'ho provata come cosa ma cosi facendo risparmi ancora di più...

29/01/2011 - Apprezzabile il fatto di contribuire al pacchetto, ma consiglio vivamente di imparare un pochetto a programmare meglio.

Un codice del genere può essere scritto risparmiando il 90% delle righe scritte da entrambi.

Se non fosse per il fatto che una patch così possa essere scaricata da utilizzatori ignari dello spreco di risorse di questa patch non sarei qui a scrivere.

Conviene cancellare tutto il bordello di codice scritto e seguire il consiglio di darkblade, aggiungendo il filtro per evitare attacchi SQL.

18/09/2012 - Ma quindi la patch scaricabile è quella rivista e sistemata di tutte le cose evidenziate dai commenti di altri programmatori?
O è ancora quella originale, programmata male?