PHP Output Question

[Wakab]

        $rating = number_format( $imgrating, 2 );
        $ratingicon = "";
        for ( $x=0; $x < round($rating); $x++ ) {
            $ratingicon .= "<img src=\"rating.gif\">";

I'm assuming that $x++ lists $ratingicon however many times. What I want to do is set this up so that each value of $x displays a different image. For example, $x=1 changes the $ratingicon to <img src=\"rating_1.gif\"> and $x=2 changes the $ratingicon to <img src=\"rating_2.gif\">. Oh, and below is the else bit.

    else {
        $rating = ['nothing'];

[drawmack]

        $rating = number_format( $imgrating, 2 );
        $ratingicon = "";
        $ratingicon = "<img src=\"rating_" . $rating . ".gif\">";

[Wakab]

I'm recieving a Parse Error. I'm assuming the $x variable must be defined in order for the file to call properly. Is it possible to list x=1 and x=2 to call their respective images, similar to the original string, to give the output I want?

[laserlight]

not sure what is a phase error.
are you sure it isnt a parse error?

[helz]

rating = number_format( $imgrating, 2 ); 
        $ratingicon = ""; 
        for ( $x=0; $x < round($rating); $x++ ) { 
            $ratingicon .= "<img src='rating_".$x.".gif'>";
        }

paste the code around the line of the parse error please

[Wakab]

$rating = 

number_format( $imgrating, 0 );  

        $ratingicon = "";  

        for ( $x=0; $x < round($rating); $x++ ) {  

            $ratingicon .= "<img src='rating_".$x.".gif'>"; 

The output is this.

<img src='rating_0.gif'> <img src='rating_1.gif'>
<img src='rating_2.gif'> <img src='rating_3.gif'>
<img src='rating_4.gif'> <img src='rating_5.gif'>
<img src='rating_6.gif'> <img src='rating_7.gif'>
<img src='rating_8.gif'> <img src='rating_9.gif'> (10)

How do I set it so it only displays one image, not a series of images adding up to x=10?

[Weedpacket]

Don't use a loop. Just set $x to whatever you want and display.

[Wakab]

$x is the number selected in a form and submitted, $ratingicon displays the average of $x, for ( $x=0; $x < round($rating); $x++ ) is a loop condition. What do I write in order for the page to display only one image, with those variables?

[Wakab]

Are you saying that I should make the conditions...

$rating =  number_format( $imgrating, 0 );   

         $ratingicon = "";
If ($x=1) { $ratingicon .= "<img src='rating_1.gif'>"; }
If ($x=2) { $ratingicon .= "<img src='rating_2.gif'>"; }
If ($x=3) { $ratingicon .= "<img src='rating_3.gif'>"; }
If ($x=4) { $ratingicon .= "<img src='rating_4.gif'>"; }
If ($x=5) { $ratingicon .= "<img src='rating_5.gif'>"; }
         else { 
         $rating = ['nothing'];

[Weedpacket]

Originally posted by Wakab
Are you saying that I should make the conditions...

1. Not if you only want one image (you'll get two if $x is 1, 2, 3, 4, or 5 - lose the .);

2. You mean ==, not = ($x=5 sets $x equal to 5 and returns true);

3. The switch statement would make the code a bit cleaner;

4. You could put the value of $x directly into the string and reduce all that to one line without needing any tests (except to make sure that $x has a sensible value).

[Wakab]

Thanks, it's resolved now.

$rating = number_format( $imgrating, 0 );
    $ratingicon = "";
    for ( $x=0; $x < round(

$rating); $x++ ) {
        $ratingicon = "<img src=\"rating_" . $rating . ".gif\">";
    }
}
else {
    $rating = $Globals['pp_lang']['none'];
}