classroom-ufersa / Gerenciamento_de_Loja_de_Joias_e_Bijuterias

Projeto da disciplina de Algoritmos e Estrutura de Dados I
1 stars 1 forks source link

Criticas construtivas #1

Open JhoanDev opened 5 months ago

JhoanDev commented 5 months ago

Feedback sobre o código em C na branch main

  1. Pequeno erro de digitação: O arquivo principal está main..c e não main.c.

  2. Múltiplas declarações de bibliotecas: Seria mais eficiente consolidar todas as bibliotecas necessárias em um único arquivo de cabeçalho, como o "funcoes.h", já que ele é incluído em todos os arquivos ".c". Isso reduziria a redundância e simplificaria o código.

  3. Faltou criar funções específicas para acessórios: Seria vantajoso ter funções específicas para acessórios e reutilizá-las para integrá-las na TAD de sessões. Isso promoveria uma maior modularidade e facilitaria a manutenção do código.

  4. Falta de portabilidade para outros sistemas operacionais: A função system("cls") só funciona no Windows, o que limita a portabilidade do programa para outros sistemas operacionais. Seria importante utilizar funções ou técnicas que sejam compatíveis com diferentes sistemas.

  5. Falta de validação dos tipos de acessórios: Seria conveniente implementar uma validação para aceitar apenas os tipos de acessórios definidos (joias e bijuterias), conforme indicado no momento do cadastro. Isso garantiria consistência nos dados e preveniria erros de entrada.

  6. Cadastro duplicado de sessões: Não deveria ser possível cadastrar duas sessões com nomes iguais. Isso pode causar confusão e inconsistência nos dados. Implementar uma verificação para garantir a unicidade dos nomes das sessões seria uma melhoria importante.

No geral, essas são algumas sugestões para aprimorar o código. Parabenizo pelo trabalho realizado!

@roscibely @marceloDev0 @ViniciusOliver13

marceloDev0 commented 5 months ago

boa noite @JhoanDev nessa parte de include eu não estava conseguindo arrumar vc poderia por favor mandar um exemplo como deveria ficar, já tentei de várias formas. Na parte de valida se a sessão já existia eu tinha feito, mas parou de funcionar

JhoanDev commented 5 months ago

Basicamente é só pegar todas as bibliotecas de todos os arquivos (as bibliotecas com <> ou seja as do próprio c) e colocar todas elas no funções.h, claro sem repetir as que repetir o nome vc apaga.