GDRCD 5 - Anagrafe con elenco

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

Commenti degli Utenti

joeilcondor

SCRIVI

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?

airon

SCRIVI

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.

darkblade

SCRIVI

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ù...

curtberth

SCRIVI

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 ^^