Witam. Mam pewien problem :

Powiedzmy że mam taką metodę klasy "imgfile" :

[PHP] pobierz, plaintext 
	public function isExistFile($imgid)
	{
		$this->filelink .= $imgid.'.jpg';
		if(file_exists($this->filelink))
		{
			return $this->filelink;	
		}
		else
		{
			return $this->filelink;
		}			
	}
[PHP] pobierz, plaintext 

Teraz w innej klasie wywołuję tą metodę :

[PHP] pobierz, plaintext 
		$upload = new imgfile;
 
			while ($row = $wynik->fetch_assoc())
			{
 
				if($akcja = $upload->isExistFile($row['imgid']))
				{
					print_r($akcja);
					array_push($this->imgtab, $row); 
				}
				else
				{
				...
                                ...
				}
			}
[PHP] pobierz, plaintext 

$row['imgid'] - zawiera kolejne id obrazków np. 1,2,3,4

Po odpaleniu :

Daje mi takie coś :

[PHP] pobierz, plaintext 
../uimages/28.jpg../uimages/28.jpg29.jpg../uimages/28.jpg29.jpg30.jpg../uimages/28.jpg29.jpg30.jpg31.jpg
[PHP] pobierz, plaintext 

Zamiast zwrócić :

[PHP] pobierz, plaintext 
../uimages/28.jpg../uimages/29.jpg../uimages/30.jpg
[PHP] pobierz, plaintext 

Gdy w pętle 'while' dodam za każdym razem - $upload = new imgfile to problem się rozwiązuje (ale po co na nowo tworzyć obiekt klasy). Dlaczego metoda zwraca sumę kolejnych $row['imgid'] ?

Przypuszczam, że chodzi o ten fragment:

[PHP] pobierz, plaintext 
$this->filelink .= $imgid.'.jpg';
[PHP] pobierz, plaintext 

Czy jak usuniesz kropkę przed znakiem "=", to działa tak jak chcesz?

Po pierwsze proszę o przeniesienie --> OOP (pomyliłem dział)

Kropka ma myć :

Pole $filelink posiada wartość :

[PHP] pobierz, plaintext 
private $filelink = '../uimages/';
[PHP] pobierz, plaintext 

Więc :

[PHP] pobierz, plaintext 
$this->filelink .= $imgid.'.jpg';
[PHP] pobierz, plaintext 

Gdy np. $imgid = 1;

Ma to dać :

[PHP] pobierz, plaintext 
../uimages/1.jpg
[PHP] pobierz, plaintext 

Niestety za drugim wywołaniem metody pole $filelink wygląda tak :

[PHP] pobierz, plaintext 
../uimages/1.jpg2.jpg
[PHP] pobierz, plaintext 

Zamiast :

[PHP] pobierz, plaintext 
../uimages/2.jpg
[PHP] pobierz, plaintext 

masz jeden obiekt w którym jest pole filelink , do którego cały czas dodajesz ciągi znaków przez tą kropkę ".=". Najprzejrzyściej jest tworzyć obiekt w pętli while. Ew. możesz dodać metodę czysc() która ustawia filelink na domyślną wartość, ale wtedy jaki jest sens istnienia obiektu.

Teraz zrozumiałem ten problem. Jak będzie się miało do wydajności jeśli za każdym razem w pętli while będzie tworzony nowy obiekt (na szczęście nie kolejny bo ten nowy będzie zastępował stary, bowiem zmienna wskazująca będzie taka sama).

Po co tworzyć nowy obiekt? Jeżeli przy inicjacji obecnego obiektu masz domyślą wartość $this->filelink, do której tylko dopisujesz nazwę pliku, zrób inaczej. W klasie stwórz zmienną $this->fileLinkDefault i mu przypisz domyślną wartość, a potem zamiast:

[PHP] pobierz, plaintext 
$this->filelink .= $imgid.'.jpg';
[PHP] pobierz, plaintext 

daj:

[PHP] pobierz, plaintext 
$this->filelink = $this->filelinkDefault.$imgid.'.jpg';
[PHP] pobierz, plaintext 

@franki01 z takim podejściem jak prezentujesz do OOP - zadaj pytanie po co w ogóle obiekt.

Przyznam się szczerze, że zamotałem się a rozwiązanie było takie proste. Doszukiwałem się sam nie wiem czego a jak @franki01 pisał tu chodziło o przypisywanie wartości do zmiennej, no ale fakt faktem w tym wypadku sens istnienia obiektu jest znikomy. Wystarczy w tamtej klasie sprawdzić "file_exists" bez tworzenia obiektu.