Identificatore di classe utilizzato all'interno della dichiarazione della classe. È una buona pratica?

Question

Durante la revisione di una delle nostre classi C++ tramite coverity, ha mostrato un messaggio di errore su una particolare classe. La classe è la seguente:

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

Qual è il significato dell'utilizzo di un identificatore all'interno della classe? È una buona pratica seguire questo?

Grazie, Mathew Liju

Answer 1

Sui compilatori gcc (v4.1 e successivi) questo non riuscirebbe a compilare con "errore: qualifica aggiuntiva". È quindi buona norma non metterlo lì!

Vedere here che discute la qualifica aggiuntiva come non legale C++.

Answer 2

G'day,

Ho sempre pensato che tipo di identificatore è stato utilizzato solo per l'attuazione e non era necessaria nella definizione della classe.

una dichiarazione di classe è

class InputRecord; 

Quello che hai c'è una definizione di classe.

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

poi nel file cpp si ha la realizzazione

InputRecord::RejectRecord() 
{ 
    ... 
} 

Answer 3

In generale, non ci sono casi in cui la disambiguazione fornito dal esplicito InputRecord:: nel codice di esempio sono suscettibili di essere qualsiasi cosa diverso da ridondante.

Se il codice sta eseguendo manipolazioni complesse in cui la classe specifica è rilevante (ad esempio lo si sta trasmettendo a una classe base con una versione ombreggiata), può essere utile semplificare il codice per renderlo esplicito.

È un po 'come usare this-> (o this. in C#/Java).

Personalmente rimuoverei tali specificatori ridondanti e troverei un altro modo per ottenere il punto.

Answer 4

Nella definizione di classe, utilizzo solo identificatori di classe se ho parametri che sono denominati in modo simile (o lo stesso) ai membri privati ​​della classe. Evita l'ambiguità e rende il codice più facile da leggere allo stesso tempo. Non riesco a pensare a un altro caso in cui avresti bisogno di usarlo.