Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Criticas construtivas #1

Open
JhoanDev opened this issue Apr 9, 2024 · 0 comments
Open

Criticas construtivas #1

JhoanDev opened this issue Apr 9, 2024 · 0 comments

Comments

@JhoanDev
Copy link

JhoanDev commented Apr 9, 2024

Feedback sobre o código em C na branch main

  1. Adicionar um arquivo .gitignore: Seria recomendável adicionar um arquivo .gitignore para evitar que os arquivos .o e os executáveis sejam incluídos no repositório do Git.

  2. Arquivo de pacientes atendidos: A ideia era a seguinte: se eu adicionei um paciente ao meu programa automáticamente ele foi atendido então não precisa desse arquivo extra.

  3. Localização da struct: Sugiro mover a definição da struct para o arquivo .c e manter apenas o typedef no arquivo .h. Isso é uma prática comum para melhorar a encapsulação e a modularidade do código, claro que irá mudar a logica de varias coisas então provavelmente seja muito trabalhoso isso.

  4. Uso do typedef: No trecho de código apresentado, há uma redundância ao utilizar struct Consultorio em vez do typedef. Como vocês já estão utilizando typedef para a struct Consultorio, não há necessidade de preceder o nome da struct com struct dentro da função.

void liberar_listas(struct Consultorio* lista) {
    struct Consultorio* temp_consultorio;
    while (lista != NULL) {
        temp_consultorio = lista;
        lista = lista->proximo;
        free(temp_consultorio);
    }
}
  1. Tratativas durante a inserção de dados: Alguns pontos no código, como a adição de especialidade, estão funcionando corretamente, mas não fornecem um retorno claro para o usuário. Seria útil fornecer mensagens de feedback para informar ao usuário que a operação foi bem-sucedida ou se ocorreu algum erro.

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

@roscibely @LeticiaVieirg @isabellylimals

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant