Ver Mensaje Individual
  #3  
Antiguo 20-09-2011
Avatar de Ñuño Martínez
Ñuño Martínez Ñuño Martínez is offline
Moderador
 
Registrado: jul 2006
Ubicación: Ciudad Catedral, Españistán
Posts: 6.000
Reputación: 25
Ñuño Martínez Tiene un aura espectacularÑuño Martínez Tiene un aura espectacular
Por cierto: estás usando un HTML muy viejo, tal vez la versión 2 o o incluso la 1. Te recomiendo que uses una más moderna. Lo mejor es HTML 5, proque es mucho más simple, menos estricta y compatible "hacia atrás".

Para que veas lo fácil que es "traduzco" tu código:
Código PHP:
<!DOCTYPE html>
<html>
<head>
 <meta lang="es" />
 <meta charset="utf-8" />
<!-- Las siguientes líneas añaden compatibilidad "hacia atrás" -->
 <meta name="lang" content="es" />
 <meta name="charset" content="utf-8" />
 <title>Consulta de usuarios</title>
 <link rel="stylesheet" type="text/css" href="estilo.css" />
<?php # La etiqueta PHP también en minúsculas.
// Incluir bibliotecas de funciones
//  include ("lib/fecha.php");

# ¡No comentes el cierre del bloque PHP!
 
?>
</head>
<body>
<h1>CONSULTA DE USUARIOS</h1>
<?php

# La "arroba" evita que MySQL escriba errores donde no deba.
$link = @mysql_connect('peopleco.readyhostingmysql.com''admin1234''pretoriano1'); 
  if (!
$link) { 
    die(
'Could not connect: ' mysql_error()); 
  } 
echo 
'<p>Connected successfully</p>'# Usa siempre etiquetas de bloque!
@mysql_select_db(administrador); 
 
//Enviar consulta
 
$instruccion="select ID,USUARIO,CLAVE,PUESTO from usuario";

# NO USES EL TRUCO DEL OR.  No hagas caso a los "gurús".
# Ofusca el código innecesariamente y no aporta nada.
# Mejor un "if" como Bob manda.
 
$consulta = @mysql_query($instruccion$link)
 if (!
$consulta)
   die (
'<p style="color: red"><strong>FALLO EN LA CONSULTA</strong></p>
<!-- No olvides mostrar el error -->
<pre>'
.mysql_error ().'</pre>
<!-- No olvides cerrar el documento. ;) -->
  </body>
</html>'
);
//Mostrar resultados de la consulta
 
$nfilas=mysql_num_rows($consulta);
 if(
$nfilas >0)
 {
  print(
"<table><tbody>\n");
  print(
"<tr>\n");
  print(
"<th>ID</th>\n");
  print(
"<th>USUARIO</th>\n");
  print(
"<th>CLAVE</th>\n");
  print(
"<th>PUESTO</th>\n");
  print (
'</tr>'); # Olvidaste cerrar la fila.
  
  
for ($i=0$i<nfilas$i++)
 {
  
$resultado=mysql_fetch_array($consulta);
  print (
"<tr>\n");
  print (
"<td>{$resultado['ID']}</td>/n");
  print (
"<td>{$resultado['USUARIO']}</td>/n");
  print (
"<td>{$resultado['CLAVE']}</td>/n");
  print (
"<td>{$resultado['PUESTO']}</td>/n");}
  print (
'</tr>'); # Olvidaste cerrar la fila.
  
# Olvidaste cerrar el "for".
}else
   
mysql_close($conexion);
 
?>
<!-- Olvidaste cerrar la tabla -->
  </tbody></table>
</body>
</html>
Puede parecerte una tontería, pero no lo es. Entre otras cosas, algunos navegadores (por ejemplo Internet Explorer 8, pero no es el único) entran en un "modo especial" si detectan un HTML anterior a HTML 4 o a XHTML 1, lo que suele traducirse en malas (o nulas) interpretaciones del CSS y el código JavaScript (de haberlo). Pásate por las W3C Schools, que no se tarda nada en leer la documentación básica del HTML, y aprendete la dirección de memoria porque sirve de mucho.

También te recomiendo que indexes y uses espacios, porque tu código es un cacao de cuidado, de ahí que olvides cerrar tablas y bloques.
__________________
Proyectos actuales --> Allegro 5 Pascal ¡y Delphi!|MinGRo Game Engine

Última edición por Ñuño Martínez fecha: 20-09-2011 a las 21:09:47.
Responder Con Cita