php function и html

Question

Когда я создаю элементы в некоторых местах, я верну html-код. Лучше ли функция возвращать массив, а затем создавать html для каждого элемента в списке или таблице после или иметь html как часть функции?

например.

function failedScansTable($eventId) 
{ 
    echo "<table class='basic-table'>"; 
    $sql = mysql_query("SELECT total_errors FROM errors WHERE id_event = $eventId"); 
    while ($row = mysql_fetch_array($sql)) 
     { 
      $totalErrors = $row['total_errors']; 
      echo "<tr class='basic-table-total'><th colspan='3'>Total Rejected Scans</th><th>$totalErrors</th></tr>"; 
     } 
    echo "</table>"; 
} 

Answer 1

• Функция должна возвращать вообще что-то вместо echo -ную
• Функция обычно не должна генерировать HTML-код, а скорее возвращать данные (например, массив ценностей es), чем тогда можно использовать в представлениях (определенные части приложения, которые генерируют только HTML). Взгляните на Model-View-Controller architecture, чтобы получить представление о том, как разделить проблемы.

• PHP был создан для использования в качестве языка шаблонного, так что вместо того, чтобы генерировать HTML путем объединения строк, вы должны использовать что-то вроде:

  <table> 
      <?php foreach (failedScansTable() as $entry): ?> 
       <tr> ... </tr> 
      <?php endforeach ?> 
  </table> 
  

• Вы не должны использовать mysql_* функции больше, так как они официально устарели. Вместо этого рассмотрите PDO.

• Всегда избегайте частей, которые вы добавляете к своему коду.
  • Вы должны бежать $eventId перед добавлением его в код SQL (mysql_real_escape_string() или, гораздо лучше, PDO)
  • Вы должны бежать $totalErrors перед добавлением его в HTML-код (htmlspecialchars())

Answer 2

Лучше, попробуйте это

function failedScansTable($eventId) 
{ 
    $string= "<table class='basic-table'>"; 
    $sql = mysql_query("SELECT total_errors FROM errors WHERE id_event = $eventId"); 
    while ($row = mysql_fetch_array($sql)) 
    { 
     $totalErrors = $row['total_errors']; 
     $string.= "<tr class='basic-table-total'><th colspan='3'>Total Rejected Scans</th><th>$totalErrors</th></tr>"; 
    } 
    $string.= "</table>"; 
    return array('html'=>$string, 'totalError'=>$totalErrors); 
} 

Answer 3

Попробуйте это:

function failedScansTable($eventId) 

{ 

$html = "<table class='basic-table'>"; 

$sql = mysql_query("SELECT total_errors FROM errors WHERE id_event = $eventId"); 

while ($row = mysql_fetch_array($sql)) 

    { 

     $totalErrors = $row['total_errors']; 

     $html.="<tr class='basic-table-total'><th colspan='3'>Total Rejected Scans</th><th>$totalErrors</th></tr>"; 

    } 

$html.="</table>"; 

return $html; 

} 

Answer 4

Если ваш код будет расти, лучше следовать шаблону дизайна, например MVC.

Поэтому лучше отделить модель от вашего вида. Массив - это ваша модель данных; и html, который представляет собой представление ваших данных, является видом.