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

[ctom]

może tak ...

Kod

$range =array(0 =>  $value1*0.34,
            50    =>    17 , // $i8
            51    =>    $value1*0.28,
            100    =>    31, // $i7
            //.....i tak dalej
            );
            
if($value1 < 0 ) {    
    $wysokosc1 = 0; // $i9
}
else {
    foreach($range as $k=>$v){
        if($value1 > $k) $wysokosc1 = $v;
    }
}

[Michael2318]

Pętla for, mnożniki tylko musisz zapisać jeszcze do tablicy.

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

[MenTo(S)]

W sumie fakt nie napisałem co za zadanie ma spełniać kod który wrzuciłem.
Otóż chodzi o to że jeżeli $wartosc ktora jest przykładowo x > 101 wtedy wartość $wysokosc ma byc $value*0.3 czyli 100*0.3.
Wtedy $wysokosc okresla mi wysokosc danego <div>.
Jako że jestem początkującym zrobiłem to jak zrobiłem ale działało .

Reasumując używając tego co podrzucił ctom niezbyt chce to działać.

[PHP] pobierz, plaintext 
	$range = array(0 => $value2*0.34, 50 => 17, 51 => $value2*0.28,
		100 => 31, 101 => $value2*0.32, 150 => 17, 151 => $value2*0.32,
		200 => 61, 201 => $value2*0.3, 250 => 76, 251 => $value2*0.3,
		300 => 91, 301 => $value2*0.3, 350 => 107, 351 => $value2*0.3,
		400 => 121);
 
	if($value2 < 0 ) {    
		$wysokosc1 = null;
	} else {
		foreach($range as $k => $v){
			if($value2 > $k) $wysokosc1 = $v;
		}
	}
[PHP] pobierz, plaintext 

Słupek pierwszy to mój kod "spaghetti"
Natomiast drugi jest to kod który podrzucił właśnie ctom

Ten post edytował MenTo(S) 1.08.2014, 11:21:59

[SmokAnalog]

Mam dziwne przeczucie, że to można zapisać w króciutkim wyrażeniu. Możesz dokładnie wytłumaczyć o co chodzi z $value1 i $value2?

[MenTo(S)]

$value1,2,3,4 w moim przypadku to będzie pobierana liczba osób z bazy danych.

[SmokAnalog]

Dlaczego wysokość rośnie tak nieregularnie, tzn. raz o 14px, raz o 15px, raz o 16px w ramach jednego przedziału? Czy to zaniedbanie czy jakieś celowe działanie? Bo coś mi się wydaje, że wystarczyłoby to zapisać jednym wyrażeniem arytmetycznym.

[MenTo(S)]

Dlaczego wysokość rośnie tak nieregularnie, tzn. raz o 14px, raz o 15px, raz o 16px w ramach jednego przedziału? Czy to zaniedbanie czy jakieś celowe działanie? Bo coś mi się wydaje, że wystarczyłoby to zapisać jednym wyrażeniem arytmetycznym.

Tak to zaniedbanie zapewne to poprawie co do kodu może rosnąc równomiernie dla każdego przedziału.

[SmokAnalog]

Najładniej byłoby określić maksymalną możliwą wysokość słupka i maksymalną liczbę użytkowników widoczną na wykresie, np.:

[PHP] pobierz, plaintext 
$maxUsers = 500;
$maxHeight = 300;
[PHP] pobierz, plaintext 

I potem na podstawie liczby użytkowników (dajmy na to $users), nadawać słupkowi odpowiednią wysokość, np.:

[PHP] pobierz, plaintext 
$height = round($users * $maxHeight / $maxUsers);
[PHP] pobierz, plaintext 

[MenTo(S)]

Najładniej byłoby określić maksymalną możliwą wysokość słupka i maksymalną liczbę użytkowników widoczną na wykresie, np.:

[PHP] pobierz, plaintext 
$maxUsers = 500;
$maxHeight = 300;
[PHP] pobierz, plaintext 

I potem na podstawie liczby użytkowników (dajmy na to $users), nadawać słupkowi odpowiednią wysokość, np.:

[PHP] pobierz, plaintext 
$height = round($users * $maxHeight / $maxUsers);
[PHP] pobierz, plaintext 

Nie przypuszczałem że można zrobić to takim krótkim kodem
Dzięki wielkie.

[ctom]

Reasumując używając tego co podrzucił ctom niezbyt chce to działać.

kod po refaktoryzacji zwraca wyniki z założeń , a to że miałeś w nich błędy oraz kwestia co z nimi robiłeś później to już inna sprawa

[AmbitnyCzłowiek]

Wykresy rysujesz w phpie? Nie prościej użyć jakiejś biblioteki js? Z własnego doświadczenia polecam Chart.js, dobrze współpracuje z JSON'em.

[SmokAnalog]

Nie prościej użyć jakiejś biblioteki js? Z własnego doświadczenia polecam Chart.js, dobrze współpracuje z JSON'em.

Biblioteka JS współpracuje z JSON-em? A po co?

[AmbitnyCzłowiek]

robisz arraya z danymi i potem tylko echujesz to

[SmokAnalog]

Nie mam pojęcia o co Ci chodzi. W JavaScript nie ma nic takiego jak echowanie.