Allowed memory size of 8388608 bytes exhausted Opcje

[KCG]

Przy dodawaniu zdjęcia mam taki błąd:
Fatal error: Allowed memory size of 8388608 bytes exhausted (tried to allocate 5200 bytes) in D:\users\trudny\motocykl\o_czesci.php on line 136
Zwiększyłem ini_set() limit pamięci na około 50MB. Na lokalnym tego nie mam.
Kod:

[PHP] pobierz, plaintext 
<?php
$image_name = imagecreatefromjpeg($img);
			$getimg = getimagesize($img);
			list($width,$height) = $getimg;
			$image = imagecreatetruecolor($width,$height);
			imagecopyresampled($image,$image_name,0,0,0,0,$width,$height,$width,$height);
			imagejpeg($image,$img);
			imagedestroy($image);
 
			//watermark
			$getimg = getimagesize($img);
			list($width,$height) = $getimg;
			$water_src = 'images/znak_wodny.png';
			$watermark = imagecreatefrompng($water_src);
			list($wwidth,$wheight) = getimagesize($water_src);
			  $image = imagecreatetruecolor($width,$height);
			  imagecopyresampled($image,$image_name,0,0,0,0,$width,$height,$width,$height);
			  imagecopy($image,$watermark,10,10,0,0,$wwidth,$wheight);
			  imagejpeg($image,$img);
			  imagedestroy($image);
 
			//thumb
			$imageThumbName = $imgName.'t.jpg';
			$widthT = 100;
			$heightT = $widthT/$width*$height;
			$imageThumb = imagecreatetruecolor($widthT,$heightT);
			$image_name = imagecreatefromjpeg($img);
			  imagecopyresampled($imageThumb,$image_name,0,0,0,0,$widthT,$heightT,$width,$height);
			imagejpeg($imageThumb,'images/gielda/moto/'.$imageThumbName);
			//new thumb
			  imagedestroy($imageThumb);
			  imagedestroy($image_name);
			$imgThumbName = $imgName.'tt.jpg';
			$imgt = imagecreatefromjpeg($img);
			$imgTH = 140;
			$imgTW = $imgTH/$height*$width;
			$srcX = ($imgTW-100)/2;
			$imgT = imagecreatetruecolor($imgTW,$imgTH);
			imagecopyresampled($imgT,$imgt,0,0,0,0,$imgTW,$imgTH,$width,$height);
			  imagedestroy($imgt);
			  $imgT2 = imagecreatetruecolor(100,140);
			  imagecopymerge($imgT2,$imgT,0,0,$srcX,0,$imgTW,$imgTH,100);
			 imagejpeg($imgT2,'images/gielda/moto/'.$imgThumbName);
			 imagedestroy($imgT2);
			  imagedestroy($imgT);
?>
[PHP] pobierz, plaintext 

[Kowalikus]

Spróbuj pobrać czy ini_set(); zadziałał poprzez ini_get('memory_limit'); Jeśli nie działa spróbuj w htaccess dodać php_value memory_limit 32M

[vtuner]

Może lepiej popracować nad optymalizacją skryptu.

[KCG]

No tak też myślałem, bo raz działa, raz nie.
vtuner masz coś konkretnego na myśli?

[.radex]

to

[vtuner]

Najpierw musiałbym znać przeznaczenie tego skryptu, żeby wiedzieć co jak pozmieniać.

[KCG]

radex_p nie pytam, co to, tylko jak :]
Skrypt dodaje zdjęcie do ogłoszenia. Jest ograniczenie do 500kB. Zapisuje je, następnie tworzy 2 miniaturki, no i to tyle.
Jak widać po move_uploaded_file() wczytuje zdjęcie i zapisuje jeszcze raz przez imagecopyresampled, bo zauważyłem, że przez to zdjęcie ma mniejszą objętość.

[vtuner]

Wiadomo, że skrypt nie wyrabia się, bo za dużo jest tych funkcji tworzących i modyfikujących zdjęcia.

Ja bym zrobił tak:
- wgrywam oryginalne zdjęcie
- przy wyświetlaniu za pomocą odpowiedniej klasy tworzę na podstawie dostarczonego linku do zdjęcia miniaturkę (o szerokości lub wysokości podanej jako następny parametr, w zależności jak sobie zrobisz klasę. Trzeba pamiętać, że klasa powinna tworzyć proporcjonalne rozmiary miniaturki). Co prawda jest utrata na jakości zdjęcia (niewielka), ale nie jest ona tak zła jak problem z pojemnością pamięci.

Myślę, że od tej strony powinieneś podejść do tego.

