[FS] N2O for Admins

[cristab]

Sim rooo

if (IsPlayerAdmin(playerid))//si admin
   {
      new TypeVoiture;//var
      TypeVoiture = GetVehicleModel(vehicleid);//var pour le Type de voiture
      if(TypeVoiture != IsInvalidNosVehicle(vehicleid))//si le type de voiture n'est pas dans la liste des invalidenos
      {
         AddVehicleComponent(vehicleid,1010);//on ajoute le nos
      }
   }

[S!m]

Salut,

Sim rooo

if (IsPlayerAdmin(playerid))//si admin
   {
      new TypeVoiture;//var
      TypeVoiture = GetVehicleModel(vehicleid);//var pour le Type de voiture
      if(TypeVoiture != IsInvalidNosVehicle(vehicleid))//si le type de voiture n'est pas dans la liste des invalidenos
      {
         AddVehicleComponent(vehicleid,1010);//on ajoute le nos
      }
   }

je ne comprend toujours pas comment le modèle de la voiture (normalement de 400 à 611 je crois) est supposé être comparé à la valeur que retourne la fonction (toujours 0 ou 1). Normalement ce test sera toujours vérifié (comme le modèle est toujours différent que 0 ou 1) et le code est en quelque sorte inutile.
D'ailleurs il serait plus judicieux de passer le modèle du véhicule en paramètre plutôt que son ID. On éviterais d'appeler GetVehicleModel plusieurs fois dans la fonction.

En gros, moi je ferais:

Code: [Sélectionner]

IsInvalidNosVehicleModel(model)//changé le nom pour + de clarté
{
#define MAX_INVALID_NOS_VEHICLES 65

new InvalidNosVehicles[MAX_INVALID_NOS_VEHICLES] =
{
417,425,430,432,435,441,446,447,448,449,450,452,453,454,
460,461,462,463,464,465,468,469,472,473,476,481,484,487,
488,493,497,501,509,510,511,512,513,519,520,521,522,523,
537,538,548,553,563,564,569,570,577,581,584,586,590,591,
592,593,594,595,606,607,608,610,611
};
for(new i = 0; i < MAX_INVALID_NOS_VEHICLES; i++)
{
if(model == InvalidNosVehicles[i])
{
return true;
}
}
return 0;

}

public OnPlayerEnterVehicle(playerid, vehicleid, ispassenger)
{
//si le joueur est admin et que son véhicule peut prendre de la nitro
if (IsPlayerAdmin(playerid) && !IsInvalidNosVehicleModel(GetVehicleModel(vehicleid)))
{
    AddVehicleComponent(vehicleid,1010);
  }
return 1;
}

enfin chacun peut faire comme il le souhaite

++Sim++

[cristab]

ah ben oui bien vue ^^

[spiirou]

Effectivement, bien vu comme l'a dit Cristab  :D

[BuBuNo]

Sim rooo

if (IsPlayerAdmin(playerid))//si admin
   {
      new TypeVoiture;//var
      TypeVoiture = GetVehicleModel(vehicleid);//var pour le Type de voiture
      if(TypeVoiture != IsInvalidNosVehicle(vehicleid))//si le type de voiture n'est pas dans la liste des invalidenos
      {
         AddVehicleComponent(vehicleid,1010);//on ajoute le nos
      }
   }

Je me demandais es ce que si je fait AddVehicleComponent sur une moto par exemple sa ne metteras pas de nos donc je pense que la fonctions est deja proteger un peu :P Donc bon moi j'aurais plutôt fait comme ceci :

if (IsPlayerAdmin(playerid))//si admin
   {
      
         AddVehicleComponent(vehicleid,1010);//on ajoute le nos
   }

Qui je pense marche :P

[Mr_C30]

GG  ;) bravo a toi :P

[Ssk]

Sim, tu devrai savoir qu'on retourne toujours le même type de données lol, toi tu retourne sois un boolean soit un integer ^^

[cristab]

Sim rooo

if (IsPlayerAdmin(playerid))//si admin
   {
      new TypeVoiture;//var
      TypeVoiture = GetVehicleModel(vehicleid);//var pour le Type de voiture
      if(TypeVoiture != IsInvalidNosVehicle(vehicleid))//si le type de voiture n'est pas dans la liste des invalidenos
      {
         AddVehicleComponent(vehicleid,1010);//on ajoute le nos
      }
   }

Je me demandais es ce que si je fait AddVehicleComponent sur une moto par exemple sa ne metteras pas de nos donc je pense que la fonctions est deja proteger un peu :P Donc bon moi j'aurais plutôt fait comme ceci :

if (IsPlayerAdmin(playerid))//si admin
   {
      
         AddVehicleComponent(vehicleid,1010);//on ajoute le nos
   }

Qui je pense marche :P

si tu arrive a mettre du nos sur une moto/bateau/avion sans faire crash le serveur ben GG a toi :)

[S!m]

Salut,

Sim, tu devrai savoir qu'on retourne toujours le même type de données lol, toi tu retourne sois un boolean soit un integer ^^

comme sa-mp a pas vraiment de boolean (seulement un système bidon de tags), true = un entier de valeur 1 si je ne m'abuse...
sinon c'est que j'ai utiliser la fonction que l'on retrouve dans le fs en changeant seulement le vehicleid à model et GetVehicleModel(vehicleid) à model. Donc, je n'ai pas remarqué les valeurs de retour...sans doute aurais-je dû y porter + attention

ps. je crois que bubu a raison. Il me parait plus logique de la part de la team d'avoir inclus le test de la validité des composantes (sur le modèle de véhicule)  dans la fonction.

++Sim++

[Watash]

J'comprends rien moi aussi, mais c'est cool mec, continue comme ça ! ;)

[cristab]

eu j'ai tester sur une moto moi le serveur crash toujours donc je pige pas du tout pourquoi vous sa ne crash pas

[spiirou]

Petit ( ou gros ) UP,
pour vous annoncer une v2,
Qui combinera ce FS avec un Autre.

Cette version sera publiée sur le 1er post ;)

EDIT  : Je vais publier l'autre FS a part, et je mettrai le lien de DL des deux FS sur les deux topics