Ricardo-1976 / CHALLENGE-DESAFIO-04

DEVSFREE
MIT License
1 stars 1 forks source link

Desafio empréstimo #1

Open WagnerAs17 opened 2 years ago

WagnerAs17 commented 2 years ago

Esse método tem uma responsabilidade única ? Como podemos isolar os cotexto? Criar mais classes ? Métodos ? app.post("/user", (request, response) => { const { name, cpf, age, uf, renda_mensal } = request.body;

const userAlreayExists = users.some((users) => users.cpf === cpf);

if(userAlreayExists) { return response.status(400).json({error: "User already exists!"}) }

if(renda_mensal <= 3000 ) { users.push({ name,cpf,age,uf,renda_mensal, emprestimo: [emprestimo] }); const emprestimo =[ "EMPRESTIMO COM GARANTIA", "Taxa: 4%", ] return response.json(users); }

if(renda_mensal > 3000 && uf === "SP") { users.push({ name,cpf,age,uf,renda_mensal, emprestimo: [emprestimo] }); const emprestimo =[ "EMPRESTIMO COM GARANTIA", "Taxa: 4%", ] return response.json(users); }

if(renda_mensal > 5000 && uf === "SP" && age < 30) { users.push({ name,cpf,age,uf,renda_mensal, emprestimo: [emprestimo] }); const emprestimo =[ "EMPRESTIMO COM GARANTIA", "Taxa: 4%", ] return response.json(users); }

if(renda_mensal > 5000) { const emprestimo =["EMPRESTIMO Consignado", "Taxa: 2%" ] users.push({name,cpf,age,uf,renda_mensal, emprestimo: [emprestimo]}); return response.json(users); }

else { return response.json({ error:"Valor não definido!" }); }

});

WagnerAs17 commented 2 years ago

Reflexão Será que podemos renomear a variavel emprestimo para o nome do emprestimo que ela representa ? image

WagnerAs17 commented 2 years ago

Dica. Lembrando que o cliente pode ter mais de um empréstimo como opção. Será que do jeito que está vai retornar mais de um empréstimo ?

Ricardo-1976 commented 2 years ago

Dica. Lembrando que o cliente pode ter mais uma avaliação como opção. Que jeito que está vai retornar mais de Será um jeito?

Não. Mas posso mudar isso

Ricardo-1976 commented 2 years ago

Reflexão Será que podemos renomear a variavel emprestimo para o nome do emprestimo que ela representa? imagem

Sim