Ver Mensaje Individual
  #14 (permalink)  
Antiguo 08/08/2007, 08:26
Avatar de GatorV
GatorV
$this->role('moderador');
 
Fecha de Ingreso: mayo-2006
Ubicación: /home/ams/
Mensajes: 38.567
Antigüedad: 18 años
Puntos: 2135
Re: Pregunta de principiante...

Primero, no utilizes $_POST[login], si tienes PHP corriendo bajo E_STRICT te arrojara varios warnings, utiliza $_POST['login'].

Segundo, lo que estas haciendo es programando estilo PHP, y no esta mal, pero no estas utilizando al 100% todo el poder de OOP, es mucho mas amplio, y como te comentaba en un inicio es dificil que alcances a ver todos los patrones que existen pero si quieres seguir en esto te recomiendo leer el libro (no ahora):

Head First - Design Patterns

Te puedes dar una muy buena idea de como hacer un buen disenio en POO.

Tercero es recomendable que a todos los metodos de tus funciones les des una visibilidad, es decir a todos pon si son public, private o protected, no los dejes sin darles una visibilidad.

Respecto al disenio general de tu clase esta bien, todavia deberias de hondar en los temas de Patrones de Disenio, por la naturaleza de PHP es dificil programar al 100% OOP en algun lado tienes que meter codigo en procedimientos, pero podrias hacer algo asi:

Código PHP:
class Pagina {
        public static function 
main() {
                  
// Aqui todo el codigo que quieras manejar como el de validar login
        
}
}


Pagina::main(); // Aqui llamamos a la funcion para que funcione "todo" 
Saludos.