[PHP]Kalkulator OOP Opcje

[miras]

Witam, czas najwyższy przerzucić się na OOP, a więc zacząłem standardowo od napisania kalkulatora i (najpierw kod)

index.php :

[PHP] pobierz, plaintext 
 
<form method="POST" action="">
	<input type="number" name="pierwsza" value=""/>
	<select name="dzialanie">
		<option value="+">+</option>
		<option value="-">-</option>
		<option value="*">*</option>
		<option value=":">:</option>
	</select>
	<input type="number" name="druga" value=""/>	
	<br><br>
	<input type="submit" value="Oblicz"/>
</form>
 
<?php 
	include('kalkulator.class.php');
	$kalkulator = new kalkulator;
	echo $kalkulator -> oblicz($_POST['pierwsza'],$_POST['druga'],$_POST['dzialanie']);
?>
[PHP] pobierz, plaintext 

kalkulator.class.php

[PHP] pobierz, plaintext 
<?php
	class kalkulator {
		public function oblicz($pierwsza, $druga, $dzialanie) { 
			$this->wynik = $pierwsza + $druga;
			 return $this->wynik;
		}
	}
 
?>
[PHP] pobierz, plaintext 

i teraz tak: powyższy kod działa, z tym, że na stałe jest wpisane dodawanie, a ja chciałbym zrobić jakieś warunki if($_POST['dzialanie']=="-") to odejmowanie itd, niestety nie wiem jak stworzyć w OOP warunki, a drugie pytanie czy ten kod, który wstawiłem ma w ogóle coś wspólnego z OOP ? ;>

Dzięki z góry...

[nospor]

niestety nie wiem jak stworzyć w OOP warunki

A co ma warunek wspólnego z OOP?? Nic.... Do robienia warunków używa się IF, niezależnie czy to OOP czy nie..... IF chyba umiesz używac?

[falcon85]

[PHP] pobierz, plaintext 
class kalkulator {
		public function oblicz($pierwsza, $druga, $dzialanie) { 
                    if ($dzialanie == "+") {
			$wynik = $pierwsza + $druga;
                    } elseif  ($dzialanie == "-") {
                       $wynik = $pierwsza - $druga; 
                    }  
                    return $wynik;
		}
	}
[PHP] pobierz, plaintext 

Ten post edytował falcon85 29.03.2013, 12:16:46

[miras]

Ok, dzięki Wam w takim razie, myślałem, że w OOP robi się to jakoś inaczej...

[devbazy]

Warunki OOP? Co masz na myśli? Warunki masz wszędzie takie same bez względu na to czy piszesz strukturalnie czy obiektowo. Warunek to warunek. Jeżeli zaś chodzi o Twoją klasę, to utwórz sobie więcej metod np. "dodaj", "odejmij" etc. W pliku index.php sprawdzaj jaki operator został wybrany i wywołuj odpowiednią metodę z klasy "kalkulator". Prosty przykład:

index.php

[PHP] pobierz, plaintext 
<form method="POST" action="">
	<input type="number" name="pierwsza" value=""/>
	<select name="dzialanie">
		<option value="+">+</option>
		<option value="-">-</option>
		<option value="*">*</option>
		<option value=":">:</option>
	</select>
	<input type="number" name="druga" value=""/>	
	<br><br>
	<input type="submit" value="Oblicz"/>
</form>
 
<?php 
	include('kalkulator.class.php');
	$kalkulator = new kalkulator;
 
        switch($_POST['dzialanie'])
        {
            case '+':
            echo $kalkulator->dodaj($_POST['pierwsza'], $_POST['druga']);
            break;
 
            case '-':
            echo $kalkulator->odejmij($_POST['pierwsza'], $_POST['druga']);
            break;
 
            // etc
        }
?>
[PHP] pobierz, plaintext 

kalkulator.class.php

[PHP] pobierz, plaintext 
<?php
	class kalkulator {
 
		public function dodaj($a, $b) { 
			// jakieś instrukcje
		}
 
		public function odejmij($a, $b) { 
			// jakieś instrukcje
		}
	}
 
?>
[PHP] pobierz, plaintext 

[g2g3]

Warunki OOP? Co masz na myśli? Warunki masz wszędzie takie same bez względu na to czy piszesz strukturalnie czy obiektowo. Warunek to warunek. Jeżeli zaś chodzi o Twoją klasę, to utwórz sobie więcej metod np. "dodaj", "odejmij" etc. W pliku index.php sprawdzaj jaki operator został wybrany i wywołuj odpowiednią metodę z klasy "kalkulator". Prosty przykład:

<?php
include('kalkulator.class.php');
$kalkulator = new kalkulator;

switch($_POST['dzialanie'])
{
case '+':
echo $kalkulator->dodaj($_POST['pierwsza'], $_POST['druga']);
break;

case '-':
echo $kalkulator->odejmij($_POST['pierwsza'], $_POST['druga']);
break;

// etc
}
?>
[/php]

Ty chyba sobie żartujesz z tym switchem.

[PHP] pobierz, plaintext 
 
class Kalkulator {
 
   protected $liczba1;
   protected $liczba2;
 
   // W komentarzu tylko liczby.
   public function __construct( $liczba1, $liczba2)
   {
        $this->liczba1 = $liczba1;
        $this->liczba2 = $liczba2;
   }
 
   public function dodawanie()
   {
        return $this->liczba1 + $this->liczba2;
   }
 
   public function odejmowanie()
   {
        return $this->liczba1 - $this->liczba2;
   }
 
   // itd.
}
 
[PHP] pobierz, plaintext 

Ten post edytował g2g3 29.03.2013, 13:11:09

[PrinceOfPersia]

switch też może być...

ważniejsze to chyba zwalidować. Coś na kształt:
$pierwsza = intval($_POST['pierwsza'])
albo
$pierwsza = floatval($_POST['pierwsza'])
wtedy upewniamy się, że na pewno będziemy operować na liczbach (bo użytkownik może cokolwiek wpisać, nawet jakieś głupoty).

Ten post edytował PrinceOfPersia 29.03.2013, 18:56:03

[devbazy]

@g2g3: widzę, że specjalista się trafił. Powiedz co jest złego w podanym przeze mnie przykładzie?

@PrinceOfPersia: zamiast korzystać z funkcji "intval()", lepiej rzutować te zmienne na inta imo.

Walidacja to podstawa, ale o to trzeba się już samemu postarać.

@Down: Podałem jeden ze sposobów rozwiązania problemu i nigdzie nie napisałem, że tak się powinno pisać obiektowo, także oszczędź sobie te durne komentarze, bo się ośmieszasz. Poza tym, interesujące jest to, że potrafisz określić czyjąś wiedzę po przeczytaniu kilku wypowiedzi na forum, zdumiewające. Polecam iść się mądrzyć gdzie indziej, panie "ekspert". Dyskusję uważam za zamknięta, nie mam zamiaru offtopować, a w dodatku szkoda mi czasu na takich delikwentów

Ten post edytował devbazy 30.03.2013, 16:30:13

[g2g3]

@g2g3: widzę, że specjalista się trafił. Powiedz co jest złego w podanym przeze mnie przykładzie?

@PrinceOfPersia: zamiast korzystać z funkcji "intval()", lepiej rzutować te zmienne na inta imo.

Walidacja to podstawa, ale o to trzeba się już samemu postarać.

To co napisałeś w ogóle mija się z pisaniem obiektowym, mądrujesz się, a wiedzy brak. Uwierz, mam większe doświadczenie.

Kolega który to propsuje chyba ma podobnie.