Witam mam mały dość problemik otóż całe 1,5 roku pisze w php ale proceduralnie , php oop znam w teori ale czas nauczyć sie pisać aplikacje już w całosci obiektowe i tutaj jest problem bo w praktyce nie zabardzo mi to wychodzi .W javie [j2se] pisze już 5miesiecy i z oop i pisaniem programów nie mam problemu ,tylko w php sam nie wiem dla czego , może dla tego z jave od poczatku uczy programowania w oop, bo inaczej nie da sie programowac w tym języku , może macie jakieś rady ?.Wiem programowania webowego nie mozna porównywac do programowania apliakcji użytkowych
Co robie zle ?

[PHP] pobierz, plaintext 
<?php
 
class  Rejestracja
{
	public  $login;
	private $haslo;
	private $haslotwo;
 
	function __construct()
{
 include_once('config_database.php');
//odbieranie danych
 $login=$_POST['login'];
 $haslo=$_POST['haslo'];
$haslotwo=$_POST['haslotwo'];
}
 
private function polaczenie()
{
$polaczenie=mysql_connect(connect,user,password)or die('connect_error'.mysql_error());
mysql_select_db(database)or die('database_error'.mysql_error());
}
private function spraw_user()
{
	polaczenie();
$zapytanielog= "SELECT * FROM user WHERE login='$login'";
$sprawdzlog= mysql_query($zapytanielog)or die(mysql_error());
$numrows=mysql_num_rows($sprawdzlog);
if($numrows > 0)
{
   echo "Podany login już istnieje";
   exit;
}
if($haslo!=$haslotwo)
{
   echo "Hasło jest błedne";
   exit;
}
}
private function rej_user()
{
	sprw_user();
$zapisrej="INSERT INTO user(`login`,`haslo`)VALUES('$login','$haslo')";
$touser=mysql_query($zapisrej);
if($touser)
{
   echo "Użytkownik dodany";
}
else
{
   echo "Użytkownik nie dodany!";
  exit;
}
}
}
$start=new Rejestracja();
//$start->sprawdz_user();
// czy może wywoływać je w konstruktorze ?, czy lepiej jako obiekty  bo w php mam ztym problem 
 
 
?>
[PHP] pobierz, plaintext 

A Prosze sobie darować złosliwe dogadywanie i gadki fizoloficzne

Nie opisujesz problemu.
Dales dluuugi wstep, ktory nic nam nie mowi na temat Twojego problemu. Co nas obchodzi ile piszesz proceduralnie a ile w oop? No pomysl. CZy nam ta wiedza jest potrzebna? Nam potrzebny jest opis Twojego problemu z jakim sie borykasz. A ty pokazales kod i zadowolony z siebie... Napisz z czym jest problem. Napisz co sie dzieje a nie powinno lub co sie nie dzieje a powinno. Znacznie ulatwi to szukanie bledu.

Oczywiscie moglbym przejrzec kod i pewnie bym cos znalazl, ale poco mam sie wczytywac, skoro od razu z opisu (ktory moze wkoncu podasz) bedzie wiadomo co moze byc nie tak.

No niestety, ale w tym topicu trzeba bylo troche pofilozofowac

Ja nie bede sie madrował bo sam słabo znam OOP ale to co moge wytknąć to
powinno byc raczej tak

[PHP] pobierz, plaintext 
<?php
public function __construct($login,$haslo,$haslotwo){
$this->login = $login;
$this->haslo = $haslo;
$this->haslotwo = $haslotwo;
// i zadnych tablic globalnych 
// i plik zainkluduj przed wywolaniem klasy a w konstruktorze utworz obiekt (chyba, nich  ktos to sprostuje)
}
?>
[PHP] pobierz, plaintext 

co to sa stale?? jesli tak to dla wyroznienia zrob je DUZYMI LITERAMI



zadnych echo w klasie return i potem wywalasz na ekran no i nie odwoluj sie do metody polaczenie (i tak zle to zrobiles) dopiero jak sobie utworzysz nowy obiekt tej klasy to wywolasz metode cos jakso tak

[PHP] pobierz, plaintext 
<?php
$obj = new Rejestracja();
$obj->polaczenie();
$obj->sprawdzenie_czy_jak_to_sie_nazywa();
?>
[PHP] pobierz, plaintext 

tu po co sprawdzasz ifem daj sobie return mysql_query($zapisrej) i dopiero po utworzeniu obiektu sprawdz co zwrocil i wal komunikat

[PHP] pobierz, plaintext 
<?php
if(obj->rej_user() == true) echo'dodany';
?>
[PHP] pobierz, plaintext 

i to tyle co chcialem napisac jakas dobra duszyczka mogla by to zweryfikowac bo sam z checia sie dowiem gdzie blednie rozumuje

Generalny błąd:


W PHP z wnętrza klasy (dowolna metoda) odwołujesz się do jej właściwości przez przez wskazanie $this->...
czyli nie:


i tak samo we wszystkich innych miejscach...

pozdrawiam.

No i panowie nadal nic nawet nie pokazuje komunikatów żadnych , nie dodaje nowego usera do bazy itd ,jak by ktoś mógł pokazać jak to prawidłowo powinno być zrobione .A ktos może też dać jakieś linki do jakiś stron które by mi pomogły w pisaniu obiektowo
Bo powiem szczerze ze oop w c++i java problemów nie mam ale w php mam ciągle

[PHP] pobierz, plaintext 
<?php
 
class  Rejestracja
{
	public  $login;
	public $haslo;
	public $haslotwo;
 
