[SAMMELTHREAD] Kleine Scripting Fragen

itsMagic. · 18. Februar 2019

Ja, entweder du löst das Problem, indem du es machst, wenn er das Fahrzeug Geparkt hat, das dass Fahrzeug sich Löscht und Neu Spawnt, oder du nutzt "SetVehiclePos".

Bei einem Respawn, setzt er das Fahrzeug auf die Kordi die unter "CreateVehicle" erstellt werden.

MfG.

Zitat von reaL.

Guten Tag, mir ist Heute wieder aufgefallen wenn ich z.b ein Privat Fahrzeug habe und ich es Umparke und alle Fahrzeuge Respawne dann steht es wieder dort wo es vorher stand außer ich Lösche es und er stelle es neu beim Parken.
Kann man dies auch irgendwie umgehen ?

Mr.Pawn · 19. Februar 2019

Hat jemand eine Funktion mit welcher man von einem Text z.B. nur die ersten 15 Buchstaben anzeigen lassen kann?

LeonMrBonnie · 19. Februar 2019

Code

new string[32] = "Dies ist ein Test!";
strmid(string, string, 0, 15);
printf(string);

Output:
Dies ist ein Te

[wiki]strmid[/wiki]

reaL. · 20. Februar 2019

Nabend,
Gibt es ein Include wo ich Abfragen kann ob es ein Motorrad/flugzeug oder sowas ist ?
Habe dazu nichts gefunden

itsMagic. · 20. Februar 2019

Für Boote z.B.

Das gleiche kannst dir ja dann noch einmal für ein Motorrad Basteln, dann hast eine Funktion ohne Include.

new Boat[] = { 472, 473, 493, 495, 484, 430, 453, 452, 446, 454, 595};
stock IsVehicleBoat(vehicleid)
{
for(new i = 0; i < sizeof(Boat); i++)
{
if(GetVehicleModel(vehicleid) == Boat[i]) return 1;
}
return 0;
}

Kaliber · 20. Februar 2019

Zitat von CodePLUS

Das gleiche kannst dir ja dann noch einmal für ein Motorrad Basteln, dann hast eine Funktion ohne Include.

So kann man das zwar lösen, aber eigentlich ist die Funktion nicht so gut.

Das liegt daran, dass:

Immer eine Schleife durchlaufen werden muss, je mehr Fahrzeuge, desto länger dauert es.
Du jedes mal eine Model Abfrage machst in der Schleife.

Besser wäre es da mit switch zu arbeiten, was als Jump-Table fungiert.

Dort wird dann direkt hingesprungen oder nicht.

Immer O(1), anstatt O(n) und du hast nur eine Model Abfrage (wobei es schon sinnvoller wäre, das Modell im eigentlichen Code in einer Variable zu speichern und dann nur zu übergeben).

Code sähe dann so aus:

C

stock IsVehicleBoat(vehicleid)
{
    switch(GetVehicleModel(vehicleid))
    {
        case 472, 473, 493, 495, 484, 430, 453, 452, 446, 454, 595:  return 1;
    }
    return 0;
}

Eventuell noch 2 Dinge als Info:

1. Es ist nicht schlimm Includes zu inkludieren, wenn sie richtig geschrieben sind. Sprich überall stock oder defines verwendet wurden. Falls man diese dann nämlich nicht nutzt, werden alle Dinge die nicht benötigt werden vom Compiler ignoriert und es gelangt nix unerwünschtes in die .amx Datei.

2.

Zitat von CodePLUS

new Boat[] = { 472, 473, 493, 495, 484, 430, 453, 452, 446, 454, 595};

So denkt der Compiler, dass es sich hierbei um eine Variable handelt.
Sprich, dass eventuell später (während der Server läuft) noch Dinge geändert werden, was ja aber nicht der Fall ist.
Zeig ihm das, indem man es als const deklariert, dadurch kann der Compiler das besser optimieren.

stock const Boat[] = { 472, 473, 493, 495, 484, 430, 453, 452, 446, 454, 595};

@Allrounder18
Schreib es mal nur so:

C

