42b60aab69e6780813a619fd59a31b10 #24
Open
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
modularização(criação de arquivos cpp para cada hpp), correção da identação, aplicação de boas práticas como a retirada de "using namespace", criação parcial do makefile
makefile concluído, criação de arquivos .o, e divisão em diretórios build, include, src e entidades dentro de src, deleção de arquivos desnecessários como pizza_marguerita hpp e cpp, e pizza_calabresa hpp e cpp, substituindo por atributos e métodos em pizza hpp e cpp, padronização do nome de variáveis em "underscore case", padronização do nome de classes com letra inicial maiúscula e substituição de variáveis com o mesmo nome por nomes diferentes.
Retirada de objetos e variáveis desnecessárias e repetidas
retirada de números mágicos e substituição deles por constantes, e retirada de comentários
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Revisão do código
• Nomenclatura padronizada, "snake_case"
• Nomes de TADs padronizados para começar com letra inicial maiúscula
• Nomes de parâmetros relacionados aos atributos
• Retirada de “using namespaces”
• Correção da indentação de todo o código, facilitando a sua compreensão e melhorando a sua legibilidade
• Retirada de variáveis desnecessárias e repetidas, sendo substituídas por uma de cada tipo(situação vista, por exemplo, no "main.cpp")
• Utilização de "this" antes da chamada de certos métodos, como forma de Boas Práticas, assegurando de que seria o método chamado
Modularização
• Separação do contrato e da implementação, separando arquivos que antes eram só "hpp", em "hpp" e "cpp"
• Separação dos arquivos em um diretório build, para arquivos ".o", em um diretório include, para arquivos "hpp", em um diretório "src", para arquivos "cpp", e , dentro desse, um diretório entidades, para arquivos "cpp" que não fossem o “main.cpp”
• Criação do arquivo makefile para facilitar a compilação e a execução do programa
Refatoração
• Mudança de atributos "public" para private, aumentando a segurança do código
• Criação de métodos "public" “get” para acessar os atributos private necessários
• Retirada de TADs desnecessários, como “pizza_de_marguerita”, e substituição deles por atributos "private", como sabor, e métodos "public" "get" para esses atributos, reduzindo o tamanho do código, mas ainda fazendo com que tenha a mesma saída, tornando-o mais eficiente
• Substituição do tipo int para bool em situações de identificação de “true” ou “false”, deixando o código mais apropriado
• Retirada de “números mágicos” e sua substituição por constantes