Problème d'anti cheat armure

[LePtitClown]

Slt

Je suis en train de faire un anti cheat pour lutter contre les cheat d'armure mais j'ai quelque soucis. Voici mon code:

Code: [Sélectionner]

new Float:armourhack[MAX_PLAYERS];

armourhack[playerid]=0;//dans OnPlayerConnect
armourhack[playerid]=0;//dans OnPlayerDisconnect


stock GetPlayerArmourhack(playerid)
{
return armourhack[playerid];
}

stock SetPlayerArmourhack(playerid,Float:armour)
{
GetPlayerArmourhack(playerid);
armourhack[playerid]= SetPlayerArmour(playerid,armour);
}

Il y a évidemment un timer qui compare la variable à GetPlayerarmour et qui banni si ces 2 valeurs ne sont pas égales. Ces 2 fonctions remplaceraient mes 2 fonctions SetPlayerArmour et GePlayerArmour pour permettre un suivi de l'armure du joueur grâce à une variable. Hélas, il y a toujours des soucis au moment de la compilation.
Des idées pour corriger ceci? Merci d'avance.

[Xartrick]

Code: [Sélectionner]

stock SetPlayerArmourhack(playerid, Float:armour) // <----------------\
{                                                               //    |
GetPlayerArmourhack(playerid);                          //    |
                                                        //    |
armourhack[playerid]= SetPlayerArmour(playerid,armour); // ---/
}

[LePtitClown]

Slt

Je ne comprend pas où tu veux en venir ???

A+ :)

[Ssk]

En gros tu créer une fonction qui s'appelle elle même, donc ça te fait une "boucle infinie"

[LePtitClown]

Slt

Oui en effet, mais moi ce que je veux c'est que lorsque je tape cette fonction, ma variable prend la valeur de la vie du joueur.

Code: [Sélectionner]

stock SetPlayerArmourhack(playerid,Float:armour)
{
SetPlayerArmour(playerid,armourhack[playerid]);
}

Il faudrait mettre ça comme ça ??

[Xartrick]

Code: [Sélectionner]

new Float:fPlayerArmour[MAX_PLAYERS] = {0.0};

stock GetPlayerArmourFromServer(playerid) {
    return fPlayerArmour[playerid];
}

stock SetPlayerArmourFromServer(playerid, Float:fArmour) {
    SetPlayerArmour(playerid, fArmour);
    
    fPlayerArmour[playerid] = fArmour;
}

stock IsArmourHacking(playerid) {
    if (GetPlayerArmour(playerid) != GetPlayerArmourFromServer(playerid)){
        return true;
    }
    
    return false;
}
Voilà un code fonctionnel et relativement complet.

[LePtitClown]

Slt

J'ai essayé de mettre ce code sur mon serveur mais je rencontre une erreur avec une de tes fonction:

Code: [Sélectionner]

stock GetPlayerArmourFromServer(playerid)
{
    return fPlayerArmour[playerid];
}

L'erreur se trouve à la ligne: return fPlayerArmour[playerid]

Le message d'erreur est:

warning 213: tag mismatch

A+ ;)

[Gilux]

L'armure c'est du type float je crois, ajoute ça dans les paramètres ;)

[LePtitClown]

Mais elle est déjà définit en Float:
new Float:fPlayerArmour[MAX_PLAYERS] = {0.0};
Je ne comprend pas où tu veux en venir ;D

[spiirou]

On ne peux pas retourner un float ? Il faut le mettre dans les parametres non ?

Code: [Sélectionner]

stock GetPlayerArmourFromServer(playerid, &Float:armure)
{
    armure = fPlayerArmour[playerid];
}
Ca se fait ca ?

[Xartrick]

Les tags ne sont pas d'une importance capitale, si vous voulez les utiliser, il faut le faire jusqu'à bout:

Code: [Sélectionner]

new Float:fPlayerArmour[MAX_PLAYERS] = {0.0};

stock Float:GetPlayerArmourFromServer(playerid) {
    return Float:fPlayerArmour[playerid];
}

stock SetPlayerArmourFromServer(playerid, Float:fArmour) {
    SetPlayerArmour(playerid, fArmour);
   
    fPlayerArmour[playerid] = fArmour;
}

stock IsArmourHacking(playerid) {
    if (GetPlayerArmour(playerid) != GetPlayerArmourFromServer(playerid)){
        return true;
    }
   
    return false;
}
Sinon, pour la ligne suivante:

Code: [Sélectionner]

new Float:fPlayerArmour[MAX_PLAYERS] = {0.0};
J'initialise toutes les valeurs du tableau à 0.0 (soit 0).

[LePtitClown]

Slt

Code: [Sélectionner]

stock Float:GetPlayerArmourHack(playerid)
En utilisant ce nouveau Float dans la fonction précédente, j'obtiens le message d'erreur ci dessous:

warning 208: function with tag result used before definition, forcing reparse
fonction avec résultat étiquette utilisée avant la définition, ce qui oblige une révision

Si vous avez des idées pour réparer ce problème, je suis preneur

A+ ;)

[Xartrick]

Le prototype doit être défini avant l'utilisation.
Met tous tes prototypes dès le début de ton script, ça sera plus simple.

Code: [Sélectionner]

stock Float:GetPlayerArmourFromServer(playerid);
stock SetPlayerArmourFromServer(playerid, Float:fArmour);
stock IsArmourHacking(playerid);

new Float:fPlayerArmour[MAX_PLAYERS] = {0.0};

stock Float:GetPlayerArmourFromServer(playerid) {
    return Float:fPlayerArmour[playerid];
}

stock SetPlayerArmourFromServer(playerid, Float:fArmour) {
    SetPlayerArmour(playerid, fArmour);
   
    fPlayerArmour[playerid] = fArmour;
}

stock IsArmourHacking(playerid) {
    if (GetPlayerArmour(playerid) != GetPlayerArmourFromServer(playerid)){
        return true;
    }
   
    return false;
}

[LePtitClown]

Slt

Merci beaucoup, ça fonctionne nickel. Par contre, je ne compred pas pourquoi je ne peux pas vérifier l'armure dans un timer.
Lorsque je laisse le timer faire, je suis ban en permanence.
Dès que je tape une cmd pour avoir un gilet, le message s'arrête

Code: [Sélectionner]

if (strcmp("/test", cmdtext, true) == 0)
{
SetPlayerArmourHack(playerid, 50);
format(string1, sizeof(string1), "var armour: %f", armourhack[playerid]);
SendClientMessageToAll(COLOR_INFO, string1);
return 1;
}
   

[spiirou]

Tu fais comment dans le Timer ? ( code ? )