Klassen-ID, die innerhalb der Klassendeklaration verwendet wird. Ist es eine gute Übung?

Question

Beim Überprüfen einer unserer C++ - Klasse durch Coverity wurde eine Fehlermeldung zu einer bestimmten Klasse angezeigt. Die Klasse lautet wie folgt:

class InputRecord 
{ 
    /* Construtor */ 
    ... 
    InputRecord::RejectRecord(); 
    ... 
    /* Destructor */ 
} 

Welche Bedeutung hat die Verwendung eines Bezeichners innerhalb der Klasse? Ist es eine gute Übung, dem zu folgen?

Danke, Mathew Liju

Answer 1

Auf gcc Compilern (v4.1 und höher) würde dies mit "error: extra qualification" nicht kompiliert werden. Es ist daher eine gute Übung, es nicht dort zu platzieren!

Siehe here, die die zusätzliche Qualifikation als nicht legale C++ diskutiert.

Answer 2

G'day,

ich diese Art von Bezeichner immer gedacht, nur bei der Umsetzung verwendet wurde, und war in der Klassendefinition nicht erforderlich.

Eine Klassendeklaration ist

class InputRecord; 

Was du hast es eine Klassendefinition ist.

class InputRecord 
{ 
    /* Construtor */ 
    ... 
    RejectRecord(); 
    ... 
    /* Destructor */ 
} 

dann in Ihrer CPP-Datei Sie die Implementierung

InputRecord::RejectRecord() 
{ 
    ... 
} 

Answer 3

Im Allgemeinen haben, gibt es keine Fälle, in denen die Begriffsklärung durch die explizite InputRecord:: in Ihrem Codebeispiel vorgesehen sind wahrscheinlich etwas sein anders als redundant.

Wenn der Code komplexe Manipulationen ausführt, bei denen die spezifische Klasse relevant ist (z. B. wenn Sie ihn an eine Basisklasse mit einer schattierten Version übergeben), kann dies die Code-Klarheit unterstützen.

Es ist ein bisschen wie this-> (oder this. in C#/Java).

Persönlich würde ich solche redundante Spezifizierer entfernen und einen anderen Weg finden, den Punkt zu vermitteln.

Answer 4

In der Klassendefinition verwende ich nur Klassenbezeichner, wenn ich Parameter habe, die ähnlich (oder gleich) für private Mitglieder der Klasse benannt sind. Es vermeidet Mehrdeutigkeiten und macht den Code gleichzeitig leichter lesbar. Ich kann mir keinen anderen Fall vorstellen, in dem du es benutzen müsstest.