Witaj Gościu! ( Zaloguj | Rejestruj )

Forum PHP.pl

 
 Reply to this topicStart new topic
> [PHP] przekształcanie elseif w switch/case
Radek41
post
Post #1





Grupa: Zarejestrowani
Postów: 15
Pomógł: 0
Dołączył: 28.11.2009

Ostrzeżenie: (0%)
-----

Witam,
zrobiłem taki banalny skrypt, który porównuje liczbę 600, zapisaną w zmiennej $aktualna ze wprowadzoną wartością $pkt i następnie pokazuje jak $aktualna się zmieniła.

Wygląda to tak

[PHP] pobierz, plaintext 
  1. <?
  2. $aktualna = 600;
  3. $pkt = $_POST['pkt'];
  4. $zmiana = $pkt - $aktualna;
  5. $suma = $aktualna + $zmiana;
  6.  
  7. if ($pkt == "") echo '';
  8. 	elseif ($pkt > $aktualna) echo 'Punktow '.$suma.' (<b><span style="color: green;">+'.$zmiana.'</span></b>).';
  9. 	elseif ($pkt < $aktualna) echo 'Punktow '.$suma.' (<b><span style="color: red;">'.$zmiana.'</span></b>).';
  10.  
  11. ?>
[PHP] pobierz, plaintext


I wszystko dla mnie jest OK, osiągnąłem oczekiwany przeze mnie efekt.

Ale nurtuje mnie czy można to samo zapisać wykorzystując instrukcję warunkową switch/case.

Wykombinowałem coś takiego, co działa źle.

[PHP] pobierz, plaintext 
  1. <?
  2. $aktualna = 600;
  3. $pkt = $_POST['pkt'];
  4. $zmiana = $pkt - $aktualna;
  5. $suma = $aktualna + $zmiana;
  6.  
  7. switch ($pkt) {
  8. 	case	$pkt == "": 
  9. 			echo '';
  10. 	case	$pkt > $aktualna: 
  11. 			echo 'Punktow '.$suma.' (<b><span style="color: green;">+'.$zmiana.'</span></b>).';
  12. 	case	$pkt < $aktualna: 
  13. 			echo 'Punktow '.$suma.' (<b><span style="color: red;">'.$zmiana.'</span></b>).'; 
  14. 			 }
  15. ?>
[PHP] pobierz, plaintext



I teraz moje pytanie - czy powyższą instrukcję zapisaną za pomocą elseif można w jakiś sposób zapisać wykorzystując switch/case? A jeśli tak, to w jaki sposób poprawić to co napisałem wyżej?

Dziękuję z góry,
Pozdrawiam

Ten post edytował Radek41 30.01.2010, 18:23:27
Go to the top of the page
+Quote Post
Blame
post
Post #2





Grupa: Zarejestrowani
Postów: 678
Pomógł: 124
Dołączył: 26.09.2009

Ostrzeżenie: (0%)
-----

Bo sprawdzanie w case musisz umieścić w nawiasach, tzn.
[PHP] pobierz, plaintext 
  1. case ($pkt > $aktualna):
[PHP] pobierz, plaintext
Go to the top of the page
+Quote Post
Radek41
post
Post #3





Grupa: Zarejestrowani
Postów: 15
Pomógł: 0
Dołączył: 28.11.2009

Ostrzeżenie: (0%)
-----

Cytat(Blame @ 30.01.2010, 18:58:05 ) *
Bo sprawdzanie w case musisz umieścić w nawiasach, tzn.
[PHP] pobierz, plaintext 
  1. case ($pkt > $aktualna):
[PHP] pobierz, plaintext


Mimo wszystko, podczas gdy używając if i nic nie wprowadzając do formularza ($pkt), nic się nie wyświetla

- zgodnie z 
[PHP] pobierz, plaintext 
  1. if ($pkt == "") echo '';
[PHP] pobierz, plaintext


to używając switch/case, pokazuje się "Punktow 0 (+-600).Punktow 0 (-600)." naraz, czyli omija nie tylko:

[PHP] pobierz, plaintext 
  1. case	($pkt == ""):
[PHP] pobierz, plaintext


Ale także źle interpretuje pozostałe warunki.

Ten post edytował Radek41 30.01.2010, 19:32:40
Go to the top of the page
+Quote Post
Blame
post
Post #4





Grupa: Zarejestrowani
Postów: 678
Pomógł: 124
Dołączył: 26.09.2009

Ostrzeżenie: (0%)
-----

