Skip to content

Criticas construtivas #1

@JhoanDev

Description

@JhoanDev

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

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions