[RISOLTO]Gdrcd, parentesi angolari

angel1989

[RISOLTO]Gdrcd, parentesi angolari postato il 11/07/2021 11:31:23 nel forum programmazione, gdrcd, open source, hosting e modificato da angel1989 il 11/07/2021 17:45:25

Salve a tutti. Ho un problema con il codice del parlato.
Talvolta, infatti, capita che giocando ci si scordi di chiude un discorso diretto. In questo caso un'azione come questa:
Vediamo cosa succede se sbaglio aprendo <poi aprendo una seconda volta <senza aver chiuso la precedente>
compare in cha così:
Vediamo cosa succede se sbaglio aprendo <poi aprendo una seconda volta
Questo perché sul db l'azione viene salvata così:
Vediamo cosa succede se invece spaglio aprendo [poi aprendo una seconda volta <senza aver chiuso la precedente]

Per ovviare ho modificato la funzione:

function gdrcd_angs($str) {
$search = [
'#\<(.+?)\>#is',
'#\<(.+?)>#is',
];
$replace = [
'[$1]',
'[$1]',
];

return preg_replace($search, $replace, $str);
}

in questa maniera:

function gdrcd_angs($str) {
$search = [
'#\<(.+?)\>#is',
'#\<(.+?)>#is',
'#\<#is',
'#\>#is',
];
$replace = [
'[$1]',
'[$1]',
'[',
']',
];

return preg_replace($search, $replace, $str);
}

Però, come potete capire non mi fa impazzire.

E' la prima volta che mi capita una cosa del genere. Smanetto con gdrcd da parecchio tempo e di solito questo inconveniente non si presenta e vorrei capire cosa io possa aver modificato. Anche perché non mi pare di aver modificato nulla:

In functions.inc.php abbiamo:

In ref_header.inc.php invece:

$chat_message=gdrcd_filter('in', gdrcd_angs($_POST['message']));
$add_chat.= ''.gdrcd_chatme($_SESSION['login'], gdrcd_chatcolor(gdrcd_filter('out',$row['testo']))).'';

Qualcuno riesce a capire dove sia il problema?

Sea of Conquest ↗
Vivi un'avventura tra i mari! Dal cuore del Mare del Diavolo, un paradiso piratesco brulicante di magia, tesori e avventure, salperai verso l'ignoto!

quota

Pagine → 1

soizora

SCRIVI

11/07/2021 12:21:54 e modificato da soizora il 11/07/2021 12:22:17

Quel comportamento che segnali con quel codice che hai evidenziato è normale, però scusami ma non riesco a capire che comportamento ti aspetteresti al suo posto. Potresti rifare l'esempio più chiaramente?

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

quota

angel1989

SCRIVI

11/07/2021 12:46:42

Nella vecchia versione di gdrcd scrivendo un'azione come:
Vediamo cosa succede se sbaglio aprendo <poi aprendo una seconda volta <senza aver chiuso la precedente>
in chat l'azione compariva così:
Vediamo cosa succede se sbaglio aprendo <poi aprendo una seconda volta <senza aver chiuso la precedente>

Niente veniva tagliato come invece succede ora. Vorrei capire il perché e come ovviare.

New World Order ↗
Diventa un agente del Nuovo Ordine Mondiale e affronta le forze occulte che minacciano il pianeta Terra in questo GDR Urban Fantasy per Mobile!

11/07/2021 13:17:57

Non mi ricordo se avevo risolto così:

/**
* Sostituisce eventuali parentesi angolari in coppia in una stringa con parentesi quadre
* @param string $str : la stringa da controllare
* @return $str con la coppie di parentesi angolari sostituite con parentesi quadre
*/
function gdrcd_angs($str) {
$search = [
'#\<#is',
'#\>#is',
];
$replace = [
'«',
'»',
];

return preg_replace($search, $replace, $str);
}

Leggendra.eu - Fantasy steampunk itinerante
https://www.leggendra.eu ↗

quota

soizora

SCRIVI

11/07/2021 14:32:22

ma in pratica se non vuoi che le parentesi angolari vengano sostituite dalle quadre, non è sufficiente non usare la funzione gdrcd_angs del tutto?

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!

quota

angel1989

SCRIVI

11/07/2021 14:42:10

forsaken ha scritto: Ho controllato e da me funziona, su functions ho scritto così:

/**
* Sostituisce eventuali parentesi angolari in coppia in una stringa con parentesi quadre
* @param string $str: la stringa da controllare
* @return $str con la coppie di parentesi angolari sostituite con parentesi quadre
*/
function gdrcd_angs($str)
{
$search = array(
'#\<(.+?)\>#is',
'#\<(.+?)>#is',
);
$replace = array(
'[$1]',
'[$1]',
);
return preg_replace($search, $replace, $str);
}

In chat appare così:

E la cosa scritta nella textarea prima di inviare era:

testo < che dovrebbe stare < nelle parentesi ma sbagliamo >

Ho provato e purtroppo non mi risolve nulla. Nel senso: è vero quel che hai scritto, ma funziona bene solo se inserisci lo spazio tra le parentesi uncinate e la scritta. Se non lo fai sei punto a capo. Mentre in Gdrcd 5.4 il problema non si presentava neppure senza spazio.

Exclusive Villa GdR ↗
Las Vegas, azzardo, vizi, soldi, intrighi... Più le sue luci sono scintillanti, più le sue ombre sono oscure. La città che non dorme mai, vi aspetta.

quota

angel1989

SCRIVI

11/07/2021 17:44:28

