Witam
mam problem z pewnym skryptem, a mianowicie w napisanym przez mojego kolege skrypcie na stronie html wyskakuja jakies dziwne bledy
ponizej zamieszczam zdjecie-to co zaznaczone na czerwowo to wlasnie tego chcialbym sie pozbyc

http://img4.imageshack.us/img4/529/uo0001co.jpg

ponizej zamieszczam kod tego skryptu:

[PHP] pobierz, plaintext 
<?php 
 
$r = $_POST['r']; //promien podstawy
$l = $_POST['l']; //dlugosc powierzchni bocznej
$h = $_POST['h']; //wysokosc stozka
 
if(!$r OR !$l OR !$h OR $r=="r" OR $h=="h" OR $l=="l") {
        echo 'Pole i objętosc stozka:<br /><br />
        r - promien podstawy<br />
        h - wysokosc stozka<br />
        l - dlugosc powierzchni bocznej<br />
        <form name="licz" action="index.php" method="post"><table cellspacing="0" cellpadding="0">
<tr>
   <td><input type="text" name="r" value="r" /></td>
</tr>
<tr>
   <td><input type="text" name="l" value="l" /></td>
</tr>
<tr>
   <td><input type="text" name="h" value="h" /></td>
</tr>
<tr>
         <td><input type="submit" value="Oblicz" /></td>
</tr>
</table></form>';
}
else echo 'Pole i objetosc stozka:<br /><br />
        r - promien podstawy<br />
        h - wysokosc stozka<br />
        l - dlugosc powierzchni bocznej<br />
        <form name="licz" action="index.php" method="post"><table cellspacing="0" cellpadding="0">
<tr>
   <td><input type="text" name="r" value="r" /></td>
</tr>
<tr>
   <td><input type="text" name="l" value="l" /></td>
</tr>
<tr>
   <td><input type="text" name="h" value="h" /></td>
</tr>
<tr>
         <td><input type="submit" value="Oblicz" /></td>
</tr>
</table></form>';
echo "Objetosc: ".(1/3)*($r*$r)*$h." &pi;<br />";
echo "Pole powierzchni bocznej: ".$r*$l." &pi;";
 
 
 ?>
[PHP] pobierz, plaintext 

jesli ktos wie co usunac lub wstawic do tego skryptu aby pozbyc sie tego co zaznaczylem na zdjeciu,to bede bardzo wdzieczny
see ya

Takimi notice'ami sie nie przejmuj i daj

[PHP] pobierz, plaintext 
error_reporting(0);
[PHP] pobierz, plaintext 

Podziękował

Oczywiscie jak chcesz wystarczy ze przed obrobieniem tych danych z $_POST dodasz if() z isset() na dane zmienne wtedy bledu nie powinno byc

Ja bym nie robił tego co zaproponował marcio. Uważam, że lepiej jest w każdym formularzu wstawić jeden input hidden np. o nazwie "form" i z wartością określającą formularz, np.

[HTML] pobierz, plaintext 
<input type="hidden" name="form" value="count">
[HTML] pobierz, plaintext 

Wtedy to, czy formularz został wysłany wystarczy sprawdzić przez

[PHP] pobierz, plaintext 
if(isset($_POST['form']) and $_POST['form'] === 'count')
[PHP] pobierz, plaintext 

Moim zdaniem można na tym etapie założyć, że formularz został wysłany prawidłowo i nie sprawdzać każdej zmiennej osobno. To, co ty zrobiłeś u siebie jest bez sensu. Najpierw przypisałeś wartości z $_POST do zmiennych, a potem sprawdzasz czy istnieją.

Po drugie zwróć uwagę, że spora część kodu powtarza się w obu przypadkach - wysłanego i niewysłanego formularza. Nie musisz wklejać tej wspólnej części dwa razy, wstaw ją poza if-else.

To juz o wiele lepiej jest sprawdzic czy zostal wyslany submit

po uruchomieniu- pierwszy raz- nie masz ustalonych danych post,
daj na poczatku

[PHP] pobierz, plaintext 
<?php
 
	  if(!isset($_POST['r'])){$_POST['r']=0;}else{}
	  if(!isset($_POST['l'])){$_POST['l']=0;}else{}
	  if(!isset($_POST['h'])){$_POST['h']=0;}else{}
 
 
 
      $r = $_POST['r']; //promien podstawy
 
 
[PHP] pobierz, plaintext 

i problem znika

celbarowicz -> po pierwsze nie powinno się przypisywać samemu czegokolwiek do $_POST. Po drugie, po co ten pusty else? Po trzecie, To że ktoś nie wpisał nic nie oznacza, że tam ma być zero. Lepiej wyświetlić formularz jeszcze raz i kazać wpisać te wartości.

croc jest sporo literatury ,które używają puste else, niekiedy jeśli nie użyjasz pustego else wyskakują błędy. ponadto dlaczego chcesz przeładowywać stronę i oglądać błędy skoro można tego uniknąć,

Programujesz od niedawna, prawda?

zaczynałem od fortranu 30 lat temu

To może 30 lat temu "wyskakiwały" błędy od braku else. Nie chcę podważać twoich kompetencji programistycznych, ale twoje rady są co najmniej krótkowzroczne. Pozbywanie się jednego problemu, żeby wywołać inny to nie jest dobry pomysł. Dane niewpisane to dane, których nie ma. Brak danych to nie zero. Jeżeli ktoś nie poda boku kwadratu, to nie znaczy, że go interesuje kwadrat o boku 0, tylko że się pomylił i należy mu to ładnie pokazać.