[GDRCD5] Form assegnazione esperienza a più utenti

Pagine → 1 2

Tibia ↗
Tibia è un MMORPG gratuito online dal 1997 e con migliaia di utenti connessi! Riscopri il divertimento old style!

26/09/2011 23:59:10

nicodem ha scritto: ... non ho capito!

Dal codice che hai riportato:
$destinat=trim($check_dest[0]);
$result=gdrcd_query("UPDATE personaggio SET esperienza = esperienza + ".gdrcd_filter('num',$_POST['px'])." WHERE nome = '".$destinat."'", 'result');

Fai effettivamente solo il primo destinatario. Quei "..." per cosa stanno?
Ci vorrebbe un ciclo for per tutti i valori che hai ottenuto splittando rispetto alla virgola.

i puntini stanno a significare "il codice continua" xD

ti ripeto.... purtroppo sto andando a tentativi, lo ammetto <.<

ho anche provato a mettere, dove dici tu

$destinat=gdrcd_filter('get',$_POST['destinatario']);

ma la questione non cambia.

Metto in altri termini la questione: vorrei fare in modo che tramite il form sotto si possano assegnare punti esperienza a più pg, i cui nomi sono da specificare in diversi form field, sul tipo di quelli presenti nel form :)

solo che non mi riesce di capire come si fa <.<"

27/09/2011 00:00:14 e modificato da leoblacksoul il 27/09/2011 00:05:51

