Refreh onclick non funzionante

cybershadow

SCRIVI

Refreh onclick non funzionante postato il 25/08/2018 20:14:18 nel forum programmazione, gdrcd, open source, hosting

Sera a tutti!
Stavo lavorando oggi su una modifica sostanziale al sistema di spesa punti esperienza di gdrcd per l'acquisto delle abilità,nel concreto cancellando quasi tutto il form pre-esistente per riscriverlo con un form ad input in php che si compone di un piccolo input type submit che una volta cliccato esegue una query mysqli aggiornando il database ed aggiungendo un punto alla abilità interessata.Tutto questo a patto che il check svolto da una if condition confermi che ci sono abbastanza punti esperienza per livellare la skill
FIn qui tutto bene:il form funziona e fila liscissimo, il problema sorge però dopo aver cliccato:infatti fino a che non si clicca una seconda volta sul submit la schermata, almeno in lato client, non si aggiorna il che è un problema, visto che in questa maniera si ha a disposizione un click extra che elude il check svolto dalla if e di cui i furbetti potrebbero approfittare.
Ho già provato ad agire usando una funzione javascript onclick sul submit(in particolare location.refresh(true),location.refresh(false), window.location.refresh(true) e location.refresh(false))ma non ha risolto il problema, visto che di fatto la pagina sembra ricaricare, ma solo in alcune sue parti(elenco presenti, messaggi etc etc) che ovviamente non includono il centro della pagina.

Qualcuno ha qualche consiglio?

Allego qui di seguito la parte interessata del codice,se mai potesse essere utile.Non me ne vogliate se è un tantinello spaghettoso u.u

for($ida=1; $ida<= 30; $ida++){



$skill_name=gdrcd_query("SELECT * FROM abilita WHERE id_abilita='$ida'");
$result=gdrcd_query("SELECT grado FROM clgpersonaggioabilita WHERE id_abilita='$ida' AND nome='".gdrcd_filter('in',$_REQUEST['pg'])."'");
if($result == 0){
$xp_necessaria=3;
}
else{
$xp_necessaria=$result['grado'] + 1;
}

?>
<form name="form" method="post">
<span ><?php

echo $skill_name['nome'], ' ' , $result['grado'] , ' (' , $skill_name['car'] , ') ' ;
?></span>
<input type="hidden" name="id_skill" id="id_skill" value="$result['grado']">
<?php if($xp_disponibile >= $xp_necessaria){ ?> <input type="submit" onclick="rEload()" name="aumenta<?php echo $ida; ?>" value="+" /><?php }?>
</form>
<?php
$skill=$_POST['id_skill'];
if(isset($_POST["aumenta$ida"])){
if($result == 0){
gdrcd_query("INSERT INTO clgpersonaggioabilita (nome, id_abilita, grado) VALUES ('".gdrcd_filter('in',$_REQUEST['pg'])."', '$ida', 1)");
gdrcd_query("UPDATE personaggio SET esperienza_spesa = esperienza_spesa +3 WHERE nome ='".gdrcd_filter('in',$_REQUEST['pg'])."'");
}
else{
gdrcd_query("UPDATE clgpersonaggioabilita SET grado = grado + 1 WHERE nome ='".gdrcd_filter('in',$_REQUEST['pg'])."' AND id_abilita = '$ida'");
gdrcd_query("UPDATE personaggio SET esperienza_spesa = esperienza_spesa + '$xp_necessaria' WHERE nome ='".gdrcd_filter('in',$_REQUEST['pg'])."'");
}
}//if isset
echo '<br>';
}
?>
<script>
function rEload() {
location.reload(true);
}
</script>

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

quota

Pagine → 1 

dueanime

SCRIVI

25/08/2018 20:21:27

Discussione spostata in "GDRCD e Open Source"

World of TanksWorld of Tanks ↗
Lanciati in epiche battaglie spalla a spalla con altri giocatori. Conquista la supremazia nel mondo dei Carri Armati!

quota

darkblade

SCRIVI

25/08/2018 20:44:44

Ma se invece fai un location.href o location.reload?

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

quota

cybershadow

SCRIVI

25/08/2018 20:52:14 e modificato da cybershadow il 25/08/2018 20:55:34

darkblade ha scritto: Ma se invece fai un location.href o location.reload?


Si,scusami, non sono pratico di javascript, con location.refresh intendevo reload.
location.href non lo conoscevo, invece, adesso provo e ti faccio sapere :D

Edit:location.href non mi è proprio utile, visto che semplicemente serve a stampare l'url della pagina che si sta visitando

