Cześć.


Nigdy się nie zastanawiałem, czy wydajniej jest zapisywać dane kilkukrotnego(2-3 razy) użytku do zmiennej, czy zwracać za pomocą metody z return. Pytanie jest w odniesieniu do OOP.

return.

Chociaż jeszcze pewnie zależy co dana metoda robi, ale jak ma zwracać to return.

A jeżeli, zwracana wartość będzie wykorzystywana przez inne metody, np:

[PHP] pobierz, plaintext 
 
private function patch() // ścieżka do pliku
 {
  return
  $this -> home_directory .$this-> type_mime_catalog[$this ->type]. $this-> slash .$this ->file['name'];
 }
 
protected function open_file () // otwarcie pliku w celu usunięcia niebezpiecznych znaków.
	{
	  $fopen = fopen($this ->patch(), 'r');
	  $content = fread($fopen, filesize($this ->patch()));
	  fclose($fopen);
	  return $content;
	 }
[PHP] pobierz, plaintext 

Niezła sieczka

W tym przypadku patch, a raczej path może być po prostu zmienną lub stałą zdefiniowaną gdzieś na początku programu... ponieważ nie wnosi za dużo.

Co innego open_file, bo jednak do czegoś służy coś zmienia itd, ale ja bym zrobił:

(nie wiem czemu masz protected, ale może gdzieś to używasz)

[PHP] pobierz, plaintext 
public function open_file($path)
{
    //tutaj reszta Twoich instrukcji
 
    return $content;
}
[PHP] pobierz, plaintext 

Tylko, że path zmienia się w zależności od poleceń użytkownika. Ścieżka path, służ do aktualnego ustalenia ścieżki dostępowej danego katalogu na serwerze i w zależności, który katalog zostanie wybrany - path przyjmie odpowiednią ścieżkę, więc definiowanie jako stała odpada. Definiowanie jako zmienna, to właśnie nie wiem, czy jest sens zapisywać dane na jednorazową interakcje z użytkownikiem.

To jest jakieś dziwne. Path generuje się sam z siebie? To po co Ci ta metoda?
Przy takim pobieraniu danych nie ma sensu zapisywać tego co zwraca metoda do zmiennej, chyba że metoda urośnie