Probleme "ispassenger"

[scott1]

un truc du genre s'est orrible alors?

Code: [Sélectionner]

if(giveplayerid == playerid) { Message(playerid,4000,"~r~Vous ne pouvez pas utilisez cette commande sur vous meme."); return 1; }
        if(!IsPlayerConnected(giveplayerid) || JoueurInfos[giveplayerid][jSpawn] != 1) { Message(playerid,4000,"~r~Ce joueur n'est pas connecte ou n'a pas encore spawn."); return 1; }
            if(!IsPlayerInAnyVehicle(playerid)){ Message(playerid,4000,"~r~Vous n'etes pas dans un vehicule de police."); return 1; }
if(!GetDistanceBetweenPlayers(playerid,giveplayerid,5.0)) { Message(playerid,4000,"~r~Ce joueur n'est pas pres de vous."); return 1; }
            new carid = GetPlayerVehicleID(playerid);
if(VehiculeInfos[carid][vType] != 8) { Message(playerid,4000,"~r~Vous n'etes pas dans un vehicule de police."); return 1; }
if(GetPlayerState(playerid) != PLAYER_STATE_DRIVER) { Message(playerid,4000,"~r~Vous n'etes pas conducteur."); return 1; }

[cristab]

sa exist un code aussi sale :o

[scott1]

Sérieusement?

Et on l'optimiserais comment?

[Xartrick]

Vraiment mal indenté, je vois également des return partout alors qu'on pourrait en utiliser qu'un seul grâce à else if.
Voilà un code bien indenté et légèrement optimisé !

Code: (Pawn) [Sélectionner]

new carid = GetPlayerVehicleID(playerid);

if(giveplayerid == playerid)
{
Message(playerid, 4000, "~r~Vous ne pouvez pas utilisez cette commande sur vous meme.");
}
else if(!IsPlayerConnected(giveplayerid) || JoueurInfos[giveplayerid][jSpawn] != 1)
{
Message(playerid, 4000, "~r~Ce joueur n'est pas connecte ou n'a pas encore spawn.");
}
else if(!IsPlayerInAnyVehicle(playerid))
{
Message(playerid, 4000, "~r~Vous n'etes pas dans un vehicule de police.");
}
else if(!GetDistanceBetweenPlayers(playerid, giveplayerid, 5.0))
{
Message(playerid, 4000, "~r~Ce joueur n'est pas pres de vous.");
}
else if(VehiculeInfos[carid][vType] != 8)
{
Message(playerid, 4000, "~r~Vous n'etes pas dans un vehicule de police.");
}
else if(GetPlayerState(playerid) != PLAYER_STATE_DRIVER)
{
Message(playerid, 4000, "~r~Vous n'etes pas conducteur.");
}
return 1;

[scott1]

Quelqu'un pourrait il m'expliquer l'intendation?

Ce que s'est et en quoi ça conciste ^^

[Brian Turner]

Sa consiste a rendre le code plus "beau" plus compréhensible, apres je sais pas si c'est pareil pour le logiciel que pour nous(plus comprehensible).

[cristab]

Oo j'avais repondut mon post ni est plus pour le compilot c'est plus lisible

mais juste un truc pourquoi ne pas regarder si le playerid remplit les condition avant le giveplayerid c'est pas logique le code

[Xartrick]

L'indentation permet une meilleur lisibilité du code présenté.
Cela donne plus envie à une personne de lire et de comprendre ton code sans avoir à le déchiffrer !


Indentation:
Très souvent utilisée en programmation, elle rend le code source plus clair et plus lisible.
- Wikipedia.org

EDIT: Voici trois code identique mais qui sont indenté différemment.
Tu pourra me dire quelle est pour toi la meilleur chose à faire: indenter ou ne pas indenter.

Code indenté

Code: (Pawn) [Sélectionner]

if(CPS_IsPlayerInCheckpoint(playerid, cpammu2)) {
AMMUBOX[playerid] = 0;

if(gTeam[playerid] != COP && gTeam[playerid] != Medic && gTeam[playerid] != SWAT && gTeam[playerid] != FBI && gTeam[playerid] != ARMY) {
if(Ammu2RecentlyRobbed == 0) {
new chances = random(500);

if(chances >= 0 && chances <= 400) {
new string[250];

RobAmmu2[playerid] = 20;
Ammu2RecentlyRobbed = 150;

format(string, sizeof(string), "%s[ID:%d] started a robbery at Ammunation!", name, playerid);
Announce(string);

return true;
}
else if(chances >= 401 && chances <= 500) {
new copmsg[250];

SendClientMessage(playerid, ERROR, "Your attempt to rob Ammunation failed!");
SendClientMessage(playerid, ERROR, "The Police are on their way to the store.");

SetPlayerWantedLevel(playerid, GetPlayerWantedLevel(playerid) + 5);

format(copmsg, sizeof(copmsg), "[HQ] %s attemped to rob Ammunation.", name);
SendRadioMessageToCops(copmsg);

Ammu2RecentlyRobbed = 100;

return true;
}
}
else if(Ammu2RecentlyRobbed > 0) return SendClientMessage(playerid, ERROR, "Ammunation has been robbed recently!");
}
else SendClientMessage(playerid, ERROR, "Your team cannot rob anything.");
}
Code non-indenté

