[OOP] Dodawanie poprzedniej wartości zmiennej - jak wyzerować Opcje

[GreeN_DG]

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'] ?

Ten post edytował GreeN_DG 9.02.2011, 16:46:34

[TheTester]

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?

[GreeN_DG]

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 

Ten post edytował GreeN_DG 9.02.2011, 16:28:27

[kalmaceta]

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.

Ten post edytował kalmaceta 9.02.2011, 17:50:06

[GreeN_DG]

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).

Ten post edytował GreeN_DG 9.02.2011, 18:25:15

[franki01]

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 

[kalmaceta]

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

Ten post edytował kalmaceta 10.02.2011, 13:01:33

[GreeN_DG]

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.