Beiträge von Kaliber

    Danke an Kaliber der es aktualisiert hat :thumbup:


    Aha...weiß ich gar nicht mehr ^^


    Nunja, ich persönlich finde das Script ziemlich schlecht, da du sehr oft leere Abfragen verwendest und du solltest mehr switch und case verwenden !


    mfg. :S

    Schreibe es so:


    ocmd:rep(playerid,params[])
    {
    new pID;
    if(sscanf(params,"u",pID)) return SCM(playerid,BLAU,"Info: /rep [ID]");
    if(!IsPlayerConnected(pID)) return 1;
    new string[64];
    SetVehicleHealth(GetPlayerVehicleID(pID),1000);
    format(string,64,"Du hast %s Sein Auto Repairt!",SpielerName(pID));
    SCM(playerid,HELLROT,string);
    format(string,64,"%s hat Dein Auto Repairt!",SpielerName(playerid));
    SCM(pID,HELLROT,string);
    return 1;
    }


    mfg. :thumbup:


    Lese mal meinen Beitrag, sowas braucht man nicht !


    mfg. :rolleyes:

    Entweder du setzt die HP auf mehrere Milliarden (ist aber im prinzip nicht godmode) oder du setzt einen timer der alle paar Sekunden die HP wieder hochsetzt.


    Das ist beides kein Godmode und auch nicht zu empfehlen :D


    Schau dir das mal an: SetPlayerTeam


    Wichtig ist dieses Zitat:

    Zitat

    Note: Players on the same team can not kill or damage each other unless they use a knife.


    mfg. :thumbup:

    So, habe mir mal den Server angeschaut ^^


    Also, auf dem Server habe ich schon einige Bugs bekannt gegeben, also es sind wirklich sehr sehr viele Vorhanden und es leggt teilweise echt extrem und das TextDraw im Hintergrund ist einfach nur fehl am Platz !


    Somit, nicht zu empfehlen :S


    mfg. :thumbup:

    dann schreib was ich besser machen kann.


    • if(dialogid == ishop)
      {
      Verwende hier auch switch und case, da es schneller ist als normale if-Abfragen ;)
    • OnDialogResponse bitte returne das mit 0 ;)
    • WeedPlanted solltest du als boolean verwenden, da die Werte nur true oder false sind, das spart an Ressourcen ;)
    • ocmd:gotoschwarz(playerid,params[])
      {
      SetPlayerPos(playerid,-2133.1331,1213.7354,47.27343);
      return 1;
      }
      Da du bei diesem Befehl den Parameter params nicht benötigst, solltest du ihn mit #pragma unused verbrauchen.
    • new Samen[MAX_PLAYERS];
      new Duenger[MAX_PLAYERS];
      new Weeder[MAX_PLAYERS];
      new weedObject[MAX_WEED];
      Du solltest die Variablen noch etwas zusammenfassen, sprich 1ne Große für alle Spieler machen mit einem 2 dimensionalen Array und enum nutzen ;)


    Jo, das sind so ein paar Sachen die mir beim überfliegen aufgefallen sind, aber schon nicht schlecht, denke jedoch, dass sich das hier eher als Include anbieten würde, da man dann es im Gamemode selbst einbauen könnte :)


    mfg. :thumbup:

    Du musst in deinem Script AddPlayerClass oder SetSpawnInfo 1x verwenden ^^


    mfg. :thumbup:


    //Edit: Da hilft man ein paar Leuten mit einem dummen GF und dann schreibt man hier auch noch public Funktionsnamen davon auf X(