Logowanie (pierwsza klasa) Opcje

[Bent]

Witam,

Dopiero zaczynam przygodę z OOP. Napisałem pierwszą taką klasę:

[PHP] pobierz, plaintext 
class Users extends mysqli
{
  private $login = 'login', $password = 'password', $submit;
 
  private function showLogForm()
  {
    ?>
	<form method="post" action="">
	<table>
	<tr>
	<td>Login: </td><td><input type="text" name="<?php echo $this -> login; ?>" /></td>
	</tr>
	<tr>
	<td>Hasło: </td><td><input type="text" name="<?php echo $this -> password; ?>" /></td>
	</tr>
	<tr>
	<td></td><td><input type="submit" name="submit" value="Zaloguj!" /></td>
	</tr>
	</table>
	</form>
	<?php
  }
 
  private function isSent($submit)
  {
    $this -> submit = $submit;
	if (isset($this -> submit))
	  return true;
	else
	  return false;
  }
 
  private function isFilled()
  {
    foreach ($_POST as $key => $value)
	{
	  if (!isset($key) || empty($value))
	    return false;
	}
	return true;
  }
 
  public function logIn()
  {
    if ($this -> isSent($_POST['submit']))
	  {
	    if ($this -> isFilled())
		  echo 'OK';
		else
		  echo 'Uzupełnij formularz!';
	  }
	else
	  {
	    $this -> showLogForm();
	  }
  }
 
}
 
$user = new Users;
$user -> logIn();
[PHP] pobierz, plaintext 

Mam 2 pytania:

1. W mojej klasie metody wyświetlające formularz i sprawdzające go są typu private, metodą, która to wszystko ogarnia jest metoda logIn(). Czy to dobry pomysł? Czy może lepiej wszystkie te metody ustawić jako public i całość sprawdzać poza klasą po utworzeniu obiektu?

2. Czy dobrym pomysłem jest tworzenie metod wyświetlających formularze czy raczej do takich rzeczy się funkcji nie tworzy?

[MateuszS]

Sam ucze sie OOP ale wg mnie nie powinno byc w zadnej metodzie ani troche HTMLa. Powinienes zrobic osobny plik .phtml z formularzem i go dolaczyc includem. No i nie wiem po co przypisales atrybutowi name wartosc ze zmiennej, zmienna, czyli jak sama nazwa wskazuje, moze sie zmieniac (jej wart.), a nie wiem w jakim celu zmieniac name inputa, a wartosci POSTa mozna przeslac osobnymi metodami typu setLogin i setPassword

Ten post edytował MateuszScirka 26.02.2010, 17:48:52