Unfertiges Roleplay Script [MySQL / 0.3.7]

In 10 Minuten startet der nächtliche Backupvorgang! Es kann währenddessen (ca. 10 Minuten) zu Einschränkungen bei der Nutzung des Forums kommen
Weitere Infos findet ihr im Thema Backup des Forums
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
  • Sehr geehrte Community,


    hiermit veröffentliche ich mein unfertiges Roleplay Script.


    Zu dem Script folgende Plugins:

    • pBlueG MySQL R39-4
    • Streamer
    • sscanf 2

    Zeilenstatus: 5351


    [tabmenu]


    [tab='Allgemeine Funktionen', 'http://icons.iconarchive.com/icons/graphicloads/100-flat/48/home-icon.png']
    [subtab='Carsystem']
    Es beinhaltet ein dynamisches Carsystem. MySQL basierend!
    Jeder Spieler kann nur ein Fahrzeug besitzen.
    [subtab='Autohaus']
    Es beinhaltet derzeit nur ein Autohaus nähe des BSN.
    Dort kann man sich ein persönliches Auto kaufen.
    [subtab='Fraktionssystem']
    Das Fraktionssystem ist noch statisch im Script eingebaut.
    Du kannst dort auswählen wo er spawnen soll.
    Welche Farbe er im Tab-Menü haben soll.
    [subtab='Haus/Biz/Firmensystem']
    Das Script besitzt ein Haus-, Business- und ein Firmensystem.
    Dies ist alles dynamisch mit MySQL gemacht worden.
    Nur es ist noch nicht vollständig! (/setinterior für Häuser)
    [subtab='Jobsystem']
    Es hat auch ein Jobsystem dieses ist auch statisch im Script eingebaut.
    Es ist im Script möglich die Namen etc. zu ändern. Und anderes.
    [tab='Jobs', 'http://icons.iconarchive.com/icons/limpa/curriculum-vitae/48/employment-icon.png']
    [subtab='Taxifahrer']
    Du kannst als Taxifahrer andere Spieler befördern aber benötigst dafür einen Führerschein!
    (/fare)
    [subtab='Busfahrer']
    Der Standart Job Busfahrer, befördere Menschen über eine bestimmte Route!
    Verdiene dein Geld ohne dass Spieler befördert werden müssen!
    [subtab='Müllmann']
    Als müllmann fährst du verschiedene Häuser ab und holst den Müll ab, am Ende bringst du es zur Base zurück und kriegst deine Belohnung am Payday.
    [subtab='Mechatroniker']
    Als Mechatroniker reparierst du aller Arten von Kfz. Die Befehle sind noch nicht reif!
    [tab='Fraktionen', 'http://icons.iconarchive.com/icons/icons-land/vista-people/48/Groups-Military-Personnel-Light-icon.png']
    [subtab='S.A.P.D.']
    Du hilfst als linke Hand der Politik und Recht im Staat San Andreas die Bürger zu beschützen und das Recht aufzuerhalten.
    [subtab='Medical Department']
    Tatütata, Eine verletze Person oder doch eine Bewusstlose Person. Wir sind da um zu Helfen!
    [subtab='Russen Mafia']
    Wir nehme Schutzgelder erpressen Politiker und und und.
    [subtab='Grove Street']
    Unser Haupteinkommen kommt über Drogenverkauf.
    [subtab='C.I.A.']
    Wir wissen alles und habe volle Immunität alles mögliche zu veranstalten um die Sicherheit des Staates aufrecht zu erhalten!
    [/tabmenu]


    [tabmenu]


    [tab='Script', 'http://icons.iconarchive.com/icons/gakuseisean/ivista-2/48/Misc-Download-Database-icon.png']
    [subtab='Informationen']
    Du kannst mit diesem Script dass machen was du möchtest. Von mir aus auch die Credits verwalten. Nur dieses Script darf nicht verkauft werden.
    Oder Editiert verkauft werden!
    [subtab='Pastebin']
    https://hastebin.com/rahihujiye.cs
    [subtab='Download']
    https://mega.nz/#!EjR2wQZB!GTf…4hsEMf7t0hLNJQNVQM1ageIRc
    [/tabmenu]


    Viel Spaß damit!


    Mit freundlichen Grüßen,
    Genadj_Andrenov

  • Codetechnisch betrachtet kompletter Müll. Abgesehen von den 256er Strings, die teilweise (z.B /kick) auch noch zu früh erstellt werden, sind da auch noch Sachen wie unter /bankneu.


    Du randomst gefühlt 30 mal in einen String und strval'st denn dann?! WTF?!?! variable = random(9999)+1 (als Beispiel) hätte es doch auch getan.


    Und weshalb mal ocmd mal strcmp wenn #pragma unused params dann geh doch gleich aufs ganze...


    Alleine bei /fpark über eine for zu gehen statt GetPlayerVehicleID(playerid); zu nutzen ist schon extrem low.


    Die Befehle /enter & /exit trau ich mich ja gar nicht anzusprechen...


    Unter OnPlayerStateChange dann der Burner... 3x == PLAYER_STATE_DRIVER...



    Und komm mir jetzt nicht mit "Ist ja unfertig das Script ich hätte ja alles ausgebessert"...


    Dennoch, danke für den Release. Muss man ja sagen. Nimm es als konstruktive Kritik, auch wenn es ein wenig schmierig klingen mag kannst du daraus lernen.

  • Es währe vom vorteil auch includes usw dabei zu packen ;)
    Vom Code her hätte man echt besser machen können aber gut jeder fängt mal klein an :)



    edit


    zu @Talykos aussage



    if(newstate == PLAYER_STATE_DRIVER)
    {
    new vID = GetPlayerVehicleID(playerid);
    for(new i=0; i<sizeof(ahCars); i++)
    {
    if(ahCars[i][id_x]!=vID)continue;
    //Verkaufsprozess
    SetPVarInt(playerid,"buyCarID",i);
    new string[256];
    format(string,sizeof(string),"Möchten sie das Fahrzeug für %i$ kaufen?",ahCars[i][c_preis]);
    ShowPlayerDialog(playerid,DIALOG_AUTOHAUS,DIALOG_STYLE_MSGBOX,"Autoverkauf",string,"Ja","Nein");
    break;
    }
    for(new mids=0; mids<sizeof(jCar); mids++)
    {
    if(IsPlayerInVehicle(playerid, jCar[mids][jv_vid]))
    {
    if(pInfo[playerid][pJob] != jCar[mids][jv_id])
    {
    RemovePlayerFromVehicle(playerid);
    SendClientMessage(playerid, COLOR_GREY, "Du bist nicht in dem jeweiligen Job!");
    return 1;
    }
    }
    }
    PlayerTextDrawShow(playerid, Tacho1[playerid]);
    PlayerTextDrawShow(playerid, Tacho2[playerid]);
    PlayerTextDrawShow(playerid, Tacho3[playerid]);
    PlayerTextDrawShow(playerid, Tacho4[playerid]);
    PlayerTextDrawShow(playerid, motorLabel[playerid]);
    if(isEngineOn(GetPlayerVehicleID(playerid)))
    {
    PlayerTextDrawSetString(playerid, motorLabel[playerid], "Motor ~g~an");
    }
    else
    {
    PlayerTextDrawSetString(playerid, motorLabel[playerid], "Motor ~r~aus");
    }
    }

    einfach das mal so machen fertig

  • Ich glaube der return war schon richtig statt des breaks. So wird beim Verkaufsfahrzeug kein Tacho angezeigt.

  • Beitrag von Selene ()

    Dieser Beitrag wurde vom Autor gelöscht ().