É errado definir funções membro dentro da própria classe? Ou é apenas uma questão de organização de código escrever os protótipos na criação da classe e depois implementa-los?
Em geral é errado mesmo.
Quando faz sentido?
- quando você usa templates
- quando é uma classe de constantes apenas
- em programas muito simples, como programas de estudante, em que se muitas vezes se define classes e
main()
e tudo em um único arquivo fonte
Imagine o caso da classe vector
, da classe list
. De iostream
. Essas classes redefinem vários operadores por exemplo. E tem muitos construtores e implementação complexa. E não se compila isso toda hora. Apenas se usa a biblioteca
Se a implementação estiver toda no header qualquer um que for usar a classe vai ler a implementação e quase nunca é o que você quer. A implementação pode não ter sido vendida, mas apenas a funcionalidade. Muita gente escreve classes como fonte de renda ;)
Se você tiver um header separado e mais o arquivo completo vai ter que manter duas vezes a mesma coisa, e isso é um risco
Se você tiver várias implementações da mesma classe em geral vai querer manter um único header com os protótipos, para sua própria segurança. E testar as implementações em vários cenários sabendo que os protótipos e estruturas são os mesmos