From 45634cfd56f9a61c066befb1f27a0497cff4bffc Mon Sep 17 00:00:00 2001 From: David Anderson Date: Fri, 8 Aug 2003 01:08:17 +0000 Subject: [PATCH] Cleaned up some inefficient code. svn path=/trunk/boinc/; revision=2033 --- html/user/profile.inc | 147 +++++++++++++++--------------------------- 1 file changed, 52 insertions(+), 95 deletions(-) diff --git a/html/user/profile.inc b/html/user/profile.inc index 945f63f2a2..2d5856f76a 100644 --- a/html/user/profile.inc +++ b/html/user/profile.inc @@ -14,6 +14,8 @@ define('MAX_DESC_LENGTH', 90); define('GALLERY_WIDTH', 7); define('GALLERY_HEIGHT', 4); +define('UOTD_THRESHOLD', 7); + $user = NULL; $profile_info = NULL; @@ -338,100 +340,59 @@ Your profile was successfully entered into our database.

// $numToDisplay indicates how many profiles to display in this table // $cols indicates how many profile summaries should be written per row // $descriptor is an optional file descriptor to write the table to. -// $pathMod is an optional path which is prepended to the image path. Useful -// when the image path is not relative to the directory from which this -// function is called. -function show_user_table($members, $offset, $numToDisplay, $cols, $descriptor=null, $pathMod=null) { +function show_user_table($members, $offset, $numToDisplay, $cols, $descriptor=null) { - // TODO: Would be nice if we could open a stream to stdout to avoid - // all the redundant $descriptor checks. Once the server is running - // PHP 5+, might want to try switching over to fprintf(...). + write_fd($descriptor, "\n"); - if ($descriptor) { - fwrite($descriptor, "
\n"); - } else { - echo "
"; - } - - $rows = ceil($numToDisplay / $cols); - $count = $offset; - $numMembers = count($members); - - for ($row = 0; $row < $rows; $row++) { - if ($count >= $numMembers) { - break; + $rows = ceil($numToDisplay / $cols); + $count = $offset; + $numMembers = count($members); + + for ($row = 0; $row < $rows; $row++) { + if ($count >= $numMembers) { + break; + } + + write_fd($descriptor, "\n"); + + for ($col = 0; $col < $cols; $col++) { + if ($count < $numMembers) { + write_fd($descriptor, ""); + $count++; + } else { + write_fd($descriptor, ""); + } + } + write_fd($descriptor, "\n"); } + write_fd($descriptor, "
"); + + // Only link an image if the user has uploaded one.; + $sql = "SELECT * FROM profile WHERE userid = " . $members[$count]; + $result = mysql_query($sql); + $profile = mysql_fetch_assoc($result); + + if ($profile['has_picture']) { + write_fd($descriptor, ""); + + } else { + write_fd($descriptor, " "); + } + + write_fd($descriptor, "
\n". get_profile_summary($profile). "
\n"); - if ($descriptor) { - fwrite($descriptor, "\n"); - } else { - echo ""; - } - - // Formatting is a table with $col columns of user summaries. - for ($col = 0; $col < $cols; $col++) { - if ($count < $numMembers) { - if ($descriptor) { - fwrite($descriptor, "
"); - } else { - echo "
"; - } - - // Only link an image if one exists. - if (file_exists($pathMod . IMAGE_PATH . $members[$count] . '_sm.jpg')) { - if ($descriptor) { - fwrite($descriptor, ""); - } else { - echo ""; - } - - } else { - if ($descriptor) { - fwrite($descriptor, " "); - } else { - echo " "; - } - } - if ($descriptor) { - fwrite($descriptor, "
"); - fwrite($descriptor, get_profile_summary($members[$count])); - fwrite($descriptor, ""); - } else { - echo "
\n", get_profile_summary($members[$count]), ""; - } - - $count++; - } - else { - // Empty entry - if ($descriptor) { - fwrite($descriptor, ""); - } else { - echo ""; - } - } - } - if ($descriptor) { - fwrite($descriptor, "\n"); - } else { - echo ""; - } - } - if ($descriptor) { - fwrite($descriptor, "\n"); - } else { - echo ""; - } } // Generates a string containing: -// 1) the name of the user with ID == $userid, with a link to a view of their profile; +// 1) the name of the user with ID == $userid, with a link to a view of their profile // 2) the first MAX_DESC_LENGTH characters from the response1 field of said user's profile. -function get_profile_summary($userid) { - $result = mysql_query("SELECT * FROM profile WHERE userid = $userid"); - $result2 = mysql_query("SELECT name FROM user WHERE id = $userid"); +function get_profile_summary($profile) { + $sql = "SELECT * FROM profile WHERE userid = " . $profile['userid']; + $result = mysql_query($sql); + $sql = "SELECT name FROM user WHERE id = " . $profile['userid']; + $result2 = mysql_query($sql); if (!$result || !$result2) { echo "Database error!"; // Change this to a standard error page. @@ -448,19 +409,15 @@ function get_profile_summary($userid) { if (strlen($row['response1']) != 0) { $temp = $row['response1']; + $description = "(\"" . sub_sentence(strip_tags($temp), ' ', MAX_DESC_LENGTH, true) . "\")"; - $description = "(\"" . substr(strip_tags($temp), 0, MAX_DESC_LENGTH); - if (strlen($row['response1']) >= MAX_DESC_LENGTH) { - $description = $description . "..."; - } - $description = $description . "\")"; } - $summary = "" . $row2['name'] . " " . $description; + $summary = "" . $row2['name'] . " " . $description; return $summary; } -// Displays a users profile (if they have one); +// Displays a user's profile (if they have one); function show_profile($userid, $verify_mode=false) { if (!$userid) { @@ -523,10 +480,10 @@ function show_profile_summary($user, $profile_info, $can_edit, $verify_mode) { echo "\n\n"; // Only display an image if the user has uploaded one; - if (!$verify_mode && file_exists(IMAGE_PATH . $user->id . '_sm.jpg')) { - echo "id , '.jpg' . "\">id , '_sm.jpg' . "\">\n"; - } else if ($verify_mode && file_exists('../html_user/' . IMAGE_PATH . $user->id . '_sm.jpg')) { - echo "id , '.jpg' . "\">id , '_sm.jpg' . "\">\n"; + if (!$verify_mode && $profile_info['has_picture']) { + echo "id , '.jpg' . "\">id , '_sm.jpg' . "\">\n"; + } else if ($verify_mode && $profile_info['has_picture']) { + echo "id , '.jpg' . "\">id , '_sm.jpg' . "\">\n"; } echo "