[PHP]Tworzenie klasy Opcje

[creovision]

Witam,
czy pomoże mi ktoś z utworzeniem prostej klasy PHP dla takiego kodu:

[PHP] pobierz, plaintext 
function getUserID($nickname) {
	$ds = mysql_fetch_array(safe_query("SELECT userID FROM ".PREFIX."user WHERE nickname='".$nickname."'"));
	return $ds['userID'];
}
 
function getUsername($userID) {
	$ds = mysql_fetch_array(safe_query("SELECT username FROM ".PREFIX."user WHERE userID='".$userID."'"));
	return $ds['username'];
}
 
function getUserEmail($userID) {
	$ds = mysql_fetch_array(safe_query("SELECT email FROM ".PREFIX."user WHERE userID='".$userID."'"));
	return getinput($ds['email']);
}
[PHP] pobierz, plaintext 

?
Bym mógł pobierać np. ID użytkownika na zasadzie $_user->id['username'];

[Geston]

Można to zrobić na kilka sposobów w zależności od tego do czego jeszcze będziesz używał tej klasy i jak.

Tak żeby robiła tylko to co wkleiłeś wyżej:

[PHP] pobierz, plaintext 
<?php #class_User.php
 
// klasa User
 
class User {
 
	public function __construct() {
 
	}
 
	public function getUserID($nickname) {
		$ds = mysql_fetch_array(safe_query("SELECT userID FROM ".PREFIX."user WHERE nickname='".$nickname."'"));
		return $ds['userID'];
	}
 
	public function getUsername($userID) {
		$ds = mysql_fetch_array(safe_query("SELECT username FROM ".PREFIX."user WHERE userID='".$userID."'"));
		return $ds['username'];
	}
 
	public function getUserEmail($userID) {
		$ds = mysql_fetch_array(safe_query("SELECT email FROM ".PREFIX."user WHERE userID='".$userID."'"));
		return getinput($ds['email']);
	}
}
 
?>
[PHP] pobierz, plaintext 

i w pliku

[PHP] pobierz, plaintext 
<?php
 
// dołączenie pliku z definicjami klas
require_once('../class_User.php');
 
// utworzenie obiektu klasy User
$user = new User();
 
$id = $user->getUserID['jakis_login'];
 
?>
[PHP] pobierz, plaintext 

[tomxx]

Spróbuj tak:

[PHP] pobierz, plaintext 
<?php
class User
{
	public $nickname;
	public $userID;
 
	function getUserID() {
		$ds = mysql_fetch_array(safe_query("SELECT userID FROM ".PREFIX."user WHERE nickname='".$this->nickname."'"));
		return $ds['userID'];
	}
 
	function getUsername() {
		$ds = mysql_fetch_array(safe_query("SELECT username FROM ".PREFIX."user WHERE userID='".$this->userID."'"));
		return $ds['username'];
	}
 
	function getUserEmail() {
		$ds = mysql_fetch_array(safe_query("SELECT email FROM ".PREFIX."user WHERE userID='".$this->userID."'"));
		return getinput($ds['email']);
	}
}
 
$_user = new User();
$_user->nickname = 'nick'; //ustawienie właściwości $nick
$_user->id = $_user->getUserID(); //przypisanie do właściwości ID użytkownika
echo $_user->id; //wyświetla ID użytkownika
 
?>
[PHP] pobierz, plaintext 

Ten post edytował tomxx 5.02.2014, 13:56:39

[creovision]

user.php

[PHP] pobierz, plaintext 
class User {
	public $username;
	public $userID;
 
	public function getID() {
		$data = mysql_fetch_array(safe_query("SELECT userID FROM ".PREFIX."user WHERE username='".$this->username."'"));
		return $data['userID'];
	}
}
[PHP] pobierz, plaintext 

index.php

[PHP] pobierz, plaintext 
$_user = new User();
$_user->username['admin'];
echo $_user->getID;
[PHP] pobierz, plaintext 

Otrzymuję błąd:

[HTML] pobierz, plaintext 
Notice: Undefined property: User::$getID in /home/system/domains/example.com/public_html/system/components/index.php on line 32
[HTML] pobierz, plaintext 

[Geston]

Zapomniałeś o " ()" po getID? To funkcja a nie zmienna.

[creovision]

Zapomniałeś o " ()" po getID? To funkcja a nie zmienna.

Tak wiem, już to zrobiłem zanim napisałeś ale zmieniło się tylko to iż nie otrzymuję błędu ale wyniku również nie ma

[Geston]

[PHP] pobierz, plaintext 
$_user->username['admin'];
[PHP] pobierz, plaintext 

Zmień na:

[PHP] pobierz, plaintext 
$_user->username = "admin";
[PHP] pobierz, plaintext 

[creovision]

[PHP] pobierz, plaintext 
$_user->username['admin'];
[PHP] pobierz, plaintext 

Zmień na:

[PHP] pobierz, plaintext 
$_user->username = "admin";
[PHP] pobierz, plaintext 

Ok, działa.
A co należy zrobić by wartości zmiennych podawać w formie

[PHP] pobierz, plaintext 
$_user->username['admin'];
[PHP] pobierz, plaintext 

?
Tak kod wygląda ładniej.

Może zamiast:

[PHP] pobierz, plaintext 
public $username;
public $userID;
[PHP] pobierz, plaintext 

