Forum PHP.pl > [PHP] Błędny wynik skryptu php

Pomoc - Szukaj - Użytkownicy - Kalendarz

Pełna wersja: [PHP] Błędny wynik skryptu php

Dym71

24.01.2010, 13:21:27

Witam Was wszystkich

Jestem tu nowy, przed chwilą się zarejestrowałem ponieważ mam problem z PHP.

Mam za zadanie zsumować kolejne wyrażenia postaci 1/n (gdzie n=1,2,3,…). Sumowanie należy zakończyć gdy różnica dwóch ostatnio dodanych składników będzie mniejsza od podanej z formularza liczby (np. 0,0001).

Napisałem kod ale gdzieś jest błąd (możliwe że wszędzie dopiero zaczynam pisać w php

) ponieważ jakiej liczby (różnicy) bym nie podał suma wychodzi mi zero.

[PHP] pobierz, plaintext 
<html>
<head>
<meta http-equiv="Content-Type" content="text/html; charset=iso-8859-2">
</head>
<body alink="#000099" bgcolor="#000000" dir="ltr" link="#000099" text="#ffffff" vlink="#990099">
<?php
$z=$_GET['z'];
$p=-1;
$suma=0;
for($i=1;$i==$p;++$i)
{
$pam=1/i;
$suma=$suma+$pam;
$pom=1/(i+1);
$roz=$pam-$pom;
if ($roz<$z)
{
$i=-1;
};
};
echo "Suma wynosi $suma";
?>
</body>
</html>
[PHP] pobierz, plaintext

Z góry dzięki za pomoc

qrzysztof

24.01.2010, 13:27:07

Dzieje się tak bo nie wygląda na to żebyś do tej pętli w ogóle wchodził.

[PHP] pobierz, plaintext 
for($i=1;$i==$p;++$i)
[PHP] pobierz, plaintext

$p jest równe -1 a $i nigdy nie będzie - więc pętla ani razu się nie wykona.

A w ogóle to brakuje $ w paru miejscach, np jest i zamiast $i. Nie wiem czy przepisywałeś to czy kopiowałeś?

Volume

24.01.2010, 13:29:05

Poza tym co napisal qrzysztof to w petli przy "i" brakuje Ci znaku $ czyli ma byc nie "i" tylko zmienna $i

Dym71

24.01.2010, 13:30:10

Cytat(qrzysztof @ 24.01.2010, 13:25:24 )

Dzieje się tak bo nie wygląda na to żebyś do tej pętli w ogóle wchodził.

[PHP] pobierz, plaintext 
for($i=1;$i==$p;++$i)
[PHP] pobierz, plaintext

Hmm, właśnie próbowałem jeszcze takie coś, testowe do 1000 i też mi nie działa

[PHP] pobierz, plaintext 
<html>
<head>
<meta http-equiv="Content-Type" content="text/html; charset=iso-8859-2">
</head>
<body alink="#000099" bgcolor="#000000" dir="ltr" link="#000099" text="#ffffff" vlink="#990099">
<?php
$z=$_GET['z'];
$p=1000;
$suma=0;
for($i=1;$i==$p;++$i)
{
$pam=1/i;
$suma=$suma+$pam;
$pom=1/(i+1);
$roz=$pam-$pom;
if ($roz<$z)
{
$i=1000;
};
};
echo "Suma wynosi $suma";
?>
</body>
</html>
[PHP] pobierz, plaintext

Teraz powinno działać ale niestety nie chodzi.

Aha to może przez to nie działa

Dzięki zaraz sprawdzę :*

qrzysztof

24.01.2010, 13:32:28

To przypisanie $i=-1 musiałbyś gdzieś poza pętlą na przykład umieścić.

Przy $i=1000 też nie zadziała - bo to wszystko jest w pętli for - a pętla for ani razu się nie wykonuje.

I zanim sprawdzisz - popraw jeszcze błąd, o którym napisał Volume.

Dym71

24.01.2010, 13:45:29

Cytat(qrzysztof @ 24.01.2010, 13:32:28 )

Poprawiłem błąd który wskazał mi Volume - dzięki.

No niestety, chyba źle rozumiem składnię pętli.
Skoro $i=1 a $p=1000 to bez modyfikacji $i w środku pętli pętla powinna wykonać się 999 razy (albo 1000 już nie pamiętam

) czy tak?
Jeżeli tak to skoro mam umieścić $i=-1 lub $i=1000 zależnie od wersji poza pętlą to jak mam ją kontrolować/przerwać (tzn. chodzi mi o przerwanie jej działania w odpowiednim momencie)?

qrzysztof

24.01.2010, 13:49:13

