[PHP] prośba o sprawdzenie klasy Opcje

[dstp]

Jak możecie napiszcie mi co sądzicie o klasie poniżej. Na razie nie jest skończona, będzie jeszcze masa innych metod. I nie wiem czemu, ale nie chce mi wyświetlić statusu jak w metodzie check_sms_code. Pod klasą przykład użycia.

[PHP] pobierz, plaintext 
<?php
 
	class Dotpay {
 
		public $dotpay_id;
		public $status;
 
		public function __construct($dotpay_id) {
			$this->dotpay_id = $dotpay_id;
		}
 
		public function check_sms_code($code, $check, $type, $del) {
			$server_ip = $_SERVER['REMOTE_ADDR'];
			if (!empty($code) AND !empty($check) AND !empty($type) AND !empty($del)) {
				$handle = fopen("http://dotpay.pl/check_code.php?id=".$this->dotpay_id."&code=".$code."&check=".$check."&type=".$type."&del=".$del."", 'r');
				$status = fgets($handle, 8);
				fclose($handle);
				$this->status = $status;
			}
			else {
				$this->error = 'Please fill in all fields';
			}
		}
 
		public function show_error() {
			return $this->error;
		}
 
	}
 
?>
 
<?php
 
	$dotpay = new Dotpay($dotpay_id = 123123);
	echo $dotpay->check_sms_code('AAA11', 'AAA12345', 'sms', 1)->status;
 
 
?>
[PHP] pobierz, plaintext 

dokładnie chodzi o tą linijke:

[PHP] pobierz, plaintext 
echo $dotpay->check_sms_code('AAA11', 'AAA12345', 'sms', 1)->status;
[PHP] pobierz, plaintext 

Ten post edytował dstp 9.12.2011, 17:29:27

[kaem]

Ale jeśli chcesz tak pobrać status jak podałeś to check_sms_code musi Ci zwrócić aktualny obiekt, czyli po prostu return $this.

Ten post edytował kaem 9.12.2011, 17:53:46

[dstp]

no dobra działa, a jak bym chciał zwrócić kilka wartości w 1 metodzie? to jak to się robi?

np. mam metode, która pobiera informacje ze z jakiejś strony i chciałbym żeby zwracała kilka wartości, np. status, id i coś tam jeszcze.

[--kaem--]

zwracasz tablicę z tymi danymi

[CuteOne]

[PHP] pobierz, plaintext 
class nowaklasa {
 
  public $eee;
  private $www;
 
  public function a() {
 
    $this -> eee = 'aaa';
    $this -> www = '1111';
  }
}
 
$n = new nowaklasa();
$n -> a();
echo $n->eee;
echo $n->www;
[PHP] pobierz, plaintext 

Ten post edytował CuteOne 9.12.2011, 18:11:53

[dstp]

CuteOne, o to mi chodziło.

A teraz tak myślę czy nie lepiej zrobić jedną metodę, żeby pobrać wartość X i następną żeby pobrać wartość Y itd... Co o tym myślicie?

[CuteOne]

Dobrze kombinujesz poczytaj o mutatorach (setXXX i getXXX)