Открытие PHP операторов

Поэтому мне просто интересно, считалось ли плохой практикой открывать и закрывать операторы PHP, и позвольте мне объяснить, что я имею в виду. Я знаю, что могу создавать переменные в начале своего кода, но мне нравится группировать вещи вместе. Я не уверен, что создание одного большого оператора PHP со всеми моими переменными лучше / хуже / такое же, как открытие и закрытие операторов PHP, аналогичное пример ниже.

<html>
<head></head> <---- HTML STUFF
<?php
(php stuff where connection to mysql db goes and other variables and errors)
?>
<body>
<html> <----- HTML stuff 
<?php
(php stuff to call a specific table from DB)
?>
<html> <----- HTML stuff 
<?php
(php stuff to call a specific table from DB)
?>
<html> <----- HTML stuff 
<?php
(php stuff to call a specific table from DB)
?>
</body>
<html>

Кстати, переменные php, о которых я говорю, - это конкретные операторы выбора из БД.

АКТУАЛЬНЫЙ КОД: или должны быть выбранные операторы в начале или в файле sep?

<table>
<tr>
<td align="left" width="200px">
Cover: Original Total
</td>
<td width="200px" align="center">
<?php
$original = "SELECT * FROM `comic_db`.`comic_db` WHERE comic_cover=\"original\"";
$orig_con = mysqli_query($comic_connect, $original);
$orig_total = mysqli_num_rows($orig_con);
echo $orig_total;
?>
</td>
</tr>
<tr>
<td width="200px" align="left">
Cover: Variants Total
</td>
<td width="200px" align="center">
<?php
$variants = "SELECT * FROM `comic_db`.`comic_db` WHERE comic_cover=\"variant\"";
$variant_con = mysqli_query($comic_connect, $variants);
$variant_total = mysqli_num_rows($variant_con);
echo $variant_total;
?>
</td>
</tr>
<tr>
<td align="left" width="200px">
Cover: Baby Totals
</td>
<td width="200px" align="center">
<?php
$baby = "SELECT * FROM `comic_db`.`comic_db` WHERE comic_cover=\"baby\"";
$baby_con = mysqli_query($comic_connect, $baby);
$baby_total = mysqli_num_rows($baby_con);
echo $baby_total;
?>
Источник

1 ответ

Решение

Я предполагаю повторный <html> теги в вашем примере являются просто заполнителями и будут на самом деле <div>s и другие элементы, составляющие фактическое содержание страницы.

Ваш пример - это то, что обычно называют "спагетти-кодом", потому что он может быстро превратиться в бесполезный беспорядок, потому что вы не можете ясно видеть обзор HTML, и при этом вы не можете видеть весь код PHP в одном месте.

Главное, что нужно иметь в виду, - это отделить логику приложения (например, запросы к вашей базе данных) от представления (HTML и логика представления, например, циклическая обработка массива для отображения его в виде списка HTML).

По крайней мере, вы захотите поместить основной PHP-код в верхнюю часть файла, как вы сказали, но было бы намного лучше, если бы он был в отдельном файле.

PS Любая книга для начинающих по PHP обсудит это подробно.

Источник
Другие вопросы по тегам