OCMD Befehl Probleme

Mister.Anonym · 2. Dezember 2020

Guten Tag, ich habe ein Problem bei einem OCMD Stats Befehl. Der Befehl funktioniert soweit auch, wenn ich mir z.B. eine Nachricht anzeigen lassen möchte. Die Perso abfrage funktioniert auch. Aber sobald ich den Perso habe und mir die Stats anzeigen lassen will, zeigt er an dieser Befehl existiert nicht. Es muss an dem stock liegen. Ich würde mich freuen, wenn ihr mir weiterhelfen könnt.

Code

ocmd:stats(playerid)
{
if(Spieler[playerid][Personalausweis] == 0)return SendClientMessage(playerid, COLOR_GRAD2, "Du besitzt keinen Personalausweis.");
statistiken(playerid, playerid);
return 1;
}

Code

stock statistiken(playerid, pID)
{
new String[1024], PremiumString[64], AdminString[64], ZusatzString[256], PersonalienString[256], BerufString[128], FinanzenString[128], AllgemeinString[256], InventarString[256], SkillsString[512], GeschlechtName[16], PartnerName[32], FraktionsName[32], BerufName[16], UnternehmensName[32], HausName[32], Float:FestgeldProzent, Jahr, Monat, Tag, Stunde, Minute, Sekunde;


//Geschlecht erkennen
if(Spieler[playerid][Geschlecht] == 1) {GeschlechtName = "Männlich";}
if(Spieler[playerid][Geschlecht] == 2) {GeschlechtName = "Weiblich";}


//Partner erkennen
if(Spieler[playerid][Partner] == 0) {PartnerName = "Niemand";}
if(Spieler[playerid][Partner] > 0)  {mysql_format(MySQL, query, sizeof(query), "SELECT * FROM `Accounts` WHERE `ID` = '%i' LIMIT 1", Spieler[playerid][Partner]); mysql_query(MySQL, query); cache_get_field_content(0, "Name", PartnerName, MySQL, 32);}


//Fraktion erkennen
format(FraktionsName, 32, "%s", Fraktion[Spieler[playerid][Frak]][Name]);


//Beruf erkennen
format(BerufName, 32, "%s", Job[Spieler[playerid][Beruf]][Name]);


//Immobilie erkennen
if(Spieler[playerid][Immobilie] == 0) {HausName = "Keine";}


if(Spieler[playerid][Immobilie] > 0)
{
for(new i=0; i<MAX_HAUS; i++)
{
if(Spieler[playerid][Immobilie] == Haus[i][ID])
{
format(HausName, 32, "%s", GetAreaName(Haus[i][EingangX], Haus[i][EingangY], Haus[i][EingangZ]));
break;
}
}
}


//Unternehmen erkennen
if(Spieler[playerid][Biz] == 0) {UnternehmensName = "Keins";}


if(Spieler[playerid][Biz] > 0)
{
for(new i=0; i<MAX_UNTERNEHMEN; i++)
{
if(Spieler[playerid][Biz] == Unternehmen[i][ID])
{
format(UnternehmensName, 32, "%s", Unternehmen[i][Beschreibung]);
break;
}
}
}


//Zinssatz erkennen
if(Spieler[playerid][FestgeldID] == 0) {FestgeldProzent = 0;}


if(Spieler[playerid][FestgeldID] > 0)
{
for(new i=0; i<sizeof(Festgeld); i++)
{
if(Spieler[playerid][FestgeldID] == Festgeld[i][ID])
{
FestgeldProzent = Festgeld[i][Zinssatz];
break;
}
}
}


//Personalien Statistiken
format(PersonalienString, sizeof(PersonalienString), "Personalien:\tGeschlecht: %s\n\t\tPartner: %s\n\n\t\tLevel: %d\n\t\tRespekt: %d/%d\n\t\tPayday: %d/60", GeschlechtName, PartnerName, Spieler[playerid][Level], Spieler[playerid][Exp], Spieler[playerid][MaxExp], Spieler[playerid][PayDay]);


//Beruf und Nebenberuf Statistiken
format(BerufString, sizeof(BerufString), "Berufe:\t\tFraktion: %s\tRang: %d\n\t\tNebenjob: %s", FraktionsName, Spieler[playerid][Rang], BerufName);


//Finanz Statistiken
format(FinanzenString, sizeof(FinanzenString), "Finanzen:\tBargeld: %d$\n\t\tKontostand: %d$\n\n\t\tGehalt: %d$\n\t\tFestgeld: %d$\tZinssatz: %.1f (Vertrag: %d)", Spieler[playerid][Bargeld], Spieler[playerid][Kontostand], Spieler[playerid][Gehalt], Spieler[playerid][FestgeldStand], FestgeldProzent, Spieler[playerid][FestgeldID]);


//Allgemein Statistiken
format(AllgemeinString, sizeof(AllgemeinString), "Statistik:\tMorde: %d\n\t\tGestorben: %d\n\t\tVerbrechen: %d\n\t\tSpielzeit: %dh\n\t\tKnast: %d\n\t\tVerwarnungen: %d/3", Spieler[playerid][Morde], Spieler[playerid][Tode], Spieler[playerid][Verbrechen], Spieler[playerid][Spielzeit], Spieler[playerid][Knastbesuche], Spieler[playerid][Verwarnungen]);


//Inventar Statistiken
format(InventarString, sizeof(InventarString), "Inventar:\tMaterialien: %d (Safe: %d)\n\t\tMeth: %d (Safe: %dg)\n\t\tGrass: %d (Safe: %dg)\n\t\tKokain: %d (Safe: %dg)\n\t\tHandy: (Nr. %d)\n\t\tImmobilie: %s\n\t\tUnternehmen: %s",
Spieler[playerid][Mats], Spieler[playerid][SafeMats], Spieler[playerid][Meth], Spieler[playerid][SafeMeth], Spieler[playerid][Grass], Spieler[playerid][SafeGrass], Spieler[playerid][Kokain], Spieler[playerid][SafeKokain], Spieler[playerid][Handynummer], HausName, UnternehmensName);


//Skills Statistiken
format(SkillsString, sizeof(SkillsString), "Skills:\t\tGrassdealer: %d\t\tExp: %d/%d\n\t\tKokaindealer: %d\t\tExp: %d/%d\n\t\tWaffendealer: %d\t\tExp: %d/%d\n\t\tBusfahrer: %d\t\tExp: %d/%d\n\t\tZugfahrer: %d\t\tExp: %d/%d\n\t\tHochseefischer: %d\tExp: %d/%d\n\t\tMüllmann: %d\t\tExp: %d/%d\n\t\tHolzfäller: %d\t\tExp: %d/%d\n\t\tDetektiv: %d\t\tExp: %d/%d\n\t\tRechtsanwalt: %d\t\tExp: %d/%d\n\t\tTrucker: %d\t\tExp: %d/%d\n\t\tPilot: %d\t\t\tExp: %d/%d",
Spieler[playerid][GrassdealerLevel], Spieler[playerid][GrassdealerExp], Spieler[playerid][GrassdealerMaxExp], Spieler[playerid][KokaindealerLevel], Spieler[playerid][KokaindealerExp], Spieler[playerid][KokaindealerMaxExp], Spieler[playerid][WaffendealerLevel], Spieler[playerid][WaffendealerExp], Spieler[playerid][WaffendealerMaxExp],Spieler[playerid][BusLevel], Spieler[playerid][BusExp], Spieler[playerid][BusMaxExp],
Spieler[playerid][ZugfahrerLevel], Spieler[playerid][ZugfahrerExp], Spieler[playerid][ZugfahrerMaxExp], Spieler[playerid][HochseefischerLevel], Spieler[playerid][HochseefischerExp], Spieler[playerid][HochseefischerMaxExp], Spieler[playerid][MuellmannLevel], Spieler[playerid][MuellmannExp], Spieler[playerid][MuellmannMaxExp], Spieler[playerid][HolzLevel], Spieler[playerid][HolzExp], Spieler[playerid][HolzMaxExp],
Spieler[playerid][DetektivLevel], Spieler[playerid][DetektivExp], Spieler[playerid][DetektivMaxExp], Spieler[playerid][AnwaltLevel], Spieler[playerid][AnwaltExp], Spieler[playerid][AnwaltMaxExp], Spieler[playerid][TruckerLevel], Spieler[playerid][TruckerExp], Spieler[playerid][TruckerMaxExp], Spieler[playerid][PilotLevel], Spieler[playerid][PilotExp], Spieler[playerid][PilotMaxExp]);


//Zusatzinhalte
if(Spieler[playerid][Admin] > 0)
{
format(AdminString, sizeof(AdminString), "{FF4000}%s (%d){FFFFFF}", AdminName(Spieler[playerid][Admin]), Spieler[playerid][Admin]);
}
if(Spieler[playerid][Premium] == 1)
{
format(PremiumString, sizeof(PremiumString), "{FF4000}Permanentes Premium{FFFFFF}");
}
if(Spieler[playerid][TempPremium] > gettime() && Spieler[playerid][Premium] == 0)
{
format(PremiumString, sizeof(PremiumString), "{FF4000}Premium (noch %s){FFFFFF}", ZeitRechnung(Spieler[playerid][TempPremium]));
}
TimestampToDate(Spieler[playerid][Registrierung], Jahr, Monat, Tag, Stunde, Minute, Sekunde, 1, 0);


if(Spieler[playerid][Admin] > 0)
{
if(Spieler[playerid][Premium] == 1 || Spieler[playerid][TempPremium] > gettime() && Spieler[playerid][Premium] == 0)
{
format(ZusatzString, sizeof(ZusatzString), "Account:\t%s\n\t\t%s\n\t\tRegistrierung: %02d.%02d.%02d - %02d:%02d:%02d", AdminString, PremiumString, Tag, Monat, Jahr, Stunde, Minute, Sekunde);
}
else
{
format(ZusatzString, sizeof(ZusatzString), "Account:\t%s\n\t\tRegistrierung: %02d.%02d.%02d - %02d:%02d:%02d", AdminString, Tag, Monat, Jahr, Stunde, Minute, Sekunde);
}
}
else
{
if(Spieler[playerid][Premium] == 1 || Spieler[playerid][TempPremium] > gettime() && Spieler[playerid][Premium] == 0)
{
format(ZusatzString, sizeof(ZusatzString), "Account:\t%s\n\t\tRegistrierung: %02d.%02d.%02d - %02d:%02d:%02d", PremiumString, Tag, Monat, Jahr, Stunde, Minute, Sekunde);
}
else
{
format(ZusatzString, sizeof(ZusatzString), "Account:\tRegistrierung: %02d.%02d.%02d - %02d:%02d:%02d", Tag, Monat, Jahr, Stunde, Minute, Sekunde);
}
}


//Statistiken zusammenfassung
format(String, sizeof(String), "%s\n\n%s\n\n%s\n\n%s\n\n%s\n\n%s\n\n%s", PersonalienString, BerufString, FinanzenString, AllgemeinString, InventarString, SkillsString, ZusatzString);
ShowPlayerDialog(pID, STATISTIKEN, DIALOG_STYLE_MSGBOX, Spieler[playerid][Name], String, "Schließen", "");
return 1;
}