Code: [Sélectionner]

if(CPS_IsPlayerInCheckpoint(playerid, cpammu2))
{
AMMUBOX[playerid] = 0;
if(gTeam[playerid] != COP && gTeam[playerid] != Medic && gTeam[playerid] != SWAT && gTeam[playerid] != FBI && gTeam[playerid] != ARMY)
{
if(Ammu2RecentlyRobbed == 0)
{
new chances = random(500);
if(chances >= 0 && chances <= 400)
{
new string[250];
RobAmmu2[playerid] = 20;
Ammu2RecentlyRobbed = 150;
format(string, sizeof(string), "%s[ID:%d] started a robbery at Ammunation!", name, playerid);
Announce(string);
return true;
}
else if(chances >= 401 && chances <= 500)
{
new copmsg[250];
SendClientMessage(playerid, ERROR, "Your attempt to rob Ammunation failed!");
SendClientMessage(playerid, ERROR, "The Police are on their way to the store.");
SetPlayerWantedLevel(playerid,GetPlayerWantedLevel(playerid)+ 5);
format(copmsg, sizeof(copmsg), "[HQ] %s attemped to rob Ammunation.", name);
SendRadioMessageToCops(copmsg);
Ammu2RecentlyRobbed = 100;
return true;
}
}
else if(Ammu2RecentlyRobbed > 0) return SendClientMessage(playerid, ERROR, "Ammunation has been robbed recently!");
}
else SendClientMessage(playerid, ERROR, "Your team cannot rob anything.");
}
Code non-indenté et mal rangé

Code: [Sélectionner]

if(CPS_IsPlayerInCheckpoint(playerid, cpammu2)){AMMUBOX[playerid] = 0;
if(gTeam[playerid] != COP && gTeam[playerid] != Medic && gTeam[playerid] != SWAT && gTeam[playerid] != FBI && gTeam[playerid] != ARMY){if(Ammu2RecentlyRobbed == 0)
{new chances = random(500);if(chances >= 0 && chances <= 400){new string[250];RobAmmu2[playerid] = 20;Ammu2RecentlyRobbed = 150;
format(string, sizeof(string), "%s[ID:%d] started a robbery at Ammunation!", name, playerid);Announce(string);
return true;}else if(chances >= 401 && chances <= 500){new copmsg[250];SendClientMessage(playerid, ERROR, "Your attempt to rob Ammunation failed!");
SendClientMessage(playerid, ERROR, "The Police are on their way to the store.");SetPlayerWantedLevel(playerid,GetPlayerWantedLevel(playerid)+ 5);
format(copmsg, sizeof(copmsg), "[HQ] %s attemped to rob Ammunation.", name);SendRadioMessageToCops(copmsg);Ammu2RecentlyRobbed = 100;
return true;}}else if(Ammu2RecentlyRobbed > 0) return SendClientMessage(playerid, ERROR, "Ammunation has been robbed recently!");
}else SendClientMessage(playerid, ERROR, "Your team cannot rob anything.");}
Tu pourra remarquer que le dernier code est très petit (en ligne) mais est complètement illisible !
PS: Code tiré aléatoirement du gamemode RuNix's Cops and Robbers codé par Raul_Ro du forum officiel de SA:MP.

[scott1]

Donc pour le serveur, le dernier est le pire?

[Xartrick]

Après être compiler, un code est un code.
Le compilateur lui, peut avoir du mal à diriger ton code mais ton serveur, une fois qu'il à un code compilé (sans erreurs), il l'exécute mais l'indentation ne l'affectera pas (à moins que je me trompe).

[scott1]

Après être compiler, un code est un code.
Le compilateur lui, peut avoir du mal à diriger ton code mais ton serveur, une fois qu'il à un code compilé (sans erreurs), il l'exécute mais l'indentation ne l'affectera pas (à moins que je me trompe).

Donc en fait je choisis la methode que je prefere.

ça n'affecte pas les performance? Ooooh grand maitre syg passe par ici ;)

[Xartrick]

Syg te conseillera d'indenter ton code, pas besoin de l'attendre ;) !

[Ssk]

En faite à la compilation leur compilateur enlève tout les "Blancs" donc que le code soit indenté ou pas ça n'affecte pas le compilateur ( sauf pour les warning d'indentation ), mais comme le dit Xartrick Syg te conseillerai d'indenter ton code.

[scott1]

L'indenter pour ma lecture, mais pas pour celle du serveur, donc pas de soucis au niveau des performance.


Je suis bisard mais cette écriture me plait donc si niveau performance aucun soucis, je la garde.

[S!m]

Salut,

effectivement il n'y a aucun soucis à l'exécution, ni à la compilation. Comme tous les autres l'ont dit, le fichier compilé n'est pas influencé par l'indentation (C'est un peu comme si tu laisse des lignes vides dans ton script pour séparer deux parties, il n'y aura pas de lignes vides dans le code compilé, de toute façon ce n'est pas du texte alors...)

Par contre, quand tu demande de l'aide, il est beaucoup plus difficile et moins tentant de t'aider avec un code non indenté et mal arrangé. Après, si ça te plait, c'est toi qui voit.

PS. comme c'est aussi plus évident de voir les niveaux de code etc... c'est plus évident de voir les erreurs d'accolade, de return mal placés....

++Sim++