Perché dovresti scegliere gli obiettivi di un incarico?

Question

mi sono imbattuto in un codice che si riduce al seguente:

enum BAR { /* enum values omitted */ } 

class Foo{ 
public: 
    void set(const BAR& bar); 
private: 
    uint32_t bits; 
}; 

void Foo::set(const BAR& bar) 
{ 
(uint32_t&)bits = bits | bar; 
} 

non capisco il punto del cast c-style nell'assegnazione di Foo :: set. Perché dovresti scegliere gli obiettivi di un incarico? Sono pazzo, o questo ha uno scopo?

Answer 1

In questo caso, non riesco a vedere alcun motivo per il cast, in quanto il cast in questione è dello stesso tipo del cast. In generale, potrebbe essere utilizzato per forzare l'utilizzo di un determinato operatore di assegnazione.

Ora ripeterò il mio mantra: se il tuo codice contiene cast, probabilmente c'è qualcosa di sbagliato nel codice o nel design e dovresti esaminare entrambi allo scopo di rimuovere il cast.

Answer 2

Se si desidera consentire solo un determinato valore (ad es., Il controllo sull'assegnazione viene eseguito come se fosse una variabile digitata come nel cast, invece di lasciare passare tutto come se si fosse allargato il rhs).

Answer 3

Non fa nulla, per quanto posso dire, anche se BAR è definito con valori al di fuori dell'intervallo uint32. Sembra un rumore per me.

Answer 4

Sono d'accordo con Neil Butterworth, il cast in questo caso non è necessario ed è un "odore di codice" definito.

Answer 5

Non posso dirlo con certezza senza conoscere lo sfondo, ma sembra che qualcuno abbia preso del codice C esistente e lo abbia avvolto in una classe C++. Il cast potrebbe essere un residuo del codice c.

Answer 6

Non sono sicuro che sia legale. un cast non è un lvalue ...

In entrambi i casi, sembra abbastanza inutile.