Operatori di conversione da sovraccarico C++

Question

Sto provando ad avere una classe che permetta il cast implicito a determinati tipi incorporati, come unsigned long int e poiché sto cercando di farlo nel modo più corretto possibile (questo è il mio primo progetto importante in C++), ho colpito un problema strano per quanto riguarda la correttezza const:

questo funziona:

#include <iostream> 

class CustomizedInt 
{ 
private: 
    int data; 
public: 
    CustomizedInt(); 
    CustomizedInt(int input); 
    operator unsigned long int() const 
    { 
     unsigned long int output; 
     output = (unsigned long int)data; 
     return output; 
    } 
}; 

CustomizedInt::CustomizedInt() 
{ 
    this->data = 0; 
} 

CustomizedInt::CustomizedInt(int input) 
{ 
    this->data = input; 
} 

int main() 
{ 
    CustomizedInt x; 
    unsigned long int y = x; 

    std::cout << y << std::endl; 

    return 0; 
} 

Ma questo:

#include <iostream> 

class CustomizedInt 
{ 
private: 
    int data; 
public: 
    CustomizedInt(); 
    CustomizedInt(int input); 
    operator unsigned long int() const; 
}; 

CustomizedInt::CustomizedInt() 
{ 
    this->data = 0; 
} 

CustomizedInt::CustomizedInt(int input) 
{ 
    this->data = input; 
} 

CustomizedInt::operator unsigned long() 
{ 
    unsigned long int output; 
    output = (unsigned long int)data; 
    return output; 
} 

int main() 
{ 
    CustomizedInt x; 
    unsigned long int y = x; 

    std::cout << y << std::endl; 

    return 0; 
} 

mi dà questo errore in Visual Studio 2010: error C2511: 'CustomizedInt::operator unsigned long(void)' : overloaded member function not found in 'CustomizedInt'

Ora, se rimuovo la parola chiave const dalla definizione dell'operatore, tutto è OK. è un insetto? Ho letto che dovrei usare la parola chiave const dopo ogni metodo/operatore (pubblico) per affermare chiaramente che non altera l'oggetto corrente in alcun modo.

Inoltre, so che definire un simile operatore può essere una cattiva pratica, ma non sono sicuro di comprendere appieno le avvertenze associate. Qualcuno potrebbe descriverli per favore? Sarebbe meglio fare una semplice definizione di un metodo pubblico chiamato ToUnsignedLongInt?

Answer 1

firma La funzione non corrisponde alla definizione di funzione.

operator unsigned long int() const; 

e

CustomizedInt::operator unsigned long() { ... } 
             ^^^ 
            const missing 

In questo caso si dovrebbe contrassegnare l'operatore di conversione come const in quanto non influisce sullo stato interno dell'oggetto.

Inoltre, utilizzare gli elenchi di inizializzazione del costruttore per inizializzare le variabili membro.

CustomizedInt::CustomizedInt() 
: data() 
{ 
} 

CustomizedInt::CustomizedInt(int input) 
: data(input) 
{ 
} 

Answer 2

Sì, se la funzione membro non influisce sullo stato logico dell'oggetto, è quindi necessario postfix con const, in modo che il compilatore lo applicherà.

Ma in questo caso, è necessario aggiungere const quando si definisce il corpo della funzione!

Answer 3

È potrebbe rimuovere il const dalla dichiarazione, ma quello che quasi certamente vuole fare è aggiungere alla definizione:

CustomizedInt::operator unsigned long() const 
{ 
    unsigned long int output; 
    output = (unsigned long int)data; 
    return output; 
} 

Answer 4

Hai solo bisogno di copiare lo stesso prototipo di funzione nell'implementazione. ie.

CustomizedInt::operator unsigned long int() const