Usaste el módulo en el primer bucle, así que úsalo de nuevo en el segundo

 print "</tr>"; print "<tr>"; for($i = 1; $i <= 50000; $i++){ $unicodeChar = "&#{$i}"; $contador--; print "<td>$i</td><td>$unicodeChar</td>\n"; if ( $i % $columnas/2 ) == 0 ) { print "</tr><tr>\n"; } } print "</tr>\n";

Muchas gracias por esta ayuda, me han solucionado!!

aquí está el código que arreglé, con tu idea :)

´´´´

 <?php $columnas = 16; $filas = 16; $word= 0; $contador = 0; print "<table border=\"1\">\n"; print "<caption>ASCII</caption>\n"; print "<tbody>\n"; print "<tr>\n"; for ($j = 1; $j <= $columnas; $j++){ if ($j%2 == 1){ print "<th>Codigo</th>\n"; }elseif ($j%2 == 0){ print "<th>Valor</th>\n"; } } print "</tr>"; for ($i = 1; $i <= $filas; $i++){ print "<tr>"; for($i = 1; $i <= 50000; $i++,$contador){ $unicodeChar = "&#{$i}"; $contador--; print "<td>" .$i. "</td>\n"; print "<td>".$unicodeChar."</td> \n"; if (($i % 8 ) == 0 ){ print "</tr><tr>"; } } print "</tr>\n"; } print "</tbody>\n"; print "</table>\n"; ?>

´´´´

Como dijo Riggsfolly , pero tal vez divida el número de columnas por 2 antes del módulo, porque hay 2 TD / iteración:

 if ( $i % ($columnas/2) ) == 0 ) { print "</tr><tr>\n"; }