[SOLUCIONADO] PHP OO Ejecicio clase Fecha

[gonzaherrera77]

Buenas comunidad, tengo el siguiente enunciado.

Crea una clase Fecha con atributos para el día, el mes y el año de la fecha.
Incluye, al menos, los siguientes métodos:

1. Constructor predeterminado con el 1-1-1900 como fecha por defecto.
2. Constructor parametrizado con día, mes y año.
3. leer() : pedirá al usuario el día (1 a 31), el mes (1 a 12) y el año (1900 a 2050).
4. bisiesto() : indicará si el año de la fecha es bisiesto o no.
5. diasMes(int) : devolverá el número de días del mes que se le indique (para el año de la fecha).
6. valida() : comprobará si la fecha es correcta (entre el 1-1-1900 y el 31-12-2050); si el día no es correcto, lo pondrá a 1; si el mes no es correcto, lo pondrá a 1; y si el año no es correcto, lo pondrá a 1900. Será un método auxiliar (privado). Este método se llamará en el constructor parametrizado y en leer() .
7. diasTranscurridos() : devolverá el número de días transcurridos desde el 1-1-1900 hasta la fecha.
8. diaSemana() : devolverá el día de la semana de la fecha (0 para domingo, ..., 6 para sábado). El 1-1-1900 fue domingo.
9. siguiente() : pasará al día siguiente.
10. anterior() : pasará al día anterior.

Esta seria la clase

Código PHP:

<?php
class Fecha {
    private $day;
    private $month;
    private $year;
    
    public function __construct($day = 1, $month = 1, $year = 1900) {
        $this->day = $day;
        $this->month = $month;
        $this->year = $year;
    }
    
    public function leer($day, $month, $year) {
        if($this->validarFecha($day, $month, $year)) {
            $this->day = $day;
            $this->month = $month;
            $this->year = $year;
        }
        else {
            echo 'La fecha no es valida!!!';
        }
    }
    
    private function validarFecha($day, $month, $year) {
        if($day > 0 && $day < 32) {
            $dayOk = true;
        }
        else {
            $dayOk = false;
        }
        
        if($month > 0 && $month < 13) {
            $monthOk = true;
        }
        else {
            $monthOk = false;
        }
        
        if($year >= 1900 && $year <= 2050) {
            $yearOk = true;
        }
        else {
            $yearOk = false;
        }
        
        if($dayOk && $monthOk && $yearOk) {
             return true;
        }
        else {
            return false;
        }
    }
    
    public function esBisiesto() {
        if($this->year % 4 == 0  && ($this->year % 100 != 0 || $this->year % 400 == 0)) {
            echo 'Es Bisiesto<br />';
        }
        else {
            echo 'No es Bisiesto<br />';
        }
    }
    
    public function diaMes($month) {
        $fecha = "$this->day-$month-$this->year";
        $timestamp = strtotime($fecha);
        $diaMes = date('t', $timestamp);
        echo 'El mes '.$month.'  tiene '.$diaMes.' dia/s';
    }
    
    public function diasTranscurridos() {
     
       $diasDefault = gregoriantojd(1, 1, 1900);
       
       $fechaActual = explode('-', date('j-n-Y'));
       $diasActual = gregoriantojd($fechaActual[1], $fechaActual[0], $fechaActual[2]);
       
       $diasTranscurridos = $diasActual - $diasDefault;
       
       echo 'Dias trancurridos de la fecha 1-1-1900 hasta la fecha actual '.$diasTranscurridos.' diasz<br />';
 
    }
    
    public function diaSemana() {
        $diaSemana = gregoriantojd(1, 1, 1900);
        $diaSemana = jddayofweek($diaSemana, 0);
        
        echo 'El dia de la fecha 1-1-1900 es '.$diaSemana;
    }
    
    public function siguiente() {
        $this->day++;
    }
    
    public function anterior() {
        $this->day--;
    }
}
 
