Solucionado (ver solução)
Solucionado
(ver solução)
2
respostas

é uma boa prática criar tantas variáveis?

Boa noite, gostaria de saber se posso atribuir os valores do form diretamente nas variáveis td, por exemplo:

nomeTd.textContent = paciente.nome;
pesoTd.textContent = paciente.peso;
alturaTd.textContent = paciente.altura;
gorduraTd.textContent = paciente.gordura;
imcTd.textContent = paciente.imc;
```

dessa forma eu evitaria criar essas variáveis:

var nome = paciente.nome;
var peso = paciente.peso;
var altura = paciente.altura;
var gordura = paciente.gordura;
var imc = paciente.imc;
```

pra só depois ter que inserir os valores nas tds:

nomeTd.textContent = nome;
pesoTd.textContent = peso;
alturaTd.textContent = altura;
gorduraTd.textContent = gordura;
imcTd.textContent = imc;
```

e também gostaria de saber se séria uma boa prática inserir a função:

calculaImc(form.peso.value, form.altura.value); 
```

diretamente na td, economizando assim duas linhas de código.

2 respostas
solução!

Oi Marcos, tudo bom?

Criar variaveis não costuma ser um problema. O ideal é que, independente da quantidade de linhas, seu código seja de facil leitura.

Essa linha:

nomeTd.textContent = paciente.nome;

Não me parece perder muita legibilidade para essas duas:

var nome = paciente.nome;
nomeTd.textContent = nome;

Nesse caso, evitar a criação da variavel nome é uma boa mesmo.

Mas, temos que ficar de olho pra não evitar a criação de uma variavel e prejudicar a legibilidade do código. De novo, legibilidade é mais importante pra manutenção do que quantidade de linhas.

Abraço!

A criação de variáveis fica legível para outros desenvolvedores darem manutenção em seu código.