	function __construct($login,$haslo,$haslotwo)
{
 include_once('config_database.php');
//odbieranie danych
 $this-> login = $login=$_POST['login'];
 $this-> haslo= $haslo=$_POST['haslo'];
$this-> haslotwo=$haslotwo=$_POST['haslotwo'];
}
 
public  function polaczenie()
{
$polaczenie=mysql_connect(connect,user,password)or die('connect_error'.mysql_error());
mysql_select_db(database)or die('database_error'.mysql_error());
}
public  function spraw_user()
{
$zapytanielog= "SELECT * FROM user WHERE login='$login'";
$sprawdzlog= mysql_query($zapytanielog)or die(mysql_error());
$numrows=mysql_num_rows($sprawdzlog);
if($numrows > 0)
{
  return "Podany login już istnieje";
   exit;
}
if($haslo!=$haslotwo)
{
 return "Hasło jest błedne";
   exit;
}
}
public  function rej_user()
{
 
$zapisrej="INSERT INTO user(`login`,`haslo`)VALUES('$login','$haslo')";
$touser=mysql_query($zapisrej);
if($touser)
{
  return "Użytkownik dodany";
}
else
{
   return  "Użytkownik nie dodany!";
  exit;
}
}
}
$start=new Rejestracja(login,haslo,haslotwo);
$start->polaczenie();
$start->spraw_user();
$start->rej_user()
?>
[PHP] pobierz, plaintext 

jezeli robisz return to musisz dac echo
czyli

[PHP] pobierz, plaintext 
<?php
echo $start->rej_user()
?>
[PHP] pobierz, plaintext 

a druga sprawa jest taka ze nieuwaznie czytales co ci napisalem

miales zrobic tak

[PHP] pobierz, plaintext 
<?php
}
$start=new Rejestracja($_POST[login],$_POST[haslo],$_POST[haslotwo]);
?>
[PHP] pobierz, plaintext 

ciezko uwierzyc ze znasz sie na OOP w jakim kolwiek jezyku skoro nawet nie potrafisz zbudowac i uzyc funkcji

skoro robisz to na klasach to po co mieszasz funckje strukturalne i te obiektowe np zasmiat mysql_query($zap);
rob $db->query($zap);

Popieram to co wyżej koledzy napisali. To nie jest OOP - to zwykły kod strukturalny ujęty w klasie.
Np. nie wiem, dlaczego używasz exit() w metodzie (pominę fakt, że się i tak nie wykona, bo wcześniej jest return) - tego się po prostu nie uzywa się wyjątków i to użytkownik kodu powinien przejąć odpowiedzialność za to, co zrobi z otrzymanym błędem.
Poprawienie ten mechanizm powinien wyglądać mniej więcej tak:

[PHP] pobierz, plaintext 
<?php
//defincja tej klasy
class{}
 
//i teraz
 
$User=new User($login,$haslo,$haslotwo);
$Registration=new Rejestracja();
// $Registration->polaczenie(); //NIE
 $db->connect(); //powiedzmy tak uprościmy
 $Registration->useConnection($db) // i przekażemy obiekt bazy do obiektu Rejestracja
//$Registration->spraw_user(); // co to znaczy spraw? Co mam sprawić? NIE
  if($Registration->isAlreadyRegistered($User))
  {
	$errorString=$Registration->getErrorString();
	die("$error");
  }
  else
  {
	//$start->rej_user() // znów dziwna konstrukcja z nazwą bez znaczenia (wyjątkiem sa nazwy dowcipne  )
	$Registration->registerUser();
	//albo
	$Registration->registerUser($User);
  }
?>
[PHP] pobierz, plaintext 

Pokazany kod, to użytkownik klasy, a właściwie 3 klas. DB, User i Registration


Dobra książka powinna pomóc.

Pozdrawiam.

Jakoś w javie nie mam problemów z oop.Jedynie w php , może to dla tego ze dopiero zaczynam programować obiektowo w tym języku :)ale nie rozumiem tego
$start=new Rejestracja($_POST[login],$_POST[haslo],$_POST[haslotwo]); ? co to niby daje ?

@dzesi - Proszę Cię - zweryfikuj swoje umiejętności dotyczące programowania obiektowego. Jeśli nie wiesz, co daje Ci taki kod, to nie wiesz też, czym jest hermetyzacja. Otóż konstrukcja o którą pytasz powoduje, że obiekt Registration jest uniezależniony od danych wejściowych. Wisi mu, czy są to POST, GET, SESSION, COOKIE, czy jeszcze inne (np. z innych obiektów). Po prostu dostaje dane i robi swoje.
Te mechanizmy są identyczne w Javie i w php, więc problem masz raczej ze składnią samego języka niż z OOP. Obiektowość jest niezależna od języka - gdybyś umiał pisać obiektowo w Javie, to w php robiłbyś to bez najmniejszego problemu.

Pozdrawiam.

wogóle ta klasa jest bez sensu


powinieneś zrobić osobne klasy dla mysql dla user i wtedy

dajesz w pliku np register.php


// register.php

[PHP] pobierz, plaintext 
<?php
 
 
	   // tu sa dane do bazyz mienne przechowujace dane takie jak haslo nazwa bazy uzytko
wnika dajemy jako global	
	include('config');
 
	// klase $sql dajemy w klasie user jako global
	$sql = new sql();
	$user = new user();
 
	if($_POST['haslo1'] == $_POST['haslo2'])
	{
		$login = $_POST['login'];
		$haslo = $_POST['haslo1'];
 
		if($user -> register($login,$haslo))
		{
			echo 'zarejestrowany';
		 }
		   else
		{
			echo 'blad';
	 }
	  else
	  {
		   echo 'blad';
		}
?>
[PHP] pobierz, plaintext 

tak bym to widział
zastosuj sie do tego piszac klase users pzdr.