php show image if exist

[james182]

using php how would i check to see if a image exist if so output it if not dont?

[Jeff Wheeler]

<?php if (file_exists('filename.jpg')) echo '<img src="filename.jpg" alt="File" title="File" />'; ?>

[james182]

<?php if (file_exists('filename.jpg')) echo '<img src="filename.jpg" alt="File" title="File" />'; ?>

not working

[Jeff Wheeler]

Err… that's descriptive.

[james182]

ok what i am trying to do is show an image that sits inside a table and if there is no image then dont show the table.

do i put the "file_exists" code in the query or before the table?

[Seb Hughes]

Nokrev dont he need the \"


so it would be

<?php if (file_exists('filename.jpg')) echo '<img src=\"filename.jpg\" alt=\"File\" title=\"File\" />'; ?>

[Jeff Wheeler]

No. When using different style quotes (single vs. double), they're unnecessary. Additionally, double quotes should be used rarely in PHP, because they're much slower and attempt to parse variables. It's faster to concatenate a variable to a single-quote string than to have one string with an embedded variable.

As for james' problem, you need to encapsulate the entire table inside the check for the image:

<?php if (file_exists('image.jpg')) { ?>
<table><stuff></stuff><stuff><img src="image.jpg" title="My image" alt="My image" /></stuff><stuff /></table>
<?php } ?>

[Seb Hughes]

No. When using different style quotes (single vs. double), they're unnecessary. Additionally, double quotes should be used rarely in PHP, because they're much slower and attempt to parse variables. It's faster to concatenate a variable to a single-quote string than to have one string with an embedded variable.

As for james' problem, you need to encapsulate the entire table inside the check for the image:

<?php if (file_exists('image.jpg')) { ?>
<table><stuff></stuff><stuff><img src="image.jpg" title="My image" alt="My image" /></stuff><stuff /></table>
<?php } ?>

Well either way would worlk, my way was correct, wasnt be all means the fastest way, but it would ahve done the job

[Krilnon]

Well either way would worlk, my way was correct, wasnt be all means the fastest way, but it would ahve done the job

You stated that you believed the escape slashes were necessary, which implies that it wasn't an alternative method and that it was a working method whereas nokrev's was 'not'. This could be compared to using:

if(1+1==2){
    echo "True";
}

//Instead of:

if(((((((((((1+1))))))))))== 2){
    echo "Also True";
} 

Both of these would work, however, it is probably obvious which one most people would suggest implementing in real life. If you had suggested your method before nokrev had, the 'alternate' method reason would have seemed more acceptable.

[Jeff Wheeler]

Err… thanks for the long-winded explanation.

[james182]

hi well it works in part but it hides both images. Here where it is going:

<?php if (file_exists('$blogimg')) { ?>
                                    <table cellpadding="0" cellspacing="0" border="0">
                                        <tr>
                                            <td width="7">
                                                <img src="images/frame_topleft.gif" width="7" height="7" alt="" border="0"><br>
                                            </td>
                                            <td width="10" background="images/frame_toptile.gif">
                                                <img src="images/trans.gif" width="1" height="7" alt="" border="0"><br>
                                            </td>
                                            <td width="7">
                                                <img src="images/frame_topright.gif" width="7" height="7" alt="" border="0"><br>
                                            </td>
                                        </tr>
                                        <tr>
                                            <td width="7" background="images/frame_lefttile.gif">
                                                <img src="images/trans.gif" width="1" height="1" alt="" border="0"><br>
                                            </td>
    <td width="10" bgcolor="#DDE9F4">
                                                <img src="images/<?=$blogimg?>.jpg" width="450" height="150" alt="" border="0"><br>
    </td>
<td width="7" background="images/frame_righttile.gif">
                                                <img src="images/trans.gif" width="1" height="1" alt="" border="0"><br>
    </td>
    </tr>
    <tr>
    <td width="7">
        <img src="images/frame_botleft.gif" width="7" height="7" alt="" border="0"><br>
    </td>
    <td width="10"  background="images/frame_bottile.gif">
    <img src="images/trans.gif" width="1" height="1" alt="" border="0"><br>
    </td>
<td width="7">
                                                <img src="images/frame_botright.gif" width="7" height="7" alt="" border="0"><br>
</td>
</tr>
</table>
<?php } ?>

[james182]

any idaes???

[kungFuChris]

<?php echo '<img src='images/$blogimg.jpg' width='450' height='150' alt='' border='0'>'; ?>

[bigmtnskier]

After decifering your code, I think this is what you want:

<?php 

// blogimg should be something like this: 
// imagename.jpg

if (file_exists("images/".$blogimg)) { 
?>
                                    <table cellpadding="0" cellspacing="0" border="0">
                                        <tr>
                                            <td width="7">
                                                <img src="images/frame_topleft.gif" width="7" height="7" alt="" border="0"><br>
                                            </td>
                                            <td width="10" background="images/frame_toptile.gif">
                                                <img src="images/trans.gif" width="1" height="7" alt="" border="0"><br>
                                            </td>
                                            <td width="7">
                                                <img src="images/frame_topright.gif" width="7" height="7" alt="" border="0"><br>
                                            </td>
                                        </tr>
                                        <tr>
                                            <td width="7" background="images/frame_lefttile.gif">
                                                <img src="images/trans.gif" width="1" height="1" alt="" border="0"><br>
                                            </td>
    <td width="10" bgcolor="#DDE9F4">
                                                <img src="images/<?php echo $blogimg; ?>" width="450" height="150" alt="" border="0"><br>
    </td>
<td width="7" background="images/frame_righttile.gif">
                                                <img src="images/trans.gif" width="1" height="1" alt="" border="0"><br>
    </td>
    </tr>
    <tr>
    <td width="7">
        <img src="images/frame_botleft.gif" width="7" height="7" alt="" border="0"><br>
    </td>
    <td width="10"  background="images/frame_bottile.gif">
    <img src="images/trans.gif" width="1" height="1" alt="" border="0"><br>
    </td>
<td width="7">
                                                <img src="images/frame_botright.gif" width="7" height="7" alt="" border="0"><br>
</td>
</tr>
</table>
<?php } ?>

You definitally had it set up pretty screwy

-Bigmtnskier

[Jeff Wheeler]

The real questions is… why all those nasty tables?