[Resolved] foreach loop and results layout

[mrjeff]

Hello,
I am trying to get each image in a 3 colum layout. The way I have implemented now is messy.

I searched through the forum and tried different things but can't get to work. The examples I have seen do not use a nested foreach loop like I have.

Am I going about looping through the results the wrong way for what I want to do?

Thankyou for any help.
Jeff

IF ($ueConfig['showbuddylistinprofile']==1)
{
$query = "SELECT buddyid FROM #__comprofiler_buddylist WHERE userid='".$user->id."' ";
$database->setQuery($query);
$result = '';
$result = $database->loadObjectList();
$total = count ($result);

if ($total>0)

{

$j=0;
$buddy='';

foreach($result as $preuser) 

{ 

$buddy[$j]=$preuser->buddyid;
$j++;

}

$passtoquery='(' . implode(',', $buddy) . ')';
$query = "SELECT id,username FROM #__users WHERE id IN ".$passtoquery." AND block!=1 ORDER BY username";
$database->setQuery($query);
$buds = $database->loadObjectList();

echo "<tr><td colspan=2 align=center align=center class=\"sectiontableheader\">Buddies of ".$user->username."</td></tr>";

echo '<table cellpadding="0" cellspacing="0" border="0" width="100%">';

foreach($buds as $bud)
{
$query_avatar = "SELECT avatar FROM mos_comprofiler WHERE user_id='".$bud->id."' AND avatarapproved=1";
$database->setQuery( $query_avatar );
$pics = $database->loadObjectList();

$path = "index.php?option=com_comprofiler&task=userProfile&user=";
$thumb_path ="images/comprofiler/tn";
$no_photo = ' alt="No Photo" border=0';

foreach ($pics as $pic)

if($ueConfig['allow_profilelink']==1) 

{
//I would like these results to go into a three colum layout
echo "<tr><td  valign=bottom colspan=2 rowspan=3><a href=".sefRelToAbs.$path.$bud->id."><img src=".$thumb_path.$pic->avatar.$no_photo."> <br/><big>".$bud->username."</big></a></td></tr>";

}
else

{
echo $bud->username."  ";
}
}
}
}

[chads2k2]

I checked out your code and it's interesting.

I see what your trying to do but just read the foreach loop manual page again. Below is a code block that I used in my latest program. It is a 2 colum layout. But I think you can use your imagination to see how to do it in threes.

<?php
          // Include the function GetGroups.
          include_once("includes/ldap.get.groups.inc.php");
          include_once("includes/search.functions.inc.php");
          // Call GetGroups passing it user information.
          if(!isset($returned_groups))
          {
              $returned_groups = GetGroups($GLOBAL_NAME, $GLOBAL_PASSWORD, $GLOBAL_DIVNUM);
          }
          // Create an array from the returned string.
          $info_array = explode(";",$returned_groups);
          // Count the amount of objects.
          $count_objects = count($info_array);
          // Counter variable.
          $i = 0;
          $internal_counter = 0;
          // Go until it equals the number of objects.
          while($i <= $count_objects)
          {
              // Define name with the array.
              $name = $info_array[$i];
              // Define description with the next part.
              $description = $info_array[$i+1];
              // Throw out name if it is blank.  I.E. the last item.
              if($name != "")
              {
                 // Check to see if the col count is at 2.
                 if($internal_counter == 2)
                 {
                     // End the row and start another.
                     echo "</tr><tr>";
                     // Reset the counter back to 0.
                     $internal_counter = 0;
                 }
                 // This looks to see if error_group_id has been set
                 // a return from error checking in the beginning
                 // of the script.
                 if(isset($error_group_id))
                 {
                    // Set group_id to an array of all
                    // the groups choosen.
                    $group_id = explode(";",$error_group_id);
                 }
                 // Search to see if the name of the group is selected
                 // or not.  array_match() return is 1 if found, 0 if not.
                 $in_name_array = array_match($name , $group_id);
                 if($in_name_array == 1)
                 {
                    // Addslashes!  The tooltip will not work if there are
                    // ''s everywhere.
                    $description = addslashes($description);
                    echo "<td width='225' height='19' valign='top'><input name='group_id[]' type='checkbox' value='$name' checked><span onMouseover=\"ddrivetip('$description')\";
 onMouseout=\"hideddrivetip()\">$name</span></td>";
                 }else{
                    // Same thing, addslashes.
                    $description = addslashes($description);
                    echo "<td width='225' height='19' valign='top'><input name='group_id[]' type='checkbox' value='$name'><span onMouseover=\"ddrivetip('<strong><font color=\'#FF0000\'>Description:</font></strong><br><strong>$de35087scription</strong>')\";
 onMouseout=\"hideddrivetip()\">$name</span></td>";
                 }
                 // Increase the internal counter.
                 $internal_counter++;
              }
              // Advance the Active Directory counter.
              $i = $i + 2;
            }
            ?>

