Pomoc - Szukaj - Użytkownicy - Kalendarz
Pełna wersja: [PHP]Kalkulator w php
Forum PHP.pl > Forum > Przedszkole
djjbenus
18.07.2012, 10:59:23
Witam,

wraca stary temat mojego kalkulatora na którego pomysł znalazłem ale teraz mam problemy. Otóż przy naciśnięciu znaku + lub - lub * / dzieli mi przez zero. Nie mam pojęcia dlaczego.


Zamieszczam kod. 


[PHP] pobierz, plaintext 
  1. <?php
  2.  
  3. function licz($tmp)
  4. {
  5. 	switch ($tmp)
  6. 	{
  7. 		case '1' : $_SESSION['wynik'] = (($_SESSION['wynik'])*10)+1; break;
  8. 		case '2' : $_SESSION['wynik'] = (($_SESSION['wynik'])*10)+2; break;
  9. 		case '3' : $_SESSION['wynik'] = (($_SESSION['wynik'])*10)+3; break;
  10. 		case '4' : $_SESSION['wynik'] = (($_SESSION['wynik'])*10)+4; break;
  11. 		case '5' : $_SESSION['wynik'] = (($_SESSION['wynik'])*10)+5; break;
  12. 		case '6' : $_SESSION['wynik'] = (($_SESSION['wynik'])*10)+6; break;
  13. 		case '7' : $_SESSION['wynik'] = (($_SESSION['wynik'])*10)+7; break;
  14. 		case '8' : $_SESSION['wynik'] = (($_SESSION['wynik'])*10)+8; break;
  15. 		case '9' : $_SESSION['wynik'] = (($_SESSION['wynik'])*10)+9; break;
  16. 		case '0' : $_SESSION['wynik'] = (($_SESSION['wynik'])*10)+0; break;
  17. 	}
  18. 	return $tmp;
  19. }
  20.  
  21. function liczy($tmp)
  22. {
  23. 	switch ($tmp)
  24. 	{
  25. 		case '1' : $_SESSION['liczba'] = (($_SESSION['liczba'])*10)+1; break;
  26. 		case '2' : $_SESSION['liczba'] = (($_SESSION['liczba'])*10)+2; break;
  27. 		case '3' : $_SESSION['liczba'] = (($_SESSION['liczba'])*10)+3; break;
  28. 		case '4' : $_SESSION['liczba'] = (($_SESSION['liczba'])*10)+4; break;
  29. 		case '5' : $_SESSION['liczba'] = (($_SESSION['liczba'])*10)+5; break;
  30. 		case '6' : $_SESSION['liczba'] = (($_SESSION['liczba'])*10)+6; break;
  31. 		case '7' : $_SESSION['liczba'] = (($_SESSION['liczba'])*10)+7; break;
  32. 		case '8' : $_SESSION['liczba'] = (($_SESSION['liczba'])*10)+8; break;
  33. 		case '9' : $_SESSION['liczba'] = (($_SESSION['liczba'])*10)+9; break;
  34. 		case '0' : $_SESSION['liczba'] = (($_SESSION['liczba'])*10)+0; break;
  35. 	}
  36. 	return $tmp;
  37. }
  38.  
  39. session_start();
  40.  
  41. if(!isset($_SESSION['wynik']))
  42. {
  43. 	$_SESSION['wynik']='0';
  44. }
  45.  
  46. if(!isset($_SESSION['liczba']))
  47. {
  48. 	$_SESSION['liczba']='1';
  49. }
  50.  
  51. /*if(!isset($_SESSION['tmp']))
  52. {
  53. 	$_SESSION['tmp'] = '0';
  54. }*/
  55.  
  56. if(isset($_POST['send']))
  57. {
  58. 	if($_POST['wynik']==0)
  59. 	{
  60. 		licz($_POST['send']);
  61. 	}
  62. 	else
  63. 	{
  64. 		liczy($_POST['send']);
  65. 	}
  66. }
  67.  
  68. if(isset($_POST['sign']))
  69. {
  70. 	switch($_POST['sign'])
  71. 	{
  72. 		case '+' :
  73. 			$_SESSION['wynik']=$_SESSION['wynik']+$_SESSION['liczba'];
  74. 		case '-' :
  75. 			$_SESSION['wynik']=$_SESSION['wynik']-$_SESSION['liczba'];
  76. 		case '*' : 
  77. 			$_SESSION['wynik']=$_SESSION['wynik']*$_SESSION['liczba'];
  78. 		case '/' : 
  79. 			$_SESSION['wynik']=$_SESSION['wynik']/$_SESSION['liczba'];
  80. 		//case '=' : $_SESSION['liczba']=0;
  81. 		case 'RESET' : 
  82. 			$_SESSION['liczba']=0; $_SESSION['wynik']=0;
  83. 	}
  84. }
  85.  
  86. ?>
  87.  
  88. <html>
  89. <head>
  90. </head>
  91. <body>
  92. 	<?php 
  93. 		echo $_SESSION['liczba']; 
  94. 		echo '<br />'; 
  95. 		echo $_SESSION['wynik']; 
  96. 	?>
  97. 	<form action="index.php" method="post">
  98. 		<fieldset style="width: 150px;">
  99. 			<input type="temp" value="<?php if($_POST['send'] == '=') { echo $_SESSION['wynik']; } else { echo $_SESSION['wynik']; } ?>" />
  100. 		</fieldset>
  101. 		<fieldset style="width: 150px;">
  102. 			<button type="submit" name="send" value="1">1</button>
  103. 			<button type="submit" name="send" value="2">2</button>
  104. 			<button type="submit" name="send" value="3">3</button>
  105. 			<button type="submit" name="sign" value="+">+</button><br/>
  106. 			<button type="submit" name="send" value="4">4</button>
  107. 			<button type="submit" name="send" value="5">5</button>
  108. 			<button type="submit" name="send" value="6">6</button>
  109. 			<button type="submit" name="sign" value="-">-</button><br/>
  110. 			<button type="submit" name="send" value="7">7</button>
  111. 			<button type="submit" name="send" value="8">8</button>
  112. 			<button type="submit" name="send" value="9">9</button>
  113. 			<button type="submit" name="sign" value="*">*</button><br/>
  114. 			<button type="submit" name="send" value="0">0</button>
  115. 			<button type="submit" name="sign" value="/">/</button>
  116. 			<button type="submit" name="send" value="=">=</button>
  117. 			<button type="submit" name="sign" value="RESET">AC</button>
  118. 		</fieldset>
  119. 	</form>
  120. </body>
  121. </html>
