[SAMMELTHREAD] Kleine Scripting Fragen

Kaliber · 18. Februar 2019

Zitat von Allrounder18

Was muss ich machen, wenn ich bei /admins angezeigt bekommen möchte , das der Admin Duty ist
Also folgendermaßen Projektleitung Allrounder18 Im Dienst. So sieht es im Dialog aus.
Ich habe wenn ich aduty bin ein [ARP] vor dem Namen stehn, dadurch kennt mein System dass ich online bin. Wie kann ich das fixen?

Was genau ist deine Frage?

Was genau geht denn nicht?

Wie sieht denn GetPlayerStatus aus?

Du beschreibst nur, was du machst, aber nicht, was du willst bzw was nicht geht...und wie es aktuell aussieht...

Allrounder18 · 18. Februar 2019

Hallo, @Kaliber!

1. Was muss ich machen damit ich wenn ich aduty bin in /admins neben dem Namen "Im Dienst" stehn hab?

2. Ich habe wenn ich aduty bin [ARP] vor dem Namen stehn übr dem Kopf und in der Tabliste, daher steht dann immer in /admins das ich offline bin. Die Listekennt den Namen nicht wegen dem {ARP].

3. mein GetPlayerStatus

Code

stock GetPlayerStatus(playerid)
{
	new string[500];
    if (IsPlayerConnected(playerid) && SpielerDaten[playerid][player_Eingeloggt])
    {
    	if(SpielerDaten[playerid][player_AFK]) { format(string, 32, "{FF0048}AFK"); return string;} 
    	if(SpielerDaten[playerid][player_AdminDienst]) { format(string, 32, "{FFFF00}Im Dienst"); return string;}
        format(string, 32, "{40FF00}Online");	
        return string;
    }	
    else
    {
    	format(string,32,"{FF0000}Offline");
    	return string; 
    }	
}

Alles anzeigen

Edit: Weitere Frage: Kann man O.3.DL Skins in einem Textdraw anzeigen lassen?
Edit: Es wird in dem Textdraw immer der Skin angezeigt den ich mit dem 0.3.L Skin ersetzt habe. ZB. habe ich mir die Skinnummer 20008 erstellt und hab dafür die Skin ID 287 genommen.

reaL. · 18. Februar 2019

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 ?

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.