classroom-ufersa / classroom-ufersa/InsertionSortEmployee

Propostas de Melhorias

Open
#12 0 comments 0 reactions 0 assignees View on GitHub
Dominant language
C
Stars
4
Forks
5
PR merge metrics
No merged PRs in 30d

Description

O menu construído na main poderia ser colocado diretamente no input, sem precisar atribuir uma variável para menu e depois imprimir com um print

poderia usar também uma estrutura melhor para estruturar o menu, garantindo que o usuário digitar o que é pedido
(essa parte foi corrigida, deixando o código mais simples, também poderia pensar em não iniciar as variáveis com none e poupar tempo de execução) o menu também foi tratado para receber apenas opções inteiras iguais a 1 , 2 ou 3.

o código original não lista os funcionários do arquivo, embora tenha a função insertion ela é utilizada apenas para uma demonstração com dados já fornecidos no modulo também não adiciona novos funcionários no arquivo txt
para corrigir seria necessário, refazer parte da estrutura do código, poderia ser aproveitado parte da estrutura de ler o arquivo e de escrever, mas seria bom a utilização de uma classe para interligar as funções, e assim automatizar as tarefas e abrindo mais possibilidades para o usuário, poderia ser pensando em implementar a parte que ler o arquivo em um modulo totalmente dedicado a listagem dos funcionários dentro da lista

No geral o código não funciona por completo manipulando os dados do arquivo, para a correção do problema como apontado seria necessário mais tempo, já que boa parte do código seria reescrito para atender a manipulação e ordenação dentro do arquivo

Contributor guide

No contributing guide indexed for this repository

Research direction

Start at main and inspect the existing insertion function, file-reading and file-writing code, and the employee list stored in the TXT file. Determine how the menu, employee listing, insertion, and file updates currently connect. Done means the program handles employee data from the file, supports adding employees, and validates menu choices 1, 2, or 3.

Written by the indexing model from the issue text.

Assessment

Tech stack
c
Domain
cli
Issue type
Feature
Difficulty
5/5
Estimated time
Over a week
Activity status
Stale
Clarity
Needs clarification
Newbie friendliness
25/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.