Fraktionsnamen ermitteln SA:MP

BreadScript · 1. August 2022

Hallo,

gerne würde ich den Fraktionsnamen einer Fraktion auslesen. Hierfür habe ich bereits folgende Funktion

Code

getFrakName(playerid)
{
    for(new i; i < sizeof(FrakInfo); i++)
    {
        if(PlayerInfo[playerid][pFraktion] == FrakInfo[i][frak_id])
        {
            return FrakInfo[i][frak_name];
        }
    }
    return 1;
}

Alles anzeigen

Allerdings zeigt er mir den Fraktionsnamen in diesem Fall garnicht bzw falsch aus.

Das Enum für die Fraktion

Code

enum FraktionEnum{
    Float:frak_x,
    Float:frak_y,
    Float:frak_z,
    Float:ifrak_x,
    Float:ifrak_y,
    Float:ifrak_z,
    Float:spawn_x,
    Float:spawn_y,
    Float:spawn_z,
    frakKasse,
    frak_interior,
    frak_id,
    frak_pickup,
    frak_pickup2,
    frak_name[256],
    Text3D:f_text,
    DoorOpen,
    h_pickup

}

Alles anzeigen

Und der Part wo der Fraktionsname geladen wird

Code

public OnFraktionLoad()
{

    new count;
    cache_get_row_count(count);
    for(new i=0; i<count; i++)
    {
        new tmp_name[128];
        cache_get_value_name(i, "fraktion", tmp_name);
        strmid(FrakInfo[id][frak_name], tmp_name, 0, sizeof(tmp_name), sizeof(tmp_name));
    }
    return 1;
}

Alles anzeigen

Hat jemand ggf eine Idee ?

Kaliber · 1. August 2022

Zitat von BreadScript

new tmp_name[128];
cache_get_value_name(i, "fraktion", tmp_name);
strmid(FrakInfo[id][frak_name], tmp_name, 0, sizeof(tmp_name), sizeof(tmp_name));

Kannst das auch direkt laden:

cache_get_value_name(i, "fraktion", FrakInfo[id][frak_name], 64);

Im Enum ändere das zu frak_name[64], denn so lange werden Fraktionsnamen ja nicht...256 Zeichen, das wäre ja Wahnsinn

C

getFrakName(playerid)
{
    new tmp[64] = "Zivilist";
    if(PlayerInfo[playerid][pFraktion] <= 0) return tmp;

    for(new i; i < sizeof(FrakInfo); i++)
    {
        if(PlayerInfo[playerid][pFraktion] == FrakInfo[i][frak_id])
        {
            return FrakInfo[i][frak_name];
        }
    }
    
    return tmp;
}

Alles anzeigen

BreadScript · 1. August 2022

Danke für deine Nachricht, bekomme nun allerdings folgenden Fehler angezeigt:

Code

inconsistent return types (array & non-array)

Kann aber nicht so genau verstehen was er genau von mir möchte da ich den Fehler noch nie gehabt habe.

ID 0 wird auch gespeichert daher ist die Abfrage ob <= 0 aus meiner Sicht sinnlos.

Allerdings zeigt er mir eben auch keinen Text an wenn ich die Abfrage starte.

Kaliber · 1. August 2022

Zitat von BreadScript

ID 0 wird auch gespeichert daher ist die Abfrage ob <= 0 aus meiner Sicht sinnlos.

Ah okay, das wusste ich nicht, dann mach es so:

C

getFrakName(playerid)
{
    new frak[64];
    for(new i; i < sizeof(FrakInfo); i++)
    {
        if(PlayerInfo[playerid][pFraktion] == FrakInfo[i][frak_id])
        {
            strcat(frak,FrakInfo[i][frak_name]);
            return frak;
        }
    }
    
    strcat(frak,"Unbekannt");
    return frak;
}

Alles anzeigen

//Edit:

Ah, mach es so.

Pawn macht da dumme Sachen mit dem Pointer

//Edit2:

Nutze strcat, bei der direkten Zuweisung, kann es zu Fehlern kommen...

[DT]midomen · 31. Oktober 2022

Nur mal aus Interesse auch wenn das veraltet ist.

Warum habt ihr nicht einfach den hier gemacht?

new frakid = PlayerInfo[playerid][pFraktion];

return FrakInfo[frakid][frak_name];

wäre deutlich weniger aufwandig und wenn die IDs konsistenz deklariert sind sollten da auch keine Probleme auftreten.

Vielleicht hab ich auch etwas in Pawn notwendiges übersehen, lange nicht mehr mit Pawn und den eigenartigen Syntaxregeln gearbeitet.

Kaliber · 1. November 2022

Zitat von [DT]midomen

wenn die IDs konsistenz deklariert sind

Da liegt der Hund begraben

Wenn dem so ist, klar, dann ist das kein Problem xD

[DT]midomen · 1. November 2022

Zitat von Kaliber

Da liegt der Hund begraben

Wenn dem so ist, klar, dann ist das kein Problem xD

Alles klar, ich hatte es nämlich nicht so umständig in meinen alten Systemen in Erinnerung, da blieb nur diese Frage offen

Whice 4. Oktober 2022