Jeśli chciałbyś żeby wykonała się 999 razy to musiałbyś mieć warunek $i<$p (czyli wykonuj pętle dopóki i$<$p). A masz tam równość (czyli wykonuj pętlę dopóki $i jest równe $p). A więc jeśli nie jest równe to nic się nie wykona ani razu.

Volume

24.01.2010, 13:57:36

Cytat

No niestety, chyba źle rozumiem składnię pętli.
Skoro $i=1 a $p=1000 to bez modyfikacji $i w środku pętli pętla powinna wykonać się 999 razy (albo 1000 już nie pamiętam ) czy tak?
Jeżeli tak to skoro mam umieścić $i=-1 lub $i=1000 zależnie od wersji poza pętlą to jak mam ją kontrolować/przerwać (tzn. chodzi mi o przerwanie jej działania w odpowiednim momencie)?

Nie bradzo o to chodzi

[PHP] pobierz, plaintext 
$p=1000;
$suma=0;
for($i=1;$i==$p;++$i)
{...
[PHP] pobierz, plaintext

pierwsze wyrazenie w petli for mowi od jakiej wartosci ma zaczac pozniejsze pokazwyanie wyniku w petli, drugie wyrazenie ile ma byc wykonanych powtorzen ale w tej postaci co Ty napisales to jest w ogole tu bez sensu gdyz ten opertor == jest to operator porownania (np. $a==$b znaczy ze prawda jeśli $a jest równe $b). W przypadku gdy np bys to zapisal w psotaci $i=$p byloby juz bardziej sensownie - ale wtedy petla by sie wykonala nieskonczona ilosc razy - dlatego trzeba jej dac do zrozumienia zeby w koncu sie zatrzymala dlatego nalezy dac zapis np. $i<$p czyli dla Twojego przykladu petla wykona sie 1000 razy jesli $p bedzi erowne 10 to 10 razy (oczywiscie wzgledem pierwszego wyrazenia jesli byloby rowne 0 ($i=0)) itd. Natomiast 3 wyrazenie mowi o ile ma sie zwiekszac wynik w petli. Takze kombinuj.

Obadaj podstawy: http://phpkurs.pl/operatory/ i http://phpkurs.pl/struktury/#struktury.for

Dym71

24.01.2010, 14:46:19

Kurcze, próbowałem z tą pętlą for i wydaje mi się że rozumiem piszę i niestety albo zero(nie przechodzi ani raz) albo jeden(przechodzi tylko raz).

Poczytałem tamte kursy i stwierdziłem że spróbuję z pętlą while.
Jest ona wykorzystywana w sytuacjach, kiedy niezbędne jest wykonywanie jakiejś operacji dopóki nie zostanie spełniony warunek.
Czyli:

[PHP] pobierz, plaintext 
<html>
<head>
<meta http-equiv="Content-Type" content="text/html; charset=iso-8859-2">
</head>
<body alink="#000099" bgcolor="#000000" dir="ltr" link="#000099" text="#ffffff" vlink="#990099">
<?php
$z=$_GET['z']; //wczytuję sobie $z z formularza
$q=0; //ustawiam $q oraz $i i zeruję sumę-profilaktycznie
$i=1;
$suma=0;
while($q<1) //pętla wykonuje się dopóki $q jest mniejsze od 1
{
$pam=1/$i; //zapamiętuje 1/1
$suma=$suma+$pam; //sumuję wynik
$pom=1/($i+1); //przewiduje następną wartość wyrażenia 1/n czyli 1/2=0,5
$roz=$pam-$pom; //liczę różnicę między 1/n1 a 1/n2 czyli 1/1 i 1/2 dla pierwszego przejścia przez pętlę
if ($roz<$z) //różnica wynosi 0,5 więc porównuję ją z różnicą wpisaną przez użytkownika, powiedzmy wpisałem 0,6 czyli różnica jest mniejsza więc $q=2 co powinno przerwać pętle
{
$q=2;
};
++$i; //zwiększam $i
};
echo "Suma wynosi $suma"; //Tu powinna wyświetlić się suma lecz po kliknięciu ok wszystko się wiesza tak jakbym zapętlił pętlę
?>
</body>
</html>
[PHP] pobierz, plaintext

Napisałem jak ja to rozumiem. Jeżeli źle bo np. w pętli trzeba zastosować inkrementację dla $q to ja nie wiem jak to zrobić. Jakbyście mogli przerobić ten kawałek kodu żeby mi działało to bym był bardzo wdzięczny i może bym zakumał. Nie wiem jakbym miał się zrewanżować

Jak ktoś mieszka we Wrocławiu to mogę piwo postawić

emtiej

24.01.2010, 14:51:56

Próbowałem to zrobić w taki sposób:

[PHP] pobierz, plaintext 
 
      <html>
      <head>
      <meta http-equiv="Content-Type" content="text/html; charset=iso-8859-2">
      </head>
      <body alink="#000099" bgcolor="#000000" dir="ltr" link="#000099" text="#ffffff" vlink="#990099">
      <?php
 
	  $liczba_podana = $_GET['liczba'];
	  $suma = 0;
	  $koniec = 0;
 
	  	for($i=1;$i>=$koniec;$i++)
		{
		echo "Wykonanie";
			$skl1 = 1/$i; //ostatni "skladnik"
			if($i!=1) // zabezpieczenie przed dzieleniem przez 0
			{
				$skl2 = 1/$i-1;  //przedostatni "skladnik"
			}
			if($i==1)
			{
				$skl2 = 1;
			}
			$suma=$suma+$skl1;
			if(($skl2-$skl1)<$liczba_podana)
			{
				break;
			}
		}
 
      echo "Suma wynosi $suma";
      ?>
      </body>
      </html>
 
[PHP] pobierz, plaintext

Coś tam zaczyna liczyć, ale przerywa na 1,5 nie wiem dlaczego, wykonuje się tylko 3 razy, dopracuj, przypatrz się może to ci pomoże, ja muszę lecieć i nie mam czasu go dokończyć

Zrobiłem to na pętli nieskończonej a w środku jest instrukcja break która przerywa jej wykonanie, kiedy warunek się spełni.

Szeszek1992

24.01.2010, 15:03:39

[PHP] pobierz, plaintext 
<?php
$dokladnosc=0.2;
$s2=1;
$suma=0;
$i=1;
$roznica=1.00000000001;//Tak na wszelki wypadek, jakby ktoś dał dokładność >1;)
while($roznica>$dokladnosc)
	{
	$suma=$suma+$s2;
	$s1=$s2;
	$i++;
	$s2=1/$i;
 
	$roznica=$s1-$s2;
	}
	echo $suma;
?>
[PHP] pobierz, plaintext

Formatowanie html'owskie zrób wg własnego uznania.

Dym71

24.01.2010, 15:28:23

Cytat(Szeszek1992 @ 24.01.2010, 15:03:39 )

[PHP] pobierz, plaintext 
<?php
$dokladnosc=0.2;
$s2=1;
$suma=0;
$i=1;
$roznica=1.00000000001;//Tak na wszelki wypadek, jakby ktoś dał dokładność >1;)
while($roznica>$dokladnosc)
	{
	$suma=$suma+$s2;
	$s1=$s2;
	$i++;
	$s2=1/$i;
 
	$roznica=$s1-$s2;
	}
	echo $suma;
?>
[PHP] pobierz, plaintext

Formatowanie html'owskie zrób wg własnego uznania.

No dzięki wielkie

Po przeczytaniu i zrozumieniu powinno śmigać, ale u mnie nie działa i może coś nie tak mam z serwerem chociaż inne skrypty działają

Mam identycznie tylko że dokładność wczytuję z formularza jako $z i niestety po kliknięciu ładuje się, myśli i myśli i nic.
Testowałeś to u siebie??

Kod:

[PHP] pobierz, plaintext 
<?php
$z=$_GET['z'];
$s2=1;
$i=1;
$suma=0;
$roznica=100;
while($roznica>$z)
{
$suma=$suma+$s2;
$s1=$s2;
$i++;
$s2=1/$i;
$roznica=$s1-$s2;
}
echo "Suma wynosi $suma";
?>
[PHP] pobierz, plaintext

Szeszek1992

24.01.2010, 15:38:31

[PHP] pobierz, plaintext 
<?php
echo '<form action="index.php" method="post"><input type="text" name="z"><input type="submit" value="Wyślij"></form><br>';
if(isset($_POST['z']))
{
if($_POST['z']>0)
{
$dokladnosc=$_POST['z'];
$s2=1;
$suma=0;
$i=1;
$roznica=1.00000000001;//Tak na wszelki wypadek, jakby ktoś dał dokładność >1;)
while($roznica>$dokladnosc)
	{
	$suma=$suma+$s2;
	$s1=$s2;
	$i++;
	$s2=1/$i;
 
	$roznica=$s1-$s2;
	}
	echo $suma;
}else echo 'Nieprawidłowe dane';//Zapętliłoby się, jeśli ktoś dałby 0	
}
?>
[PHP] pobierz, plaintext

Dorobiłem formularz do swego kodu;) Działa na pewno, testowałem. Nie zapomnij, że w PHP część ułamkową oddzielamy kropką, a nie przecinkiem.

