Return, jaki sens, jakie zadania? Opcje

[teez]

Witam!

Właśnie rozpoczynam naukę OOP z książką PHP obiekty, wzorce, narzędzia. Po paru lekcjach otrzymałem m.innymi taki kod:

[PHP] pobierz, plaintext 
<?php
class ShopProduct {
	public $numPages;
	public $playLength;
	public $title;
	public $producerMainName;
	public $price;
 
	function __construct($title, $firstName, $price, $numPages = 0, $playLength = 0){
		$this->title				= $title;
		$this->producerMainName		= $firstName;
		$this->price 				= $price;
		$this->numPages				= $numPages;
		$this->playLength			= $playLength;
	}
	function getProducer() {
		return "{$this->producerMainName}";
	}
	function getSummaryLine() {
		$base = "{$this->title} ( {$this->producerMainName} )";
		return;
	}
}
 
 
?>
[PHP] pobierz, plaintext 

I teraz mam pytanie, po co return? Do czego tu służy? Jakie ma zadanie? Pewnie błaha sprawa, ale przeszukując google natrafiam jedynie na zagraniczne witryny. Pozdrawiam.

[buliq]

hmm, metoda jest niepełna. Co chcesz zrobić z tym $base? Co to za próba wywołania?

$this->title ( {$this->producerMainName} )

To przecież nie ma sensu.

[teez]

Mniejsza o kod, chodzi mi o zadania return. Kiedy go używać i do czego głównie służy.

Trochę potestowałem i użyłem return do

[PHP] pobierz, plaintext 
	function drukuj(){
		return "{$this->title}</br>".
			   "{$this->producerMainName}</br>".
			   "{$this->price}</br>".
			   "{$this->numPages}</br>";
	}
[PHP] pobierz, plaintext 

Lecz, czy ma ono jeszcze jakieś inne zadania?

Ten post edytował teez 28.02.2013, 18:05:02

[viking]

Do natychmiastowego przerwania wykonywania funkcji i zwrócenia wartości. Jeśli tak wygląda kod w kursie to go zwyczajnie wyrzuć do kosza.

[teez]

Kod w kursie tak nie wygląda. Sam po prostu pomieszałem. Ok, temat do zamknięcia.

[buliq]

Polubiłbym post wyżej.

To co wyrzuca return uzyskujesz w miejscu wywołania, musisz tego jeszcze użyć jakoś np echo $this->drukuj();

[Fifi209]

I takie coś:

[PHP] pobierz, plaintext 
return "{$this->producerMainName}";
[PHP] pobierz, plaintext 

Polecałbym zmienić na:

[PHP] pobierz, plaintext 
return $this->producerMainName;
[PHP] pobierz, plaintext 

Unikniesz wielu kłopotów, a poza tym... Doczytaj o używaniu cudzysłowów "

[matix]

A ja od siebie dorzucę:

[PHP] pobierz, plaintext 
function getSummaryLine() {
	return sprintf("%s (%s)", $this->title, $this->producerMainName);
}
[PHP] pobierz, plaintext 

Cóż za code review:D

Ten post edytował matix 18.03.2013, 09:19:58

[Fifi209]

@up
To akurat jest poprawne, mam na myśli sposób formatowania.

[thek]

True... Poprawny i często stosowany, zwłaszcza w przypadku wyjątków.