[FS] Achat d'arme via menu | Mon premier FS |

[ipowergraphiix]

Salut,
Je viens de finir mon premier FS potable , il faut dire que je n'ai pour le moment pas de très grande compétence en Pawn , ça fait que quelques semaines que j'ai commencé.

Le script à été réalisé à 100% (Aucune plagia ni autre) par moi même je me suis fixé des objectifs et j'ai tenté de les atteindre , si des personnes pense que j'ai copié collé un autre script ils ont tord. Il n'y à pas 1000 façons de faire la même chose !

Donc mon filterscript est un script qui permet d'acheter des armes avec un prix définit, il est donc possible pour vous d'ajouter d'autres armes facilement

Afin d'aficher le menu InGame vous devez taper /armes et naviguer dans le menu sans plus attendre je vous livre le lien de téléchargement (PasteBin) , Je vous demande juste de ne pas enlever les credits mais vous pouvez l'utiliser sur n'importe quel de vos serveur tant que mon nom reste dans les commentaires  :P
 

V1.0

http://pastebin.com/d2aNxUfL

V1.5

http://pastebin.com/jNWcSWyf

- Création d'une fonction permettant de regrouper le tout
- Ajout de crédit
- Ajout de la commande /gg via une define : // #define COMMANDE_TEST //Si la commande est décommenté alors via la commande /gg pour effectuer vos tests vous pourrez vous spawn 999999$ !

V2.0 Prévision

- Création d'un switch

Crédit

- Gilux pour son idée , son explication et la vitesse dont il ma répondu.  :)
- Vuki pour m'avoir débloqué :-)

* J'aimerai si possible que vous me donnez des avis constructif sur la façon à laquelle je me suis pris pas du genre"Waa c cro bien t on truk ta vu"

[Gilux]

waaa c kro bi1 ton truk tavu 

Plus sérieusement, bien joué, c'est un script classique pour un premier, bien indenté, des defines... pas mal.
Par contre je te conseille 2 choses :

Au lieu de faire des if(listitem == ...), utilise la structure switch, ça te permettra de gagner quelques lignes dans ton code.

Et surtout, tu utilises pour chaque arme ce code :

Code: [Sélectionner]

    if(GetPlayerMoney(playerid) >= 500)
    {
                 GivePlayerMoney(playerid,-500);
                 GivePlayerWeapon(playerid, 30, 500);
                 SendClientMessage(playerid, COLOR_VERTE, "Vous avez bien acheté votre AK-47");
     }
     else
     {
                 SendClientMessage(playerid, COLOR_ROUGE, "Vous n'avez pas assez de sous");
     }
Tu peux créer une fonction qui regroupe les instructions et la condition, comme ça pour chaque arme tu pourras remplacer ces 10 lignes par une seule, par exemple une fonction DonnerArme(playerid,argent,arme,munitions,message);
En plus, tu pourras utiliser tes switch sans accolades, donc gagner encore des lignes.
Je te laisse essayer tout ça si tu veux l'améliorer :P

[ipowergraphiix]

Salut Gilux,
Je te remercie de ta réponse pour la fonction DonnerArme j'y vais vraiment pas pensé  :) !
Sinon pour le switch j'avais déjà essayé mais j'ai rencontré des problèmes avec.  :-X

Donc prévision pour la seconde version
* Création d'une fonction permettant l'ajout des armes , des prix et des textes plus rapidement qui sera nommé VendreArme.
* Peut être : refonte des "listitem" en un switch.
Autres :

** Modification du sujet initiale + Ajout de Gilux dans les crédit.

[[KoD]Jhonny]

Salut,
Comme a pu le dire Gilux c'est un script classique mais bien joué quand même pour ton premier script :)

[Kiloutre]

Au début, j'ai cru que c'était mon FS 
En tous cas bravo a toi et merci du partage