public AdminList(playerid)
{
    new string[700] = "{FFFFFF}Rang\t{FFFFFF}Name\t{FFFFFF}Onlinestatus",adminrang,name[MAX_PLAYER_NAME],pID,rows;
    cache_get_row_count(rows);
    for(new i; i<rows; i++)
    {
        cache_get_value_name_int(i,"Admin",adminrang);
        if(adminrang < 3) continue;
        cache_get_value_name(i,"Name",name,sizeof(name));
        pID = GetPlayerIdFromName(name);
        format(string, sizeof(string), "%s\n%s\t%s\t%s",string, GetAdminName(adminrang), name, GetPlayerStatus(pID));
    }
    ShowPlayerDialog(playerid, DIALOG_ADMINS, DIALOG_STYLE_TABLIST_HEADERS, "{FF0000}Austrian Roleplay {FFFFFF}- Team",string,"{FFFFFF}Abbrechen","");
    return 1;
}

Alles anzeigen

In der Funktion GetPlayerStatus ändere mal die String-Größe von 500 auf 32.

Selbst 32 wäre noch zu groß, 16 würde da vollkommen ausreichen, musst die nicht immer so riesig machen xD

Und nutze da nicht GetPlayerIdFromName, sondern meine Funktion:

C

stock GetPlayerFromAdminName(const pName[])
{
    new playerid,n[MAX_PLAYER_NAME] = "[ARP]";
    strcat(n,pName),sscanf(n, "u", playerid);
    return playerid;
}

Allrounder18 · 20. Februar 2019

Es funktioniert leider mit deiner Funktion auch nicht. @Kaliber

Kaliber · 20. Februar 2019

Zitat von Allrounder18

Es funktioniert leider mit deiner Funktion auch nicht. @Kaliber

Kannst du mal den Command zeigen, mit dem du Aduty gehst?

Allrounder18 · 20. Februar 2019

Code

ocmd:aduty(playerid)
{
    new nachricht[128],string[128],SERVERTAG[500];
    format(SERVERTAG,sizeof(SERVERTAG),"%s",ServerInfo[0][server_Tag]);
    if(!IsAdmin(playerid,2))return SendNoPerms(playerid);
    if(!SpielerDaten[playerid][player_AdminDienst])
    {
        format(string,50,"[%s]%s",SERVERTAG,SpielerDaten[playerid][player_Name]);
        SetPlayerName(playerid,string);
        format(nachricht,128,"<< %s hat sich als %s angemeldet! >>",SpielerDaten[playerid][player_Name],GetAdminRang(playerid));
        SendClientMessageToAll(WEISS , nachricht);
        //SetPlayerScore(playerid,GetPlayerLevel(playerid));
        SendInfoMessage(playerid,"Du bist nun im Admindienst");
        SpielerDaten[playerid][player_AdminDienst] = true;
        //TextDrawShowForPlayer(playerid,Reportdraw[0]);
        //extDrawShowForPlayer(playerid,Reportdraw[1]);
        switch(SpielerDaten[playerid][player_Admin])
        {
            case 2: aDuty[playerid] = Create3DTextLabel("Probe Supporter",0xFE2EF7FF,30.0,40.0,50.0,MAX_STREAM_NAME_DISTANCE,-1,1), SetPlayerColor(playerid, ROT);
            case 3: aDuty[playerid] = Create3DTextLabel("Supporter",0xFE2EF7FF,30.0,40.0,50.0,MAX_STREAM_NAME_DISTANCE,-1,1), SetPlayerColor(playerid, ROT);
            case 4: aDuty[playerid] = Create3DTextLabel("Administrator",0x585858FF,30.0,40.0,50.0,MAX_STREAM_NAME_DISTANCE,-1,1), SetPlayerColor(playerid, ROT);
            case 5: aDuty[playerid] = Create3DTextLabel("Developer",0xDF01A5FF,30.0,40.0,50.0,MAX_STREAM_NAME_DISTANCE,-1,1), SetPlayerColor(playerid, ROT);
            case 6: aDuty[playerid] = Create3DTextLabel("Manager",0x00a4b3FF,30.0,40.0,50.0,MAX_STREAM_NAME_DISTANCE,-1,1), SetPlayerColor(playerid, ROT);
            case 7: aDuty[playerid] = Create3DTextLabel("Projektleiter",0xFF0000FF,30.0,40.0,50.0,MAX_STREAM_NAME_DISTANCE,-1,1), SetPlayerColor(playerid, ROT);
        }
        Attach3DTextLabelToPlayer(aDuty[playerid],playerid,0.0,0.0,0.6);
    }
    else if(SpielerDaten[playerid][player_AdminDienst])
    {
        format(string,24,"%s",SpielerDaten[playerid][player_Name]);
        SetPlayerName(playerid,string);
        SetPlayerColor(playerid, WEISS);
        format(nachricht,128,"<< %s hat sich als %s abgemeldet! >>",SpielerDaten[playerid][player_Name], GetAdminRang(playerid));
        SendClientMessageToAll(WEISS, nachricht);
        SendInfoMessage(playerid,"Du bist nun nicht mehr im Admindienst");
        Delete3DTextLabel(aDuty[playerid]);
        //TextDrawHideForPlayer(playerid,Reportdraw[0]);
        //TextDrawHideForPlayer(playerid,Reportdraw[1]);
        SpielerDaten[playerid][player_AdminDienst] = false;
    }
    return 1;
}