Alles anzeigen

Mogly1 · 2. Dezember 2020

Bau dir ein paar print's mit ein, und schau wo es endet. bei so viel Code mit Spieler Infos. wird es schwer da durch zu sehen

palwal · 2. Dezember 2020

Die Function braucht keine 2 Parameter da beide sowieso den selben Inhalt haben! Behebe es.

Wenn es danach immernoch nicht funktioniert, so kommentiere zur Debugzwecken alles aus, löse Part für Part von den Kommentaren und schaue ab wann es funktioniert..würde ich dir als Debugmethode empfehlen.

Xegrima · 2. Dezember 2020

versuche es mal so

Code

ocmd:stats(playerid,params[])
{
    if(Spieler[playerid][Personalausweis] == 0)return SendClientMessage(playerid, COLOR_GRAD2, "Du besitzt keinen Personalausweis.");
    statistiken(playerid, playerid);
    return 1;
}

Code

stock statistiken(playerid, pID)
{
    new String[1024], PremiumString[64], AdminString[64], ZusatzString[256], PersonalienString[256], BerufString[128], FinanzenString[128], AllgemeinString[256], InventarString[256], SkillsString[512], GeschlechtName[16], PartnerName[32], FraktionsName[32], BerufName[16], UnternehmensName[32], HausName[32], Float:FestgeldProzent, Jahr, Monat, Tag, Stunde, Minute, Sekunde;
    //Geschlecht erkennen
    if(Spieler[playerid][Geschlecht] == 1) {GeschlechtName = "Männlich";}
    if(Spieler[playerid][Geschlecht] == 2) {GeschlechtName = "Weiblich";}
    //Partner erkennen
    if(Spieler[playerid][Partner] == 0) {PartnerName = "Niemand";}
    if(Spieler[playerid][Partner] > 0)  {mysql_format(MySQL, query, sizeof(query), "SELECT * FROM `Accounts` WHERE `ID` = '%i' LIMIT 1", Spieler[playerid][Partner]); mysql_query(MySQL, query); cache_get_field_content(0, "Name", PartnerName, MySQL, 32);}
    //Fraktion erkennen
    format(FraktionsName, 32, "%s", Fraktion[Spieler[playerid][Frak]][Name]);
    //Beruf erkennen
    format(BerufName, 32, "%s", Job[Spieler[playerid][Beruf]][Name]);
    //Immobilie erkennen
    if(Spieler[playerid][Immobilie] == 0) {HausName = "Keine";}
    if(Spieler[playerid][Immobilie] > 0)
    {
        for(new i=0; i<MAX_HAUS; i++)
        {
            if(Spieler[playerid][Immobilie] == Haus[i][ID])
            {
                format(HausName, 32, "%s", GetAreaName(Haus[i][EingangX], Haus[i][EingangY], Haus[i][EingangZ]));
                break;
            }
        }
    }
    //Unternehmen erkennen
    if(Spieler[playerid][Biz] == 0) {UnternehmensName = "Keins";}
    if(Spieler[playerid][Biz] > 0)
    {
        for(new i=0; i<MAX_UNTERNEHMEN; i++)
        {
            if(Spieler[playerid][Biz] == Unternehmen[i][ID])
            {
                format(UnternehmensName, 32, "%s", Unternehmen[i][Beschreibung]);
                break;
            }
        }
    }
    //Zinssatz erkennen
    if(Spieler[playerid][FestgeldID] == 0) {FestgeldProzent = 0;}
    if(Spieler[playerid][FestgeldID] > 0)
    {
        for(new i=0; i<sizeof(Festgeld); i++)
        {
            if(Spieler[playerid][FestgeldID] == Festgeld[i][ID])
            {
                FestgeldProzent = Festgeld[i][Zinssatz];
                break;
            }
        }
    }
    //Personalien Statistiken
    format(PersonalienString, sizeof(PersonalienString), "Personalien:\tGeschlecht: %s\n\t\tPartner: %s\n\n\t\tLevel: %d\n\t\tRespekt: %d/%d\n\t\tPayday: %d/60", GeschlechtName, PartnerName, Spieler[playerid][Level], Spieler[playerid][Exp], Spieler[playerid][MaxExp], Spieler[playerid][PayDay]);
    //Beruf und Nebenberuf Statistiken
    format(BerufString, sizeof(BerufString), "Berufe:\t\tFraktion: %s\tRang: %d\n\t\tNebenjob: %s", FraktionsName, Spieler[playerid][Rang], BerufName);
    //Finanz Statistiken
    format(FinanzenString, sizeof(FinanzenString), "Finanzen:\tBargeld: %d$\n\t\tKontostand: %d$\n\n\t\tGehalt: %d$\n\t\tFestgeld: %d$\tZinssatz: %.1f (Vertrag: %d)", Spieler[playerid][Bargeld], Spieler[playerid][Kontostand], Spieler[playerid][Gehalt], Spieler[playerid][FestgeldStand], FestgeldProzent, Spieler[playerid][FestgeldID]);
    //Allgemein Statistiken
    format(AllgemeinString, sizeof(AllgemeinString), "Statistik:\tMorde: %d\n\t\tGestorben: %d\n\t\tVerbrechen: %d\n\t\tSpielzeit: %dh\n\t\tKnast: %d\n\t\tVerwarnungen: %d/3", Spieler[playerid][Morde], Spieler[playerid][Tode], Spieler[playerid][Verbrechen], Spieler[playerid][Spielzeit], Spieler[playerid][Knastbesuche], Spieler[playerid][Verwarnungen]);
    //Inventar Statistiken
    format(InventarString, sizeof(InventarString), "Inventar:\tMaterialien: %d (Safe: %d)\n\t\tMeth: %d (Safe: %dg)\n\t\tGrass: %d (Safe: %dg)\n\t\tKokain: %d (Safe: %dg)\n\t\tHandy: (Nr. %d)\n\t\tImmobilie: %s\n\t\tUnternehmen: %s",
    Spieler[playerid][Mats], Spieler[playerid][SafeMats], Spieler[playerid][Meth], Spieler[playerid][SafeMeth], Spieler[playerid][Grass], Spieler[playerid][SafeGrass], Spieler[playerid][Kokain], Spieler[playerid][SafeKokain], Spieler[playerid][Handynummer], HausName, UnternehmensName);
    //Skills Statistiken
    format(SkillsString, sizeof(SkillsString), "Skills:\t\tGrassdealer: %d\t\tExp: %d/%d\n\t\tKokaindealer: %d\t\tExp: %d/%d\n\t\tWaffendealer: %d\t\tExp: %d/%d\n\t\tBusfahrer: %d\t\tExp: %d/%d\n\t\tZugfahrer: %d\t\tExp: %d/%d\n\t\tHochseefischer: %d\tExp: %d/%d\n\t\tMüllmann: %d\t\tExp: %d/%d\n\t\tHolzfäller: %d\t\tExp: %d/%d\n\t\tDetektiv: %d\t\tExp: %d/%d\n\t\tRechtsanwalt: %d\t\tExp: %d/%d\n\t\tTrucker: %d\t\tExp: %d/%d\n\t\tPilot: %d\t\t\tExp: %d/%d",
    Spieler[playerid][GrassdealerLevel], Spieler[playerid][GrassdealerExp], Spieler[playerid][GrassdealerMaxExp], Spieler[playerid][KokaindealerLevel], Spieler[playerid][KokaindealerExp], Spieler[playerid][KokaindealerMaxExp], Spieler[playerid][WaffendealerLevel], Spieler[playerid][WaffendealerExp], Spieler[playerid][WaffendealerMaxExp],Spieler[playerid][BusLevel], Spieler[playerid][BusExp], Spieler[playerid][BusMaxExp],
    Spieler[playerid][ZugfahrerLevel], Spieler[playerid][ZugfahrerExp], Spieler[playerid][ZugfahrerMaxExp], Spieler[playerid][HochseefischerLevel], Spieler[playerid][HochseefischerExp], Spieler[playerid][HochseefischerMaxExp], Spieler[playerid][MuellmannLevel], Spieler[playerid][MuellmannExp], Spieler[playerid][MuellmannMaxExp], Spieler[playerid][HolzLevel], Spieler[playerid][HolzExp], Spieler[playerid][HolzMaxExp],
    Spieler[playerid][DetektivLevel], Spieler[playerid][DetektivExp], Spieler[playerid][DetektivMaxExp], Spieler[playerid][AnwaltLevel], Spieler[playerid][AnwaltExp], Spieler[playerid][AnwaltMaxExp], Spieler[playerid][TruckerLevel], Spieler[playerid][TruckerExp], Spieler[playerid][TruckerMaxExp], Spieler[playerid][PilotLevel], Spieler[playerid][PilotExp], Spieler[playerid][PilotMaxExp]);
    //Zusatzinhalte
    if(Spieler[playerid][Admin] > 0)
    {
        format(AdminString, sizeof(AdminString), "{FF4000}%s (%d){FFFFFF}", AdminName(Spieler[playerid][Admin]), Spieler[playerid][Admin]);
    }
    if(Spieler[playerid][Premium] == 1)
    {
        format(PremiumString, sizeof(PremiumString), "{FF4000}Permanentes Premium{FFFFFF}");
    }
    if(Spieler[playerid][TempPremium] > gettime() && Spieler[playerid][Premium] == 0)
    {
        format(PremiumString, sizeof(PremiumString), "{FF4000}Premium (noch %s){FFFFFF}", ZeitRechnung(Spieler[playerid][TempPremium]));
    }
    TimestampToDate(Spieler[playerid][Registrierung], Jahr, Monat, Tag, Stunde, Minute, Sekunde, 1, 0);
    if(Spieler[playerid][Admin] > 0)
    {
        if(Spieler[playerid][Premium] == 1 || Spieler[playerid][TempPremium] > gettime() && Spieler[playerid][Premium] == 0)
        {
            format(ZusatzString, sizeof(ZusatzString), "Account:\t%s\n\t\t%s\n\t\tRegistrierung: %02d.%02d.%02d - %02d:%02d:%02d", AdminString, PremiumString, Tag, Monat, Jahr, Stunde, Minute, Sekunde);
        }
        else
        {
            format(ZusatzString, sizeof(ZusatzString), "Account:\t%s\n\t\tRegistrierung: %02d.%02d.%02d - %02d:%02d:%02d", AdminString, Tag, Monat, Jahr, Stunde, Minute, Sekunde);
        }
    }
    else
    {
        if(Spieler[playerid][Premium] == 1 || Spieler[playerid][TempPremium] > gettime() && Spieler[playerid][Premium] == 0)
        {
            format(ZusatzString, sizeof(ZusatzString), "Account:\t%s\n\t\tRegistrierung: %02d.%02d.%02d - %02d:%02d:%02d", PremiumString, Tag, Monat, Jahr, Stunde, Minute, Sekunde);
        }
        else
        {
            format(ZusatzString, sizeof(ZusatzString), "Account:\tRegistrierung: %02d.%02d.%02d - %02d:%02d:%02d", Tag, Monat, Jahr, Stunde, Minute, Sekunde);
        }
    }
    //Statistiken zusammenfassung
    format(String, sizeof(String), "%s\n\n%s\n\n%s\n\n%s\n\n%s\n\n%s\n\n%s", PersonalienString, BerufString, FinanzenString, AllgemeinString, InventarString, SkillsString, ZusatzString);
    ShowPlayerDialog(pID, STATISTIKEN, DIALOG_STYLE_MSGBOX, Spieler[playerid][Name], String, "Schließen", "");
    return 1;
}

