get Funktion gibt nicht den richtigen wert über

Hallo,

ich würde gerne mit getFraktion(id) den Namen der Fraktion angezeigt bekommen. Allerdings zeigt er mir nichts davon Richtig an. Hat einer eine Idee woran es liegt ?

Code:

enum FraktionEnum{
    Float:frak_x,
    Float:frak_y,
    Float:frak_z,
    Float:ifrak_x,
    Float:ifrak_y,
    Float:ifrak_z,
    frakKasse,
    frak_interior,
    frak_id,
    frak_pickup,
    frak_pickup2,
    frak_name[128],
    Text3D:f_text,
    DoorOpen
}
new FrakInfo[100][FraktionEnum];


getFraktion(id)
{
    return FrakInfo[id][frak_name];
}

Laden der Namen

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

Das komische für mich ist, sobald ich die Fraktion manuell Abfrage ohne get Funktion also wie folgt

print(FrakInfo[id][frak_name])

zeigt er den richtigen Wert an wenn ich die ID entsprechend ersetze

Tatsächlich hatte ich einen kleinen Denkfehler. Nun mehr läuft dass ganze wie folgt ab,

new string[128];
format(string, sizeof(string), "Du hast %s zum Leader der Fraktion %s gemacht.", getPlayerName(playerid), getFraktion(playerid));
SendClientMessage(playerid, COLOR_RED, string);

Folgendes Ergebnis erhalte ich hiermit

 Du hast testb zum Leader der Fraktion Stestb gemacht.

Üblicherweise sollte dort wo Stestb steht die Stadtverwaltung als Wert hinterlegt sein. Warum kommt mein Benutzername auf einmal dazwischen ?

Meine Funktion zu getPlayerFraktion(playerid) sieht wie folgt aus:

getFraktion(playerid)
{
    if(PlayerInfo[playerid][pLoggedIn])
    {
         new FraktionID = PlayerInfo[playerid][pFraktion];
         return FrakInfo[FraktionID][frak_name];
    }
    return 1;
}

Zitat von Kaliber

Mach das if und das return 1 weg

Bringt leider auch nicht den gewünschten Erfolg. Hab ich eventuell einen Fehler im Enum selbst ?

Ich muss in jedenfall mit

strmid(FrakInfo[id][frak_name], tmp_name, 0, sizeof(tmp_name), sizeof(tmp_name));

arbeiten. Andernfalls ist der Wert dauerhaft null und überhaupt nicht mehr aufrufbar. Das ganze geht ja an sich mit unnötigen umwegen aber warum nicht mit einer funktion ?

Zitat von RaPTiiC

BreadScript da ich mein Fraktionssystem auch in einem Enum habe, nutze ich folgende Funktion um meinen Fraktionsnamen auslesen zu lassen:

[pFraktion] ist in meinem "SpielerEnum"

Code

CMD:checkme(playerid, params[])
{
    new string[128];
    SCME(playerid, -1,"Du bist in der Fraktion: %s",FrakInfo[PlayerInfo[playerid][pFraktion]][f_name]);
    return 1;
}

Weiß nicht ob du die Funktion SCME drin hast ansonsten wäre es bei dir ungefähr wie folgt:

Code

CMD:checkme(playerid, params[])
{
    new string[128];
    format(string,sizeof(string),"Du bist in der Fraktion: %s",FrakInfo[PlayerInfo[playerid][pFraktion]][f_name]);
    SCME(playerid, -1,string);
    return 1;
}

Alles anzeigen

In diesem Fall habe ich auch überhaupt kein Problem damit, allerdings muss es ja eine Möglichkeit geben dass ganze zu vereinfachen.