Pacchetto: GDRCD 5 - Anagrafe con elenco
Descrizione: La patch sostituisce l'anagrafe nativo di Gdrcd5. Invece che un menu a tendina con tutti i personaggi, viene visualizzato l'elenco delle lettere dell'alfabeto, e quindi l'elenco per iniziale del nome, con il link per l'invio diretto della posta.
Completa di istruzioni.
Codice commentato per eventuali personalizzazioni del singolo programmatore.
Autore: bukama
Data: 21/01/2011
Dimensione: 3,05 Kb
Registrati per scaricare il pacchetto
Lascia Commento Invia un Pacchetto
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?
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.
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ù...
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 ^^