[1.7.10 -> 1.12] Synchroniser ses données avec DBSynchronizer

BrokenSwing

D’après ce que j’ai compris, si un changement de données s’effectue sur le client, alors les données sont aussi modifiés sur le serveur. Ça peut être utile dans certains cas mais dans la majorité des cas cela va donner une faille exploitable par les joueurs pour tricher

Cl00e9ment

Oui effectivement. Je n’y avais pas pensé.
Peut être que je peux régler ce problème en faisant en sorte que le serveur doit au préalable autoriser tel ou tel client à accéder à tel ou tel donnée. De ce fait les clients n’auront accès qu’aux données dont le server leur a autorisé l’accès. Les autres données seront disponibles qu’en lecture seule.

Qu’en pensez-vous ?

BrokenSwing

Tu fais un système d’approbation du changement de donnée. Une fonction du style :


public boolean canClientChangeData(EntityPlayerMP player, DBFolder currentDatas, DBFolder changedDatas)

En gros par défaut ça retourne toujours faux (parce qu’il vaut mieux interdire l’écriture des données par le client par défaut). Si la fonction renvoie vrai les données actuelles sont remplacées par les nouvelles, sinon c’est les anciennes qui restent. Au niveau de l’emplacement de la fonction je te laisse l’intégrer comme bon te semble. Tu peux tout à fait mettre la fonction dans le DBFolder en changeant le prototype, enfin bon, voilà.

Cl00e9ment

Ok, c’est bien à ça que je pensais. Merci.
Je vais m’y mettre tout de suite !

LavaPower

Salut, je suis devant un nouveau problème. Je veux me servir de DBSynchronizer pour faire une sorte de cooldown sur mon block :
Voici la classe de mon bloc où on bloque l’activation si le cooldown n’est pas fini : https://pastebin.com/K0kGvmjD
Et la classe de ma Tile Entity où le cooldown descend : https://pastebin.com/xwBZzxBA

Le problème c’est que le cooldown ne descend jamais. Le bloc me dit toujours d’attendre 1 seconde (sauf la première fois)

Cl00e9ment

Alors je vois déjà plusieurs problèmes :

Tu génère un nom à tes cooldows en fonction de la position du bloc, sauf que selon ta façon de faire, un bloc situé en (x:10 y:63 z:2) génèrera le même nom qu’un bloc situé en (x:106 y:3 z:2). Je te conseile donc de séparer tes coordonnées par un espace:

String key = this.xCoord +" "+ this.yCoord +" "+ this.zCoord;

Ensuite je vois que tu met plusieurs fois le même bout de code qui sert à générer la clé. Utilise ce bout de code qu’une seul fois : stock la valeur dans une variable et le tour est joué, c’est plus optimisé et plus propre.

Dans ta classe TE, ton if (je ne parle pas du else) ne sert à rien, en effet, si le cooldown est égal à 0 alors tu le met à 0.

Donc je te laisse régler ça. Ensuite tu pourra mettre un System.out.println (“test”); dans la méthode updateEntity() pour vérifier qu’elle est bien appelée.

LavaPower

J’ai modifié ce que tu m’as dis de modifier.
J’ai mis le System.out.println et comme je le pensais, la fonction est bien appelée.

Cependant, toujours rien

Cl00e9ment

Tu peux me renvoyer ton code modifié stp ?

Cl00e9ment

Ok c’est bon j’ai trouvé.
Comme quoi des fois c’est juste des fautes d’inattention qui font chercher longtemps .

Tu n’utilises pas la même clé en fonction de tes classes :
Dans la classe Tappe tu utilises “TrappeCooldown”, et dans la classe TE tu utilises “MarteauCooldown”.

Je tiens à rajouter que si tu n’as pas besoin de ton cooldown à la fois sur le server et sur le client alors tu n’as pas besoin de passer par ma library et ça sera plus propre.

LavaPower

Merci ça marche nickel !