Fonction gerer que par le client et non par le serveur

robin4002

Bonjour,

Peux-tu envoyer le code en rapport avec ta gestion d’xp ?

ZunF1x

	public void addXp(int value) {
		xp += value;
		
		if (getManager().xp >= getManager().maxXp + 1) {
			getManager().xp = 0;
			
			if (getManager().level == this.number) {
				getManager().level++;
				this.number++;
				getManager().maxXp += 500;
			}
		}
	}

package fr.zunf1x.icemc.packets;

import fr.zunf1x.icemc.utils.ExperienceManager;
import io.netty.buffer.ByteBuf;
import net.minecraft.entity.player.EntityPlayerMP;
import net.minecraftforge.fml.common.network.simpleimpl.IMessage;
import net.minecraftforge.fml.common.network.simpleimpl.IMessageHandler;
import net.minecraftforge.fml.common.network.simpleimpl.MessageContext;

public class PacketJobsXP implements IMessage {
	
	private int xp;
	private int maxXp;
	private int level;
	
	public PacketJobsXP() {
		
	}
	
	public PacketJobsXP(int xp, int maxXp, int level) {
		this.xp = xp;
		this.maxXp = maxXp;
		this.level = level;
	}
	
	@Override
	public void fromBytes(ByteBuf buf) {
		this.xp = buf.readInt();
		this.maxXp = buf.readInt();
		this.level = buf.readInt();
	}

	@Override
	public void toBytes(ByteBuf buf) {
		buf.writeInt(this.xp);
		buf.writeInt(this.maxXp);
		buf.writeInt(this.level);
	}
	
	public static class Handler implements IMessageHandler<PacketJobsXP, IMessage> {
		@Override
		public IMessage onMessage(PacketJobsXP message, MessageContext ctx) {
			EntityPlayerMP player = ctx.getServerHandler().player;
			ExperienceManager.getManager().addXp(250);
			return null;
		}
	}
}

			if (button.id == 0) {
				IceMC.network.sendToServer(new PacketJobsXP(ExperienceManager.getManager().xp, ExperienceManager.getManager().maxXp, ExperienceManager.getManager().level));
			}

robin4002

Ta fonction addXp tu l’appel où et elle se trouve dans quoi comme classe ?

Parce que là j’ai l’impression que tu essaies de synchro des valeurs calculé sur le client vers le serveur, ce qui est une très mauvaise idée puisqu’un joueur pourrait envoyer un fausse valeur pour tricher.
Tu dois mettre en place la logique inverse : le serveur doit faire tout les calcules et la valeur doit être synchronisé avec le client.
Si tu as une capabilité pour l’xp, le plus simple est de synchro les valeurs à la connexion du joueur, quand il rejoint le monde (tu peux utiliser l’event LivingJoinWorldEvent pour ça).

ZunF1x

il faut faire une capability?

robin4002

Pas obligatoirement mais c’est mieux. Tu gère comment ton xp actuellement et la sauvegarde de sa valeur ?

ZunF1x

je n’ai pas encore de sauvegarde mais j’ajoute l’xp a une variable dans mon code

robin4002

Ok donc c’est probablement il semble variable qui tu as dans une classe, pas étonnant que rien ne fonctionne côté serveur du-coup (tout le monde aura la même valeur). Il te faut une capability pour faire les choses correctement.

ZunF1x

Okay merci ^^

UtopiePhysique

Bonjour,

Ne pas oublier qu’il ne faut rien gérer côté client, surtout pas de fonction qui attribut quelque chose sans sécurité côté serveur, sinon 3 minutes suffisent pour que votre mod devient “le mod à ne surtout pas ajouter sur le serveur”, comme blibliocraft et beaucoup d’autres mods qui ont eu de très grosses faille car ils gèrent côté client strictement sans vérification supplémentaire côté serveur et POUF.

C’était juste un rappel au cas où

ZunF1x

Okay merci!