illegal Aufruf von nicht-statischer Member-Funktion

Question

Ich bin unten mit dieser Funktion Problemen mit:

char* GetPlayerNameEx(int playerid) 
{ 

    char Name[MAX_PLAYER_NAME], i = 0; 

    GetPlayerName(playerid, Name, sizeof(Name)); 

    std::string pName (Name); 

    while(i == 0 || i != pName.npos) 
    { 
     if(i != 0) i++; 
     int Underscore = pName.find("_", i); 
     Name[Underscore] = ' '; 
    } 
    return Name; 
} 

Erklärung:

char* GetPlayerNameEx(int playerid); 

Nutzung:

sprintf(string, "%s", CPlayer::GetPlayerNameEx(playerid)); 

Nun mein Problem hier ist

Persönliche Informationen entfernt n.

Wenn dies etwas zu tun hat, was ich bezweifle, ist diese Funktion in einem "Class" Header (Deklaration) enthalten.

Auch ich habe keine Ahnung warum, aber ich kann nicht die "Code" Box richtig überpassen.

Answer 1

Sie können diese Funktionen als statisch nicht schaffen (ohne viel Zwicken), weil Sie die Daten einer bestimmten Instanz zu ändern versuchen. Um dies zu beheben Ihr Problem:

class CPlayer 
{ 
public: 
    // public members 

    // since you are operating on class member data, you cannot declare these as static 
    // if you wanted to declare them as static, you would need some way of getting an actual instance of CPlayer 
    char* GetPlayerNameEx(int playerId); 
    char* GetPlayerName(int playerId, char* name, int size); 
private: 
    // note: using a std::string would be better 
    char m_Name[MAX_PLAYER_NAME]; 
}; 

// note: returning a string would be better here 
char* CPlayer::GetPlayerNameEx(int playerId) 
{ 
    char* Name = new char[MAX_PLAYER_NAME]; 
    memset(Name, MAX_PLAYER_NAME, 0); 
    GetPlayerName(playerId, m_Name, sizeof(m_Name)); 
    std::string sName(m_Name); 
    std::replace(sName.begin(), sName.end(), '_', ' '); 
    ::strncpy(sName.c_str(), Name, MAX_PLAYER_NAME); 
    return Name; 
} 
// in your usage 
CPlayer player; 
// ... 
sprintf(string, "%s", player.GetPlayerNameEx(playerid)); 

Answer 2

CPlayer::GetPlayerNameEx(playerid) 

Sie können nicht den Umfang (::) Operator für eine Klasse-Typ verwenden, um eine Funktion aufzurufen, wenn es eine statische Funktion ist. Um eine Funktion für ein Objekt aufzurufen, müssen Sie zuerst den Speicher für dieses Objekt erstellen (indem Sie eine CPlayer-Variable irgendwo einfügen) und dann die Funktion für dieses Objekt aufrufen.

Statische Funktionen sind global und speziell nicht mit Member-Variablen der Klasse (außer sie sind auch statisch), was sie gültig macht, ohne den Bereich einer tatsächlichen Objektinstanz aufzurufen.

Answer 3

Illegaler Aufruf der nicht statischen Elementfunktion bedeutet, dass Sie versuchen, die Funktion aufzurufen, ohne ein Objekt der Klasse zu verwenden, die die Funktion enthält.

Die Lösung sollte sein, die Funktion zu einer statischen Funktion zu machen.

Dies ist in der Regel, was den Fehler verursacht C2352:

class MyClass { 
    public: 
     void MyFunc() {} 
     static void MyFunc2() {} 
}; 

int main() { 
    MyClass::MyFunc(); // C2352 
    MyClass::MyFunc2(); // OK 
} 

Wenn es statisch zu machen für Sie keine Option ist, dann haben Sie eine Instanz der Klasse CPlayer zu erstellen.

So:

CPlayer myPlayer; 
myPlayer.GetPlayerNameEx(playerid);