Mi primera clase, consejos

[neodani]

Buenas,

Estoy metiéndome en el mundo de la clases en PHP pero no se si voy bien o no...

Por ejemplo he creado esto para escribir mis propios ficheros de log

Código php:

<?
// Clase para escribir en ficheros
class RWfilelog {
    function abrir ($fichero,$modo) {
    $this -> abrir = fopen($fichero,$modo);
    }
    function escribir($texto) {
    $this -> escribir = fputs($this->abrir,date('Y-m-d H:m:s').' '.$texto."\r\n");
    }
    function cerrar() {
    $this -> cerrar = fclose($this -> abrir);
    }
}
// Usando la clase
$fichero = new RWfilelog; //instanciar
$fichero -> abrir('prueba.log','a');
$fichero -> escribir('Esta es una linea');
$fichero -> escribir('segunda linea');
$fichero -> escribir('Escribiendo la fecha');
$fichero -> cerrar();
 
?>
 

¿La veis bien? funcionar funciona.

Pero no se si estas hace falta hacerlo así
$this -> cerrar = fclose($this -> abrir);

o con esto ya vale
fclose($this -> abrir);

Y si tengo que definir variables al inicio de la clase :S

Muchas gracias de antemano!

[MarioAraque]

Siempre que declares una clase, segun lo que hagas, te conviene guardar variables.

Aparte tenes que crear un constructor. Osea, cuando vos definis una clase, seguro que es para abrirla (fopen) y luego trabajar con ella, y cerrarlo, no? Entonces porque mejor no abrir el archivo cuando declaras la clase? Y te ahorras la linea $fichero->abrir()...

Si haces lo q vos escribiste, al cerrar el archivo lo volves a abrir, por lo que no va a hacer lo que vos quieras, aparte abrir tiene 2 parametros, por lo que te va a tirar un error de sintaxis.

Te conviene guardar la ruta del archivo en una variable, declarala dentro de la clase.

private $_nombre;

Y cuando hagas el constructor, asignasela!

Saludos.

[pateketrueke]

pues, tu código esta bien...

claro, no siempre hay que asignar todo a variables, al menos si no tienen un uso posterior...

y si, definitivamente deberias definir las variables como privadas o protegidas en la clase, sería lo ideal...

suerte!!

[GatorV]

Por otro lado recuerda que en PHP5, que es la versión actual de PHP es necesario definir el acceso de las funciones, si no las defines se toman como publicar por defecto.

[neodani]

Iniciado por GatorV

Por otro lado recuerda que en PHP5, que es la versión actual de PHP es necesario definir el acceso de las funciones, si no las defines se toman como publicar por defecto.

A qué se refiere cuando "es necesario definir el acceso de las funciones"

Muchas gracias

[MarioAraque]

El se refiere a que tenes que declarar el alcance de las funciones. Pueden ser publicas (public), protegidas (protected) o privadas (private).

Vos no declaraste el alcance de las funciones, por lo que todas se asumen q son publicas, no creo q eso cause ningun problema, pero creo que deberias buscar por internet una definicion de clase simple ... porque te falta definir constructores y eso.

Saludos.

[neodani]

Iniciado por MarioAraque

El se refiere a que tenes que declarar el alcance de las funciones. Pueden ser publicas (public), protegidas (protected) o privadas (private).

Vos no declaraste el alcance de las funciones, por lo que todas se asumen q son publicas, no creo q eso cause ningun problema, pero creo que deberias buscar por internet una definicion de clase simple ... porque te falta definir constructores y eso.

Saludos.

De acuerdo, Gracias ;)