Problème avec mon anticheat cheat SERVER SIDE

[Syg]

Reste maintenant à savoir si ma proposition corrige le problème.
D'ailleurs, je me rappelle qu'à l'époque, j'avais aidé Guigui a corrigé un problème similaire sur le SFTDM concernant l'argent. C'était il y a 2 ou 3 ans et je me rappelle plus ce qu'on avait fait mais surement quelque chose d'équivalent.

++
Syg

[Ssk]

ça m'étais aussi arrivé mais je me souviens plus comment j'ai réglé le problème lol

[Winklevows]

Bonjour,
Je pensais que mon topic avait été abandonné mais je suis retombé dessus par hasard et je vois que non finalement  :)

Étant en plein bac actuellement je vais pas pouvoir vérifié ton code maintenant Syg...

Avant sa, je me rappel (si je me souvient bien) que lorsque je donnais de l'armure avec ma fonction, la variable SSArmor[playerid] était égal à 100 mais lorsque le timer "checkais", l'armure était reset car la variable SSArmor[playerid] était égal à 0 (quelque fois elle était bien était égal à 100 donc l'armure n'était pas reset, par conséquence l'armure était bien donné).

[Winklevows]

Bonjour,

Alors j'ai une mauvaise nouvelle, ton code ne fonctionne pas plus Syg.

Mais j'ai trouvé la solution: Utilisé un booléen qui indique à l'anticheat si le joueur peut ou non utiliser l'armure.

Code: [Sélectionner]

stock SetPlayerArmourEx(playerid, Float:varArmor)
{
CanUseArmour[playerid] = 1; // Indique que le joueur peut utiliser l'armure qu'il a acheté / trouvé
SSArmor[playerid] = varArmor;
return SetPlayerArmour(playerid,varArmor);
}
Ensuite, au lieu d'utiliser un Timer, j'ai préféré utilise OnPlayerUpdate qui est, selon moi, plus adapté pour un anti-cheat bien que mon propos peut être discutable.

Code: [Sélectionner]

public OnPlayerUpdate(playerid)
{
if(IsSpawned[playerid] == 1 && AdminLevel[playerid] == 1337)
{
new Float:tArmor;
GetPlayerArmour(playerid, tArmor);
if(tArmor != SSArmor[playerid]) /* Premier contrôle */
{
/* On récupère l'armure pour un double controle */
GetPlayerArmour(playerid, tArmor);
if(tArmor < SSArmor[playerid]) /* Deuxième contrôle */
{
SSArmor[playerid] = tArmor;
}
else if(tArmor > SSArmor[playerid])
{
if(CanUseArmour[playerid] == 1) // Si le joueur peut utiliser l'armure qu'il a acheté/trouvé
{
CanUseArmour[playerid] = 0;
SSArmor[playerid] = tArmor; // alors on met à jour la variable qui n'avait pas été mise à jour pour une raison inconnue ??
}
else // Sinon cela signifie qu'il a cheaté donc:
{
new username[MAX_PLAYER_NAME], string[150];
GetPlayerName(playerid,username,sizeof(username));
format(string,sizeof(string),"[ANTI-CHEAT]"ECOLOR_WHITE" %s(%d) a tenté de regénérer son armure.", username,playerid);
SendClientMessageToAllAdmins(string);
SetPlayerArmour(playerid,SSArmor[playerid]);
}
}
}
}
return 1;
}
Et sa fonctionne d'après mes tests !

Je pense qu'en réalité la variable SSArmor n'avait pas le temps de se mettre à jour car elle était reset aussitôt, j'ai remarqué sa car le bug était plus flagrant en utilisant OnPlayerUpdate qu'avec un timer (qui est exécuté moins souvent);

Que pensez vous de ma solution ?

Merci

[Ssk]

Dans la première condition tu test si le joueur est admin au level 1337, cela veut donc dire que les autres joueurs qui ne sont pas admins pourront cheater sans problème.

[Winklevows]

S'était pour utiliser mon compte administrateur et mes commandes admins pour me donner de l'armure lors de mes tests, j'ai juste oublié de remplacé 1337 par 0 quand je vous ai collé mon code  :happy