Beiträge von Kaliber

  • Grand Theft Auto eSports Selfmade

    Wenn wir schon mal dabei sind habe ich direkt noch ein paar Tipps :D


    Um ein bisschen Ressourcen zu sparen kann man Variablen anstatt so:


    new SkinAuswahl[MAX_PLAYERS];
    new Adminlogin[MAX_PLAYERS];
    new Float:Gotox[MAX_PLAYERS];
    new Float:Gotoy[MAX_PLAYERS];
    new Float:Gotoz[MAX_PLAYERS];


    besser so deklarieren:


    new SkinAuswahl[MAX_PLAYERS],
    Adminlogin[MAX_PLAYERS],
    Float:Gotox[MAX_PLAYERS],
    Float:Gotoy[MAX_PLAYERS],
    Float:Gotoz[MAX_PLAYERS];


    Leere Callbacks am besten einfach rausschmeißen...


    Sachen wie sowas:


    if(Adminlogin[playerid] == 0)
    {
    SendClientMessage(playerid,Rot,"Du bist nicht als Administrator Eingeloggt!");
    return 1;
    }


    Kann man auf sowas kürzen:


    if(Adminlogin[playerid] == 0) return SendClientMessage(playerid,Rot,"Du bist nicht als Administrator Eingeloggt!");


    Aber ansonsten sieht das Script eigentlich ganz gut aus, 8.5/10 ! :)


    mfg. :thumbup:


    //Edit: Rechtschreibung

  • Grand Theft Auto eSports Selfmade

    Hätte da ein paar Verbesserungsvorschläge:


    Anstatt sowas:


    dcmd_cleartext(playerid,params[])
    {
    #pragma unused params
    new String[128];
    if(PlayerInfo[playerid][pAdministrator] >= 2)
    {
    if(Adminlogin[playerid] == 0)
    {
    SendClientMessage(playerid,Rot,"Du bist nicht als Administrator Eingeloggt!");
    return 1;
    }
    SendClientMessageToAll(Weiß," ");
    SendClientMessageToAll(Weiß," ");
    SendClientMessageToAll(Weiß," ");
    SendClientMessageToAll(Weiß," ");
    SendClientMessageToAll(Weiß," ");
    SendClientMessageToAll(Weiß," ");
    SendClientMessageToAll(Weiß," ");
    SendClientMessageToAll(Weiß," ");
    SendClientMessageToAll(Weiß," ");
    SendClientMessageToAll(Weiß," ");
    SendClientMessageToAll(Weiß," ");
    SendClientMessageToAll(Weiß," ");
    SendClientMessageToAll(Weiß," ");
    SendClientMessageToAll(Weiß," ");
    SendClientMessageToAll(Weiß," ");
    SendClientMessageToAll(Weiß," ");
    SendClientMessageToAll(Weiß," ");
    SendClientMessageToAll(Weiß," ");
    SendClientMessageToAll(Weiß," ");
    SendClientMessageToAll(Weiß," ");
    SendClientMessageToAll(Weiß," ");
    SendClientMessageToAll(Weiß," ");
    SendClientMessageToAll(Weiß," ");
    SendClientMessageToAll(Weiß," ");
    format(String,sizeof(String),"((Administrator: %s hat den Chatlog gecleart))",SpielerName(playerid));
    SendClientMessageToAll(Rot,String);
    return 1;
    }
    else
    {
    SendClientMessage(playerid,Rot,"Du bist kein Administrator!");
    }
    return 1;
    }


    einfach sowas:


    dcmd_cleartext(playerid,params[])
    {
    #pragma unused params
    new String[128];
    if(PlayerInfo[playerid][pAdministrator] >= 2)
    {
    if(Adminlogin[playerid] == 0)
    {
    SendClientMessage(playerid,Rot,"Du bist nicht als Administrator Eingeloggt!");
    return 1;
    }
    for(new i; i<24; i++) SendClientMessageToAll(Weiß, " ");
    format(String,sizeof(String),"((Administrator: %s hat den Chatlog gecleart))",SpielerName(playerid));
    SendClientMessageToAll(Rot,String);
    return 1;
    }
    else
    {
    SendClientMessage(playerid,Rot,"Du bist kein Administrator!");
    }
    return 1;
    }


    und Quini, die mysql_query Systeme von Maddin sind nicht die besten, da er jede Variable einzeln in die Datenbank einträgt...das geht mit einer längeren query viel besser ;)


    mfg. :thumbup:

  • Scripting [Günstige Preise]

    Zitat von no.Ob

    Sei mir nicht böse Quini, wenn man keine Zeit sowie das nötige Wissen dazu hat sich einen Server einzurichten und sich ein Script zurecht zu schreiben, sollte man es bleiben lassen. Es bringt doch nichts wenn man von der Materie nichts versteht, aber alles haben will.


    MfG noobs Meinung ...


    Okay, das ist eine Meinung, aber nehmen wir mal an, dass der Scripter noch ein Anfänger ist und dann kann Quini ihm unter die Arme greifen, indem er ein Grundgerüst für sie/ihn erstellt ;)


    In diesem Sinne, viel Erfolg Quini :)


    mfg. :thumbup:

  • Wie erstelle ich einen /accept medic?

    Zitat von Acrio

    Ok. Habe mir schon sowas gedacht :). Ich muss doch nur ne Ich glaube Variable hinzufügen das es funktioniert oder? Und wegen dem Code:


    SetPlayerCheckpoint(playerid, zX, zY, zZ, 5);
    Das ist die Errorzeile.


    Also der Code da ist absolut richtig :D


    Nein, im Prinzip brauchst du keine weitere Variable aber einen Array *hust*[MAX_PLAYERS]*hust* ;)


    mfg. :thumbup:

  • Wie erstelle ich einen /accept medic?

    Zitat von Acrio

    Aber mir fällt gerade auf. Was wenn 2 oder 3 Leute gleichzeitig /service medic machen? Dann wird die Variable doch überschrieben und nur der letze kann Hilfe bkommen oder sehe ich das falsch?


    Jap, das ist aber absicht, damit du nicht nur C&P machst :)


    Musst nur paar Sachen abändern und den Code den du mir geschickt hast, damit kann ich nichts anfangen, tut mir leid :whistling:


    mfg. :thumbup: