Что делать с ошибкой PHP Fatal error: Allowed memory size exhausted?

Question

[Ambal89]

Доброго времени суток, товарищи! Подскажите пожалуйста,столкнулся с проблемой, до этого не было такого. Сделал сайт на opencart 2.3, на локальном компьютере открывается всё отлично, быстро и без проблем. Начал переносить на хостинг - пошли проблемы, а именно:
Сайт постоянно падает, а лог ошибок выдает
PHP Fatal error: Allowed memory size of 134217728 bytes exhausted (tried to allocate 33072 bytes) in /home/c/cw1709/public_html/system/library/image.php on line 26

Судя по написанному, сделал вывод, что не хватает памяти из-за огромных дурацких картинок (клиент настаивал, чтобы они были огромные и без потери качества) которые весили от 2-х до 8 метров. Начал ужимать картинки, ужал все до 150кб-800кб. По идее, это уже реальный размер. На сайте всего 140 товаров, у каждого по одной картинке. Сайт стал работать лучше, но иногда всё равно вылетает с этой ошибкой. Писал хостеру с просьбой увеличить память memory_limit - сказал, чтобы губы не раскатывал. Как выйти из данной ситуации? всего 140 товаров, а если было бы 5000?

Comments

[Владислав Лысков]

memoty_limit катастрофически мало, попробуй в корне сайта создать php.ini c memory_limit 256M, или в htaccess прописать, но многие хостеры блочат

[Ambal89]

Владислав Лысков, Да, у таймвеба заблочило, так и осталось 128М, а у джино изменилось. Жаль, что джино не очень подходит для ИМ. Может подскажете какой-нибудь хостинг не сильно дорогой но с такой возможностью?

[Владислав Лысков]

Ambal89, сам хостился у таймвеба 2 года и именно по этой причине (memory_limit) переехал на бегет, там поменять можно самостоятеятельно

[Ambal89]

Владислав Лысков, Огромное спасибо!

Answer 1

[Андрей Саныч]

пробуй менять memory_limit через htaccess. Если не сработает - шли нахер хостера

Comments

[Ambal89]

Попробовал, потом проверил phpinfo - на таймвебе остался стандартный размер 128М, на джино изменился - стал 256М и всё стало работать хорошо. Просто странно, неужели 128М не хватает для работы сайта с 140 позициями...

[Ambal89]

Ваш ответ чуть позже перенесу в решения, пока посмотрю, вдруг кто-нибудь подскажет еще какое-то удивительное решение)

[Андрей Саныч]

Ambal89, public_html/system/library/image.php on line 26

что у вас там на 26 строке?

[Ambal89]

Андрей Саныч,

if ($this->mime == 'image/gif') {
				$this->image = imagecreatefromgif($file);
			} elseif ($this->mime == 'image/png') {
				$this->image = imagecreatefrompng($file);
			} elseif ($this->mime == 'image/jpeg') {
				$this->image = imagecreatefromjpeg($file); //26 строка
			}
		} else {
			exit('Error: Could not load image ' . $file . '!');
		}

Видимо, какая-то генерация картинок или аватарок

[ThunderCat]

Ambal89, К - качество кода. При каждом заходе программно генерятся превьюшки. Т.к. картинки большие - вся эта свистопляска нехило грузит память. Есть подозрение что в методе который использует этот созданный объект изображения $this->image = NULL в конце работы с картинкой не предусмотрен.

[Ambal89]

ThunderCat, Есть какие-нибудь варианты? Это, скорее всего, воду мутит плагин лупы для картинок, но с моими знаниями php я не смогу его исправить.
PS конечно, постоянная генерация превьюх это прискорбно...

[ThunderCat]

Ambal89, надо смотреть код, там где картинка полностью изменена и готова к отдаче клиенту в виде изображения, объект обработки картинки зануляется, иначе он висит в памяти, известный "баг" гд библиотеки. Поищите в гугле на предмет "gd memory".

[Ambal89]

ThunderCat, Посмотрел в гугле - сложно понять, куда что добавлять.
На всякий случай прикладываю код, может увидите сразу, в чем косяк.

spoiler

<?php
class Image {
	private $file;
	private $image;
	private $width;
	private $height;
	private $bits;
	private $mime;

	public function __construct($file) {
		if (file_exists($file)) {
			$this->file = $file;

			$info = getimagesize($file);

			$this->width  = $info[0];
			$this->height = $info[1];
			$this->bits = isset($info['bits']) ? $info['bits'] : '';
			$this->mime = isset($info['mime']) ? $info['mime'] : '';

			if ($this->mime == 'image/gif') {
				$this->image = imagecreatefromgif($file);
			} elseif ($this->mime == 'image/png') {
				$this->image = imagecreatefrompng($file);
			} elseif ($this->mime == 'image/jpeg') {
				$this->image = imagecreatefromjpeg($file);
			}
		} else {
			exit('Error: Could not load image ' . $file . '!');
		}
	}

	public function getFile() {
		return $this->file;
	}

	public function getImage() {
		return $this->image;
	}

	public function getWidth() {
		return $this->width;
	}

	public function getHeight() {
		return $this->height;
	}

	public function getBits() {
		return $this->bits;
	}

	public function getMime() {
		return $this->mime;
	}

	public function save($file, $quality = 90) {
		$info = pathinfo($file);

		$extension = strtolower($info['extension']);

		if (is_resource($this->image)) {
			if ($extension == 'jpeg' || $extension == 'jpg') {
				imagejpeg($this->image, $file, $quality);
			} elseif ($extension == 'png') {
				imagepng($this->image, $file);
			} elseif ($extension == 'gif') {
				imagegif($this->image, $file);
			}

			imagedestroy($this->image);
		}
	}

	public function resize($width = 0, $height = 0, $default = '') {
		if (!$this->width || !$this->height) {
			return;
		}

		$xpos = 0;
		$ypos = 0;
		$scale = 1;

		$scale_w = $width / $this->width;
		$scale_h = $height / $this->height;

		if ($default == 'w') {
			$scale = $scale_w;
		} elseif ($default == 'h') {
			$scale = $scale_h;
		} else {
			$scale = min($scale_w, $scale_h);
		}

		if ($scale == 1 && $scale_h == $scale_w && $this->mime != 'image/png') {
			return;
		}

		$new_width = (int)($this->width * $scale);
		$new_height = (int)($this->height * $scale);
		$xpos = (int)(($width - $new_width) / 2);
		$ypos = (int)(($height - $new_height) / 2);

		$image_old = $this->image;
		$this->image = imagecreatetruecolor($width, $height);

		if ($this->mime == 'image/png') {
			imagealphablending($this->image, false);
			imagesavealpha($this->image, true);
			$background = imagecolorallocatealpha($this->image, 255, 255, 255, 127);
			imagecolortransparent($this->image, $background);
		} else {
			$background = imagecolorallocate($this->image, 255, 255, 255);
		}

		imagefilledrectangle($this->image, 0, 0, $width, $height, $background);

		imagecopyresampled($this->image, $image_old, $xpos, $ypos, 0, 0, $new_width, $new_height, $this->width, $this->height);
		imagedestroy($image_old);

		$this->width = $width;
		$this->height = $height;
	}

