Skip to content

PR de Correção#21

Open
jvpalves wants to merge 38 commits intocorrecao-projetofrom
master
Open

PR de Correção#21
jvpalves wants to merge 38 commits intocorrecao-projetofrom
master

Conversation

@jvpalves
Copy link

Observações sobre o projeto

Shidoshi93 and others added 30 commits March 30, 2021 20:20
parte de criar estudante, professor e turma criada, junto com seus re…
Implementado funcionalidade de troca de turma.
…ora. Também criei um script que roda uma função que cria as tabelas.
Feito refatoramento nos códigos das funcionalidades existentes até ag…
feito o primeiro desafio, pegar nome dos estudantes por turma
Resolvido bug no script de criar tabelas.
Implementado funcionalidade de pegar idade.
Refatorado funcionalidade de pegar estudantes por turma.
Implementado funcionalidade de pegar professor pela classe.
Implementado funcionalidade de pegar alunos por hobby.
Implementado funcionalidade de remover aluno de turma.
Implementado funcinalidade de remover estudante do curso.
IMplementado funcionalidade de remover professor da turma.
Implementado funcionalidade de alterar módulo da turma.
Copy link
Author

@jvpalves jvpalves left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Oi, gente! O projeto de vocês foi considerado acima do esperado. Meus parabéns!

Toda a fundação do módulo 4 parece ter sido muito bem absorvida, e deixa vocês prontíssimos para o módulo 5. Parabéns especiais às funções auxiliares que foram criadas, foram realmente uma mão na roda para os tratamentos de dados que foram feitos. Uma dica é que usar o DayJS para lidar com datas talvez possa ajudar! Até o próximo projeto

Comment on lines +9 to +14
export const getAge = async (idUser: string, tableName: string) => {
const birthdate = await connection.raw(`
SELECT birthdate
FROM ${tableName}
WHERE id = "${idUser}"
`)
Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

O cálculo da idade podia ser feito direto no banco, sem problemas

FOREIGN KEY (class_id) REFERENCES Class(id)
);
`)
console.log('Funfou')
Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

eu ri desse funfou hahaha, mas é uma trocar para algo como "Tabelas criadas com sucesso"

Comment on lines +12 to +14
if (req.body.type_class === 'Noturna') {
if (!req.body.name.includes('-na-night')) {
throw new Error('Nomes de turmas noturnas precisam terminar com -na-night.')
Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

sugestao: podia ser apenas um if, com &&

Comment on lines +35 to +36
if (error.message.includes('type_class')) {
res.status(422).send('Este tipo de turma não existe.')
Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

muito bom!

try {
const id = Number(req.params.id)

if(isNaN(id) === true) throw new Error("Invalid Id.")
Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

pode ser contraído para if(isNaN(id))

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

Successfully merging this pull request may close these issues.

3 participants

Comments