Bu kodu hakkında ne düşünüyorsunuz?

Ben bu html php ile karışık olduğunu biliyorum. Nasıl php html ayırabilirsiniz? Başka ne bu kodun kalitesini artırmak için yapılması gereken?

Umarım burada bazı php uzmanlar bana yardımcı olabilir.

Edit:

Sadece bu özel kod iş mantığı randevuları, yaratılış silme vs vs ile ilgili her şeyi halleder AppointmentManager sınıfı gibi ayrı dosyalar halinde zaten olduğunu eklemek istedim

Ben buraya koymak kod parçası (manage.php) sadece randevular ayrıntılarını görüntüler ve onları iptal, düzenleme için olanak sağlar.

Şimdi, biz bunu ekran mantık veya sunum mantığı ya da başka bir şey olsun, bu sayfayı dediğimiz emin değilim .... ama ne olur ben bilmeniz gereken; s mümkün daha fazla ayrı html ve php için?

Ben bu dosyanın dışında html bu parça almak için örneğin bir şey yapabilir miyim ...

<script language="JavaScript" src="ajax/core2.js"></script>

<h2 align='center'> Appointment Control Panel V 1.0</h2>
<h2 align='center'>Your Next Appointments</h2>  
<div id=detail></div>

Ve bunun benzer kalanı ...... bu uygulamayı maintian ve genişletmek için kolay, böylece ben tamamen ayrı html ve php demek.

Ben Smarty kullanmak istemiyorum, ben bu gereksiz şeyleri zorlaştırıyor düşünüyorum.

I denilen bu html işlemek bazı php fonksiyonlarını oluşturmak ve belki displayhtml.php gibi dosya ortak fonksiyonları içine koyabilirsiniz?

Herhangi bir başka daha iyi bir öneriniz muhtemelen?

<?php
/*
 * manage.php
 *
 * Author: Bob :  2010-11-11
 *
 */

ob_start();

$page_title = "Appointments Management";

require_once('appts/coreincs.inc');
require_once('appts/pagetop.inc');

$am = AppointmentManager::getInstance(); 
$appts = $am->getPendingAppointments($g_userID);  
?>

<script language="JavaScript" src="ajax/core2.js"></script>

<h2 align='center'> Appointment Control Panel V 1.0</h2>
<h2 align='center'>Your Next Appointments</h2>  
<div id=detail></div>

<?php 
if ($appts === NULL)    
{   
  echo <<<EOEMPTY
<p align='center'>
  You have no upcoming appointments.
</p>
EOEMPTY;
}
else
{
  /**
   * List the appointments.  Start the table.
   */
  echo <<<EOT
<table align='center' width='80%' border='0' cellspacing='0'  
       cellpadding='3' class='apptTable'>  
<tr>
  <td width='25%' class='apptDispHeader'>When:</td>
  <td width='30%' class='apptDispHeader'>Title:</td>  
  <td width='15%' class='apptDispHeader'>Action:</td> 
  <td width='25%' class='apptDispHeader'>Where:</td>
</tr>     
EOT;

  /**  
   * Zip through all the appointments and print them out. 
   */

  foreach ($appts as $appt) 
  {
    //if ($appt->StartTime->sameDay($appt->EndTime)) 
    //{

      $start = $appt->FullTimestart; 
      $end = $appt->FullTimeend;       

    echo <<<EOAPPT    
<tr>    
  <td>$start ==> $end</td>    
  <td>    
  <div id=title_{$appt->AppointmentID} loaded="t_{$appt->AppointmentID}">   
  <div id=title_{$appt->AppointmentID}_t_{$appt->AppointmentID}>    
    <a class='apptDispLink'
       href='showappt.php?aid={$appt->AppointmentID}'>
    {$appt->Title}
    </a> 
 </div>
  </div>
  </td> 

  <td><a onclick=Core2.loadXMLDoc('ajaxrequest.php?aid={$appt->AppointmentID}',['edit'],['detail'],['edit_form_{$appt->AppointmentID}'],"reLoad","hide_all")>Edit</a> | <a onclick=Core2.loadXMLDoc('cancel.php?aid={$appt->AppointmentID}',['delete'],['title_{$appt->AppointmentID}'],['cancel_{$appt->AppointmentID}'],"reLoad","hide_all")>Cancel</a></td>  
  <td>  
    {$appt->Location}   
  </td>   
</tr>   
EOAPPT;    
  }    

  /**   
   * Close out the Table:    
   */

  echo <<<EOT   
</table>    
<iframe name=main width="0" height="0" marginwidth="1" marginheight="1" scrolling="no" border="0" frameborder="0"></iframe> 

EOT;
}   

require_once('appts/pagebottom.inc');  
ob_end_flush();

?>