Beiträge von Kaliber

Irgendwie bringst du da den Index für VehicleInfo ständig durcheinander.

Mal nutzt du FrakFahrzeugInfo[vehicleid][frakfahrzeug_VehicleID] und mal nur die vehicleid...

Was willst du denn jetzt nutzen?

Kein wunder dass da nichts funktioniert

Und der String ist auch am Anfang leer, du kannst kein leeres 3DTextLabel erstellen :c

ocmd:fk1(playerid,params[])
{
    if(GetPlayerState(playerid) != PLAYER_STATE_DRIVER)return SendErrorMessage(playerid,"Du bist nicht der Fahrer eines Fahrzeuges");
    new v = GetPlayerVehicleID(playerid),frakVeh = FrakFahrzeugInfo[v][frakfahrzeug_VehicleID];
    if(IsNull(params))
    {
        //Hier hat er nur /fk eingegeben...
        if(VehicleInfo[frakVeh][KennungObj] != Text3D:0) Delete3DTextLabel(VehicleInfo[frakVeh][KennungObj]); //Falls es bereits existiert, check ob du es auf 0 oder INVALID gesetzt hast...das solltest du anpassen
        else SendInfoMessage(playerid,"/fk [Kennung]"); //Wenn es nicht existiert, gibts auch nichts zu zerstören und die Nachricht kommt
        return 1;
    }
    new string[200],Float:pos[3];


    format(VehicleInfo[v][KennungZahl],50, params);


    VehicleInfo[frakVeh][KennungObj] = Create3DTextLabel(params, WEISS, 0, 0, 0, 10,-1,0);


    GetVehicleModelInfo(GetVehicleModel(frakVeh), VEHICLE_MODEL_INFO_SIZE, pos[0], pos[1], pos[2]);
    Attach3DTextLabelToVehicle(VehicleInfo[frakVeh][KennungObj],frakveh, -pos[0]/2.23, -pos[1]/2.23, 0.1);


    format(string,sizeof(string),"%s-%s",VehicleInfo[v][Kennung],VehicleInfo[v][KennungZahl]);
    Update3DTextLabelText(VehicleInfo[frakVeh][KennungObj], WEISS, string);
    return 1;
}

Soo könntest du es z.B. machen..

Aber ich weiß nicht genau, was genau du machen willst bzw welche Indizes richtig sind, da ich deinen Code nicht kenne

Auch in keinem Include/Filterskript?

Zitat von Allrounder18

Die TextLabels werden im Script nirgends destroyed sondern nur wenn der Spieler sich ausloggt aus irgend einem Grund.

Das klingt irgendwie Paradox

Dann such doch mal nach [wiki]Delete3DTextLabel[/wiki], eventuell zerstörst du ja das nicht gezielt, aber ganz viele auf einmal...

Zitat von Allrounder18

wird die Kennung destroyed

Ja...wie vermutet.

Dann musst du diese wieder beim connecten erstellen

Oder hoffst du da irgendwie auf Black-Magic?

Zitat von Allrounder18

So wird es attached

Das hilft mir nicht.

Wichtig wäre, wann es verwendet wird...und ob es noch andere Funktionen gibt, die das Label beeinflussen.

Zitat von Allrounder18

jedoch wird dieses nach einem rejoin nicht mehr angezeigt

Dann machst du irgendwas falsch.

Entweder du attachst es nochmal mit einer falschen id für den spieler oder du löschst es beim connect/disconnect.

Tatsache ist, dass es serverseitig gesteuert wird, somit hat reconnecten keinen Einfluss auf das Label

Schau zum Beispiel mal hier: https://www.php-einfach.de/mysql-tutorial/daten-einfuegen/

Aktuel erstellst du nur eine Variable mit dem Query (was du nicht so tuen solltest, sondern nutze dafür die prepare Funktion) und anschließend die execute Funktion

Erst dann, wird dein Query auch abgesendet.

Zitat von Allrounder18

So sind die Tausenderpunte definiert:

Schreibs mal so:

stock FormatNumber(geld, const szChar[] = ".")
{
    new str[24],i;
    valstr(str,geld),i=(geld > 0) ? strlen(str)-3 : strlen(str)-4;
    for(; i>0; i-=3) strins(str,szChar,(geld > 0) ? i : i+1,sizeof(str));
    return str;
}

Abgesehen davon, dass dein Code nur so vor Redundanz und Unübersichtlichkeit strotzt, liegt der Fehler vermutlich daran, dass du einfach 2x gespawnt wirst.

Nach OnPlayerRequestClass wird er vermutlich gespawnt, das musst du abfangen, damit das nicht 2x aufgerufen wird.

Ja, lösch die erste Abfrage einfach mit dem strval, die ist unnötig

Zitat von Dr. Frauenarzt

!strval(inputtext)

Das prüft, ob die Zahl 0 ist...

Was wolltest du da abfragen?

Ehh kannst du über https://jsfiddle.net/ deinen Code teilen, wie das so aussieht?

Dann wäre es um einiges leichter dir zu helfen, da wir uns das dann besser vorstellen können und direkt deinen Code updaten können

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];

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.

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.

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:

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.

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.

Zitat von Kaliber

pID = GetPlayerIdFromName(name);

Und du hast sicher diese Zeile durch:

pID = GetPlayerFromAdminName(name);

ersetzt?

Versuchs mal mit der Funktion:

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

Zitat von Allrounder18

Es funktioniert leider mit deiner Funktion auch nicht. @Kaliber

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

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:

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:

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;
}

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:

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