Si es posible evita escribir código donde mezcles bloques de HTML/PHP. Es la sintaxis más horrible que conozco, produciendo programas complicados para analizar y depurar.
Propongo para estos casos una técnica muy sencilla que consiste en lo siguiente:
- Trabajar todo dentro de un bloque PHP.
- Para las partes HTML creas una variable (en este caso la he llamado
$html
), y concatenas en esa variable todo lo que necesites. Nótese que esta variable se puede usar en varias partes del código, mediante el operador de concatenación .=
. Es como ir recogiendo en ella todo el HTML posible.
- Todo el contenido que se vaya a poner dentro de
$html
debe estar rodeado por comillas dobles "...contenido"
, esto permitirá escribir $variables
dentro del bloque sin necesidad de unir con el .
cortando el ritmo.
- Para el contenido que lleva comillas, como las etiquetas
name, class
, etc. debes escapar las mismas con \
.
- El final solamente se hace
echo $html;
y todo listo.
Esa simpleza produce un código totalmente distinto, elegante, legible, fácil de depurar, sin tantas aperturas y cierres al estilo <?php ... ?> echo "Hola" ?> <select> <?php while () {}
, cuando tienes un código así con cientos o miles de líneas y hay un problema, es imposible analizarlo a simple vista.
Aplicando lo dicho, tu código quedaría así. Otra mejora que he incluido aquí ha sido evaluar el dato de la sesión con un ternario, así sabemos si existe y al mismo tiempo lo guardamos en una variable para luego usar esa variable.
<?php
$dominio=!empty($_SESSION['dominioselect']) ? $_SESSION['dominioselect'] : NULL;
if ($dominio) {
$html="<input type=\"text\" id=\"nomb\" value=\"$dominio\" name=\"Nomb\" placeholder=\"Dominio\" class=\"form-control dAceptado\"> <br>";
}else{
$html="<input type=\"text\" id=\"nomb\" name=\"Nomb\" placeholder=\"Dominio\" class=\"form-control\"> <br>";
}
$html.="<select id=\"ext\" name=\"Ext\" class=\"custom-select\">
<option value=\"com\">.com</option>
</select>";
echo $html;
?>
<?php }else{ ?>
.