Una ayudita con OOP

[gus_anomaly]

Muchachos, estoy viendo PHP con Objetos... aqui algo que hice, pero desde ya que no funciona, me dan una ayudita!?

Código:

<?php

include ('config.inc.php');

class Conexion{
	private $serv;
	private $user;
	private $pass;
	private $base;
	private $conect;
	
	function __construct($serv, $user, $pass, $base){
		$this->serv = $serv;
		$this->user = $user;
		$this->pass = $pass;
		$this->base = $base;
		$this->conectar();
	}

	private function conectar(){
		$this->conect = mysql_connect ($this->host, $this->user, $this->pass)or die ("error en conexion");
		mysql_select_db ($this->base, $this->conect);
	}
}

class Datos{
	
	function mostrar(){
		$this->resultado = mysql_query("SELECT titulo, contenido FROM post ORDER BY id ASC", $this->conect);
		$resu = mysql_fetch_array($this->resultado,MYSQL_ASSOC);
		
		while($resu){
			echo " ".$resu['titulo']." - ".$resu['contenido']." <br/>";
		}

	}
}

?>

En el index.php (ya contiene el include de la clase) va lo siguiente:

Código:

$mostrar = new Datos();
$mostrar->mostrar();

Bueno, muchas gracias, no me insulten mucho!
Gustavo.

[maycolalvarez]

¿te sale algún error?.

lo que si veo es que la clase Datos no tiene ningún miembro $this->conect ya que en ningún momento creas la instancia de la clase conexion y en ningún momento se la asignas como valor del miembro a instancia de la clase Datos $mostrar

[gus_anomaly]

Gracias por responder, aqui los errores:

Warning: mysql_query(): supplied argument is not a valid MySQL-Link resource in C:\AppServ\www\gustavo\site\class.inc.php on line 29

Warning: mysql_fetch_array(): supplied argument is not a valid MySQL result resource in C:\AppServ\www\gustavo\site\class.inc.php on line 30

El $this->conect esta en la funcion mysql_query...

Saludos.

[maycolalvarez]

exacto, pero $this->conect NO pertenece como Miembro de la clase Datos, por lo tanto no devolverá Nada, recuerde que POO dista en algunos puntos de la programación estructurada, existe un mayor nivel de acceso y ámbito de las variables como parte de la encapsulación

[gus_anomaly]

aja, ok... entonces uso herencia? Es decir, uso... asi:

class Datos extends Conexion{
}

O paso a public el atributo $conect ???

????
Gracias.

[maycolalvarez]

depende de como lo quieras implementar, podrías consultar patrones de diseño.

[gus_anomaly]

Es una prueba porque estoy aprendiendo a usar PHP... realmente hay cosas que no las tengo presentes.

Si puede ser una ayuda mas puntual.

Muchas gracias!
Gustavo.

[GatorV]

No puedes heredar por heredar más bien tendrías que agregar un getter a tu clase Conexión para obtenerla desde tu clase datos, aunque inyección de dependencias es lo mejor:

Código PHP:

class Datos {
       private $conexion;
 
       public function setConexion($conexion) {
                 $this->conexion = $conexion;
       }
 
       public function mostrar() {
               $resultado = mysql_query("SELECT titulo, contenido FROM post ORDER BY id ASC", $this->conexion);
        $resu = mysql_fetch_array($resultado,MYSQL_ASSOC);
        
        while($resu){
            echo " ".$resu['titulo']." - ".$resu['contenido']." <br/>";
        }
       }
}
 
class Conexion {
        public function getConexion() {
                  return $this->conect;
        }
}
 
$conexion = new Conexion(/* */);
$datos = new Datos();
$datos->setConexion($conexion);
$datos->mostrar();
 

[gus_anomaly]

Gracias!!!
Sigo con el mismo error, pero la idea de usar setter y getter no la habia tenido en cuenta...

Warning: mysql_query(): supplied argument is not a valid MySQL-Link resource in C:\AppServ\www\gustavo\site\class.inc.php on line 37

Warning: mysql_fetch_array(): supplied argument is not a valid MySQL result resource in C:\AppServ\www\gustavo\site\class.inc.php on line 38

Muchas gracias!
Gustavo.

[GatorV]

Perdon sería $datos->setConexion($conexion->getConexion());

[gus_anomaly]

ahhh, excelente... ahi funciona!
No me muestra los datos que esperaba de la query, seguro algo hice mal... pero ahora me pongo a ver que pasa yo... mi problema era lograr hacer que cierre el circuito... que lo esta!

Muchisimas gracias!
Gustavo.