E' preferibile usare la sintassi che hai trovato nell'esempio quando un metodo deve fare solo (e sottolineo solo) assegnazioni.
In questa maniera il metodo della classe (che in questo caso è il costruttore) viene implicitamente definito come funzione "inline", migliorando l'efficienza dell'eseguibile generato.

Sarebbe buona norma dividere l'interfaccia dall'implementazione, separandole anche fisicamente usando files con estensione .h e .cpp.
Scegliere fra leggibilità ed efficienza è responsabilità del programmatore, che deve valutare quale sia la soluzione migliore volta per volta.