2011-10-09 6 views
0

내가 검색 하루 종일을 보냈습니다 그냥 캔트 그것을 할 방법을 찾을 수가이 코드는 더 깨끗할까요 ?? 올바른 상태로 보입니까?

데이터베이스에서 데이터를 끌어와 4 열 중 하나에 데이터를 삽입하기 위해이 코드를 ..이

수 ..

<?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>';    

    ?> 

코드는 바로 상태를 보이는 .. 다음 열 등을위한 그 다음 마 에코 - 이상적으로, 나는 데이터베이스에서 모든를 선택하려면 다음 위치를 배열 1의 열 ID를 가지고 가져 나는 더 잘하니?

+2

당신은 코드 개선을 요구하고 있는가? 그런 다음 http://codereview.stackexchange.com/을 사용해야합니다. – vstm

+0

템플릿 엔진이 아무에게도 해를 끼치 지 않습니다 ... 백만 에코의 효율성은 물론 코드의 유지 관리 가능성에도 의문을 제기합니다. –

+0

나는 그것에 대해 생각하고 있었지만 배우기 위해 처음부터 코드화하고 싶었다. 나는 다음 2 가지를 사용하여 uni를 사용할 여유가 없다. Google 및 Stack Overflow :) –

답변

3

사용 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 
+0

+1하고 싶지만 15 명이 필요합니다. 답변 해 주셔서 감사합니다! :) –

0

어쩌면

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); 
//.... 
관련 문제