if von bis

Einklappen
X
 
  • Filter
  • Zeit
  • Anzeigen
Alles löschen
neue Beiträge

  • 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

  • #2
    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.
    Zuletzt geändert von h3ll; 14.04.2013, 19:08.

    Kommentar


    • #3
      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
      Zuletzt geändert von cbixx; 14.04.2013, 19:14.

      Kommentar


      • #4
        Zitat von cbixx Beitrag anzeigen
        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.

        Kommentar


        • #5
          Zitat von h3ll Beitrag anzeigen
          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

          Kommentar


          • #6
            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

            Kommentar


            • #7
              Zitat von Melewo Beitrag anzeigen
              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?

              Kommentar


              • #8
                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.

                Kommentar

                Lädt...
                X