[PHP] pobierz, plaintext 
var $username;
var $userID;
[PHP] pobierz, plaintext 

?

Ten post edytował creovision 5.02.2014, 14:20:47

[markuz]

Kod nie musi wyglądać tylko działać Poza tym nikt nie stosuje tego typu odniesień do parametrów klas..

[Geston]

Żeby było jeszcze ładniej to zrób jeszcze inaczej, bo ogólnie to komplikujesz sobie pracę, pobierasz dane z tabeli userID na podstawie nickname po to aby na podstawie userID pobrać kolejne dane z tej samej tabeli. Używając klasy możesz skrócić sobie pracę.

[PHP] pobierz, plaintext 
<?php #class_User.php
 
// klasa User
 
class User {
 
	private $userID;
	private $username;
	private $email
 
	public function __construct($nickname = "") {
		// konstruktor
 
		if ($nickname != "") {
			$ds = mysql_fetch_array(safe_query("SELECT userID, username, email FROM ".PREFIX."user WHERE nickname='".$nickname."'"));
			$this->userID = $sd['userID'];
			$this->username = $sd['username'];
			$this->email = getinput($sd['email']);
		}
	}
 
	public function __get($arg) {
		// funkcja zwracająca wartości zmiennych
 
		return $this->$arg;
	}
}
 
?>
[PHP] pobierz, plaintext 

i pliczek

[PHP] pobierz, plaintext 
<?php
 
// dołączenie plików z definicjami klas
require_once('../class_User.php');
 
$nickname = "admin" // tak czy inaczej pobrana wartość z formularza przypisana zmiennej nickname
 
// utworzenie obiektu klasy User
$user = new User($nickname);
 
$id = $user->__get("userID");
$username = $user->__get("username");
$email = $user->__get("email");
 
?>
[PHP] pobierz, plaintext 

Tworząc obiekt podajesz mu od razu nickname który pobrałeś czy to z formularza czy sam przypisałeś zmiennej. Konstruktor go przejmuje i z jego pomocą pobiera z bazy danych potrzebne pola od razu gdy tylko obiekt jest utworzony, nie trzeba wywoływać żadnych dodatkowych funkcji. Później tylko z pomocą funkcji __get() odwołujesz się do kolejnych zmiennych prywatnych w klasie.

[creovision]

Kod nie musi wyglądać tylko działać Poza tym nikt nie stosuje tego typu odniesień do parametrów klas..

Jakiego typu ? "var" ?

Żeby było jeszcze ładniej to zrób jeszcze inaczej, bo ogólnie to komplikujesz sobie pracę, pobierasz dane z tabeli userID na podstawie nickname po to aby na podstawie userID pobrać kolejne dane z tej samej tabeli. Używając klasy możesz skrócić sobie pracę.

[PHP] pobierz, plaintext 
<?php #class_User.php
 
// klasa User
 
class User {
 
	private $userID;
	private $username;
	private $email
 
	public function __construct($nickname = "") {
		// konstruktor
 
		if ($nickname != "") {
			$ds = mysql_fetch_array(safe_query("SELECT userID, username, email FROM ".PREFIX."user WHERE nickname='".$nickname."'"));
			$this->userID = $sd['userID'];
			$this->username = $sd['username'];
			$this->email = getinput($sd['email']);
		}
	}
 
	public function __get($arg) {
		// funkcja zwracająca wartości zmiennych
 
		return $this->$arg;
	}
}
 
?>
[PHP] pobierz, plaintext 

i pliczek

[PHP] pobierz, plaintext 
<?php
 
// dołączenie plików z definicjami klas
require_once('../class_User.php');
 
$nickname = "admin" // tak czy inaczej pobrana wartość z formularza przypisana zmiennej nickname
 
// utworzenie obiektu klasy User
$user = new User($nickname);
 
$id = $user->__get("userID");
$username = $user->__get("username");
$email = $user->__get("email");
 
?>
[PHP] pobierz, plaintext 

Tworząc obiekt podajesz mu od razu nickname który pobrałeś czy to z formularza czy sam przypisałeś zmiennej. Konstruktor go przejmuje i z jego pomocą pobiera z bazy danych potrzebne pola od razu gdy tylko obiekt jest utworzony, nie trzeba wywoływać żadnych dodatkowych funkcji. Później tylko z pomocą funkcji __get() odwołujesz się do kolejnych zmiennych prywatnych w klasie.

Z tym, że aktualnie mam 5 tabel w których przechowuję dane użytkowników i za pomocą tej klasy chciałbym wyciągać te dane.

[Geston]

To już sam najlepiej wiesz jakie masz tabele oraz co i jak potrzebujesz z nich pobierać. Miłej zabawy.

[creovision]

To już sam najlepiej wiesz jakie masz tabele oraz co i jak potrzebujesz z nich pobierać. Miłej zabawy.

Ok, zrobiłem to w tej formie i działa

[PHP] pobierz, plaintext 
class User {
 
	private $username;
 
	public function __construct($userID = "") {
		if(!empty($userID)) {
			$user = mysql_fetch_array(safe_query("SELECT username FROM ".PREFIX."user WHERE userID={$userID}"));
			$this->username = $user['username'];
		}
	}
 
	public function get($arg) {
		return $this->$arg;
	}
}
[PHP] pobierz, plaintext 

Dzięki wielkie za pomoc !
Serdecznie pozdrawiam.

Ten post edytował creovision 5.02.2014, 14:34:44