[Roger_Ramjet]

Yes you are going about this all the wrong way.

Not going to repeat myself, just look at my posts and code in this thread

PS I hacked the code on my last post in about 3 mins so it is untested and you can expect some basic typos, but the theory and the logic is sound. (but my function name is color coded so 'slice' may be a reserved word?? so rename it if it is)

[Roger_Ramjet]

PS Why are so many of you obsessed with using objects to process query results?? Object are great when you need them ,but what advantage do they have when itterating through a result set? None that I know of, plenty of memory overhead though.

So I'm going to provoke someone into justifying themselves by calling you obssessive-compulsives with an object-retention problem, or gnurds for short.

[mrjeff]

Than you for the suggestions and help!
I see how it should be done.
Jeff

Here is a different way that I solved the issue with a div float left, and it works perfect!

IF ($ueConfig['showbuddylistinprofile']==1)
{
$query = "SELECT buddyid FROM #__comprofiler_buddylist WHERE userid='".$user->id."' ";
$database->setQuery($query);
$result = '';
$result = $database->loadObjectList();
$total = count ($result);

if ($total>0)

{

$j=0;
$buddy='';

foreach($result as $preuser) 

{ 

$buddy[$j]=$preuser->buddyid;
$j++;

}

$passtoquery='(' . implode(',', $buddy) . ')';
$query = "SELECT id,username FROM #__users WHERE id IN ".$passtoquery." AND block!=1
ORDER BY username";
$database->setQuery($query);
$buds = $database->loadObjectList();

echo "<tr><td colspan=2 align=center align=center
class=\"sectiontableheader\">Buddies of ".$user->username."</td></tr>";

echo '<tr><td colspan=2 align=center align=center>';

foreach($buds as $bud)
{
$query_avatar = "SELECT avatar FROM mos_comprofiler WHERE user_id='".$bud->id."' AND
avatarapproved=1";
$database->setQuery( $query_avatar );
$pics = $database->loadObjectList();

$path = "index.php?option=com_comprofiler&task=userProfile&user=";
$thumb_path ="images/comprofiler/tn";
$no_photo = ' alt="'.$bud->username.'" border=0';

foreach ($pics as $pic)

if($ueConfig['allow_profilelink']==1) {

                                                    if($pic->avatar!='') {
                                                    echo "<div style=\"float:left; padding:4px;\"><a
href=".sefRelToAbs.$path.$bud->id."><img
src=".$thumb_path.$pic->avatar.$no_photo."><br/>".$bud->username."</a></div>";
                                                        } else {
                                                        echo "<div style=\"float:left; padding:4px;\"><a
href=".sefRelToAbs.$path.$bud->id."><img
src=\"components/com_comprofiler/images/english/tnnophoto.jpg\" border=0
alt=\"no photo\"><br/>".$bud->username."</a></div>";
}
}
}
}
}