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.

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.

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?

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

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

Polubiłbym post wyżej.

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

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 "

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

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

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