La mezcla de contenido HTML / PHP abriendo y cerrando bloques produce un código verdaderamente horrible y difícil de analizar o de depurar.
Para casos como el tuyo, puede ser interesante el uso de printf()
o de sprintf()
. Funcionan igual, con la diferencia de que con la primera imprimes in situ, y con la segunda almacenas el contenido en variables, por ejemplo si estás concatenando o creando un array o un objeto.
Así en lugar de esto:
<a href="mostrar_notarias.php?documento_seleccionado=<?php echo $documento_seleccionado; ?>&nombre=<?php echo $nombre; ?>">Continuar</a></button>
Tendríamos:
printf('<button>
<a href="mostrar_notarias.php?documento_seleccionado=%s&nombre=%s">Continuar</a>
</button>',
$documento_seleccionado,
$nombre);
He agregado la etiqueta <button>
de apertura, la cual no aparece.
Y en lugar de:
echo '<td><a href="mostrar_horas.php?documento_seleccionado=' . $fila['id'] . '&nombre=' . $fila['nombre'] . '¬aria_seleccionada=' . $fila['id'] . '" class="btn btn-outline-warning"><h6>Continuar</h6></a></td>';
Tendríamos:
printf('<td>
<a href="mostrar_horas.php?documento_seleccionado=%s&nombre=%s¬aria_seleccionada=%s" class="btn btn-outline-warning"><h6>Continuar</h6></a>
</td>',
$fila['id'],
$fila['nombre'],
$fila['id']);
Como vez, tanto printf()
como sprintf()
funcionan poniendo marcadores donde va cada dato, y al final pasas los datos en la posición en que deban ir.
En cuanto a tu código PHP, podemos darle claridad usando el operador de fusión de null (PHP 7 o superior) o bien operadores ternarios en su defecto.
<?php
$doc = $_GET['documento_seleccionado'] ?? NULL;
$nombre = $_GET['nombre'] ?? NULL;
$id = $_GET['notaria_seleccionada'] ?? NULL;
if ($doc && $nombre && $id) {
// Conecta a la base de datos
$pdo = new PDO('mysql:host=localhost;dbname=notaria', 'root', '');
// Recoge el ID del documento seleccionado del formulario
// Ejecuta una consulta para obtener los datos del documento seleccionado
$stmt = $pdo->prepare('SELECT * FROM notarias WHERE id = :id');
$stmt->bindParam(':id', $id, PDO::PARAM_INT);
$stmt->execute();
$documento = $stmt->fetch();
}
?>
Es un código más limpio. Te aconsejo que evites el uso de variables con nombres kilométricos. Tal es el caso de $documento_seleccionado
, con nombrarla $doc
se salva la comprensión del código y se gana mucho en legibilidad. Le puedes llamar $pepitoDeLosPalotesYMasCosas
o lo que quieras, pero demasiada verbosidad innecesaria hace que el código sea infumable muchas veces.