$fecha = new Fecha();
$fecha->leer(1,1,2014);
$fecha->diasTranscurridos();
$fecha->diaSemana();
$fecha->anterior();
?>
 

En el metodo diasTranscurridos() tuve que hacer una conversión de gregoriano a juliano, por es menor que la fecha del timestamp.

Al igual que al metodo diaSemana().

Queria saber como plantiarian esta misma clase algunos de ustedes ya que yo recien estoy empezando y debe tener algunos errores. Tambien alas propiedades que estoy usando adentro de la clase como son $diaSemana, $diasDefault, $fechaActual, ect. Se hacen propiedades publicas?

¡Saludos!

[mortiprogramador]

Saludo
Pues en la función diaMes, pondría por default algún mes,
y llamaría a la función de validación de fecha en el constructor (como en el requerimiento)

En cuanto a la conversión, sí es mejor, porque curiosamente haciendo esto
date('Y',mktime(0,0,0,1,1,1900));
devuelve 1970, ¿será un bug de php o falta algo más?
(Y sobra la z en la respuesta del método)

Finalmente, en las funciones de siguiente y anterior,
usaria $this->day + 1 y $this->day - 1 respectivamente.

Esto porque sí se deja como está actualmente, al llamar a siguiente(),
mostraría la misma fecha, pues se está incrementando luego de mostrar la fecha,
debería ser antes, pero al estar trabajando con el valor de $this->day directamente
al llamar a anterior() se obtendrá el valor de siguiente() por la misma razón,
se está decrementando después de mostrarse.

Esto se solucionaría haciendo las operaciones antes, pero sí por algún motivo
se llama primero a anterior() y luego a siguiente() los resultados van a cambiar
por las operaciones sobre el atributo de la clase.

[gonzaherrera77]

Muchas gracias por su tiempo voy hacer los cambios mencionados, y las variables que estoy usando en la clase Fecha no hay ningun problema si no la declaro con el metodo private, public o protected?

¡Saludos!

[mortiprogramador]

Saludo.
Pues las variables están declaradas como private.
Sí se hace referencia a las otras variables que están
siendo usadas como auxiliares (Por ej dayOk),
sí posteriormente se hará algo con ellas,
lo mejor sería declararlas como privadas y accederlas
por un método.

Finalmente, en la función de validarFecha se podría omitir
recibir los valores por la función, y hacer uso de los atributos de la clase,
pues tanto las variables como esta función son privadas.
(Esto haría que la función leer asignara primero los valores)

[gonzaherrera77]

Saludo.
Pues las variables están declaradas como private.
Sí se hace referencia a las otras variables que están
siendo usadas como auxiliares (Por ej dayOk),
sí posteriormente se hará algo con ellas,
lo mejor sería declararlas como privadas y accederlas
por un método.

Finalmente, en la función de validarFecha se podría omitir
recibir los valores por la función, y hacer uso de los atributos de la clase,
pues tanto las variables como esta función son privadas.
(Esto haría que la función leer asignara primero los valores)

Podria ser asi? o no estoy entendiendo vien el concepto.

Código PHP:

  <?php
public function leer($day, $month, $year) {
   $this->day = $day;
   $this->month = $month;
   $this->year= $year;
 
   $this->validarFecha();
}
 
public function validarFecha() {
   if($this->day > 0 && $this->day <=31) {
         $dayOk = true;
   }
   else {
         $dayOk = false;
   }
   if($this->month > 0 && $this->month <=12) {
         $monthOk = true;
   }
   else {
         $monthOk = false;
   }
   if($this->year > 0 && $this->year <=12) {
         $yearOk = true;
   }
   else {
         $yearOk = false;
   }
 
   if(!$dayOk, !$monthOk, !$yearOk) {
      echo 'Fecha no valida...';
 
      $this->day = 1;
      $this->month = 1;
      $this->year = 1900;
   }
}
?>   

¡Saludos!

[mortiprogramador]

Saludo.
Sí, así tal cual.

[gonzaherrera77]

Gracias por todos tus consejos!!!

¡Saludos!