[PHP] pobierz, plaintext
Necsord
18.07.2012, 11:11:28
Nie wnikam w reszte kodu, ale w poniższym switch'u brakuje break'ów.
[PHP] pobierz, plaintext 
  1. if(isset($_POST['sign']))
  2. {
  3. 	switch($_POST['sign'])
  4. 	{
  5. 		case '+' :
  6. 			$_SESSION['wynik']=$_SESSION['wynik']+$_SESSION['liczba'];
  7. 		case '-' :
  8. 			$_SESSION['wynik']=$_SESSION['wynik']-$_SESSION['liczba'];
  9. 		case '*' : 
  10. 			$_SESSION['wynik']=$_SESSION['wynik']*$_SESSION['liczba'];
  11. 		case '/' : 
  12. 			$_SESSION['wynik']=$_SESSION['wynik']/$_SESSION['liczba'];
  13. 		//case '=' : $_SESSION['liczba']=0;
  14. 		case 'RESET' : 
  15. 			$_SESSION['liczba']=0; $_SESSION['wynik']=0;
  16. 	}
  17. }
[PHP] pobierz, plaintext
djjbenus
18.07.2012, 11:25:37
Rozwiązało to problem niepotrzebnego dzielenia przez zero. Jednak do zmiennej $_SESSION['liczba'] nic nie jest zapisywane. W czym tkwi problem w moim rozumowaniu?
Necsord
18.07.2012, 11:55:30 
[PHP] pobierz, plaintext 
  1. if(isset($_POST['send']))
  2. {
  3. 	if($_POST['wynik']==0)
  4. 	{
  5. 		licz($_POST['send']);
  6. 	}
  7. 	else
  8. 	{
  9. 		liczy($_POST['send']);
  10. 	}
  11. }
