最受欢迎

这段代码能更简洁吗??它看起来是一个正确的状态

Could this code be cleaner?? It looks a right state?

本文关键字：一个 状态 看起来 段代码 代码 简洁 | 更新日期: 2023-09-27

我有这个代码从数据库中提取数据，并将数据插入到4列之一，

我已经找了一整天了，就是找不出该怎么做。

理想情况下，我想从数据库中选择所有，然后在那里的fetch数组有一个列id为1 - echo，然后为下一列的样本等。

<?php

            echo '<div class="column grid_3 clearfix" id="column0" >';
            echo '&nbsp;';
            $user_sites_0=mysqli_query($connection, "SELECT * FROM user_sites WHERE column_id='0' ORDER BY sort_no");
            if(!$user_sites_0) {
                echo 'No sites added, please <a class="addsite" href="#">add one now</a>';
            }
            else {
                while($user_site_0=mysqli_fetch_array($user_sites_0))
                {
                    $id = stripslashes($user_site_0['id']);
                    $site_name = stripslashes($user_site_0['site_name']);
                    $site_address = stripslashes($user_site_0['site_address']);
                    $site_desc = stripslashes($user_site_0['site_desc']);
                    $site_category = stripslashes($user_site_0['site_category']);
                    $getImage = 'http://immediatenet.com/t/s?Size=1024x768&URL='.$site_address;
                    echo '<div class="dragbox" id="item'.$id.'">';
                    echo '<h2 class="h2handle">'.$site_name.' <span class="close"><a href="#" id="'.$id.'" class="delete_button"><img src="assets/img/closepanel.png"></a></span></h2>';
                    echo '<div class="dragbox-content" ';
                    if($user_site_0['collapsed']==1)
                        echo 'style="display:none;" ';
                    echo '>';
                    echo '<p><a class="sitelink" href="' . $site_address . '" title="'.$site_name.'"><img src="'.$getImage.'" alt="'.$site_name.'" title="'.$site_name.'"/></a>';
                    echo '<p>' . $site_category . '</p>';
                    echo '<p>' . $site_address . '</p>';
                    echo '<p>' . $site_desc . '</p>';
                    echo'   </div>
                    </div>';
                }
            }
            echo '</div>';

    ?>
    <?php

            echo '<div class="column grid_3 clearfix" id="column1" >';
            echo '&nbsp;';
            $user_sites_1=mysqli_query($connection, "SELECT * FROM user_sites WHERE column_id='1' ORDER BY sort_no");
            if(!$user_sites_1) {
                echo '';
            }
            else {
                while($user_site_1=mysqli_fetch_array($user_sites_1))
                {
                    $id = stripslashes($user_site_1['id']);
                    $site_name = stripslashes($user_site_1['site_name']);
                    $site_address = stripslashes($user_site_1['site_address']);
                    $site_desc = stripslashes($user_site_1['site_desc']);
                    $site_category = stripslashes($user_site_1['site_category']);
                    $getImage = 'http://immediatenet.com/t/s?Size=1024x768&URL='.$site_address;
                    echo '<div class="dragbox" id="item'.$id.'">';
                    echo '<h2 class="h2handle">'.$site_name.' <span class="close"><a href="#" id="'.$id.'" class="delete_button"><img src="assets/img/closepanel.png"></a></span></h2>';
                    echo '<div class="dragbox-content" ';
                    if($user_site_1['collapsed']==1)
                        echo 'style="display:none;" ';
                    echo '>';
                    echo '<p><a class="sitelink" href="' . $site_address . '" title="'.$site_name.'"><img src="'.$getImage.'" alt="'.$site_name.'" title="'.$site_name.'"/></a>';
                    echo '<p>' . $site_category . '</p>';
                    echo '<p>' . $site_address . '</p>';
                    echo '<p>' . $site_desc . '</p>';
                    echo'   </div>
                    </div>';
                }
            }
            echo '</div>';

    ?>
    <?php

            echo '<div class="column grid_3 clearfix" id="column2">';
            echo '&nbsp;';
            $user_sites_2=mysqli_query($connection, "SELECT * FROM user_sites WHERE column_id='2' ORDER BY sort_no");
            if(!$user_sites_2) {
                echo '';
            }
            else {
                while($user_site_2=mysqli_fetch_array($user_sites_2))
                {
                    $id = stripslashes($user_site_2['id']);
                    $site_name = stripslashes($user_site_2['site_name']);
                    $site_address = stripslashes($user_site_2['site_address']);
                    $site_desc = stripslashes($user_site_2['site_desc']);
                    $site_category = stripslashes($user_site_2['site_category']);
                    $getImage = 'http://immediatenet.com/t/s?Size=1024x768&URL='.$site_address;
                    echo '<div class="dragbox" id="item'.$id.'">';
                    echo '<h2 class="h2handle">'.$site_name.' <span class="close"><a href="#" id="'.$id.'" class="delete_button"><img src="assets/img/closepanel.png"></a></span></h2>';
                    echo '<div class="dragbox-content" ';
                    if($user_site_2['collapsed']==1)
                        echo 'style="display:none;" ';
                    echo '>';
                    echo '<p><a class="sitelink" href="' . $site_address . '" title="'.$site_name.'"><img src="'.$getImage.'" alt="'.$site_name.'" title="'.$site_name.'"/></a>';
                    echo '<p>' . $site_category . '</p>';
                    echo '<p>' . $site_address . '</p>';
                    echo '<p>' . $site_desc . '</p>';
                    echo'   </div>
                    </div>';
                }
            }
            echo '</div>';              
    ?>
    <?php

            echo '<div class="column grid_3 clearfix" id="column3">';
            echo '&nbsp;';
            $user_sites_3=mysqli_query($connection, "SELECT * FROM user_sites WHERE column_id='3' ORDER BY sort_no");
            while($user_site_3=mysqli_fetch_array($user_sites_3))
            {
                $id = stripslashes($user_site_3['id']);
                $site_name = stripslashes($user_site_3['site_name']);
                $site_address = stripslashes($user_site_3['site_address']);
                $site_desc = stripslashes($user_site_3['site_desc']);
                $site_category = stripslashes($user_site_3['site_category']);
                $getImage = 'http://immediatenet.com/t/s?Size=1024x768&URL='.$site_address;
                echo '<div class="dragbox" id="item'.$id.'">';
                echo '<h2 class="h2handle">'.$site_name.' <span class="close"><a href="#" id="'.$id.'" class="delete_button"><img src="assets/img/closepanel.png"></a></span></h2>';
                echo '<div class="dragbox-content" ';
                if($user_site_3['collapsed']==1)
                    echo 'style="display:none;" ';
                echo '>';
                echo '<p><a class="sitelink" href="' . $site_address . '" title="'.$site_name.'"><img src="'.$getImage.'" alt="'.$site_name.'" title="'.$site_name.'"/></a>';
                echo '<p>' . $site_category . '</p>';
                echo '<p>' . $site_address . '</p>';
                echo '<p>' . $site_desc . '</p>';
                echo'   </div>
                </div>';
            }
            echo '</div>';              
    ?>