PS $dokladnosc możesz zamienić na $z

Dym71

24.01.2010, 16:10:11

Cytat(Szeszek1992 @ 24.01.2010, 15:38:31 )

[PHP] pobierz, plaintext 
<?php
echo '<form action="index.php" method="post"><input type="text" name="z"><input type="submit" value="Wyślij"></form><br>';
if(isset($_POST['z']))
{
if($_POST['z']>0)
{
$dokladnosc=$_POST['z'];
$s2=1;
$suma=0;
$i=1;
$roznica=1.00000000001;//Tak na wszelki wypadek, jakby ktoś dał dokładność >1;)
while($roznica>$dokladnosc)
	{
	$suma=$suma+$s2;
	$s1=$s2;
	$i++;
	$s2=1/$i;
 
	$roznica=$s1-$s2;
	}
	echo $suma;
}else echo 'Nieprawidłowe dane';//Zapętliłoby się, jeśli ktoś dałby 0	
}
?>
[PHP] pobierz, plaintext

Dorobiłem formularz do swego kodu;) Działa na pewno, testowałem. Nie zapomnij, że w PHP część ułamkową oddzielamy kropką, a nie przecinkiem.

PS $dokladnosc możesz zamienić na $z

Dzięki za wszystko, niestety u mnie nie działa a próbowałem na krasnalu, webserw i easyphp. W identycznej formie po wpisaniu liczby i kliknięciu ok pojawia mi się nowy formularz i nic więcej a gdy do niego wpiszę i kliknę wyślij to jest pusto - nie ma nic.