No ale ja mimo wszystko poszedłbym w stronę zoptymalizowania if a nie przerabiania tego na switch bo to IMHO jest bez sensu.
Ja bym to zrobił tak:
[PHP] pobierz, plaintext 
  1. <?
  2. $aktualna = 600;
  3. if (!empty($_POST['pkt']){ //1
  4. $pkt = $_POST['pkt'];
  5. $zmiana = $pkt - $aktualna;
  6. $suma = $aktualna + $zmiana;
  7.  
  8. if ($pkt > $aktualna) echo 'Punktow '.$suma.' (<b><span style="color: green;">+'.$zmiana.'</span></b>).';
  9. 	elseif ($pkt < $aktualna) echo 'Punktow '.$suma.' (<b><span style="color: red;">'.$zmiana.'</span></b>).';
  10. }
  11. else{ //2
  12. }
  13. ?>
[PHP] pobierz, plaintext

Na tę chwilę wydaje mi się to najprostszym rozwiązaniem.
1. Sprawdzamy czy $_POST['pkt'] nie jest puste. Jeśli tak to robimy sprawdzenie.
2. Tu wrzucamy kod który się wykona gdy jednak będzie.
Go to the top of the page
+Quote Post
Radek41
post
Post #5





Grupa: Zarejestrowani
Postów: 15
Pomógł: 0
Dołączył: 28.11.2009

Ostrzeżenie: (0%)
-----

Dzięki za kod, napewno z niego skorzystam. (IMG:style_emoticons/default/smile.gif)

Nawet nie chciałem korzystać ze switch/case, intryguje mnie tylko, czy da się to w ten sposób zapisać. (IMG:style_emoticons/default/tongue.gif)

Ten post edytował Radek41 30.01.2010, 19:54:35
Go to the top of the page
+Quote Post
Fifi209
post
Post #6





Grupa: Zarejestrowani
Postów: 4 655
Pomógł: 556
Dołączył: 17.03.2009
Skąd: Katowice

Ostrzeżenie: (0%)
-----

A umiecie używać switch ?

[PHP] pobierz, plaintext 
  1. switch($zmienna) {
  2.  
  3. case 'wartosc':
  4. // czynnosc
  5. break;
  6.  
  7. default:
  8. // domyslnie
  9. break;
  10. }
[PHP] pobierz, plaintext


Druga sprawa:
[PHP] pobierz, plaintext 
  1. if ($pkt == "") echo '';
[PHP] pobierz, plaintext


A empty ?

Ogólnie:
[PHP] pobierz, plaintext 
  1.  
  2. if (!empty($pkt)) {
  3. if ($pkt > $aktualna) {
  4. // wieksza
  5. }else{
  6. // mniejsza
  7. }
  8. }else{
  9. //pusto
  10. }
[PHP] pobierz, plaintext


Go to the top of the page
+Quote Post
Blame
post
Post #7





Grupa: Zarejestrowani
Postów: 678
Pomógł: 124
Dołączył: 26.09.2009

Ostrzeżenie: (0%)
-----

No wszystko fajnie tylko twój skrypt z if'ami nie uwzględnia jednej, chyba najważniejszej rzeczy a mianowicie:
Cytat(Radek41 @ 30.01.2010, 18:11:05 ) *
[...]i następnie pokazuje jak $aktualna się zmieniła.

bo else wykona się nawet jak będą identyczne, więc i tak trzeba dodać jeszcze jeden warunek. Można by pokombinować z operatorem różności "<>" ale z tego co widzę to autor chce uzyskać różny kolor w zależności czy $aktualna jest mniejsza czy większa.
Go to the top of the page
+Quote Post
Radek41
post
Post #8





Grupa: Zarejestrowani
Postów: 15
Pomógł: 0
Dołączył: 28.11.2009

Ostrzeżenie: (0%)
-----

Cytat(Blame @ 31.01.2010, 08:59:36 ) *
No wszystko fajnie tylko twój skrypt z if'ami nie uwzględnia jednej, chyba najważniejszej rzeczy a mianowicie:

bo else wykona się nawet jak będą identyczne, więc i tak trzeba dodać jeszcze jeden warunek. Można by pokombinować z operatorem różności "<>" ale z tego co widzę to autor chce uzyskać różny kolor w zależności czy $aktualna jest mniejsza czy większa.


Dodałem już to wcześniej, tutaj wstawiłem troszkę skrócony kod. (IMG:style_emoticons/default/winksmiley.jpg)

Mimo wszystko dziękuję za zwrócenie na to uwagi.
Go to the top of the page
+Quote Post
« Następny starszy · Przedszkole · Następny nowszy »
 

Reply to this topicStart new topic
2 Użytkowników czyta ten temat (2 Gości i 0 Anonimowych użytkowników)
0 Zarejestrowanych:

 

Tryb wyświetlania: Standardowy · Przełącz na: Linearny+ · Przełącz na: Drzewo

Śledź ten temat · Wyślij temat na e-mail · Wydrukuj ten temat · Subskrybuj to forum

RSS Aktualny czas: 25.08.2025 - 19:41
Powered By IP.Board © 2025  IPS, Inc.
All changes by PHP.pl Administrators
Hosting zapewnia NQ.pl hosting, trac, svn