Witam
uczę się php z książki wykonywałm kolejne zadanie czy dany rok jest przestępny
oto skrypt ponoć w 7 linijce jest błąd jak go poprawić
proszę o pomoc

[PHP] pobierz, plaintext 
<?php
$answer =  is_leapyear(2000);
 
if($answer) {
	echo &#092;"rok 200 jest rokiem przestępnym
&#092;";
	} else {
		echo &#092;"rok 2000 nie jest rokiem przestępnym
&#092;";
		}
		/*Zastosowanie domyślnej wartości parametru */
		$answer = is_leapyear() ;
		if ($answer) {
			echo &#092;"rok 2004 jest rokiem przestępnym
&#092;";
} else {
echo &#092;"rok 2004 nie jest rokiem przestępnym
&#092;";
}
?>
[PHP] pobierz, plaintext 

[PHP] pobierz, plaintext 
<?php
$answer =  is_leapyear(2000);
 
if($answer) echo "rok 200 jest rokiem przestępnym"; 
else echo "rok 2000 nie jest rokiem przestępnym";
	   
 /*Zastosowanie domyślnej wartości parametru */
$answer = is_leapyear() ;
if ($answer) echo "rok 2004 jest rokiem przestępnym";
else
echo "rok 2004 nie jest rokiem przestępnym";
 
?>
[PHP] pobierz, plaintext 

~neverever jak już komuś poprawiasz to poprawiaj tak jak zwyczaje i standardy nakazują:

[PHP] pobierz, plaintext 
<?php
 
$answer =  is_leapyear(2000);
 
if ($answer) {
	echo "rok 200 jest rokiem przestępnym";
} else {
	echo "rok 2000 nie jest rokiem przestępnym";
}
	   
 /*Zastosowanie domyślnej wartości parametru */
$answer = is_leapyear() ;
 
if ($answer) {
	echo "rok 2004 jest rokiem przestępnym";
} else {
	echo "rok 2004 nie jest rokiem przestępnym";
}
 
?>
[PHP] pobierz, plaintext 

chyba źle :0
Chyba chodzi o to ,że tej funkcji nie mam ( dobrze rozumuje ?)

jak uruchamiam to na serwerze to wyskakuje
Fatal error: Call to undefined function is_leapyear() in /var/www/test/1.20.php on line 6

Tak, chodzi o to ,że taka funkcja nie istnieje

Zawsze myślałem że nawias jest konieczny TYLKO w przypadku gdy w warunku albo pętli występuje więcej niż jedno polecenie.
Jeżeli standardem jest wciskanie nawiasów wszędzie tam gdzie są one zwyczajnie zbędne to sory, nie wiedziałem o tym.
Będe zobowiązany za link do manuala albo innej strony na której jest opisany ten przypadek.

[PHP] pobierz, plaintext 
<?php
if ( function_exists('is_leapyear') ) {
	$answer = is_leapyear(2000);
	if ($answer == true)
		echo 'rok 2000 jest rokiem przestępnym';
	else
		echo 'rok 2000 nie jest rokiem przestępnym';
 
	 /*Zastosowanie domyślnej wartości parametru */
	$answer = is_leapyear();
	if ($answer == true)
		echo 'rok 2004 jest rokiem przestępnym';
	else
		echo 'rok 2004 nie jest rokiem przestępnym';
}
else
	echo 'Funkcja is_leapyear() nie istnieje.';
 
# teraz nie bedzie bledu :)
?>
[PHP] pobierz, plaintext 

Niech modek skasuje pow. przyklad bardzo prosze.

dziękuję bardzo
możecie tylko opisać co wstawiliście i do czego służy ?
bo ja świeżak jestem jeśli o php chodzi
P.S
tylko wyskakuje
Funkcja is_leapyear() nie istnieje.
a nie wynik

Jeśli jej nie napisałeś, to nic dziwnego że nie istnieje. Jeśli uważasz, że ją napisałeś, to sprawdź, czy w nazwie nie ma literówki.

a możesz mi pokazac jak to powiino byc dobrze napisane aby wszystko działało

Jasne, musisz gdzieś (przed wywołaniem funkcji) napisać:

[PHP] pobierz, plaintext 
<?php
function is_leapyear($year = 2004)
{
   // tutaj umieszczasz warunek sprawdzający czy rok jest przestępny czy nie
   // funkcja wywołana bez argumentów sprawdzi przestępność w roku 2004
}
?>
[PHP] pobierz, plaintext 

[PHP] pobierz, plaintext 
<?php
function is_leapyear($rok=NULL) {
	# true => zwykly
	# false => przestepny
	if ($rok >= 4) {
		if (is_float($rok / 4))
			return true;
		if (is_int($rok / 100)) {
			if (is_int($rok / 400))
				return false;
			return true;
		}
	}
	trigger_error('Krytyczny i w dodatku fatalny blad funkcji is_leapyear() <a href=\"http://php.pl\" target=\"_blank\">http://php.pl</a> ', E_USER_ERROR
}
?>
[PHP] pobierz, plaintext 

Mam nadzieje ze nie zrypalem tych warunkow, pozno jest

@up
Co prawda chciałam, żeby autor sam zrobił tą funkcję, ale w sumie albo ja coś mieszam z latami przestępnymi, albo wystarczy:

[PHP] pobierz, plaintext 
<?php
function is_leapyear($year = 2004)
{
	if(is_int($year) && $year % 4 === 0)
		return true;
	return false;
 
}
?>
[PHP] pobierz, plaintext 

Tyle że u mnie false jest w momencie nieprzestępnego roku, a true przestępnego.

2up

Rok jest zwykły jeśli nie dzieli się przez 4,
Rok jest przestępny jeśli dzieli się przez 4, chyba że ...
- ... dzieli się przez 100 - zwykły, chyba że
- ... dzieli się przez 400, jest wtedy przestępny.

wiec twoja funkcja jest nieco... sredniowieczna...

Faktycznie o tym nie pomyślałam, zapewne ze względu na późną porę, zwracam honor Imo jednak sprawdzanie modulo jest lepsze niż sprawdzanie czy liczba po podziale jest floatem czy intem.

Nie napisałem, że te nawiasy są konieczne, napisałem, że są standardem dobrego kodowania. Takie standardy nie są ujęte w dokumentacji.
Ale proszę bardzo:Code Conventions for the Java Programming Language
Identyczne wytyczne dobrych zwyczajów tyczą się wszystkich języków gdzie nawiasy klamrowe (curly braces lub brackets) służą do wydzielania bloków kodu i są ujęte we wszystkich konwencjach.
Zresztą przejrzyj sobie dowolne źródła jakiegoś otwartego projektu (ZendFramework, Symfony, ...) w większości zwyczaj ten jest stosowany, pomimo tego, że składania języka tego nie wymaga.

Oo, znalazłem kolejne:PHP coding guidelines :: Code layout :: Including braces