Commande de réparation véhicule

[Sismix84]

bonsoir, je me suis fait une commande pour réparer son véhicule

if (strcmp("/rep", cmdtext, true, 10) == 0) { SetVehicleHealth(playerid, 1000); GivePlayerMoney(playerid, -1500); SendClientMessage(playerid, COLOR_GREEN, "Kit de réparation acheté."); return 1 }

mais il y a une erreur et je n'arrive pas a la résoudre

C:\Program Files\Rockstar Games\GTA San Andreas\samp022server.win32\gamemodes\Sismix84.pwn(743) : error 001: expected token: ";", but found "}"

est ce que quelqu'un peut m'aider ?

[NatiVe™]

Voila pour toi :

Code: [Sélectionner]

if (strcmp(cmdtext, "/rep" , true, 4) == 0){
 
SetVehicleHealth(playerid, 1000);
GivePlayerMoney(playerid, -1500);
SendClientMessage(playerid, COLOR_GREEN, "Kit de réparation acheté.");

return 1;
}
AK NV.

[Sismix84]

euh je ne voit pas ce que tu as changé dans mon script mais ça marche  :laugh

Merci

PS : tu as changer quoi au fait ? :blink

EDIT : a oui je voit, le ";" à la fin de "return 1", mais comment j'ai fait pour oublier ça ?  :-[ quel boulet que je suis  :bangin

[NatiVe™]

EDIT : a oui je voit, le ";" à la fin de "return 1"

Nan pas que ca !

J'ai changer ca :

Code: [Sélectionner]

   if (strcmp("/rep", cmdtext, true, 10) == 0)   
Par ca :

Code: [Sélectionner]

   if (strcmp( cmdtext, "/rep", true, 4) == 0)   
J'ai deplacer le cmdtext et j'ai mis  true, 4 au lieu de true, 10

Il y a 3 lettre donc true, 4

Et c'est a cause de ca que l'on voi que tu n'as pas créer la commande mais que c'est du copier / coller !


AK NV.

[UrbanGhetto]

Sismix ton code n'est pas très efficace car si le joueur a 0$ il ira dans les -1500$ je ne vais pas te donner la correction du code complet mais juste un indice :

Code: [Sélectionner]

GetPlayerMoney(playerid) A toi de voir  :)

[Syg]

Arsenik, ne serais-tu pas devenu un spécialiste du strcmp ?  :)
Le coup du 4 à la place du 10 : chapeau, c'est le genre d'erreur con qu'on met 3 heures à trouver.
Mais juste pour ton information : ce n'était pas la peine d'inverser "/rep" et cmdtext .... ça revient au même.

++
Syg

[NatiVe™]

Chacun a sa maniere de Coder ;)

AK NV.

[Sismix84]

Merci Arsenik, tu as résond, j'ai fait du copier/coller mais d'une autre commande de mon script.

Merci aussi à toi Urban, je connais la suite.

PS : je n'ai pas très bien compris la partie du code true, 4, je ne comprend pas en quoi elle consiste  :blink.

[spykerc8]

À indiquer le nombre de caractères de la commande pour vérifier si elle fait la même longueur.
('fin me semble que c'est ça, Syg confirmera ou infirmera)

[NatiVe™]

J'aurais dit aussi :

À indiquer le nombre de caractères de la commande pour vérifier si elle fait la même longueur.

et

Toujours mettre 1 chiffre de plus pour le / !

AK NV.

[Syg]

C'est pas tout à fait ça.

La fonction strcmp, comme son nom l'indique sert à comparer des chaînes de caractères (string compare).
Les paramètres sont donc les deux chaînes à comparer, puis un indicateur pour savoir si on fait la différence entre majuscule et minuscule et enfin, dernier paramètre, le nombre de caractères à comparer.
Et dans ce cas, la longueur des deux chaînes importe peu, pourvu qu'elles fassent toutes les deux au moins 4 caractères.

Dans le cas qui nous occupe, on veut comparer ce qu'a saisi le joueur (cmdtext) avec la chaîne "/rep" qui contient 4 caractères : le /, le r, le e et le p.
Donc strcmp (cmdtext, "/rep", true, 4);
NB : la valeur true du troisième paramètre indique que l'on ne veut pas faire la différence entre majuscules et minuscules (ce qui veut dire que /REP, /Rep, /reP, ... sont toutes équivalentes).

Mais attention, avec le test ci dessus, si le joueur saisi "/reparer", "/repair", "/repetition" ou "/repense à tes vacances",  la commande sera exécutée puisqu'on ne compare que le 4 premier caractères.

Pour être sur que notre script ne prenne en compte que la commande "/rep", il faudrait utiliser la ligne suivante :
strcmp (cmdtext, "/rep", true);
Le 4 à disparu. Ceci indique au script qu'il doit faire la comparaison jusqu'au dernier caractère de la chaîne la plus courte.

Mais laisser le 4 permet d'accepter les commandes "/rep", "/repair", "/reparation", "/reparer" qui sont toutes valables.

++
Syg

[UrbanGhetto]

Syg nous apprend un nouveau truc comme d'habitude  :)

[R@f]

dommage que syg n'est pas mon prof d'informatique  :laugh

[NatiVe™]

Lol

" /repense à tes vacances "

Je pense pas que les PlayerS vont aller faire cette commande pour repare leurs vehicule lool !  :laugh

AK NV.

[Sismix84]

Merci beaucoup Syg pour tes explications, et sans oublier Arsenik et Urban :D