Skrócenie kodu Opcje

[MenTo(S)]

Witam, potrzebuje skrócić poniższy kod sam sobie jakoś nie mogę z tym poradzić.
Z góry dzięki za pomoc.

[PHP] pobierz, plaintext 
	$i = 121;
	$i2 = 107;
	$i3 = 91;
	$i4 = 76;
	$i5 = 61;
	$i6 = 47;
	$i7 = 31;
	$i8 = 17;
	$i9 = 0;
 
	if($value1 < 0){
		$wysokosc1 = $i9;
	}
	if($value1 > 0){
		$wysokosc1 = $value1*0.34;
	}
	if($value1 > 50){
		$wysokosc1 = $i8;
	}
	if($value1 > 51){
		$wysokosc1 = $value1*0.28;
	}
	if($value1 > 100){
		$wysokosc1 = $i7;
	}
	if($value1 > 101){
		$wysokosc1 = $value1*0.32;
	}
	if($value1 > 150){
		$wysokosc1 = $i6;
	}
	if($value1 > 151){
		$wysokosc1 = $value1*0.32;
	}
	if($value1 > 200){
		$wysokosc1 = $i5;
	} 
	if($value1 > 201){
		$wysokosc1 = $value1*0.3;
	}
	if($value1 > 250){
		$wysokosc1 = $i4;
	}
	if($value1 > 251){
		$wysokosc1 = $value1*0.3;
	}
	if($value1 > 300){
		$wysokosc1 = $i3;
	}
	if($value1 > 301){
		$wysokosc1 = $value1*0.3;
	}
	if($value1 > 350){
		$wysokosc1 = $i2;
	}
	if($value1 > 351){
		$wysokosc1 = $value1*0.3;
	}
	if($value1 > 399){
		$wysokosc1 = $i;
	}
[PHP] pobierz, plaintext 

Ten post edytował MenTo(S) 1.08.2014, 01:20:40

[SmokAnalog]

Ale paskudne kodzisko. Kod spaghetti pełną gębą.

Zacznij od nazwania zmiennych po ludzku, bo jakoś nie wiem co to jest $value1 czy $i5. Pomysł kolegi ctom jest dobry, ale ja bym poszedł o krok dalej. Jak widać w tych warunkach występuje inna wartość dla równości (mimo, że zapisanej jako >). Wartości równe tym przedziałowym otrzymują wysokość jako wartość stałą, a wszystkie pośrodku jako iloczyn $value1 (aha!) i jakiejś stałej. Uwzględniłbym tę zależność w projektowaniu tablicy rang, nadając jej kształt na zasadzie:

[PHP] pobierz, plaintext 
$range = array(
	// ...
 
	101 => array(
		'fixed'      => 31,
		'multiplier' => 0.32
	),
 
	// ...
);
[PHP] pobierz, plaintext 

Ale najpierw napisz co ten potwór robi.