Forum PHP.pl

Witam,

PHP uczę się trzeci dzień, także jestem totalnym laikiem, tym bardziej, że wcześniej nie miałem do czynienia z językiem obiektowym (tylko HTML i CSS).

Do rzeczy: napisałem sobie prosty "kalkulator". Wygląda to tak, że jest formularz z dwoma polami i trzeba wpisać do nich dwie liczby (do każdego pola po jednej), a następnie wybrać z inputa select czy chcemy dodać/odjąć/pomnożyć/podzielić liczbę.

Fajnie- jest prymitywne, ale działa Żeby to trochę rozwinąć chciałem dodać funkcję preg_match, żeby sprawdzić czy w obu polach jest przynajmniej jedna liczba. Część odpowiadającą za liczenie dałem jako funkcję. No i się posypało- wyskakuje error. Proszę o pokazanie mi gdzie leży błąd i ewentualną wskazówkę jak go usunąć.

[HTML] pobierz, plaintext 
<form action="calc.php" method="post">
 
Liczba 1 <input type="text" name="jeden"><br>
Liczba 2 <input type="text" name="dwa"><br>
<select name="option">
	<option value="dodawanie">Dodawanie</option>
	<option value="odejmowanie">Odejmowanie</option>
	<option value="mnozenie">Mnożenie</option>
	<option value="dzielenie">Dzielenie</option>
</select><br>
 
<input type="submit" value="Wyslij"><br>
 
</form>
[HTML] pobierz, plaintext 

[PHP] pobierz, plaintext 
<?php
 
$_POST['jeden'] = $jeden;
$_POST['dwa'] = $dwa;
$_POST['option'] = $option;
 
 
if (preg_match('@[0-9]{1,}@',$jeden && $dwa)){
oblicz();
}
 
else{
echo('wpisz liczby!');
}
 
function oblicz(){
 
if ($option == "dodawanie"){
$suma = $jeden + $dwa;
echo("Suma to $suma");
}
 
else if ($option == "odejmowanie"){
$roznica = $jeden - $dwa;
echo("Roznica to $roznica");
}
 
else if ($option == "mnozenie"){
$iloczyn = $jeden * $dwa;
echo("Iloczyn to $iloczyn");
}
 
else {
$iloraz = $jeden / $dwa;
echo("Iloraz to $iloraz");
}
}
 
?>
[PHP] pobierz, plaintext 

Zamierzenie jest takie, że wymagane jest wypełnienie obu pól formularza dowolnymi liczbami. Jeśli warunek zostanie spełniony, ma zostać wykonana funkcja oblicz, jeśli nie- ma pokazać się komunikat "Wpisz liczby!".

Jeszcze jedno pytanie- czy pogrubiony poniżej fragment kodu jest poprawny? Chodzi o te &&.
if (preg_match('@[0-9]{1,}@',$jeden && $dwa))

Z góry dziękuję za pomoc i pozdrawiam.

http://php.net/manual/en/function.is-numeric.php tutaj poszukaj odp.

Dzięki za cenną radę co do sprawdzania błędów

Wiem, że problem leży w funkcji oblicz() i domyślam się, że brakuje jej parametrów, ale nie mam pojęcia jakie zmienne wpisać pomiędzy nawiasy, żeby skrypt działał.

Bez użycia funkcji (przy użyciu instrukcji warunkowej w instrukcji warunkowej ) skrypt działa dobrze, ale chciałbym się dowiedzieć co robię źle.

Swoją drogą takie używanie "ifa w ifie" jest poprawne?

Dzięki.

Nie mogę zrozumieć tylko jakie to ma znaczenie czy jest

[PHP] pobierz, plaintext 
$_POST['zmienna'] = $zmienna;
[PHP] pobierz, plaintext 

czy

[PHP] pobierz, plaintext 
$zmienna = $_POST['zmienna'];
[PHP] pobierz, plaintext 

Przecież x = y to y = x.
Czyli jak zamienię z $_POST na $zmienna to potem muszę używać już tylko drugiej wersji?

Ten post edytował stefan09 28.09.2011, 20:24:31

Wszystko jasne. Jeszcze raz dzięki

Jakby ktoś pytał, to najpierw radzę dobrze zaznajomić z operatorami bo w PHP znak równości (=) nie oznacza równości więc:

[PHP] pobierz, plaintext 
//to
$_POST['jeden'] = $jeden;
//nie jest to samo co to
x = y;
[PHP] pobierz, plaintext 

= - przypisanie prawej do lewej
== - lewa równa z prawą i odwrotnie
=== - lewa identyczna z prawą i odwrotnie