Обсуждение качества кода Datso Gallery - DatsoGallery

switch(f) {
case 0:
 return "false";
 break;
case 1:
 return "true";
 break;
case 2:
 return "Фуясе О_о";
 break;
default:
 return "Ненене, Дэвид Блэйн, ненене!!";
}

<a href='<?php echo JRoute::_("index.php?option=com_datsogallery&amp;func=detail&amp;catid=".$row->catid."&amp;id=".$row->id."&amp;Itemid=".$Itemid); ?>'>
    <img src='<?php echo $thumbnailpath.$row->imgthumbname; ?>' style='<?php echo $ad_imgstyle; ?>' alt='' /></a>
    </td>
    <td width='100%' valign='top'><strong><?php echo $row->imgtitle; ?></strong>
    <span class="small">
                  <?php
                    if($ad_showdetail)
                      $picdate=strftime("%d.%m.%Y %H:%M",$row->imgdate);
                    echo "<br /><strong>"._DG_DATE_ADD."</strong>: $picdate ";
                    echo "<br /><strong>"._DG_HITS."</strong>: $row->imgcounter ";
                    if($ad_showrating) {
                      if($row->imgvotes>0) {
                        $fimgvotesum=number_format(intval($row->imgvotesum)/intval($row->imgvotes),2)*20;
                      }
                      else{
                        $fimgvotesum="0";
                      }
                      echo "<br /><strong>"._DG_RATING."</strong>:
                      <span class='inline-rating'>
                      <ul class='star-rating small-star' style='width: 50px'>
                      <li class='current-rating' style='width: ".$fimgvotesum."%'></li>
                      <li class='star' style='width: 20%'></li>
                      <li class='star' style='width: 40%'></li>
                      <li class='star' style='width: 60%'></li>
                      <li class='star' style='width: 80%'></li>
                      <li class='star' style='width: 100%'></li>
                      </ul>
                      </span> / $row->imgvotes";
                    }
                    if($ad_showcomment) {
                      $db->setQuery("select cmtid from #__datsogallery_comments where cmtpic='".$row->id."'");
                      $comments_result=$db->query();
                      $comments=mysql_num_rows($comments_result);
                      echo "<br /><strong>"._DG_COMMENTS."</strong>: $comments";
                    }
                  ?>
    </span>

$output='<table cellspacing="1" cellpadding="0" border="0" width="100%">';
      if(@ $rows[0]->parent) {
        $output.='<tr><td align="left" colspan="'.$colspan.'" class="sectiontableheader">'._DG_SUBCATEGORIES.'</td></tr>';
      }
      else
        if(@ $rows[0]->cid) {
          $output.='<tr><td align="left" colspan="'.$colspan.'" class="sectiontableheader">'._DG_CATEGORIES.'</td></tr>';
        }
        if($num_rows)
          for($row_count=0; $row_count<($num_rows/$ad_ncsc); $row_count++) {
            $output.='<tr class="sectiontableentry2">';
            for($i=0; $i<$ad_ncsc; $i++) {
              $cur_name=@ $rows[$index];
              $output.='<td align="center" valign="middle">';
              if(@ $cur_name->cid) {
                $output.='<a href="'.JRoute::_("index.php?option=com_datsogallery&func=viewcategory&catid=".$cur_name->cid."&Itemid=".$Itemid).'">';
              }
              if(!@ $cur_name->cid) {
                $output.='</td><td>&nbsp;</td>';
              }
              else{
                $catid=$cur_name->cid;
                $query="select *, c.access from #__datsogallery as p "." left join #__datsogallery_catg as c on c.cid=p.catid "." where ".($catid?" ( p.catid in (".$catid.") )":'')." and p.published = '1' and p.approved='1' and c.access<='".$user->get('aid')."' "." || ".($catid?" ( c.parent in (".$catid.") )":'')." and p.published = '1' and p.approved='1' and c.access<='".$user->get('aid')."' "." order by rand() limit 1";
                $db->setQuery($query);
                $rows2=$db->loadObjectList();
                $row2=&$rows2[0];
                $db->setQuery($query);
                $count=$db->loadResult();
                if($count>0) {
                  $output.='<img src="'.$thumbnailpath.$row2->imgthumbname.'" style="'.$ad_imgstyle.'" title="'._DG_OPEN_CAT.'" alt="" /></a></td>';
                  $output.='<td class="sectiontableentry2" align="left" valign="top" width="'.$cw.'">';
                }
                else
                  if(GetThumbsInCats($cur_name->cid)) {
                    $output.='<img src="'.$thumbnailpath.GetThumbsInCats($cur_name->cid).'" style="'.$ad_imgstyle.'" title="'._DG_OPEN_CAT.'" alt="" /></a></td>';
                    $output.='<td class="sectiontableentry2" align="left" valign="top" width="'.$cw.'">';
                  }
                  else
                    if(!$count) {
                      $output.='<div style="background:url('.JURI::base().'/components/com_datsogallery/images/blank.gif) no-repeat;display:block;background-position: 50% 50%;width:'.$ad_thumbwidth.'px;height:'.$ad_thumbheight.'px"></div></td>';
                      $output.='<td class="sectiontableentry2" align="left" valign="top" width="'.$cw.'">';
                    }
              }
              if($cur_name&&($count)||(GetThumbsInCats(@ $cur_name->cid))) {
                $output.='<a href="'.JRoute::_("index.php?option=com_datsogallery&func=viewcategory&catid=".@$cur_name->cid."&Itemid=".$Itemid).'">';
                $output.='<strong>'.@ $cur_name->name.'</strong></a>';
              }
              else
                if($cur_name) {
                  $output.='<strong>'.$cur_name->name.'</strong>';
                }
                if(@ $cur_name->name) {
                  $output.='<br /><span class="small">('.GetNumberOfLinks($cur_name->cid).')';
                  if($ad_showinformer) {
                    $output.='&nbsp;'.GetNewPics($cur_name->cid);
                  }
                  $output.='</span>';
                }
                $output.='<br />'.@ $cur_name->description.'</td>';
              $index++;
            }
            $output.='</tr>';
      }
      $output.='</table>';

function showPictures($option, &$rows, &$clist, &$slist, &$search, &$pageNav)
  {
    global $mainframe;
    $db = & JFactory :: getDBO();
    require (JPATH_COMPONENT.DS.'config.datsogallery.php');

    $db->setQuery( "ALTER TABLE `#__datsogallery` ADD `imgdownloaded` int(11) NOT NULL default '0' AFTER `imgcounter`");
    if (!$db->query()) {}
    $db->setQuery( "ALTER TABLE `#__datsogallery_comments` ADD `cmtmail` varchar(100) NOT NULL default '' AFTER `cmtname`");
    if (!$db->query()) {}
    ?>

  <form action="index2.php" method="post" name="adminForm">
    <table>