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.

Nie, nie jest poprawny. Trzeba zrobić dwa warunki:

[PHP] pobierz, plaintext 
if(preg_match('@[0-9]{1,}@',$jeden) && preg_match('@[0-9]{1,}@',$dwa)) {
 
}
[PHP] pobierz, plaintext 

PS. Następnym razem radzę przeczytać co wyskakuje za błąd, mimo, że jest po angielsku to w 90% wszystko nam serwer pokazuje, w jakij linijce i ewentualnie jaka funkcja. A jak nie możesz dojśc co to za problem wyskoczył, najlepiej skopiować i wkleić go tutaj i podać kawałek kodu z linijką, która występuje w błędzie.

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?

[PHP] pobierz, plaintext 
$_POST['jeden'] = $jeden;
$_POST['dwa'] = $dwa;
$_POST['option'] = $option;
[PHP] pobierz, plaintext 

Prawdopodobnie chcesz przypisać z formularza danych do zmiennych, więc powinieneś zrobić to odwrotnie czyli

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

twoja funkcja oblicz, powinna przyjąć jakieś parametry i w tej funkcji pracujesz na jej parametrach, czyli:

[PHP] pobierz, plaintext 
oblicz($jeden, $dwa, $option);
[PHP] pobierz, plaintext 

zamiast preg_match, uzyj is numeric, a wczesniej sprawdź czy taka zmienna istnieje, tzn:

[PHP] pobierz, plaintext 
if(isset($_POST['jeden'])
   echo 'istnieje';
[PHP] pobierz, plaintext 

w funkcji oblicz chyba lepiej użyć switch-a niż if, elseif w twoim przypadku.

Ppozdro

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?

znaczenie ma takie, ze na początku skryptu zmienna $jeden jest pusta.
Skoro robisz:

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

To do zmiennej odebranej z formularza przypisujesz pusta zmienną.

a jeśli dasz odwrotnie to do zmiennej $jeden przypisujesz zmienna odebrana z formularza.

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