CrossoutCrossout ↗
Tuffati nel mondo di Crossout, il gioco MMO d'azione post apocalittico! Costruisci le tue speciali macchine da combattimento!

quota

darkblade

SCRIVI

25/08/2018 23:45:55

location.href = '/nuova-pagina.html';
Questo fa un redirect alla pagina che metti tu.

World of WarshipWorld of Warship ↗
MMO gratuito con frenetiche battaglie navali ambientate nel ventesimo secolo. Salpa con la tua nave ed affronta i nemici!

quota

cybershadow

SCRIVI

26/08/2018 02:25:49 e modificato da cybershadow il 26/08/2018 02:32:33

darkblade ha scritto: location.href = '/nuova-pagina.html';
Questo fa un redirect alla pagina che metti tu.


allora, sembrava apparentemente non funzionare, poi ho pensato che il problema era nel fatto che non mettessi il tag <script> nell'head del documento, ho provveduto e funziona.
ALmeno in parte, perché la pagina tutta ora presena il refresh, ma la skill interessata resta al grado attuale, lasciando di fatto il bug irrisolto :/

Lineage IILineage II ↗
Entra in uno sconfinato mondo fantasy dominato da razze in contrasto tra loro. Scatena i tuoi poteri in uno dei Mmo più famosi al mondo!

quota

darkblade

SCRIVI

26/08/2018 06:54:55

Ora ti basta farti una select ad inizio pagina che si calcola il valore aggiornato, così quando farai il reload della pagina dovrebbe ritrovarsi il valore nuovo e non quello vecchio.

Storie di AgarthiStorie di Agarthi ↗
Un Varco si apre davanti a te, un mondo tra i mondi è a portata di mano. Lasciati alle spalle le certezze, inizia l'avventura!

quota

cybershadow

SCRIVI

26/08/2018 10:12:33 e modificato da cybershadow il 26/08/2018 11:13:32

darkblade ha scritto: Ora ti basta farti una select ad inizio pagina che si calcola il valore aggiornato, così quando farai il reload della pagina dovrebbe ritrovarsi il valore nuovo e non quello vecchio.


Ci ho provato, ma come immaginavo non ha funzionato, visto che la query select che mi va a prendere il grado della skill è inserita dentro un for loop che itera id_abilita tante volte quante sono le skill in database, dandogli il valore di $i all'interno del loop.Chiaramente estraendo dal loop la query il tutto non funziona.

A questo punto stavo pensando di studiarmi un po' lAJAX e riscrivere tutto con quello, ma avrei due domande.
Innanzitutto, risolverebbero il mio problema o probabilmente me lo ripresenterebbero?
E secondariamente, visto che immagino che come javascript in generale AJAX lavori almeno per parte in lato client, quanto sarebbe pericolosa a livello di sicurezza una soluzione simile?

EDIT:Ho anche provato a mettere il codice della query in una variabile e poi ad eseguire la query richiamando la variabile, ma il risultato è che il loop non funziona uguale

CRSED: F.O.A.D.CRSED: F.O.A.D. ↗
Brutale sparatutto MMO in cui vince l'ultimo giocatore che resta in vita! Il giocatore sarà accompagnato da armi realistiche e magie!

quota

leoblacksoul

SCRIVI

26/08/2018 18:17:41

Il tuo problema non è un bug del sistema o del browser, è proprio un errore nella logica di quello che hai scritto. Quindi usare dei trucchi come "location.reload" per aggirarlo non risolverà il problema.

L'errore principale è molto semplice: il codice che aggiorna il grado della skill è posizionato in fondo alla pagina.
Dato che sia la query SELECT sia la stampa del form avvengono *prima* della query di UPDATE è ovvio che al primo submit i dati mostrati dalla pagina siano errati, dato che vengono stampati prima di venire aggiornati. Il problema non è che la pagina non si aggiorna (anzi la pagina di aggiorna perfettamente) è che gli stai facendo stampare delle informazioni vecchie.
Sposta la query di update fuori dal for principale e all'inizio della pagina e vedrai che le informazioni stampate saranno giuste.


Aggiungo una cosa: il tuo if che controlla se il giocatore ha abbastanza exp da spendere puoi farlo al momento della stampa (come stai facendo), ma *DEVI* anche rifarlo prima della query di UPDATE: altrimenti un giocatore che vuole fregarti potrebbe sempre fare F5 e aggiungersi tutti i gradi che vuole.

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

quota

cybershadow

SCRIVI

26/08/2018 18:50:36

