[RISOLTO][GDRCD 5.6] Px settimanali al login + Tetto massimo a update

Pagine → 1

cbr89

04/03/2023 19:21:11

Ciao!
così, a prima vista c'è un errore nell IF statment

if($PARAMETERS['settings']['week_exp'] = 'ON')

questo va cambiato così

if($PARAMETERS['settings']['week_exp'] == 'ON')

Poi, credo che ce ne sia uno possibile pure più sotto

if($chat['azione'] >= 4

secondo me, andrebbe fatto un cast implicito per sicurezza

if( (int) $chat['azione'] >= 4

Vedi che succede! e fammi sapere

Brancix
Aka Claudio, sviluppatore di
https://www.fervm.it

bunnyboo

06/03/2023 12:02:49

Oltre a quello che indica cbr89, mi viene un dubbio sul controllo effettuato, cioè questa parte:

if($newdata == $data){

Questo controllo non comporta che se il giocatore non logga nel giorno preciso dato dall'ultima exp + 1 settimana, salti tutto per com'è fatto il codice?
Potrei sbagliarmi, ma a logica credo possa essere un problema.

Nothing is static. Everything is evolving. Everything is falling apart.

- Professoressa della penna rossa (cit.) -

backwaters

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

06/03/2023 15:21:41

@cbr89: ho inserito le correzioni, appena vedo se funzionano ti do conferma. Intanto grazie :)

bunnyboo ha scritto: Questo controllo non comporta che se il giocatore non logga nel giorno preciso dato dall'ultima exp + 1 settimana, salti tutto per com'è fatto il codice?

Esatto. Tra l'altro, se per qualsiasi motivo x non assegna nulla una settimana, non assegna più nulla nella nelle settimane a seguire e il sistema salta. Ho aggiunto l'else per aggiornare comunque la data di ultima_exp per ovviare il problema, ma non sono sicura sia sufficiente.
Ho fatto varie prove modificando il controllo con le comparazioni, ma a quel punto mi assegna i px giornalmente. Onestamente, non ho idea di come impostarlo.

cbr89

07/03/2023 15:48:45 e modificato da cbr89 il 07/03/2023 15:49:48

Ciao Backwaters,
ho dato uno sguardo allo script che hai fatto.
Secondo me ci sono un paio di errori logici dentro, io ho provato a farti uno script che dovrebbe darti il risultato che speri.

dato che non ricordo bene la sintassi della funzione PHP diff su un oggetto date, ti chiedo di verificare quella roba.

if($PARAMETERS['settings']['week_exp'] == 'ON') {
/*ESPERIENZA SETT*/
$row = gdrcd_query("SELECT esperienza, ultima_exp FROM personaggio WHERE nome = '".$_SESSION['login']."' LIMIT 1");

$ultimaexp = new Date(gdrcd_filter('out', $row['ultima_exp']));
$oggi = new Date("now");

/*se dall'ultimo aggiornamento sono passati da 7 a più giorni*/
if( $oggi->diff($ultimaexp)->days >= 7 ){

/*questa query era potenzialmente sbagliata.
Nel senso che, se uno va in 4 chat qualunque, manda un messaggio tu gli aumenti gli exp. L'ho modificata per estrarre il numero di azioni per stanza nell'ultima settimana da ora. */
$chat = gdrcd_query("SELECT COUNT(mittente) AS azione FROM chat WHERE DATE_SUB(NOW(), INTERVAL 1 WEEK) >= ora AND tipo = 'P' AND mittente = '".$_SESSION['login']."' GROUP BY stanza HAVING COUNT(*) > 4", 'result');

if(gdrcd_query($chat, 'num_rows') > 0 ) {
/*Se è stato tornato un numero di righe, vuol dire che il pg ha fatto almeno 4 azioni nella stessa chat nella settimana */
gdrcd_query("UPDATE personaggio SET esperienza = esperienza + 1, ultima_exp = NOW() WHERE nome = '".$_SESSION['login']."'");
}
}
}

Nota che, quando e se creerai questa colonna ti consiglio di mettere come data_ultimo_exp oggi o il giorno in cui inserirai questa modifica, in modo che più o meno tutti, partano dalla stessa situazione.

per i test, come chiaro, dovrai forzare a mano un valore minore di una settimana nella data ultimo exp, e anche in almeno 4 azioni della stessa stanza!
Non uccidermi se ci sono errori di sintassi, non ho proprio avuto tempo di verificare su un IDE decente

Fammi sapere!

Brancix
Aka Claudio, sviluppatore di
https://www.fervm.it

backwaters