[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 

[!*!]

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