Как заключить повторяющийся код в функцию?

Question

[newaitix]

$prc=3899;
$prefix1='88Qn|Q*wcIa5';
$prefix2='66{Rvi#wy?r?#1';
$width=200;
$typ='grp';
$cur='uk';
$rrc=3906;
$hdt=2;

$code=urlencode(base64_encode(json_encode([
	'encode'	=>	$prefix1.$prc.$prefix2,
	'width'		=>	$width,
	'typ'		=>	$typ,
	'cur'		=>	$cur,
	'rrc'		=>	$rrc,
	'hdt'		=>	$hdt
])));

echo"<img src='text_to_img.php?w=$code'>";

text_to_img.php

<?php
$encod_array=$_GET['w'];
$decode_array=json_decode(base64_decode(urldecode($encod_array)));

$width=$decode_array->{'width'};
$encode=$decode_array->{'encode'};
$type=$decode_array->{'typ'};
$cur=$decode_array->{'cur'};
$rrc=$decode_array->{'rrc'};
$height=$decode_array->{'hdt'};

$prefix1='88Qn|Q*wcIa5';
$prefix2='66{Rvi#wy?r?#1';
$font_file='arial.ttf';
$lan=array(
		'uk'=>' грн.',
		'ru'=>' грн.',
		'en'=>' hrn.'
	);

if(
	isset($width)&&!empty($width)&&
	isset($encode)&&!empty($encode)&&
	isset($type)&&!empty($type)&&
	isset($cur)&&!empty($cur)&&
	isset($height)&&!empty($height)
){
	$p1_test=strripos($encode,$prefix1);
	$p2_test=strripos($encode,$prefix2);

	if($p1_test===false||$p2_test===false)exit;
	if(gettype($lan[$cur])==NULL)exit;
	if($type!='grp'&&$type!='txt')exit;
	if($width==0)exit;

	$p1_split=str_replace($prefix1,"",$encode);
	$p2_split=str_replace($prefix2,"",$p1_split);

	$price_smash=strrev(implode(' ',str_split(strrev($p2_split),3)));
	$rrc_smash=strrev(implode(' ',str_split(strrev($rrc),3)));

	$price_ready=$price_smash.$lan[$cur];
	$rrc_ready=$rrc_smash.$lan[$cur];

	$price_ln=strlen($p2_split);

function test(){
	$w_1_symbol=13.9;
	$image=imagecreatetruecolor($width,26);
	$background=imagecolorallocate($image,255,255,255);
	$color=imagecolorallocate($image,57,181,74);
	imagefilledrectangle($image,0,0,299,99,$background);
	imagefttext($image,17,0,($width-$price_ln*$w_1_symbol-42)/2,21,$color,$font_file,$price_ready);
	return $image;
}

	if($height==2){
		if($type=='grp'){
			$image=test();
		}
		if($type=='txt'){
			$w_1_symbol=7;
			$image=imagecreatetruecolor($width,15);
			$background=imagecolorallocate($image,255,255,255);
			$color=imagecolorallocate($image,0,0,0);
			imagefilledrectangle($image,0,0,299,99,$background);
			imagefttext($image,9,0,($width-$price_ln*$w_1_symbol-27)/2,12,$color,$font_file,$price_ready);
		}
	}
	if($height==3){
		if($type=='grp'){
			$w_1_symbol=13.9;
			$image=imagecreatetruecolor($width,47);
			$background=imagecolorallocate($image,255,255,255);
			$color=imagecolorallocate($image,57,181,74);
			imagefilledrectangle($image,0,0,299,99,$background);
			imagefttext($image,17,0,($width-$price_ln*$w_1_symbol-42)/2,40,$color,$font_file,$price_ready);
		}
	}
	if($height==4&&isset($rrc)&&!empty($rrc)){
		if($type=='grp'){
			$w_1_symbol=13.9;
			$image=imagecreatetruecolor($width,47);
			$background=imagecolorallocate($image,255,255,255);
			$color=imagecolorallocate($image,57,181,74);
			imagefilledrectangle($image,0,0,299,99,$background);
			imagefttext($image,17,0,($width-$price_ln*$w_1_symbol-42)/2,40,$color,$font_file,$price_ready);
			if($rrc){
				$w_1_symbol=7;
				$color=imagecolorallocate($image,153,0,0);
				imagefttext($image,10,0,($width-$price_ln*$w_1_symbol-27)/2,14,$color,$font_file,$rrc_ready);
				$rtrgtfre='_';
				$gerfkerng='';
				for($i=0;(strlen($rrc)*7+27)/7>$i;$i++)
					$gerfkerng=$gerfkerng.$rtrgtfre;
				$gerfkerng=$gerfkerng.$rtrgtfre;
				imagefttext($image,9,0,($width-$price_ln*$w_1_symbol-27)/2-3,8,$color,$font_file,$gerfkerng);
			}
		}
	}
	header('Content-Type:image/png');
	imagepng($image);
	imagedestroy($image);
}
?>

вызывается
$image=test();
Почему не работает ? Как правильно написать чтоб не плодить повторяющийся код ?

Comments

[DieZz]

А вы пробелами принципиально не пользуетесь? :)

[newaitix]

DieZz, Да. Мне не удобно читать с пробелами код.

[DieZz]

newaitix, точтонаписаноувасчитатьоченьсложно-всесливаетсявединуюстроку

[DieZz]

А можно узнать, ради любопытства, что значит переменная $gerfkerng?

[alex-1917]

браза, твой код уже можно ужать в ДВА раза, начал бы с этого!!!

[newaitix]

alex-1917, Как?

[alex-1917]

newaitix, начни с этой строчки:

$background=imagecolorallocate($image,255,255,255);

Найди эту строку и удали три такие строки, начиная с конца
Она явно лишняя там. И т.д.
Думаешь, я тебе буду жевать весь код? Такую кашу сам вари.

Answer 1

[DieZz]

Проблема в области видимости. Ознакомьтесь php.net/manual/ru/language.variables.scope.php
Упрощенно ваша функция не видит переменных $width, $image и т.д. Эти переменные надо передавать как аргументы функции

function someFunction($width)
{
    return ++$width;
}

либо расширять область видимости до глобальной

function someFunction()
{
    global $width;
    // do something with $width
}

что в принципе не очень хорошо. Вариант с аргументами функции предпочтительнее, но в вашем случае их получится много, что тоже нехорошо

Comments

[alex-1917]

подозреваю, что половина аргументов таки не меняется от раза к разу)))

но с другой стороны, почему бы не сделать так))):
var args = [];

[DieZz]

Таскать массивы в качестве аргументов тоже не круто. Но в данном случае, думаю, можно.

[newaitix]

Странно. Я уже пробовал делать таким образом. Но не получалось. В этот раз получилось.

[alex-1917]

DieZz, в такой каше это один из двух единственных вариантов, к бабке не ходи...