Databases
StackOverflow2013
Postsdbo
PO
 

Note that there are some explanatory texts on larger screens.

plurals
  1. PO
    primarykey
    data
    text
    <p>Look what you are trying to do is select the image that matches the name of the user in the current row of your result set. So you will set your image file variable as appropriate for each row and you are sending that to the browser.</p> <p>For starters and can see the probability of case issues here. Are all user names forced to lower case and all image names forced to lower case? If this is on a linux box that is a land mine on windows not so much, but this should be taken into account.</p> <p>It will set an image name for <strong>each row</strong> of your queries result set so it will look like:</p> <p>[image] [comments]<br> [image] [comments]<br> [image] [comments]<br></p> <p>if you have three rows in your result set.'</p> <p>Personally I avoid all of the turning php on and off all over the place. Concat a single string and then simply echo it out for each row. So I would code it like so:</p> <pre><code>&lt;? $commentCount = 0 ; $sql = "SELECT comment_user FROM comments"; $result = mysql_query($sql) or die(mysql_error()); while ($row = mysql_fetch_assoc($result)) { $commentCount++ ; $pathFile = $pathImg . $row['comment_user'] . ".png"; if (!file_exists($pathFile)) { $pathFile = $pathImg . "guest.png"; } echo "&lt;img src=\"". $pathFile ."\" alt=\"". $row['comment_user'] ."\"\&gt;&lt;br&gt;"; } </code></pre> <p>So I have eliminated a lot of things from your code example like counters etc. You don't really need to check and see if there are rows since the while loop simply will not execute of there are no rows so you will simply have a question of comment with no subordinate comments and it will <strong>only</strong> send the image link if there are comments.</p> <p>No if it were me doing this I would create an avatar file name is the user table and store the path to those as part of the system configuration which would be part of the global set of variables that are always present. Your query would then join in the users table and the image name or guest image would be in your result set. A bit more complex but <strong>much</strong> cleaner and it simplifies your code.</p> <p>One of the reasons I don;t like dynamic typing. $row was being mutated to an array of ALL the rows..</p>
    singulars
    1. This table or related slice is empty.
    1. PODisplay image on php mysql blog based on username of post
      singulars
      1. PTQuestion
    1. PTAnswer
    1. USFlyingGuy
    1. USFlyingGuy
    plurals
    1. This table or related slice is empty.
    1. This table or related slice is empty.
    1. PODisplay image on php mysql blog based on username of post
      singulars
      1. PTQuestion
    1. This table or related slice is empty.
    1. VO
      singulars
      1. PO
      1. This table or related slice is empty.
      1. VTUpMod
    2. VO
      singulars
      1. PO
      1. This table or related slice is empty.
      1. VTAcceptedByOriginator
    1. COFlyingGuy, based on my edit to my question above, how would your code work now for this, it is still displaying 6 guest images followed by the admin image followed by 4 guest images (again adding a new guest image to the row everytime i make a new post and a new comment on that new post). How would I go about doing your way of: "create an avatar file name is the user table and store the path to those as part of the system configuration which would be part of the global set ...."? Thanks for the help!
      singulars
      1. PO
      1. USKGDD
    2. COSome questions:
      singulars
      1. PO
      1. USFlyingGuy
    3. COOk, so something is missing here. limit the problem. How many rows does your query return? If it returns 6 then you should get 6 lines of text with 6 avatar images, nothing more nothing less with the code I showed you.
      singulars
      1. PO
      1. USFlyingGuy
 

Querying!

 
Guidance
A row detail

Detail views are divided into sections. All the information in the data section comes from columns in the selected row. The other sections display data from other, related rows.

Related data can be related in a to-one or a to-many fashion. Captions of data related in a to-many fashion link to a list view showing a filtered view of the table.

Try moving around until you find a non-empty to-many entry and click on the label to get to one. You can move back to the root by clicking on the database name in the header.

SQuiL has stopped working due to an internal error.

If you are curious you may find further information in the browser console, which is accessible through the devtools (F12).

Reload