Anfänger Release [Selfmade]

Wichtiger Hinweis: Bitte ändert nicht manuell die Schriftfarbe auf schwarz sondern belasst es bei der Standardeinstellung. Somit tragt ihr dazu bei dass euer Text auch bei Verwendung unseren dunklen Forenstils noch lesbar ist!

Tipp: Ihr wollt längere Codeausschnitte oder Logfiles bereitstellen? Benutzt unseren eigenen PasteBin-Dienst Link
  • Es ist ein Anfänger Script und "nicht" Spiel bereit Vorhanden ist


    -Gps System
    -Drive In
    - LSPD
    -SFPD
    -FBI
    -Taxi
    -Medic
    -Müllmann
    -Postbote
    -Dedektiv
    -Jobs im Rh hohlbar aber nur die 3 Funktionstüchtig.
    -Überall Fraktion Autos
    -Bank System(Man kann es überall machen mit /atm
    -Eingänge
    Wetter System (verschiedenes Wetter in anderen Städten
    -Payday (jede volle Stunde
    Ping Kicker(ab 200 Kann man ändern.
    Afk/Back System
    Fahrzeuge an ihrem Spawn Punkt setzbar mit /carback
    Haussystem
    Tachosystem
    Tank System
    Motor System
    Autohaus(verbuggt)
    Gangfight(Nicht getestet nach dem Tut gemacht)
    Report System(Nicht getestet nach dem Tut gemacht)


    Vielleicht bringe ich wieder paar Updates raus aber bin mich nicht sicher


    Pastebin: http://pastebin.com/68RM42m2
    Download:http://www.mediafire.com/?4818izdc68dzwk3


    Viel Spaß,


    Ich stehe gerne für Support bereit per Pn oder Im Thread. Ich nehme gerne Verbesserungsvorschläge an. Kritik ist auch erwünscht.


    Viel Spaß damit.


    Mit freundlichen grüßen Tobi aka magnussoft


    Kritik ist erwünscht sowie Verbesserungsvorschläge.


    dankee..



    Wer die neue Version will meldet sich per Private Nachricht an mich

    7 Mal editiert, zuletzt von magnussoft ()

  • Was ist darin bitte Komisch damit man es Verbessern kann?!?

  • Hallo,
    also Selfmade ist das ja schonmal nicht!
    Das Register System und die 3 Fraktionen waren schon drinne!
    Mind. 1000 Zeilen sind aus dem MrMonat Script!
    3/10 P.

    Warum fällt immer die Wand um, wenn ein Mantafahrer dagegen fährt? - Der Klügere gibt nach!


  • :P Ich habe auch mit MrMonat angefangen, nur weil ich bei dir Kritik gesagt habe brauchst du mich hier nicht Runter machen.


    LoL Glückwunsch ja die 3 Fraktionen waren drinne und mit irgendwelchen Befehlen? Mit Irgendwelchen Autos. Nein, er hat sie "nur" erstellt was 1 Minute arbeit ist eine zu erstellen aber ohne Commands. Und ja es ist das Register System von Mr.Monat und? Jeder Fängt damit an. Ich kanns auch gerne Umschreiben und so tun als ob ich keine Hilfe bräuchte etc. Nein. Ich lasse es so. Ich nehm gerne deine Kritik an. Aber das ist ja mal Gelaber.....

  • Homer, das kann man ruhig netter sagen ;)
    Außerdem hat er mehr Zeilen als du!
    Das ist doch mal was :)


    Bekommst von mir mal 7/10, Gründe:
    - Ich sehe kein /invite oder /uninvite ^^ Das kommt aber bestimmt in der nächsten Version (hoffe ich)
    - Payday hättest du aber gleich Individuell machen können, sprich für jeden User einzeln, wenn dieser 60 min gespielt hat ;)
    - Zeile 254: Was hat das zu verloren ohne { und }? for (new i=0; i< MAX_VEHICLES; i++)
    - Zeile 263 - 268: Das geht Resourccensparender:

    for (new i=0; i< MAX_VEHICLES; i++)
    {
    GetVehicleParamsEx(i,engine,lights,alarm,doors,bonnet,boot,objective);
    SetVehicleParamsEx(i,VEHICLE_PARAMS_OFF,lights,alarm,doors,bonnet,boot,objective);
    Motor[i] = false;
    }
    Einfach nurfor (new i=0; i< MAX_VEHICLES; i++) { Motor[i] = false; }

    Unkompetent. Das neue dynamisch.



    ihr seit schon lustig postet erros aber nicht die zeilen wo sie sind dann können wir euch auch nicht helfen^^


    Lernt Scripten, Leute, und macht eure Augen auf!


    _______________________________________
    50 Beiträge: [x] 300 Beiträge: [x]
    100 Beiträge: [x] 500 Beiträge: [x]
    150 Beiträge: [x] 1000 Beiträge: [x]
    200 Beiträge: [x]
    2000 Beiträge: []
    250 Beiträge: [X] Boardaktivitäten: Profil


    [size=36]_______________________________________

    /Edit: Schmerzt es dir eigentlich nicht so ein Müll zu schreiben 8|

  • Ich finds nicht so toll :/
    - Viele Schreibfehler ("Du hast dir ein Essen gekauft")
    - nicht sehr Ressourcenschonend (new tmp[200]; unter OnPlayerCommandText)
    - unübersichtlich (ocmd und strcmp?)
    Anstatt immer "new spieler[MAX_PLAYERNAME]; GetPlayerName(...)" könntest du auch einfach eine Variable new PlayerName[MAX_PLAYERS][MAX_PLAYERNAME] erstelle, die du dann bei OnPlayerConnect formatierst.

  • oder einfach ein stock ;)



    stock PName(playerid)
    {
    new plainame[MAX_PLAYER_NAME];
    GetPlayerName(playerid,plainame,sizeof(plainame));
    return plainame;
    }


    Die Rechschreibfehler sind da extra, kann ja sein, dass er PlayerName schon hat oder so :)

    Unkompetent. Das neue dynamisch.



    ihr seit schon lustig postet erros aber nicht die zeilen wo sie sind dann können wir euch auch nicht helfen^^


    Lernt Scripten, Leute, und macht eure Augen auf!


    _______________________________________
    50 Beiträge: [x] 300 Beiträge: [x]
    100 Beiträge: [x] 500 Beiträge: [x]
    150 Beiträge: [x] 1000 Beiträge: [x]
    200 Beiträge: [x]
    2000 Beiträge: []
    250 Beiträge: [X] Boardaktivitäten: Profil


    [size=36]_______________________________________

    /Edit: Schmerzt es dir eigentlich nicht so ein Müll zu schreiben 8|

  • Zitat

    oder einfach ein stock


    und was soll das für einen Sinn haben? Die GetPlayerName-Funktion wird dann trotzdem jedes mal aufgerufen. Es reicht doch wenn die Funktion nur einmal beim Connecten aufgerufen wird.

  • Danke euch beiden :)  Haxler für dich sollte das Script auch net sein eher für Anfänger :) aber ich danke dir für die Tipps Achte ich bei der nächsten Version :) Rechtschreibfehler werde ich verbessern :)

  • do.de - Domain-Offensive - Domains für alle und zu super Preisen
  • Genau da liegt das Problem. Dann lernen die Anfänger wie man ressourcen-fressend scriptet.



    //global:
    new PlayerName[MAX_PLAYERS][MAX_PLAYERNAME];
    // OnPlayerConnect:
    GetPlayerName(playerid,PlayerName[playerid],MAX_PLAYERNAME);
    jetzt kannst du das so verwenden:
    new s[50];
    format(s,50,"Du heißt: %s",PlayerName[playerid]);

  • Als Anfängerscript ist es doch ziemlich gut geeignet!

  • Kritik ist erwünscht sowie Verbesserungsvorschläge.


    mfg tobi.

    Einmal editiert, zuletzt von magnussoft ()

  • haha :)


    Schau mal wie viel Zeilen MrMonat hat und meins


    JA ICH habe mit MrMonat angefangen jeder fängt so an? Und ist es schlimm. ?


    Danke für die Kritik.


    P.S Ich will sehn wie du nur paar Fraktionen machst.

  • Dixon ich liebe dich.


    Ab und zu ne :) du weißt wie ich es meine :D


    P.S Ich schreib dir ma ne Pn

  • Danke Jay_S für deinen doch so sinnvollen und hilfreichen Beitrag...


    Ansich find ich das script nicht schlecht, aber man kann es auch kürzer machen
    z.b.:
    stock FBICar(vehicleid)
    {
    for(new i = 0; i < sizeof(FBICars); i++)
    if(vehicleid== FBICars[i]) return 1;
    return 0;
    }
    funktioniert bei mir tadellos


    und warum hast du strget und sscanf drin?
    entscheid dich für eins...


    //Edit: falls wieder einer mekert
    stock FBICar(vehicleid)
    {
    for(new i = 0; i < sizeof(FBICars); i++) { if(vehicleid== FBICars[i]) return 1; }
    return 0;
    }
    alles das gleiche :thumbup:

    "Bevor ich mir Informationen aus der "Bild" hole,
    werde ich anfangen, Wahlergebnisse danach vorauszusagen,
    neben welchen Busch unsere Katze gepinkelt hat."

    Margarete Stokowski

  • Ah okay, dank dir


    mal ein guter Beitrag. Nur so kann ich noch mehr lernen.


    Mit freundlichen grüßen Tobi.

  • do.de - Domain-Offensive - Domains für alle und zu super Preisen