po usunięciu 2 linijki kodu:

[PHP] pobierz, plaintext 
echo '<form action="index.php" method="post"><input type="text" name="z"><input type="submit" value="Wyślij"></form><br>';
[PHP] pobierz, plaintext

i wpisaniu liczby w mój formularz i kliknięciu ok wszystko znika (to samo tylko że nie ma 2 formularzu).

Nie wiem w czym rzecz. Może jakieś złe ustawienie PHP, opcja register_globals ma być aktywna?

emtiej

24.01.2010, 16:59:47

Skrypt śmiga, register_globals dla bezpieczeństwa wszędzie powinna być na OFF ustawiona z powodów bezpieczeństwa, możliwe że twoje ustawienie PHP nie daje możliwości używania skróconych nazw $_POST[''], spróbuj $HTTP_POST_VARS[''];

Dla testu cały kod prócz formularza weź sobie w komentarz, mianowicie:

[PHP] pobierz, plaintext 
      <?php
      echo '<form action="index.php" method="post"><input type="text" name="z"><input type="submit" value="Wyślij"></form><br>';
      /*    
       if(isset($_POST['z']))
      {
      if($_POST['z']>0)
      {
      $dokladnosc=$_POST['z'];
      $s2=1;
      $suma=0;
      $i=1;
      $roznica=1.00000000001;//Tak na wszelki wypadek, jakby ktoś dał dokładność >1;)
      while($roznica>$dokladnosc)
      {
      $suma=$suma+$s2;
      $s1=$s2;
      $i++;
      $s2=1/$i;
       
      $roznica=$s1-$s2;
      }
      echo $suma;
      }else echo 'Nieprawidłowe dane';//Zapętliłoby się, jeśli ktoś dałby 0
      }
      */
      ?>
[PHP] pobierz, plaintext

a za wyświetlaniem formularza daj

[PHP] pobierz, plaintext 
echo $_POST['z']." - skrócona nazwa tablicy <br />";
 
echo $HTTP_POST_VARS['z']." - długa nazwa tablicy";
[PHP] pobierz, plaintext

Jak przetestujesz to daj znać co się dzieje

Szeszek1992

24.01.2010, 17:05:40

Tak dla pewności: Nazwa pliku skryptu musi być taka sama jak nazwa podana w <form>, czyli np. index.php

Dym71

24.01.2010, 17:54:26

Niestety nawet dając tylko
echo $_POST['z']." - skrócona nazwa tablicy <br />";

echo $HTTP_POST_VARS['z']." - długa nazwa tablicy";

Nie wyświetla mi tego co wpisałem w formularzu. Wyświetla tylko:
- skrócona nazwa tablicy
- długa nazwa tablicy

bez żadnej wartości zmiennej $z

Wrzuciłem tutaj trzy pliki:
zad7.htm z formularzem
zad7.php plik z kodem
screen.jpg screen po wpisaniu wartości i kliknięciu ok

Jak ktoś może je sprawdzić to bym był bardzo wdzięczny.

Szeszek1992

24.01.2010, 18:06:44

W zad7.php popraw to, co masz w <form>(zmień z index.php na zad7.php).
Poza tym wszystko działa

Dym71

24.01.2010, 19:27:40

Dziękuję Wam wszystkim za pomoc, wszystko śmiga. Naprawdę bardzo dobre forum, miła atmosfera bo mili ludzie - nie to co na innych "forach" sami prowokatorzy. Pozdrawiam i jeszcze raz dzięki

To jest wersja lo-fi głównej zawartości. Aby zobaczyć pełną wersję z większą zawartością, obrazkami i formatowaniem proszę kliknij tutaj.