	public function watermark($watermark, $position = 'bottomright') {
		switch($position) {
			case 'topleft':
				$watermark_pos_x = 0;
				$watermark_pos_y = 0;
				break;
			case 'topcenter':
				$watermark_pos_x = intval(($this->width - $watermark->getWidth()) / 2);
				$watermark_pos_y = 0;
				break;
			case 'topright':
				$watermark_pos_x = $this->width - $watermark->getWidth();
				$watermark_pos_y = 0;
				break;
			case 'middleleft':
				$watermark_pos_x = 0;
				$watermark_pos_y = intval(($this->height - $watermark->getHeight()) / 2);
				break;
			case 'middlecenter':
				$watermark_pos_x = intval(($this->width - $watermark->getWidth()) / 2);
				$watermark_pos_y = intval(($this->height - $watermark->getHeight()) / 2);
				break;
			case 'middleright':
				$watermark_pos_x = $this->width - $watermark->getWidth();
				$watermark_pos_y = intval(($this->height - $watermark->getHeight()) / 2);
				break;
			case 'bottomleft':
				$watermark_pos_x = 0;
				$watermark_pos_y = $this->height - $watermark->getHeight();
				break;
			case 'bottomcenter':
				$watermark_pos_x = intval(($this->width - $watermark->getWidth()) / 2);
				$watermark_pos_y = $this->height - $watermark->getHeight();
				break;
			case 'bottomright':
				$watermark_pos_x = $this->width - $watermark->getWidth();
				$watermark_pos_y = $this->height - $watermark->getHeight();
				break;
		}
		
		imagealphablending( $this->image, true );
		imagesavealpha( $this->image, true );
		imagecopy($this->image, $watermark->getImage(), $watermark_pos_x, $watermark_pos_y, 0, 0, $watermark->getWidth(), $watermark->getHeight());

		imagedestroy($watermark->getImage());
	}

	public function crop($top_x, $top_y, $bottom_x, $bottom_y) {
		$image_old = $this->image;
		$this->image = imagecreatetruecolor($bottom_x - $top_x, $bottom_y - $top_y);

		imagecopy($this->image, $image_old, 0, 0, $top_x, $top_y, $this->width, $this->height);
		imagedestroy($image_old);

		$this->width = $bottom_x - $top_x;
		$this->height = $bottom_y - $top_y;
	}

	public function rotate($degree, $color = 'FFFFFF') {
		$rgb = $this->html2rgb($color);

		$this->image = imagerotate($this->image, $degree, imagecolorallocate($this->image, $rgb[0], $rgb[1], $rgb[2]));

		$this->width = imagesx($this->image);
		$this->height = imagesy($this->image);
	}

	private function filter() {
        $args = func_get_args();

        call_user_func_array('imagefilter', $args);
	}

	private function text($text, $x = 0, $y = 0, $size = 5, $color = '000000') {
		$rgb = $this->html2rgb($color);

		imagestring($this->image, $size, $x, $y, $text, imagecolorallocate($this->image, $rgb[0], $rgb[1], $rgb[2]));
	}

	private function merge($merge, $x = 0, $y = 0, $opacity = 100) {
		imagecopymerge($this->image, $merge->getImage(), $x, $y, 0, 0, $merge->getWidth(), $merge->getHeight(), $opacity);
	}

	private function html2rgb($color) {
		if ($color[0] == '#') {
			$color = substr($color, 1);
		}

		if (strlen($color) == 6) {
			list($r, $g, $b) = array($color[0] . $color[1], $color[2] . $color[3], $color[4] . $color[5]);
		} elseif (strlen($color) == 3) {
			list($r, $g, $b) = array($color[0] . $color[0], $color[1] . $color[1], $color[2] . $color[2]);
		} else {
			return false;
		}

		$r = hexdec($r);
		$g = hexdec($g);
		$b = hexdec($b);

		return array($r, $g, $b);
	}
}

[ThunderCat]

Ambal89, навскидку, все функции корректно завершаются imagedestroy, однако как все работает в комплексе, сказать сложно. Тут уже от вызовов этого объекта зависит. Сори, задача объемная, в 2 клика вопрос не решить. Как оно его использует - загадка, там пол проекта у вас наверно его пользует, где конкретно косяк только дебаг покажет. Пробуйте расставлять дебаг и смотреть размер занимаемой скриптами памяти по ходу выполнения кода, найдете где у вас большое потребление можно будет уже что-то править.

[Ambal89]

ThunderCat, Хорошо, спасибо большое!