Mach ihr euren Script ordentlich ?

  • Hab dir in der Zeile kommentiert, was falsch ist :D


    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;
    }

  • Variante nummero 1


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

    Sie brauchen Hilfe bei Linux oder Windows Servern? Wissen nicht, wie man am besten ein Forum o. a. installiert? Dann schreiben Sie mir eine PN und ich versuche Ihnen zu helfen!

  • Ich bevorzug zwar die 1. Variante, machs mir aber bei der Variablen Bezeichnung aus irgendeinem Grund ohne es zu wollen immer extremst schwer ô.0


    Wobei ich zugeb ab und zu auch mal bei kleineren Abfragen usw. die 3. Variante zu benutzen. ^^

  • 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?