c61827f282943fa6d8e3b8d450515274

[pietroffernandes]

Com relação a nomeação das variáveis e métodos é interessante, para uma melhor documentação do código, priorizar um único formato como a camel notation. Alguns nomes estavam desnecessariamente em caixa alta ou muitos extensos. Como o nome do método calcula_BONIFICACAO_GERENTE, que foi alterado para calculaBonificacao.
Sobre o using namespace não acredito ser uma boa prática o seu uso pois pode dificultar o entendimento do código, por isso acrescentei std:: as devidas funções.
Havia um código como comentário irrelevante em venda.hpp que foi apagado. Tinha alguns pequenos problemas de indentação e redundâncias como public: 2 vezes que também foram corrigidos. Além de pedaços de código não utilizados.
As variáveis dentro das classes foram colocadas como privadas, que garante que elas serão preservadas, o que é muito importante em OO. Os objetos criados no heap são destruídos assim que não são mais utilizados. Em todas as oportunidades os métodos foram marcados com o contrato const. (Ainda podem ser feitas alterações como essas em Venda.hpp e Gerente.hpp; os cabeçalhos também poderiam ter sido divididos em .cpp's para separar as declarações e as implementações dos métodos)