Implementa los getter y setter en el archivo .h, ¿esto no es incorrecto?
Ni correcto ni incorrecto. Depende del contexto. Generalmente se separa la declaración en archivo de cabecera (.h
o .hpp
) y definición en archivo de código (.cpp
) para seguir los principios de encapsulamiento ya que ésto permite ocultar los detalles de implementación al usuario de la clase.
Pero un getter y un setter habitualmente ocupan una línea de código o poco más y tienen un comportamiento bastante obvio, así que ocultar los detalles de implementación de esas funciones en concreto es como ocultar un secreto a voces.
Los archivos de cabecera (.h
o .hpp
) suelen ser la interfaz pública de una clase; si desarrollaeses una librería sobre la que entregas tan sólo el código ya compilado (.lib
o .dll
) necesitarías las cabeceras para indicar al usuario de tu librería cómo usar tu clase.
Tenía entendido que acá solo iban los encabezados de los procedimientos.
En un archivo de cabecera puedes poner tanto las declaraciones (encabezados de procedimientos) como las definiciones (el cuerpo de los procedimientos), no hay ninguna restricción al respecto, tan sólo las restricciones impuestas por las necesidades de tu proyecto.
Por otro lado, los Data Types se que se implementan como las clases.
Esto es falso, una clase se implementa, un tipo de datos puede ser una clase, estructura, unión, enumerado o alias... los cuatro primeros pueden tener implementación mientras que el alias no se implementa: es. Otros objetos son también tipos de datos pero de una complejidad diferente como plantillas y lambdas; las primeras (salvo excepciones) sólo disponen de implementación (carecen, salvo excepciones, de definición) y las lambdas son tan sólo definición sin declaración.
¿A éstos tambièn les corresponde un archivo ".h"?
El archivo de cabecera, en general, contiene las declaraciones; pero no hay nada que impida que contengan también las implementaciones (definiciones).