leoblacksoul ha scritto: L'errore principale è molto semplice: il codice che aggiorna il grado della skill è posizionato in fondo alla pagina.
Dato che sia la query SELECT sia la stampa del form avvengono *prima* della query di UPDATE è ovvio che al primo submit i dati mostrati dalla pagina siano errati, dato che vengono stampati prima di venire aggiornati. Il problema non è che la pagina non si aggiorna (anzi la pagina di aggiorna perfettamente) è che gli stai facendo stampare delle informazioni vecchie.
Sposta la query di update fuori dal for principale e all'inizio della pagina e vedrai che le informazioni stampate saranno giuste.


Ah, ecco, questo mi spiega un po' di cose.
A questo punto in sorgente posiziono la stampa sotto l'update e poi la riposiziono sopra nella visualizzazione grafica con il css.
Grazie, mi sei stato davvero di aiuto :D


leoblacksoul ha scritto: Aggiungo una cosa: il tuo if che controlla se il giocatore ha abbastanza exp da spendere puoi farlo al momento della stampa (come stai facendo), ma *DEVI* anche rifarlo prima della query di UPDATE: altrimenti un giocatore che vuole fregarti potrebbe sempre fare F5 e aggiungersi tutti i gradi che vuole.


Eh, di questo mi ero accorto, e infatti ho provveduto a correggerlo.

World of WarshipWorld of Warship ↗
MMO gratuito con frenetiche battaglie navali ambientate nel ventesimo secolo. Salpa con la tua nave ed affronta i nemici!

quota

Pagine → 1 

Rispondi alla Discussione Segui Discussione Inoltra Discussione Forum Programmazione, GDRCD, Open Source, Hosting Elenco Forum

Articoli, Interviste e altre Risorse!

Regno di Drams
Regno di Drams
Intervista alla gestrice del GdR fantasy medievale Regno di Drams
Età e GdR
Età e GdR
Divertiti senza limiti... i GdR online come forma di intrattenimento per tutte le età
Orde En Macht
Orde En Macht
Intervista alle gestrici di Orde En Macht il play by chat ad ambientazione moderna con tinte distopiche!
New Old Tale
New Old Tale
Intervista ai gestori del gioco di ruolo via chat ad ambientazione manga New Old Tale Magi Parallel Universe

Pubblicità

Ultime dal Portale

news The Eyrie GdR: Halloween ad Harrenhal

annuncio piromee ha aperto un annuncio di ricerca: Momentum Incantatio ricerca Tester

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

risposta Il gestore di Portus: Harry Potter Gioco di Ruolo ha risposto alla recensione di dreamboy

news Black Sails Chronicles: ON - Casa delle Aste

news RAID Shadow Legends: Summon Boost e Champion Chase

news Storie di Esperia: Uno strappo nella notte

Hero Wars 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!

news Crystal Tokyo: Ingresso nella Via Magica

forum alexandros si è accreditato come gestore di Shinobi no Saisei - Naruto x Boruto GDR

risposta Il gestore di Naruto x Boruto GDR - New Rebirth ha risposto alla recensione di jimcho

risposta Il gestore di Naruto x Boruto GDR - New Rebirth ha risposto alla recensione di theduelist92

Seconda Era Seconda Era → Vaste terre coperte di ombre e misteri, dove ogni eroe scrive la storia del Mondo di Eidel. Seconda Era, Play By chat

news War Thunder: Chieftain 900: Capo della Royal Ordnance

gioco Il portale raggiunge i 17.200 giochi catalogati! Grazie a tutti!

news Hero Wars: Anomalia dei Pets! 🐾

news New Orleans: [On Game] Casting per i Cantanti!

Crossout Crossout → Tuffati nel mondo di Crossout, il gioco MMO d'azione post apocalittico! Costruisci le tue speciali macchine da combattimento!

Pubblicità

Articoli

AI e GdR AI e GdR - Abbiamo chiesto a due AI di presentarsi e spiegarci cosa possono fare per i giocatori di ruolo online. Scopriamo cosa ci rispondono!

Shannara Shannara - Recensione del GDR ambientato nel mondo creato da Terry Brooks

Guida RpG Guida RpG - Una guida per iniziare a giocare ai giochi di ruolo (RPG)

Raja Dunia Raja Dunia - Intervista allo staff del play by chat fantasy Le Cronache di Raja Dunia!

Personaggi Badass Personaggi Badass - He-Man lo fa meglio! Costruiamo insieme un personaggio Badass!

Giochi Ultima Giochi Ultima - Lista dei giochi online ambientati nel mondo di Sosaria

The Hobbit The Hobbit - The Hobbit tra fiaba e mito: un viaggio verso la crescita