Witam, dobrze napisałem ten skrypt? Ma on wyświetlić date i godzine, a gdy będzie 12.24 wyświetli info, że jest wigilja.

[PHP] pobierz, plaintext 
$dzien = date("d");
$miesiac = date("m");
$rok = date("Y");
$data = getdate(time());
print ("Info z dnia: <BR />");
if ($dzien == 24)
{
	if ($miesiac == 12)
	{
		print ("Dzisiaj jest $dzien $miesiac $rok <BR />");
		print ("Wigilia");
		printf ("%d:%d", $data["hours"], $data["minutes"]);
	}
}
else
{
	print ("Dzisiaj jest $dzien $miesiac $rok");
}
[PHP] pobierz, plaintext 

A dziala? Znaczy ze dobrze
Czy optymalnie? Srednio na jeza

przenosze

Rozwijając odpowiedź poprzednika, spokojnie ta częśc kodu:

[PHP] pobierz, plaintext 
if ($dzien == 24)
{
	if ($miesiac == 12)
	{
		print ("Dzisiaj jest $dzien $miesiac $rok <BR />");
		print ("Wigilia");
		printf ("%d:%d", $data["hours"], $data["minutes"]);
	}
}
else
{
	print ("Dzisiaj jest $dzien $miesiac $rok");
}
[PHP] pobierz, plaintext 

może byc uproszczona np. tak:

[PHP] pobierz, plaintext 
print ("Dzisiaj jest $dzien $miesiac $rok <BR />");
if ($dzien == 24 && $miesiac == 12)
{
		print ("Wigilia");
		printf ("%d:%d", $data["hours"], $data["minutes"]);
}
[PHP] pobierz, plaintext 

To tak na dobry początek.

A teraz masz mindfucka ode mnie

[PHP] pobierz, plaintext 
$teraz = new DateTime();
$wigilia = new DateTime('24 December');
 
echo 'Info z dnia: <BR />';
echo 'Dzisiaj jest ' . $teraz->format("Y-m-d");
if ( $teraz->diff($wigilia)->days == 0 ) {
	echo '<br />Wigilia<br />';
	echo $teraz->format("H:i") . '<br />';
}
[PHP] pobierz, plaintext 

@Turson poniżej - prawda, o jeden apostrof za dużo. Dzięki za zwrócenie uwagi. Poprawiłem.

@up
Syntax error