[PHP] pobierz, plaintext


W formularzu nie widze żadnego pola "wynik", chyba chodziło tobie o $_SESSION. Swoja drogą masz dużo notice'ów "undefined index" ( Temat: PHP Notice Undefined index ).
peter13135
18.07.2012, 11:58:57
Po co robisz takie bycze switche ?Przecież to w jednej linijce da się zrobić.
Tomplus
18.07.2012, 11:58:59
Upewnij się czy po prostu funkcjach licz/liczy zapisują się informacje do sesji liczba.
peter13135
18.07.2012, 12:00:15
A ja se tak myślę... najpierw masz funkcje, które zapisują coś do sesji, a potem dopiero session_start() tongue.gif
SmokAnalog
18.07.2012, 12:04:24
Kod Twojego kalkulatora jest straszny. Ale to mnie najbardziej rozbawiło: (pomijając nawet fakt że te funkcje są źle skonstruowane działając bezpośrednio na zmiennej sesji)
[PHP] pobierz, plaintext 
  1. switch ($tmp)
  2. {
  3. 	case '1' : $_SESSION['wynik'] = (($_SESSION['wynik'])*10)+1; break;
  4. 	case '2' : $_SESSION['wynik'] = (($_SESSION['wynik'])*10)+2; break;
  5. 	case '3' : $_SESSION['wynik'] = (($_SESSION['wynik'])*10)+3; break;
  6. 	case '4' : $_SESSION['wynik'] = (($_SESSION['wynik'])*10)+4; break;
  7. 	case '5' : $_SESSION['wynik'] = (($_SESSION['wynik'])*10)+5; break;
  8. 	case '6' : $_SESSION['wynik'] = (($_SESSION['wynik'])*10)+6; break;
  9. 	case '7' : $_SESSION['wynik'] = (($_SESSION['wynik'])*10)+7; break;
  10. 	case '8' : $_SESSION['wynik'] = (($_SESSION['wynik'])*10)+8; break;
  11. 	case '9' : $_SESSION['wynik'] = (($_SESSION['wynik'])*10)+9; break;
  12. 	case '0' : $_SESSION['wynik'] = (($_SESSION['wynik'])*10)+0; break;
  13. }
  14. return $tmp;
[PHP] pobierz, plaintext

I jeszcze jeden podobny switch. Przecież wystarczy:
[PHP] pobierz, plaintext 
  1. if(preg_match('#^\d$#', $tmp)) {
  2. 	$_SESSION['wynik'] = (($_SESSION['wynik'])*10)+$tmp;
  3. }
  4. return $tmp;
[PHP] pobierz, plaintext

Zresztą sama idea sesji dla tego kalkulatora wydaje mi się średnio trafiona. Wystarczyłby input hidden.
djjbenus
18.07.2012, 12:07:11
@peter13135

Próbowałem ale mi nie wyszło dlatego jest jak jest. Jeśli masz jakiś pomysł to chętnie spróbuje.

Hmmm wydawało mi się że nie ma znaczenia gdzie zadeklaruje funkcje.

@Tomplus

Niby funkcja licz oddaje to co powinno być ale już funkcja liczy nie spełnia założeń.

@SmokAnalog

Wydawało mi się że tak ponieważ przy zwykłym formularzu mi się nie udało skontruować nic konkretnego.
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.
Invision Power Board © 2001-2025 Invision Power Services, Inc.