[PHP] obiektowe - czy idę w dobrym kierunku? Opcje

[Th0e]

Witam! Zacząłem się uczyć programowania obiektowego. Zrobiłem grę w nożyce papier, którą starałem się napisać obiektowo.

Moje pytanie - czy idę w dobrym kierunku?

[PHP] pobierz, plaintext 
<?php
class gra
{	
	function wybierz()
	{
		echo '<form method="POST">
		<select name="wybierz">
			<option>Kamien</option>
			<option>Nozyce</option>
			<option>Papier</option>
		</select>
		<input type="submit" value="wyslij">
		</form>
		';
	}
	function walka()
	{
 
		$wrog = rand(1,3);
		$ty = $_POST['wybierz'];
 
		switch($wrog)
		{
			case 1:
			$wrog ="Nozyce";
			break;
 
			case 2:
			$wrog ="Kamien";
			break;
 
			case 3:
			$wrog ="Papier";
			break;
		}
		if($ty == "Nozyce" && $wrog =="Papier" OR $ty =="Kamien" && $wrog=="Nozyce" OR $ty =="Papier" && $wrog=="Kamien")
		{
			echo'Wybrałeś '.$ty.'&nbsp; &nbsp; Natomiast przeciwnik wybrał '. $wrog .'<br /> Wygrałeś!';
		}
		else
		{
			echo'Wybrałeś '.$ty.'&nbps; &nbsp; Natomiast przeciwnik wybrał '. $wrog .'<br /> Przegrałeś...';
		}
	}
}
 
$gra = new gra();
 
$gra->wybierz();
$gra->walka();
?>
[PHP] pobierz, plaintext 

[Szymciosek]

[PHP] pobierz, plaintext 
<?php
class Gra //nazwy klas z dużej litery
{	
	public function wybierz() //podajemy modyfikator metody
	{
		echo '<form method="POST">
		<select name="wybierz">
			<option>Kamien</option>
			<option>Nozyce</option>
			<option>Papier</option>
		</select>
		<input type="submit" value="wyslij">
		</form>
		';
	}
 
	public function walka()
	{
 
		$wrog = rand(1,3);
		$ty = $_POST['wybierz'];
 
		switch($wrog)
		{
			case 1:
			    $wrog ="Nozyce";
			    break;
 
			case 2:
			    $wrog ="Kamien";
			    break;
 
			case 3:
			    $wrog ="Papier";
			    break;
		}
 
		if($ty == "Nozyce" && $wrog =="Papier" || $ty =="Kamien" && $wrog=="Nozyce" || $ty =="Papier" && $wrog=="Kamien") //Nie ma OR i AND, jest za to: OR = || i AND = &&
		{
			echo 'Wybrałeś '.$ty.'&nbsp; &nbsp; Natomiast przeciwnik wybrał '. $wrog .'<br /> Wygrałeś!'; //spacja po echo
		}
		else
		{
			echo 'Wybrałeś '.$ty.'&nbps; &nbsp; Natomiast przeciwnik wybrał '. $wrog .'<br /> Przegrałeś...';
		}
	}
}
 
$gra = new Gra(); //odwołujemy się do klasy, którą poprawiłem na dużą literę
 
$gra->wybierz();
$gra->walka();
?>
[PHP] pobierz, plaintext 

Ogólnie jest ok, uwagi masz w komentarzach.
Modyfikatory stosujemy żeby określić możliwość dostępu do metod klasy z zewnątrz.

[viking]

//Nie ma OR i AND, jest za to: OR = || i AND = &&

A co to za teoria?
http://www.php.net/manual/en/language.oper....precedence.php

&& i || są silniejsze niż AND i OR. W pewnych sytuacjach ich odpowiednie użycie ma sens. Do takiego kodu klasa nie jest potrzebna wcale. Raczej można by wydzielić klasę do obsługi parametrów od użytkownika czyli POST, GET i odpowiednia walidacja, walka ($input1, $input2) gdzie input to był by wybór pochodzący od różnych użytkowników. Ze zmiennej $wrog można by zrobić jakieś stałe a wręcz prosi się o eksperymentalny SPLEnum (przy okazji poznasz SPL) - wtedy zamiast ciągle powtarzać papier, kamień, nożyce (łatwo zrobić literówkę) tylko np WROG::1

[!*!]

Moje pytanie - czy idę w dobrym kierunku?

Nie. Metoda (brakuje też odpowiednich flag np. public private, protected) w klasie ma zwracać wynik jako return, a nie wyświetlać go przez echo.

Ten post edytował !*! 20.04.2013, 09:36:50