Probleme "ispassenger"

[vardanega]

Bonsoir, je voudrais que tout les joueurs puissent entrez en passager dans les véhicules de faction mais il ne peuvent pas entrez du tout dans les véhicules protégé a part les membres de l'equipe a laquelle appartient le véhicule.

Voici mon code:

Code: [Sélectionner]

if(vehicleid <= 10)
{
    if(PlayerInfo[playerid][pMember] != 1 || PlayerInfo[playerid][pLeader] != 1 && !ispassenger)
    {
    SendClientMessage(playerid, 0xFF0000FF, "Vous n'avez pas les clefs de ce véhicule !");//VEHICULE STARS
    RemovePlayerFromVehicle(playerid);
    TogglePlayerControllable(playerid, 1);
    return 1;
}
else if(PlayerInfo[playerid][pMember] == 2 || PlayerInfo[playerid][pLeader] == 2 && ispassenger)
{
    return 1;
}
}



Je voudrais seulement que les joueurs puissent entrez en passager.

Merci beaucoup

Vincent

[cristab]

Code: [Sélectionner]

if(vehicleid <= 10)
{
   if(!ispassenger)
   {
      if(PlayerInfo[playerid][pMember] != 1 || PlayerInfo[playerid][pLeader] != 1)
      {
         SendClientMessage(playerid, 0xFF0000FF, "Vous n'avez pas les clefs de ce véhicule !");//VEHICULE STARS
RemovePlayerFromVehicle(playerid);
TogglePlayerControllable(playerid, 1);
return 1;
      }
      else if(PlayerInfo[playerid][pMember] == 2 || PlayerInfo[playerid][pLeader] == 2)
      {
         return 1;
      }
   }
}
tente comme ceci par contre le titre de ton sujet n'as rien avoir avec ton probleme

[vardanega]

Effectivement sa fonctionne ! merci beaucoup !

[scott1]

Le premier code n'était pas juste? ou alors j'ai mal compris

[Pixect]

Salut,

Non scott, tu avais raison, le 1er code était faux, et celui de cristab est juste. ;)

++
Lloyd

[scott1]

Code: [Sélectionner]

if(!ispassenger)
   {
      if(PlayerInfo[playerid][pMember] != 1 || PlayerInfo[playerid][pLeader] != 1)
      {

s'est la meme chose que

Code: [Sélectionner]

if(PlayerInfo[playerid][pMember] != 1 || PlayerInfo[playerid][pLeader] != 1 && !ispassenger)
    {

Non? Ou alors je vais pas bien mais pour moi s'est la même chose

[cristab]

c'est censer oui mais les RP sont tellement bien optimiser qu'apparement sa n'as pas le temps de lire toutes les condition donc autant soulager le serveur avec une seul condition pour commencer

[scott1]

Cristab fait nous un cours d'optimisation ^^

[cristab]

^^ je pense que syg a deja fait un tuto la dessus apres le plus simple et de penser comment rendre le plus simple possible la lecture d'un code est ce en lui mettant plein de condition a ne plus finir ou est ce de lui faire comprendre que si Z on fait E et si A on fait X :)

[scott1]

J'irais faire une petite recherche alors

[cristab]

voila pour toi  http://www.gtaonline.fr/forums/index.php/topic,6250.0.html

[scott1]

merci bien car "optimisation" ne donnais guère de résultats
Double post

LES CONDITIONS

Les conditions sont la clé d'un code non linéraire. Elles permettent d'orienter le script en fonction de ce qui se passe sur le serveur.
Les conditions s'utilisent avec les mots clés (ou instructions) suivants : if, while, do/while et for.
Les conditions peuvent très vite devenir très compliquées à formuler et il ne faut pas hésiter à les éclater en plusieurs conditions.
Par exemple, on veut tester pour chaque joueur qui est connecté si il est passager d'un véhicule ou si il est à pied.
On pourrait écrire :

Code: [Sélectionner]

for (i=0 ; i<MAX_PLAYERS ; i++)
{
   if (IsPlayerConnected (i) && (IsPlayerInAnyVehicle (i) && GetPlayerState (i) == PLAYER_PASSENGER) || GetPlayerState (i) == PLAYER_ON_FOOT))
   {
      /* On fait quelque chose */
   }
}On voit très vite dans ce bout de code qu'il est très facile de se tromper dans la condition (parenthèses, opérateurs).

Il vaut donc mieux écrire :

Code: [Sélectionner]

for (i=0 ; i<MAX_PLAYERS ; i++)
{
   if (IsPlayerConnected (i))
   {
      if (IsPlayerInAnyVehicle (i))
      {
         if (GetPlayerState (i) == PLAYER_PASSENGER)
         {
            /* On fait quelque chose */
         }
      }
      else if (GetPlayerState (i) == PLAYER_ON_FOOT)
      {
         /* On fait quelque chose */
      }
   }
}NB : Ici, on tombe dans un autre travers où l'on est obligé d'utiliser deux fois le même code (/* On fait quelque chose */) mais ce n'est qu'un exemple

Il ne parle pas de rapidité, juste d'erreur que l'on pourrai faire,

J'irais faire un tour du coté de Y_Less

[cristab]

on peu oui faire des erreure mais si tu met des printf partout tu verras que de temps a autre il y en a qui sont sauter

[scott1]

tendis qu'avec la methode de syg non?

[cristab]

beaucoup moin car a la premier condition le code s'arrete ou continue si il doit continuer