Alles anzeigen

Kaliber · 20. Februar 2019

Zitat von Kaliber

pID = GetPlayerIdFromName(name);

Und du hast sicher diese Zeile durch:

pID = GetPlayerFromAdminName(name);

ersetzt?

Versuchs mal mit der Funktion:

C

stock GetPlayerFromAdminName(const pName[])
{
    new playerid,n[MAX_PLAYER_NAME] = "["#SERVERTAG#"]";
    strcat(n,pName),sscanf(n, "u", playerid);
    return playerid;
}

Dr. Frauenarzt · 20. Februar 2019

Sorry das ich kurz mich einmische.

SERVERTAG[500]

warum ist dir Lange auf 500 wen es 5 sind [ARP].

Meiner Meinung nach auch ziehmlich Verschwendung.

Kaliber · 20. Februar 2019

Oh, gar nicht gesehen, dass das eine Variable ist

Dachte es wurde so gelöst:

#define SERVERTAG "ARP"

Dann vergiss meine "neue" Funktion und prüf einfach, ob ARP wirklich 1:1 gleich geschrieben ist.

Allrounder18 · 20. Februar 2019

Jetzt werden alle als Offline angezeigt obwohl man Online ist und bei dem ersten in der Liste wird der Rang nicht angezeigt

Kaliber · 20. Februar 2019

Zitat von Allrounder18

Jetzt werden alle als Offline angezeigt obwohl man Online ist und bei dem ersten in der Liste wird der Rang nicht angezeigt

Ahja, stimmt, machs mal so:

C

stock GetPlayerFromAdminName(const pName[])
{
    new playerid,n[MAX_PLAYER_NAME] = "[ARP]";
    strcat(n,pName),sscanf(n, "u", playerid);
    return (playerid==INVALID_PLAYER_ID) ? GetPlayerIdFromName(pName) : playerid;
}

Und es wird kein Rang angezeigt?

Das liegt dann daran, dass er einen ungültigen besitzt?!

Welchen Wert hat denn der Rang in der DB, prüf das doch mit deinem Rang-System gegen.

Allrounder18 · 20. Februar 2019

Danke, es geht. In der DB stand n falscher Wert.

Wie kann ich machen das ich 14 Ränge hab so geht es nicht fraktion_Rang[14][50]

PS: Die Frage wegen 0.3.DL Skins weiter oben hat sich von selbst erledigt.

Kaliber · 20. Februar 2019

Zitat von Allrounder18

Wie kann ich machen das ich 14 Ränge hab so geht es nicht fraktion_Rang[14][50]

Du musst deine if-Abfragen vermutlich anpassen und dann einfach nur die Rangnamen ergänzen, dann sollte das funktionieren

Wenn dich nicht genau weißt, was...musst du leider dein System nochmal posten mit Rängen etc.

Allrounder18 · 20. Februar 2019

Code

enum FraktionDatenEnum {
	fraktion_ID,
	fraktion_Name[255],
	fraktion_Tag[50],
	fraktion_Farbe[2],
	fraktion_Rang[14][50],
	fraktion_Gehalt[14]
}

Ich will die Rangnamen nicht i Script definieren.

Kaliber · 20. Februar 2019

Zitat von Allrounder18

Ich will die Rangnamen nicht i Script definieren.

Ja...und jetzt?

Wenn du sagst, etwas geht nicht, dann beschreibe genau, was nicht geht, wo es geladen wird bzw wie es generell funktionieren sollte.

Einfach mit einem enum, kann man überhaupt nichts anfangen.

Allrounder18 · 20. Februar 2019

Ich möchte im Enum nicht FrakRang1, FrakRang2, etc schreiben sondern alles in einem .

Kaliber · 20. Februar 2019

Das geht so nicht, da du sonst eine 4 Dimensionale Variable benötigen würdest und das kann dieser Pawn-Compiler nicht.

Es gibt ein Update, der kann das.

Alternativ, musst du das in eine andere Variable tun, sowas wie:

new FrakRang[MAX_FRAKS][MAX_RÄNGE][MAX_RANG_LEN];