[PHP]Dlaczego w linijce 6 jest bład, Proszę o pomoc prz naprawnieniu go ... Opcje

[efect1]

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 

Ten post edytował Piniek 23.07.2008, 22:13:52

Powód edycji: poprawiłem bbcode // Dodałem tag

[neverever]

[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 

[mike]

~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 

[efect1]

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

Ten post edytował efect1 23.07.2008, 21:51:26

[Xniver]

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

[neverever]

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

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.

[-gox-]

[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 (IMG:http://forum.php.pl/style_emoticons/default/smile.gif) bardzo prosze.

[efect1]

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 (IMG:http://forum.php.pl/style_emoticons/default/smile.gif)
P.S
tylko wyskakuje
Funkcja is_leapyear() nie istnieje.
a nie wynik (IMG:http://forum.php.pl/style_emoticons/default/sad.gif)

Ten post edytował efect1 23.07.2008, 22:24:16

[Shili]

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.

[efect1]

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

Ten post edytował efect1 23.07.2008, 22:42:53

[Shili]

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 

[-gox-]

[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> (IMG:http://forum.php.pl/style_emoticons/default/tongue.gif) ', E_USER_ERROR
}
?>
[PHP] pobierz, plaintext 

Mam nadzieje ze nie zrypalem tych warunkow, pozno jest (IMG:http://forum.php.pl/style_emoticons/default/smile.gif)

[Shili]

@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.

Ten post edytował Shili 23.07.2008, 22:52:18

[-gox-]

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...

[Shili]

Faktycznie o tym nie pomyślałam, zapewne ze względu na późną porę, zwracam honor (IMG:http://forum.php.pl/style_emoticons/default/winksmiley.jpg) Imo jednak sprawdzanie modulo jest lepsze niż sprawdzanie czy liczba po podziale jest floatem czy intem.

[mike]

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.

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:

Braces are used around all statements, even single statements, when they are part of a control structure

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:

Braces should always be included when writing code using if, for, while etc. blocks. There are no exceptions to this rule, even if the braces could be omitted. Leaving out braces makes code harder to maintain in the future and can also cause bugs that are very difficult to track down.

PHP coding guidelines :: Code layout :: Including braces