eh, se vuoi fare una cosa limitata la logica è quella. Cioè gli metti fissi, chessò, 5 posti per dare esperienza a 5 utenti (come avevi pensato tu, basta copiare 5 volte i campi). Ovviamente la chiave sta nel dare nomi diversi a tutti gli elementi, altrimenti si sovrascrvono a vicenda quando arrivano su PHP (ecco perchè si setta solo l'ultimo).
Volendolo fare un po' più avanzato, ci metti un pochino di javascript; parti da un solo elemento per assegnare exp, e poi cliccando un bottone aggiungi altri campi uguali (ma con id e name diversi!) al volo, usando javascript.
Ecco un esempio, l'avevo usato come guida la prima volta che avevo provato ad aggiungere dinamicamente elementi con javascript. Se usi Jquery è infinitamente più semplice, questo esempio è javascript puro: http://www.webdeveloper.com/forum/showthread.php?t=71813 ↗

EDIT: Per gestire comodamente la cosa su PHP quando ti arrivano i dati, basta che dai dei nomi di questo tipo ai tuoi campi:
<input name="exp[1][pg]" />
<input name"exp[1][px]" />
<input name="exp[1][causale]"/>
ovviamente facendo andare avanti il numero per ogni gruppo di campi. Così su PHP ti ritroverai con la variabile $_POST['exp'] che sarà un array, che contiene in ogni suo elemento un altro array; il quale avrà gli elementi 'pg', 'px' e 'causale' che contengono i tuoi dati

---
Non chiedetemi aiuto in privato per questioni di programmazione; chiedete sul forum e eventualmente vi risponderò lì.
http://gdrportal.eu ↗ - Hosting per GDR

RAID Shadow Legends ↗
Plasma i tuoi eroi fino a farli diventare vere e proprie armi viventi e assembla i più epici gruppi d'assalto mai visti finora!

27/09/2011 00:23:47

la soluzione tramite JS sembra interessante... appena fatta quella "brutale" tramite php la provo u.u

ho provato a cambiare i nomi come suggerito, ma... non mi aggiorna l'esperienza ç_ç" ho anche provato a cambiare la query iniziale come suggerito da nicodem, ma la questione non cambia. Forse che il problema sia in come è scritto il "submit"?

27/09/2011 00:38:51

provo a farti un esempio di codice, nella versione base senza JS.

<?php
if(isset($_POST['submit'])){
if(($_SESSION['permessi']>=GAMEMASTER)){
foreach($_POST['exp'] as $item){
$pg=gdrcd_filter('get',$item['pg']);
$exp=gdrcd_filter('get',$item['px']);
$caus=gdrcd_filter('get',$item['causale']);
//Qui usa $pg, $px e $caus per costruire la tua query di inserimento punti EXP
}
}
}
?>
<form method="post" action="<?php echo $_SERVER['PHP_SELF'];?>">
<fieldset>
<legend>Utente1</legend>
<label>Nome Utente: </label><input type="text" name="exp[1][pg]" />
<label>Esperienza: </label><input type="text" name="exp[1][px]" />
<label>Causale</label><input type="text" name="exp[1][causale]" />
</fieldset>
<fieldset>
<legend>Utente2</legend>
<label>Nome Utente: </label><input type="text" name="exp[2][pg]" />
<label>Esperienza: </label><input type="text" name="exp[2][px]" />
<label>Causale</label><input type="text" name="exp[2][causale]" />
</fieldset>
<input type="hidden" name="submit" value="1" />
<input type="submit" name="Invia" />
</form>

---
Non chiedetemi aiuto in privato per questioni di programmazione; chiedete sul forum e eventualmente vi risponderò lì.
http://gdrportal.eu ↗ - Hosting per GDR

Hero Wars ↗
Costruisci la tua squadra di eroi leggendari e domina il campo di battaglia! Strategia, tattica e potenza si scontrano in questo RPG ricco di azione!

27/09/2011 12:59:28

grazie mille a tutti! ho risolto! *_*

ho messo la prima parte del codice così:

<?php
if(isset($_POST['submit'])){
if(($_SESSION['permessi']>=GAMEMASTER)){
foreach($_POST['exp'] as $item){
$pg=gdrcd_filter('get',$item['pg']);
$exp=gdrcd_filter('get',$item['px']);
$caus=gdrcd_filter('get',$item['causale']);
$result=gdrcd_query("UPDATE personaggio SET esperienza = esperienza + ".gdrcd_filter('num',$exp)." WHERE nome = '".$pg."'", 'result');
/*Registro l'operazione*/
$query = "INSERT INTO log (nome_interessato, autore, data_evento, codice_evento ,descrizione_evento) VALUES ('".gdrcd_capital_letter(gdrcd_filter('in',$pg))."', '".$_SESSION['login']."', NOW(), ".PX.", '(".gdrcd_filter('num',$exp).' px) '.$caus."')"; mysql_query($query);
print '<div class="warning">'.gdrcd_filter('out',$MESSAGE['warning']['done']).'</div>';
} else {
print '<div class="warning">'.gdrcd_filter('out',$MESSAGE['warning']['camt_do']).'</div>';
}

/*Avviso l'utente*/
if ($_SESSION['login']!=$_POST['nome']){
$query="INSERT INTO messaggi (mittente, destinatario, spedito, testo, oggetto) VALUES ('".$_SESSION['login']."', '".$pg."', NOW(), 'Ti sono stati assegnati ".gdrcd_filter('num',$exp).' px con la seguente motivazione: '.$caus."', '".gdrcd_filter('in',$MESSAGE['interface']['skills']['oggetto'])."')";
}
mysql_query($query);

}
}
}
?>

Solo, senza quelle parti che ho messo in grassetto. Credo di non sapermici raccapezzare con le parentesi, ma.... vabbè xD

altra cosa che ho cambiato è stato indicare, nel form, come action l'indirizzo del form [insomma: action="main.php?page=servizi_assegnazione_px"], invece che come indicato da Leo (la pagina mi dava errore). Curiosità "formativa": come mai? E in quali casi si usa come action quanto indicato da Leo?

vino_veritas

Enlisted ↗
Guida la tua squadra di soldati in combattimenti su larga scala, con fanteria, veicoli corazzati e aerei della IIa Guerra Mondiale!

27/09/2011 13:23:19 e modificato da vino_veritas il 27/09/2011 13:24:33

$_SERVER['PHP_SELF'] è semplicemente l'indirizzo dello script attualmente in esecuzione. Tu invece hai bisogno della stringa di query (per intenderci, quella a partire dal punto interrogativo in poi nell'action); leo intendeva che dovevi utilizzare nell'action:

$_SERVER["PHP_SELF"] . "?page=servizi_assegnazione_px"

:-)

War Thunder ↗
Aviazione, veicoli corazzati e flotte militari della seconda guerra mondiale. Parteciperai a tutte le principali battaglie!

27/09/2011 14:26:42

