Skip to main content
se añadieron 215 caracteres en el cuerpo
Origen Enlace
A. Cedano
  • 95.2k
  • 31
  • 145
  • 257
$query = array(
                'documento_seleccionado'=>$documento_seleccionado,
                'nombre'=>$nombre
              );

printf('<button>
          <a href="mostrar_notarias.php?documento_seleccionado=%s&nombre=%s">Continuar<%s">Continuar</a>
        </button>',
        $documento_seleccionado,
        $nombrehttp_build_query($query));
$query = array(
                'documento_seleccionado'=>$fila['id'],
                'nombre'=>$fila['nombre'],
                'notaria_seleccionada'=>$fila['id']
              );

printf('<td>
         <a href="mostrar_horas.php?documento_seleccionado=%s&nombre=%s&notaria_seleccionada=%s"%s" class="btn btn-outline-warning"><h6>Continuar</h6></a>
        </td>',
       $fila['id'],
       $fila['nombre'],
       $fila['id']http_build_query($query));
printf('<button>
          <a href="mostrar_notarias.php?documento_seleccionado=%s&nombre=%s">Continuar</a>
        </button>',
        $documento_seleccionado,
        $nombre);
printf('<td>
         <a href="mostrar_horas.php?documento_seleccionado=%s&nombre=%s&notaria_seleccionada=%s" class="btn btn-outline-warning"><h6>Continuar</h6></a>
        </td>',
       $fila['id'],
       $fila['nombre'],
       $fila['id']);
$query = array(
                'documento_seleccionado'=>$documento_seleccionado,
                'nombre'=>$nombre
              );

printf('<button>
          <a href="mostrar_notarias.php?%s">Continuar</a>
        </button>',
        http_build_query($query));
$query = array(
                'documento_seleccionado'=>$fila['id'],
                'nombre'=>$fila['nombre'],
                'notaria_seleccionada'=>$fila['id']
              );

printf('<td>
         <a href="mostrar_horas.php?%s" class="btn btn-outline-warning"><h6>Continuar</h6></a>
        </td>',
        http_build_query($query));
Origen Enlace
A. Cedano
  • 95.2k
  • 31
  • 145
  • 257

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'] . '&notaria_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&notaria_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.