Alles anzeigen

palwal · 2. Dezember 2020

statistiken(playerid, playerid); was fällt auf

**Andosius** · 2. Dezember 2020

Zitat von palwal

statistiken(playerid, playerid); was fällt auf

Das dient dazu um auch Dritten die Stats zeigen zu können.

Macht ja keinen Sinn zwei Funktionen für playerid und pID zu bauen.

varrez · 2. Dezember 2020

~~Beim Befehl:~~

~~statistiken(playerid, playerid); zu statistiken(playerid);~~

~~Falls an dritten gezeigt werden soll, pid des dritten übergeben und nicht die eigene playerid~~

~~und:~~

~~stock statistiken(playerid, pID) zu stock statistiken(playerid)~~

**Andosius** · 2. Dezember 2020

Zitat von varrez

Beim Befehl:

statistiken(playerid, playerid); zu statistiken(playerid);

Falls an dritten gezeigt werden soll, pid des dritten übergeben und nicht die eigene playerid

und:

stock statistiken(playerid, pID) zu stock statistiken(playerid)

Alles anzeigen

Das macht überhaupt keinen Sinn, da du sonst gar nicht weißt wessen Stats angezeigt werden sollen.

Ihr redet komplett am Thema vorbei, das hat hiermit nicht zu tun...

varrez · 2. Dezember 2020

Zitat von Andosius

Das macht überhaupt keinen Sinn, da du sonst gar nicht weißt wessen Stats angezeigt werden sollen.

Ihr redet komplett am Thema vorbei, das hat hiermit nicht zu tun...

Du hast Recht, denkfehler von mir. Sorry

OCMD Befehl Probleme

breadfish.de 12. März 2022

Ähnliche Themen

[jTuT] MySQL R41-4 (Installation, XAMPP, Verwendung, Bedienung & Registrations-System)

ADuty Befehl

Veh und RepVeh Befehl