Ten post edytował vtuner 28.02.2008, 20:42:49

[LonelyKnight]

- przy wyświetlaniu za pomocą odpowiedniej klasy tworzę na podstawie dostarczonego linku do zdjęcia miniaturkę (o szerokości lub wysokości podanej jako następny parametr, w zależności jak sobie zrobisz klasę. Trzeba pamiętać, że klasa powinna tworzyć proporcjonalne rozmiary miniaturki). Co prawda jest utrata na jakości zdjęcia (niewielka), ale nie jest ona tak zła jak problem z pojemnością pamięci.

Nie jestem pewien czy dobrze Ciebie zrozumiałem - proponujesz autorowi aby nie tworzył miniaturki przy uploadzie tylko przy każdym wywołaniu zdjęcia? W każdej chwili kiedy miniaturka będzie potrzebna ma ją wtedy tworzyć? Jeśli tak to pomysł jest.... brrr... niedobry Metoda na zabicie serwera

Co do problemu to na forum przewijał się x razy.

Poza tym http://www.google.pl/search?q=+Allowed+memory+size+of

Ten post edytował LonelyKnight 28.02.2008, 21:34:20

[KCG]

No co do tego tworzenia przy wyświetlaniu też nie jestem pewien, a w skrypcie raczej tylko jest to, co potrzebne... No chyba, że jakoś łatwiej można to zrobić.
LonelyKnight, jak szukałem to głównie piszą, żeby zwiększyć memory_limit, który już zwiększyłem. Na serwerze na którym jest błąd mam chyba 8 czy 16 MB ramu. Jest darmowy, docelowo będzie na płatnym, np. home.pl daje max 80MB, z tego, co piszą. Może wtedy problemu nie będzie?

[LonelyKnight]

Nie powinno być. Operacje na obrazkach zabierają dużo pamięci a jeśli nie możesz zwiększyć dostępnej pamięci to zrobisz niewiele. Proponuję Ci poczekać do momentu aż przeniesiesz sie na nowy serwer. Jeśli problem będzie dalej istniał, chociaż w to wątpię, to będziesz mógł zacząć kombinować, teraz chyba nie ma sensu.

[KCG]

W takim razie dzięki Chciałem się upewnić po prostu. Jak coś, to napiszę.

[Kocurro]

Jak już kolega wspomniał ale niedopowiedział:

1) Wgrywasz plik do odpowiedniego katalogu, obcinając go jak należy i rekompresując
2) Na wyświetlanie miniaturek tworzysz wrappera
3) Gdy wrapper otrzyma zapytanie o miniaturke sprawdza w cache'u czy ją ma
4) Jeśli ją ma to wysyła
5) Jeśli nie ma to tworzy i zapisuje w cache'u a następnie wysyła do klienta

Do tego daj ograniczenie by nikt nie mógł Ci wgrać zdjęcia o większym rozmiarze pliku niż jakiś ludzki rozmiar oraz od razu przed robieniem operacji sprawdzaj czy wymiary x, y są zgodne z jakimiś założeniami np. nie więcej niż v, w.

I jakby to powiedział mój ulubiony aktor w ulubionym filmie: nie ma ch*** fujarki we wsi

pozdr.

ps: jeśli i to nie poskutkuje to zmień serwer bo ten Cię oszukuje

ps2: a po ki diabeł robisz to:

[PHP] pobierz, plaintext 
<?php
 
$image_name = imagecreatefromjpeg($img);
			$getimg = getimagesize($img);
			list($width,$height) = $getimg;
			$image = imagecreatetruecolor($width,$height);
			imagecopyresampled($image,$image_name,0,0,0,0,$width,$height,$width,$height);
			imagejpeg($image,$img);
			imagedestroy($image);
?>
[PHP] pobierz, plaintext 

Przecież to nic nie robi - w teorii zrobi rekompresje ale ... nie zwalniasz zasobu $image_name chociażby ...

Dalej znowu tworzysz zasoby i ich nie zwalniasz ...

Przepisz to od nowa i wrzuć tutaj a powiem Ci czy dobrze robisz czy nie - wybacz, że samemu nie przepiszę Ci teraz ale siedzę w pracy.

pozdr.

Ten post edytował Kocurro 29.02.2008, 11:12:17

[KCG]

8,20,31,32,40,44,45 linie w których są usuwane zasoby, bo jeśli dobrze rozumiem chodzi o image_destroy()?
Co do ps2 to zdjęcie ma mniejszy rozmiar niż jak się tylko zapisze. (przynajmniej tak mi wychodziło)
Mógłbyś mi przybliżyć wrapper i cache?