Vi ringrazio per le risposte e scusate se ho aperto una nuova discussione per qualcosa di già presente. Ho cercato prima di aprire questa ma non ho trovato nulla.

Per quanto la vostra soluzione in effetti funzioni, me ne hanno suggerito un'altra che per quanto ci riguarda funziona meglio. Infatti volevamo qualcosa che non modificasse le parentesi angolari in fase di stampa sullo schermo.

Questo si può ottenere modificando ref_header.inc.php nel caso P dello switch del tipo di azione da così:

$add_chat.=': ';
$add_chat.= ''.gdrcd_chatme($_SESSION['login'], gdrcd_chatcolor(gdrcd_filter('out',$row['testo']))).'';

a così:

$add_chat.=': ';
$testo_pulito = htmlspecialchars($row['testo']);
$add_chat.= ''.gdrcd_chatme($_SESSION['login'], gdrcd_chatcolor(gdrcd_angs($testo_pulito))).'';

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

quota

soizora

SCRIVI

11/07/2021 18:02:51

Okkei h finalmente capito il punto della discussione. Praticamente il problema è che non ti colora in grassetto le scritte e si limita a sostituirti con parentesi quadre le parentesi angolari? Bhe per forza, ho confrontato il codice che hai messo tu:

function gdrcd_angs($str) {
$search = [
'#\<(.+?)\>#is',
'#\<(.+?)>#is',
];
$replace = [
'[$1]',
'[$1]',
];

return preg_replace($search, $replace, $str);
}

con quello di una qualsiasi versione di gdrcd precedente:

function gdrcd_chatcolor($str)
{
$search = array(
'#\<(.+?)\>#is',
'#\[(.+?)\]#is',
);
$replace = array(
'<$1>',
'<$1>',
);
return preg_replace($search, $replace, $str);
}

Non pè difficile rendersi conto del perchè del comportamento: I due codici fanno due cose completamente diverse. Il primo si limita solo a sostituire parentesi ang con quadre, il secondo invece sostituisce sia le angolari che le quadre con il marker html più consono per dare l'effetto colorato.

Francamente non so cosa pensare delle ultime versioni di gdrcd, non è il primo strafalcione madornale che trovo di questo tipo.

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

quota

angel1989

SCRIVI

11/07/2021 20:16:42 e modificato da angel1989 il 11/07/2021 20:19:39

soizora ha scritto: Okkei h finalmente capito il punto della discussione. Praticamente il problema è che non ti colora in grassetto le scritte e si limita a sostituirti con parentesi quadre le parentesi angolari? Bhe per forza, ho confrontato il codice che hai messo tu:

function gdrcd_angs($str) {
$search = [
'#\<(.+?)\>#is',
'#\<(.+?)>#is',
];
$replace = [
'[$1]',
'[$1]',
];

return preg_replace($search, $replace, $str);
}

con quello di una qualsiasi versione di gdrcd precedente:

Non pè difficile rendersi conto del perché del comportamento: I due codici fanno due cose completamente diverse. Il primo si limita solo a sostituire parentesi ang con quadre, il secondo invece sostituisce sia le angolari che le quadre con il marker html più consono per dare l'effetto colorato.

Francamente non so cosa pensare delle ultime versioni di gdrcd, non è il primo strafalcione madornale che trovo di questo tipo.

No, temo proprio di non essermi spiegato bene XD

Con gdrcd 5.5 senza modifiche se io invio in chat questa azione:

testo <che dovrebbe stare <nelle parentesi ma sbagliamo>

Questa parte:

non viene stampata. Perché nel db l'azione viene salvata così:

testo [che dovrebbe stare <nelle parentesi ma sbagliamo]

Quindi io per non perdere pezzi di azioni a causa di errori di battitura dei giocatori ho aggiunto alla funzione che sostituisce alla coppie di < > le [ ] che anche una singola < o una singola > venissero tramutate in parentesi quadre aperta o chiusa. Così in chat mi stampava questo:

testo <che dovrebbe stare [nelle parentesi ma sbagliamo>

Quindi ero conscio di cosa avessi fatto, solo che non mi piaceva come soluzione e sono venuto qui a chiedere se qualcuno ne avesse un'altra.

Credo che il problema si trovi in una commistione di filtro dell'html alto e la chiusura dei tag. Anche se non avrei idea di che modifiche apportare per risolverlo alla radice.

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

12/07/2021 14:13:20

Purtroppo, non esiste una soluzione semplice al problema.

Per cercare di correggere in corsia tutti i possibili strafalcioni degli utenti in termini di apertura e chiusura tag andrebbero ripensate le funzioni esistenti e, laddove è possibile, implementare una funzionalità di preview dell'azione in modo che l'utente possa notare subito gli errori e correggerli prima di inviare.

In ogni caso, senza andare troppo a scomodare le regexr, personalmente adotterei la soluzione di @haruka .

Questa è quella proposta da Lei. La puoi riscrivere in questo modo, per allinearla con le parentesi quadre.

/**
* Sostituisce eventuali parentesi angolari in coppia in una stringa con parentesi quadre
* @param string $str : la stringa da controllare
* @return $str con la coppie di parentesi angolari sostituite con parentesi quadre
*/
function gdrcd_angs($str) {
$search = [
'#\<#is',
'#\>#is',
];
$replace = [
'[',
']',
];

return preg_replace($search, $replace, $str);
}

È ovvio che non risolve il problema e che ti lascia le impurità in azione, ma lo trovo un poco più versatile.

Kasa.