[PHP] Wyślwietlanie daty Opcje

[Bieniu]

Witam

Prosiłbym o sprawdzenie gdzie mam błąd w kodzie

[PHP] pobierz, plaintext 
<?php
 
class Data
{
  private $dzien;
  private $miesiac;
  private $rok;
 
  private function Walidacja()
  {
    if($this->dzien <= 1 AND $this->dzien >= 31)
    {
      echo "this->dzien - Nieprawidłowy numer dnia<br/>";
      return false;
    }
    else
    {
      switch($this->miesiac)
      {
        case 1:
          if($this->rok / 2 > 0 OR $this->dzien < 29)
          {
            echo "Luty w roku przestępnym może mieć tylko 29 dni<br/>";
            return false;
	  }
	  elseif($this->rok / 2 != 0 OR $this->dzien >= 28)
	  {
	    echo "Luty w roku NIE przestępnym może mieć tylko 28 dni<br/>";
	    return true;
	  }
	  break;
 
	case 4:
	case 6:
	case 9:
	case 12:
          if($this->dzien = 30)
          {
            echo "Miesiąc MiesiacNaTekst() ma tylko 30 dni!<br/>";
            return false;
          }
          break;
      }
    }        
 
    if($this->miesiac <= 1 AND $this->miesiac >= 12)
    {
      echo "this->miesiac - Nieprawidłowy numer miesiąca<br/>";
      return true;
    }
 
    if($this->rok <= 2010)
    {
      echo "this->rok - Uwzględniany jest tylko rok począwszy od 2010<br/>";
      return true;
    }
 
    return true;
  }
 
  private function MiesiacNaTekst()
  {
    switch($this->miesiac)
    {
      case "1": return styczeń; break;
      case "2": return luty; break;
      case "3": return marzec; break;
      case "4": return kwiecień; break;
      case "5": return maj; break;
      case "6": return czerwiec; break;
      case "7": return lipiec; break;
      case "8": return sierpień; break;
      case "9": return wrzesień; break;
      case "10": return paĽdziernik; break;
      case "11": return listopad; break;
      case "12": return grudzień; break;
    }
  }
 
  public function WyswietlDane()
  {
    if($this->Walidacja())
    {
      echo "dzień = $this->dzien, miesiąc = MiesiacNaTekst(), rok = $this->rok<br/>";
      return true;
    }
    else
      return false;
  }
 
  public function __construct($dzien, $miesiac, $rok)
  {
    $this->dzien = $dzien;
    $this->miesiac = $miesiac;
    $this->rok = $rok;
  }
}
 
class DataNotatka extends Data
{
  private $notatka;
 
  public function __construct($dzien, $miesiac, $rok, $notatka)
  {
    parent::__construct($dzien, $miesiac, $rok);
    $this->notatka = $notatka;
  }
 
  public function WyswietlDane()
  {
    if(parent::WyswietlDane())
      echo "<b>this->notatka</b><br/>";
    echo "<hr/>";
  }
}
 
$data1 = new DataNotatka(37,12,2011,"Urodziny Basi");
$data2 = new DataNotatka(17,13,2011,"Urodziny Kasi");
$data3 = new DataNotatka(16,7,2008,"Urodziny Marysi");
$data4 = new DataNotatka(29,2,2011,"Urodziny Krysi");
$data5 = new DataNotatka(29,2,2012,"Urodziny Zosi");
$data6 = new DataNotatka(31,6,2011,"Urodziny Mieci");
$data7 = new DataNotatka(18,1,2012,"Imieniny Monisi");
 
$data1->WyswietlDane();
$data2->WyswietlDane();
$data3->WyswietlDane();
$data4->WyswietlDane();
$data5->WyswietlDane();
$data6->WyswietlDane();
$data7->WyswietlDane();
 
?>
[PHP] pobierz, plaintext 

[erix]

I co się niby ma robić?

[lobopol]

I jaki błąd?

[Bieniu]

Ma wyświetlać datę daną numerycznie
(dzień, miesiąc, rok) oraz powiązać z tą datą notatki tekstowe.

Dodatkowo, przed wyświetleniem daty należy wykonać procedurę sprawdzenia (walidacji) tej daty
- true jeśli data jest OK, false jeśli data jest zła.

[vokiel]

@erix może to upgrade dla checkdate (IMG:style_emoticons/default/wink.gif)

@Bieniu błąd masz przede wszystkim projektowy. Poza tym:

• Nieprawidłowy operator w warunku w linii 11 (powinno być OR)
• Sprawdzanie roku przestępnego (nie każdy z parzystą ostatnią liczbą jest przestępny - rok 1900 nie był i 2100 też nie będzie)
• Wyświetlanie komunikatów przez echo
• Niepotrzebne użycie case
• Na twardo wpisane warunki (np na rok 2010)

[lobopol]

No i co nie działa, nie będziemy przecież na darmo analizować całego kodu, wysil się trochę.