if von bis

[cbixx]

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:

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]

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:

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]

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]

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]

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]

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.

// 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]

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.

// 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]

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.