vino_veritas ha scritto: $_SERVER['PHP_SELF'] è semplicemente l'indirizzo dello script attualmente in esecuzione. Tu invece hai bisogno della stringa di query (per intenderci, quella a partire dal punto interrogativo in poi nell'action); leo intendeva che dovevi utilizzare nell'action:

$_SERVER["PHP_SELF"] . "?page=servizi_assegnazione_px"

:-)

Interessante... non lo sapevo! (me scema)

grazie ancora a tutti ^^ ora che ho risolto nel modo più brutale, provo anche ad ingegnarmi con la soluzione tramite JS :D

27/09/2011 15:27:37

Sviste notturne v.v
Stai attenta che gli utenti inseriti nel form esistano. Il tuo codice non tiene conto di eventuali errori, se qualcuno inserisce un utente inesistente viene comunque registrata l'assegnazione nei log.
La parte che hai messo in grassetto non ci stava perchè stai cercando di mettere un else dopo un foreach.
$_POST['nome'] cos'è? Nel nostro form non c'è nessun campo chiamato così. Per avvisare ogni utente che gli è stata aggiunta esperienza devi fare quel controllo dentro al foreach, con la condizione if($_SESSION['login']!=$item['pg']). Possibilmente mettendo il mysql_query dentro all'if, non fuori.

---
Non chiedetemi aiuto in privato per questioni di programmazione; chiedete sul forum e eventualmente vi risponderò lì.
http://gdrportal.eu ↗ - Hosting per GDR

War Thunder ↗
Aviazione, veicoli corazzati e flotte militari della seconda guerra mondiale. Parteciperai a tutte le principali battaglie!

27/09/2011 17:33:35 e modificato da irenes il 27/09/2011 17:40:56

leoblacksoul ha scritto: Sviste notturne v.v
Stai attenta che gli utenti inseriti nel form esistano. Il tuo codice non tiene conto di eventuali errori, se qualcuno inserisce un utente inesistente viene comunque registrata l'assegnazione nei log.
La parte che hai messo in grassetto non ci stava perchè stai cercando di mettere un else dopo un foreach.
$_POST['nome'] cos'è? Nel nostro form non c'è nessun campo chiamato così. Per avvisare ogni utente che gli è stata aggiunta esperienza devi fare quel controllo dentro al foreach, con la condizione if($_SESSION['login']!=$item['pg']). Possibilmente mettendo il mysql_query dentro all'if, non fuori.

$_POST['nome'] è un refuso del vecchio codice :) non avevo notato la sua presenza, grazie per avermelo fatto notare! :)

allora, a questo punto, visto che ci sono, domando: come fare per tenere conto di eventuali errori, magari avvisando dell'inesistenza del pg di cui si è sbagliato a scrivere il nome? v.v

e poi c'è una questione che ho potuto verificare ora, purtroppo <.<

allora: se uno dei field dove vanno messi i nomi dei pg resta vuoto (ovvero, in esempio: io ho due field in cui mettere potenzialmente due nomi di pg, ma ne sfrutto solo uno), mi "parte" lo stesso un aggiornamento del log sul DB, ovviamente aggiornamento a... nessuno, visto che il pg non esiste. Il fatto è che mi parte anche il messaggio automatico di avviso di ricezione px, ovviamente. Peccato che, oltre ad inserire il messaggio in DB, non esistendo il pg, il sistema messaggi segnali la presenza di un messaggio a tutta l'utenza, senza che però qualcuno, aprendo la posta, si ritrovi il messaggio xD

27/09/2011 19:25:51 e modificato da leoblacksoul il 27/09/2011 19:28:01

Ovviamente XD
Questo bug che hai trovato è un conseguenza diretta dell'assenza del controllo degli errori che abbiamo menzionato.

Fare il controllo è molto semplice, basta fare una query per controllare che il pg esista prima di tutto. Per evitare il caos, te lo scrivo in una funzione a parte, anche se gdrcd fa scarso uso (nullo?) di funzioni all'interno dei suoi file inclusi.

---
Non chiedetemi aiuto in privato per questioni di programmazione; chiedete sul forum e eventualmente vi risponderò lì.
http://gdrportal.eu ↗ - Hosting per GDR