à chaque commande meme juste, SERVER:Unknow Command

[feldbrau]

Met un return 1; a la fin de chaque else if, ca marchera ;)

[Nikko™]

ah oui pas bete ;)

++
nikko

[[StAfF]MaN]

oui lol, pourquoi on se fait ch*** pour ça alors que c'est tout simple  :laugh

[feldbrau]

Content d'avoir pu aider ;)

[pinouf42]

Merciiiii sa marche j'ai meme pu mettre commande invalide
Merci beaucoup

[Bayshore]

Pas forcement !

NV.

Comment ça ? ...

[Syg]

Je répond pour Native à Alexandre :
En fait, on peut avoir envie que certaines commandes ne soit visible que par certains joueurs.
Par exemple, la commande /ban qui ne doit être utilisée que par les admins.
Soit lorsqu'un joueur non admin l'utilise on lui affiche un message du genre "Vous n'êtes pas admin" soit on lui fait croire que le commande n'existe pas :

Code: [Sélectionner]

if (strcmp(cmdtext, "/ban", true) == 0)
{
if (IsPlayerAdmin (playerid))
{
/* Code pour bannir un joueur */
...
return 1;
}
else
{
return 0;
}
}
++
Syg

[[StAfF]MaN]

Ha Ha !! ça c'est trop fort !! Allé paf dans les dents les petits curieux qui pensent connaitre la commande ^^ (même si elle est belle et bien sur le serveur :p )

[Nikko™]

Pk tu dit ca ???
Nempeche, je savais pas qu'on pouvait metre un reuturn 1 ou 0 dans une commande. cets pas mal pour dissimuler les comandes admin ^^

++
nikko

[[StAfF]MaN]

je dis ça parce que j'y avais pas pensé et que c'est bien pratique pour faire croire que cette commande existe pas alors que, si, elle existe ^^

[Bayshore]

Je répond pour Native à Alexandre :
En fait, on peut avoir envie que certaines commandes ne soit visible que par certains joueurs.
Par exemple, la commande /ban qui ne doit être utilisée que par les admins.
Soit lorsqu'un joueur non admin l'utilise on lui affiche un message du genre "Vous n'êtes pas admin" soit on lui fait croire que le commande n'existe pas :

Code: [Sélectionner]

if (strcmp(cmdtext, "/ban", true) == 0)
{
if (IsPlayerAdmin (playerid))
{
/* Code pour bannir un joueur */
...
return 1;
}
else
{
return 0;
}
}
++
Syg

Ah, ok je vois. :happy

Je voyait pas car moi j'"englobe" mes commandes admins dans une condition pour regarder son niveau admin, ça sert à rien de vérifier si le joueur à tapé une commande admin alors qu'il ne l'est pas. :happy

Code: (pawn) [Sélectionner]

if(pInfo[playerid][admin])
{
if(!strcmp(cmd, "/cmd", true))
{
// ...
return 1;
}
}

++
Alexandre

[Syg]

J'ai pris le statut d'admin comme un exemple mais ça pourrait être en fonction d'autre chose (team, level).
Si tu n'as que deux status (joueur/admin), tu peux faire comme tu as dit, c'est effectivement plus efficace (en terme de temps de traitement).
Par contre, lorsque tu as plusieurs niveau d'administration, c'est plus facilement gérable de mettre le test dans chaque commande.

++
Syg

[Bayshore]

Enfait j'ai plusieurs niveau d'administration, là j'ai retiré le >= 1. ;D

Si j'englobe toutes les conditions admin (de 1 à 6), pour ensuite en faire 6 conditions dans une, est-ce qu'il y aura un gain de temps d'éxécution pour les nons-admins ? :)

++
Alexandre

[Syg]

A mon avis si il y a un gain, il ne sera pas sensible.
Un test sur un variable droit prendre moins d'1 milliseconde.
Donc à moins d'avoir un millier de tests, tu ne verras pas la différence.

Par contre, les test sur les strcmp sont beaucoup plus long (on compare caractère par caractère la commande saisie avec des chaines).
Dans ce cas, l'emploi de else avant chaque if et la fait d'essayer de mettre les commandes en commençant par la plus utilisée peut peut-être faire gagner quelques centaines de millisecondes.
Reste à savoir quelles sont les commandes les plus utilisées.

++
Syg

[Bayshore]

Un test sur un variable droit prendre moins d'1 milliseconde.
[...]
Par contre, les test sur les strcmp sont beaucoup plus long (on compare caractère par caractère la commande saisie avec des chaines).

Donc il vaut mieux englober les strcmp par des if(pInfo), non ? :happy

Sinon, bonne idée celle de mettre des else if en placant les plus utilisées en premiers. :happy
Je pense que je vais faire ça. :)

++
Alexandre