PHP-Scripte PHP-Tutorials PHP-Jobs und vieles mehr

PHP-Scripte PHP-Tutorials PHP-Jobs und vieles mehr (https://www.php-resource.de/forum/)
-   PHP Developer Forum (https://www.php-resource.de/forum/php-developer-forum/)
-   -   if von bis (https://www.php-resource.de/forum/php-developer-forum/103940-if-von-bis.html)

cbixx 14-04-2013 19:59

if von bis
 
Hallo.

Ich lese aus der DB das level ( bzw. Points ).
Nun möchte ich das alle levels die über 99 sind 1.jpg als bild haben
alle die über 199 sind 2.jpg
alle die über 299 sind 3.jpg
etc.

Mein Code ist wie folgt:
PHP-Code:

function level($level
    {
        if (
$level 99)             {return ">img src=\"images/rank/1.jpg\" width=\"31px\" height=\"31px\" alt=\"\">";}
        else if (
$level 199)         {return ">img src=\"images/rank/2.jpg\" width=\"31px\" height=\"31px\" alt=\"\">";}
        else if (
$level 299)         {return ">img src=\"images/rank/3.jpg\" width=\"31px\" height=\"31px\" alt=\"\">";}
        else if (
$level 399)         {return ">img src=\"images/rank/4.jpg\" width=\"31px\" height=\"31px\" alt=\"\">";}
        else if (
$level 499)         {return ">img src=\"images/rank/5.jpg\" width=\"31px\" height=\"31px\" alt=\"\">";}
        else if (
$level 699)         {return ">img src=\"images/rank/6.jpg\" width=\"31px\" height=\"31px\" alt=\"\">";}
        else if (
$level 799)         {return ">img src=\"images/rank/7.jpg\" width=\"31px\" height=\"31px\" alt=\"\">";}
        else if (
$level 899)         {return ">img src=\"images/rank/8.jpg\" width=\"31px\" height=\"31px\" alt=\"\">";}
        else if (
$level 999)         {return ">img src=\"images/rank/9.jpg\" width=\"31px\" height=\"31px\" alt=\"\">";}
        else if (
$level 1099)     {return ">img src=\"images/rank/10.jpg\" width=\"31px\" height=\"31px\" alt=\"\">";}
        else if (
$level 1199)     {return ">img src=\"images/rank/11.jpg\" width=\"31px\" height=\"31px\" alt=\"\">";}
        else if (
$level 1299)     {return ">img src=\"images/rank/12.jpg\" width=\"31px\" height=\"31px\" alt=\"\">";}
        else if (
$level 1399)     {return ">img src=\"images/rank/13.jpg\" width=\"31px\" height=\"31px\" alt=\"\">";}
        else if (
$level 1499)     {return ">img src=\"images/rank/14.jpg\" width=\"31px\" height=\"31px\" alt=\"\">";}
        else if (
$level 1599)     {return ">img src=\"images/rank/15.jpg\" width=\"31px\" height=\"31px\" alt=\"\">";}
    } 

Allerdings zeigt er mir immer das erste Bild ( 1.jpg ) an auch wenn man ( ich ) mehr als 500 Points habe

h3ll 14-04-2013 20:04

Falsche Reihenfolge. Jeder Level über 99 (auch 1600) liefert bei der ersten Abfrage TRUE, somit wird PHP nie in den else-Zweig kommen.

Davon abgesehen ist der Code unnötig aufgebläht. Warum nicht so:

PHP-Code:

function get_ranking_image_for_level($level) {
    
$rank floor($level 100);
    return 
'<img src="http://example.org/images/rank/' $rank '.jpg" width="31" height="31" alt="" />';


Die "px" haben bei width und height übrigens nichts verloren.

Außerdem ist deine Funktion irreführend benannt. Ein Funktionsname sollte immer ein Zeitwort beinhalten, weil eine Funktion ja etwas tut und der Name sollte dies auch aussagen.

cbixx 14-04-2013 20:12

Danke für die antwort,
das mit den functions namen, wenn ich nur ein paar functions brauche bennene ich sie immer kurz.

Sollte ich aber viele haben, so bekommen sie auch eine genauere beschreibung.

ein / am ende meines img tags ist nicht nötig, da ich mit HTML5 arbeite

h3ll 14-04-2013 20:19

Zitat:

Zitat von cbixx (Beitrag 665856)
Danke für die antwort,
das mit den functions namen, wenn ich nur ein paar functions brauche bennene ich sie immer kurz.

Es ist egal, ob es 5 oder 5000 Funktionen sind. Code muss immer lesbar bleiben. Und auch wenn du sagst, dir selber ist das egal (weil du dir lieber unnötig das Leben schwer machen willst, oder warum auch immer), sobald du Code in einem Forum postest, sollte er für alle Programmierer lesbar sein und nicht nur für dich selber. Das ist hier in dem Fall nicht wirklich kritisch, aber schlechte Gewohnheiten behält man meist.

cbixx 14-04-2013 20:22

Zitat:

Zitat von h3ll (Beitrag 665857)
Es ist egal, ob es 5 oder 5000 Funktionen sind. Code muss immer lesbar bleiben. Und auch wenn du sagst, dir selber ist das egal (weil du dir lieber unnötig das Leben schwer machen willst, oder warum auch immer), sobald du Code in einem Forum postest, sollte er für alle Programmierer lesbar sein und nicht nur für dich selber. Das ist hier in dem Fall nicht wirklich kritisch, aber schlechte Gewohnheiten behält man meist.

Werde mir dies merken, und in zukunft functions besser bennen :)

Melewo 14-04-2013 20:45

Am besten Du benennst die mit foo() und bar(), ist aber nicht unbedingt ernst gemeint. Ein eindeutiger Name ist schon besser. Auch ging es darum, den Code besser zu gestalten. Wenn Du den Code so beibehalten möchtest, so solltest Du von-bis-Bereiche abfragen, denke ich mir.

PHP-Code:

// so:

($level 99  and $level <= 199)
(
$level 199 and $level <= 299)
(
$level 299 and $level <= 399)

// oder:

($level 99  and $level 200)
(
$level 199 and $level 300)
(
$level 299 and $level 400


h3ll 14-04-2013 20:51

Zitat:

Zitat von Melewo (Beitrag 665859)
Am besten Du benennst die mit foo() und bar(), ist aber nicht unbedingt ernst gemeint. Ein eindeutiger Name ist schon besser. Auch ging es darum, den Code besser zu gestalten. Wenn Du den Code so beibehalten möchtest, so solltest Du von-bis-Bereiche abfragen, denke ich mir.

PHP-Code:

// so:

($level 99  and $level <= 199)
(
$level 199 and $level <= 299)
(
$level 299 and $level <= 399)

// oder:

($level 99  and $level 200)
(
$level 199 and $level 300)
(
$level 299 and $level 400


IMHO nur unnötige Redundanzen. Warum sollte man das wollen?

Melewo 14-04-2013 21:09

Ist doch nur für den Fall gedacht, dass cbixx seinen bisherigen Code beibehalten und nur ergänzen möchte. Klar geht das, so wie mit Deinem Beispiel, auch kürzer.


Alle Zeitangaben in WEZ +2. Es ist jetzt 17:47 Uhr.

Powered by vBulletin® Version 3.8.2 (Deutsch)
Copyright ©2000 - 2019, Jelsoft Enterprises Ltd.
Search Engine Friendly URLs by vBSEO 3.3.0
[c] ebiz-consult GmbH & Co. KG