Mach ihr euren Script ordentlich ?

Hab dir in der Zeile kommentiert, was falsch ist

public OnPlayerStateChange(playerid, newstate, oldstate)
{
new modelid = GetPlayerVehicleID(playerid);
if(GetPlayerState(playerid) == PLAYER_STATE_DRIVER)
{
if(modelid == 472 || modelid == 473 || modelid == 493 || modelid == 595 || modelid == 484 || modelid == 430 || modelid == 453 || modelid == 452 || modelid == 446 || modelid == 454 )
{
if(PlayerInfo[playerid][pBootschein] == 0)
{
SendClientMessage(playerid, ROT, " Du hast keine Bootschein");

}
}
else if(modelid == 592 || modelid == 577 || modelid == 511 || modelid == 512 || modelid == 593 || modelid == 520 || modelid == 553 || modelid == 476 || modelid == 519 || modelid == 460 || /* hier fehlte ein || */ modelid == 513 || modelid == 548 || modelid == 425 || modelid == 417 || modelid == 487 || modelid == 488 || modelid == 497 || modelid == 563 || modelid == 447 || modelid == 469 ) // 777
{
if(PlayerInfo[playerid][pFlugschein] == 0)
{
SendClientMessage(playerid, ROT, " Du hast keinen Flugschein !");
}
}
else
{
SendClientMessage(playerid, ROT, " Du hast keinen Führerschein, ");
SendClientMessage(playerid, ROT, " Solltest du erwischt werden, ");
SendClientMessage(playerid, ROT, " musst du mit Konsequenzen rechnen. !");
}
}
return 1;
}

Ich bevorzuge auch Variante Nr.1

Zitat von InternetInk aka Benjamin

Ich bevorzuge auch Variante Nr.1

*sign*

Ich mach immer die erste und kommentiere immer die abschnitte z.b bei /afk und back mach ich oben drüber

Zitat

//== Afk und Back System==//

Variante nummero 1

Einfach am übersichtlichsten.
Und soviel Zeit mehr brauch man auch nich wie bei Variante 3

Ich bevorzuge Nr. 2
Damit komm ich besser zurecht.

1. Version

ich bin in ordentlich

sind eh die meisten

ich machs so ähnlich wie bei Variante 1
bei Variante 3 würde ich iwann nicht mehr durchblicken

Ich nehm Variante 4.
if(strcmp(cmdtext,"/teleports",true)==0) {
if(pi[playerid][admin]>=1) {
SendClientMessage(playerid,gelb,"|________Teleports________|");
SendClientMessage(playerid,blau,"bar, aldi, 4dragons, lvf, area51, sfpd, spawn, sff, adminbase, lsf");
SendClientMessage(playerid,blau,"lspd, startower, RSAbase, SCbase, taxibase, gangpehbase, BKAbase"); }
else { SendClientMessage(playerid,rot,"Dein Admin-Level ist nicht hoch genug"); }
return 1;
}
PS: @ 8D.: Dir ist es also egal, ob du sitrbst, Hauptsache jemand arbeitet am Script weiter?