Entrar Ainda não tem acesso?
Solucionado (ver solução)
Solucionado
(ver solução)
1
resposta

Redundância na criação de variável?

Referente ao curso PHP I: Orientação a objetos com sua linguagem preferida, no capítulo Evoluindo a listagem de produtos e atividade Mãos na massa: evoluindo o formulário de alteração

por Douglas Alcântara de Araújo Cata Preta
| 430.7k xp | 12 posts
Desenvolvedor .NET Júnior

Na opinião do instrutor no exercício 13 ele sugere incluir no arquivo "produto-altera-formulario.php" o seguinte trecho de código:

$selecao_usado = $produto->usado ? "checked='checked'" : "";
$produto->usado = $selecao_usado;

Porém, no exercício 11, existe no arquivo "produto-formulario-base.php" o trecho:

<?php
$usado = $produto->usado ? "checked='checked'" : "";
?>

Como o "produto-altera-formulario.php" usa o método "require_once("produto-formulario-base.php")" não é reduntante a criação de $selecao_usado e $usado?

Outra questão é que não entendi a razão do trecho de código abaixo. Se ao acessar o "altera-produto.php" o atributo $produto->usado já é setado em uma condicional if(array_key_exists('usado', $_POST)).

$produto->usado = $selecao_usado;
1 resposta
solução!
por André Chaves
| 1967.1k xp | 2579 posts
Instrutor Lider técnico

Oi Douglas, tudo bom?

Nesse caso, realmente daria pra reduzir o código com o if ternário, como você comentou evitando até de ter que repassar esse valor pro $produto->usado.

Você pode sugerir uma melhoria pra esse capitulo aqui =)

Mas, não declarar esse código por ter importado no include/require pode ser um tiro no pé pra manutenção do software. Imagina uma pessoa pegando esse sistema sem nunca ter visto, não ficaria tão evidente que aquela variável veio do include e de qual include ela veio.

Esse código precisa ser feito no formulário para trazer a tela preenchida com os dados certinho. No altera-produto.php a gente ta pegando esses dados da tela e jogando pro banco de dados. São momentos diferentes da aplicação.

Abraço!