代码看起来是正确的状态。我能做得更好吗?

使用for并更改代码，一些代码:

<?php
for ($i=0; $i<4; $i++)
{
  echo '<div class="column grid_3 clearfix" id="column'.$i.'" >';
  echo '&nbsp;';
  $user_sites=mysqli_query($connection, "SELECT * FROM user_sites WHERE column_id='".$i."' ORDER BY sort_no");
  if(!$user_sites) {
    echo 'No sites added, please <a class="addsite" href="#">add one now</a>';
  }
  else
  {
    while($user_site=mysqli_fetch_array($user_sites))
    {
       ... // Do it yourself

maybe

    function get_site($column_id){
    $str = "";
     $user_sites = mysqli_query($connection, "SELECT * FROM user_sites WHERE column_id='".$column_id."' ORDER BY sort_no");
    if(!$user_sites_0) {
                echo 'No sites added, please <a class="addsite" href="#">add one now</a>';
            }
            else { 
            while($user_site_0=mysqli_fetch_array($user_sites_0))
                {
                    $id = stripslashes($user_site_0['id']);
                    $site_name = stripslashes($user_site_0['site_name']);
                    $site_address = stripslashes($user_site_0['site_address']);
                    $site_desc = stripslashes($user_site_0['site_desc']);
                    $site_category = stripslashes($user_site_0['site_category']);
                    $getImage = 'http://immediatenet.com/t/s?Size=1024x768&URL='.$site_address;
                    $str.= '<div class="dragbox" id="item'.$id.'">';
                    $str.= '<h2 class="h2handle">'.$site_name.' <span class="close"><a href="#" id="'.$id.'" class="delete_button"><img src="assets/img/closepanel.png"></a></span></h2>';
                    $str.= '<div class="dragbox-content" ';
                    if($user_site_0['collapsed']==1)
                        echo 'style="display:none;" ';
                    $str.= '>';
                    $str.= '<p><a class="sitelink" href="' . $site_address . '" title="'.$site_name.'"><img src="'.$getImage.'" alt="'.$site_name.'" title="'.$site_name.'"/></a>';
                    $str.= '<p>' . $site_category . '</p>';
                    $str.= '<p>' . $site_address . '</p>';
                    $str.= '<p>' . $site_desc . '</p>';
                    $str.='   </div>
                    </div>';
                }
            }
            $str.='</div>';
            return $str;}
echo '<div class="column grid_3 clearfix" id="column0">';
echo '&nbsp;';
echo oget_site(0);
echo '<div class="column grid_3 clearfix" id="column1">';
echo '&nbsp;';
echo oget_site(1);
//....
相关文章：
最新更新
www.56WenDa.com 2012-2023 | php问答网 | php学习