El principal problema que estás sufriendo es debido a que estás disminuyendo el valor de $x
(el número de elementos que deben mostrarse en la primera línea y límite superior de tu bucle externo) dentro del bucle interno:
for ($i = 0; $i < $x; $i++) {
for ($u = -3; $u <= $i; $u++) {
echo $x--;
}
}
Eso hace que cada iteración del bucle exterior $x
valga 4+n unidades menos. Por eso nos encontraremos con números que faltan o incluso números negativos:
15: 151413121110987654321
22: 22212019181716151413121110987654321
12: 1211109876543210-1-2
14: 14131211109876543210
16: 1615141312111098765432
Para corregirlo debes mostrar el índice del bucle interior en el echo
interno y, además, generar un retorno del carro o nueva línea (<br />
) al finalizar cada iteración del bucle exterior.
Además, es recomendable ir de $x
a 1
(decrementando el valor de $i
) y en el bucle interior hacer lo mismo desde $i
a 1
(decrementando el valor de $u
):
for ($i = $x; $i > 0; $i--) {
for ($u = $i; $u > 0 ; $u--) {
echo $u;
}
echo "<br />", PHP_EOL;
}
También puedes hacer el trabajo mediante bucles while
anidados:
while (($i = $x--) > 0) {
while ($i > 0) {
echo $i--;
}
echo "<br />", PHP_EPL;
}
También puedes hacerlo con un único echo
, sin bucles, usando array_map()
, range()
e implode()
:
echo implode(array_map(function($i) {
return implode(range($i, 1, -1), "");
}, range($x, 1, -1)), "<br />". PHP_EOL);
Volviendo a recomendarte no generar HTML mediante un echo
grande, tu código completo podría ser:
<h2><u>Pirámide Inversa de Números:</u></h2>
<form action='ejercicio9.php' method='post'>
Introduce un número del 10 al 40: <input type='text' name='x' value=''>
<br /><br />
<input type='submit' value='Mostrar'>
</form>
<hr />
<?php
if (($x = intval($_POST['x'])) >= 10 && $x <= 40) {
?><h2>Número correcto</h2><?php
while (($i = $x--) > 0) {
while ($i > 0) {
echo $i--;
}
echo "<br />", PHP_EOL;
}
} else {
?><h2 style="color: red;">Número incorrecto</h2><?php
}