[PHP]Liczba spełniająca warunki Opcje

[Daiszman]

czy ktoś może mnie oświecić
Mam liczbę $interval musi ona spełnić warunki że jest miedzy np -1 a -10 oraz jest przypisana do $doc = P

[PHP] pobierz, plaintext 
if($interval >= 0 && $doc['progress'] === "P") { echo'<div ><img src="ikony/1.png" width="15px" hight="15px";></div>';
				} elseif($interval <= -1 && $doc === "P") { echo'<div class="zegar" ><img src="ikony/2.png" width="15px" hight="15px";>'.$interval.'</div>';
				} elseif($interval <= -10 && $doc === "P") { echo'<div class="zegar" ><img src="ikony/3.png" width="15px" hight="15px";>'.$interval.'</div>';
				} elseif($interval <= -15 && $doc === "P") { echo'<div class="zegar" ><img src="ikony/4.png" width="15px" hight="15px";>'.$interval.'</div>';
				} elseif($interval <= -20 && $doc === "P") { echo'<div class="zegar" ><img src="ikony/5.png" width="15px" hight="15px";>'.$interval.'</div>';
				} elseif($interval <= -45 && $doc === "P") { echo'<div class="zegar" ><img src="ikony/6.png" width="15px" hight="15px";>'.$interval.'</div>';
				} else echo '<div>N/A</div>';
[PHP] pobierz, plaintext 

chciałem dać elseif($interval <= -15 && $interval >=-10 && $doc === "P" ale wysypuje
co robie źle bardzo źle , moja wina?

Ten post edytował Daiszman 9.03.2020, 10:45:17

[trueblue]

[PHP] pobierz, plaintext 
$doc['progress'] === "P"
[PHP] pobierz, plaintext 

[PHP] pobierz, plaintext 
$doc === "P"
[PHP] pobierz, plaintext 

Po poprawieniu warto sprawdzić jakie wartości faktycznie mają zmienne (w tym $interval).

[nospor]

Procz tego co napisal trueblue to klania sie jeszcze prosta logika

Przeciez to
} elseif($interval <= -1 && $doc === "P") { echo'<div class="zegar" ><img src="ikony/2.png" width="15px" hight="15px";>'.$interval.'</div>';
} elseif($interval <= -10 && $doc === "P") { echo'<div class="zegar" ><img src="ikony/3.png" width="15px" hight="15px";>'.$interval.'</div>';
} elseif($interval <= -15 && $doc === "P") { echo'<div class="zegar" ><img src="ikony/4.png" width="15px" hight="15px";>'.$interval.'</div>';
} elseif($interval <= -20 && $doc === "P") { echo'<div class="zegar" ><img src="ikony/5.png" width="15px" hight="15px";>'.$interval.'</div>';
} elseif($interval <= -45 && $doc === "P") { echo'<div class="zegar" ><img src="ikony/6.png" width="15px" hight="15px";>'.$interval.'</div>';
nie ma zadnego sensu.

Zalozmy ze $interval wynosi -50. Zakladam, ze bys chcial by sie wyswietlil ostatni IF, ale wyswietli sie juz pierwszy IF z tej listy bo -50 jest tez mniejsze od -1/ Czyli generalnie zaden z IFow procz tego pierwszego nigdy sie nie zalapie

[Daiszman]

[PHP] pobierz, plaintext 
$doc['progress'] === "P"
[PHP] pobierz, plaintext 

[PHP] pobierz, plaintext 
$doc === "P"
[PHP] pobierz, plaintext 

Po poprawieniu warto sprawdzić jakie wartości faktycznie mają zmienne (w tym $interval).

przybiera poprawnie

brakuje mi tej prostej logiki :|

[kreatiff]

Musisz sprawdzać warunki w odwrotnej kolejności, bo tak jak pisze nospor, jeśli $interval spełnia np. ostatni warunek to spełnia też każdy poprzedni, w tym ten pierwszy i na nim właśnie skończy się przyrównywanie. Czyli nigdy do tych kolejnych warunków nie dojdzie.

[PHP] pobierz, plaintext 
$html = '<div>N/A</div>';
 
if ($doc === 'P') {
	$class = ' class="zegar"';
	if ($interval <= -45) $ikona = 6;
	elseif ($interval <= -20) $ikona = 5;
	elseif ($interval <= -15) $ikona = 4;
	elseif ($interval <= -10) $ikona = 3;
	elseif ($interval <= -1) $ikona = 2;
	else {
		$ikona = 1;
		$class = '';
		$interval = '';
	}
 
	$html = sprintf('<div%s><img src="ikony/%d.png" width="15px" hight="15px";>%s</div>', $class, $ikona, $interval);
}
 
echo $html;
[PHP] pobierz, plaintext 

[vokiel]

Przy takiej ilości if/else warto by było dla czytelności zamienić to na switch/case.