Funciones en get y set

[saumal20]

ola...espero ke me puedan ayudar.

Yo tengo estas dos funciones:
1.-

<?php

class Datos{
private $nombrehostConsultas;
private $nombreusuarioConsultas;
private $passusuarioConsultas;
private $nombrebdConsultas;

//Consntructor
function __construct(){

}

//Metodos get y set
function getnombrehostConsultas(){
return $this -> nombrehostConsultas = obtenerDatos(1);
//return obtenerDatos(1);
}


function getnombreusuarioConsultas(){
return $this -> nombreusuarioConsultas = obtenerDatos(2);
}


function getpassusuarioConsultas(){
return $this -> passusuarioConsultas = obtenerDatos(3);
}

function getnombrebdConsultas(){
return $this -> nombrebdConsultas = obtenerDatos(4);
}


function obtenerDatos($valor){
$archivo="..\xml\info.xml";

if(file_exists($archivo)) {
$xml = simplexml_load_file($archivo);
if($xml) {
foreach ($xml->dato as $dato) {
if($valor<1 || $valor>4){
echo 'Dato incorrecto';
}
if($valor==1){
return $dato->nombrehost;
}
if($valor==2){
return $dato->nombreusuario;
}
if($valor==3){
return $dato->passusuario;
}
if($valor==4){
return $dato->passusuario;
}
}
} else {
echo "Sintaxis XML invalida";
}
} else {
echo "Error abriendo $archivo";
}

}
?>

La 2da:
<?php
include 'Datos.php';

$a = new Datos();
echo $a ->getnombrehostConsultas();
//echo $a -> obtenerDatos(5);
?>

Bien como ven, yo intento llamar a getnombrehostonsultas() para asi obtener el valor. pero al momento de llamarla en el arhivos datos.php me lanza el error:

atal error: Call to undefined function obtenerdatos() in...

Algo estoy haciendo mal ...ojala puedan ayudarme

[spider_boy]

Solo veo que te falta una llave que cierra a la clase Datos.

[maturano]

- Por favor, cuando coloques código, hazlo con las etiquetas correspondientes para resaltado de sintaxis y así poderlo analizar mejor.
- Haz tu código leíble. Indéntalo adecuadamente realmente es una gran diferencia.
- Lo mismo, coloca el operador de visibilidad a tus métodos, así como con sus propiedades.
- Que mejor si siguieras un estándar de estilización de código (Zend, PEAR son los más populares).

- Tu problema es, como dice el error, querer utilizar una función no definida. Estás dentro de una clase, y ahí defines un método que estará asociado a la instancia del objeto. Nuevamente, así como lo haces con las propiedades, debes utilizar el operador $this para hacer la referencia al método del propio objeto.

- Por último, no es recomendable hacer la asignación y el retorno de un valor en una misma sentencia, se presta a errores y mal-interpretaciones de lectura de código; ¿regresas el resultado de la asignación o el valor de la variable?.


Todo resumido en que es diferente esto:

Código PHP:

class Datos{
private $nombrehostConsultas;
 
//Metodos get y set
function getnombrehostConsultas(){
return $this -> nombrehostConsultas = obtenerDatos(1);
//return obtenerDatos(1);
}
}
 

A esto (sin mencionar los comentarios):

Código PHP:

class Datos
{
    private $_nombreHostConsultas;
 
    public function getNombreHostConsultas()
    {
        // Tu error en concreto está aquí
        $this->_nombreHostConsultas = $this->obtenerDatos(1);
 
        return $this->_nombreHostConsultas;
    }
}
 

[GatorV]

Tema movido desde PHP a PHP orientado a objetos

[saumal20]

Iniciado por maturano

- Por favor, cuando coloques código, hazlo con las etiquetas correspondientes para resaltado de sintaxis y así poderlo analizar mejor.
- Haz tu código leíble. Indéntalo adecuadamente realmente es una gran diferencia.
- Lo mismo, coloca el operador de visibilidad a tus métodos, así como con sus propiedades.
- Que mejor si siguieras un estándar de estilización de código (Zend, PEAR son los más populares).

- Tu problema es, como dice el error, querer utilizar una función no definida. Estás dentro de una clase, y ahí defines un método que estará asociado a la instancia del objeto. Nuevamente, así como lo haces con las propiedades, debes utilizar el operador $this para hacer la referencia al método del propio objeto.

- Por último, no es recomendable hacer la asignación y el retorno de un valor en una misma sentencia, se presta a errores y mal-interpretaciones de lectura de código; ¿regresas el resultado de la asignación o el valor de la variable?.


Todo resumido en que es diferente esto:

Código PHP:

class Datos{
private $nombrehostConsultas;
 
//Metodos get y set
function getnombrehostConsultas(){
return $this -> nombrehostConsultas = obtenerDatos(1);
//return obtenerDatos(1);
}
}
 

A esto (sin mencionar los comentarios):

Código PHP:

class Datos
{
    private $_nombreHostConsultas;
 
    public function getNombreHostConsultas()
    {
        // Tu error en concreto está aquí
        $this->_nombreHostConsultas = $this->obtenerDatos(1);
 
        return $this->_nombreHostConsultas;
    }
}
 

oks...muxas graxias.Puse asi xq solo copie y pegue mi codigo y salio asi, mas bien pido disculpas...graxias nuevamente y saludos