Комментарии пользователя Sanovskiy

Как исправить ошибку в коде?

Sanovskiy @Sanovskiy

Люба, я посмотрел в этот код. Знаете, я сейчас рефакторю очень старый код. Года эдак 2008-2009.
Он местами ещё на PHP4. Так вот в нем мне было проще разбираться чем в вашем скрипте. понимаю, что вы ещё учитесь, но учитесь вы по совершенно неправильным материалам.
Основные недостатки вашего кода

1. винегрет из PHP/HTML
Есть множество способов отделить представление от бизнес-логики. Т.е. сначала вы прогоняете всю логику с выборками из БД, собираете, например, массив с данными, необходимыми для сборки страницы, и передаёте его в сборщик страницы. Самый простой сборщик просто напросто подключает шаблон phtml

public function render(string $template = null)
    {
        if (!$this->beforeRender()){
            return false;
        }

        if (is_null($template)) {
            $template = $this->layout;
        }

        if (!$this->templateExist($template)) {
            return 'Can\'t render not existing view ' . $template;
        }

        ob_start();
        include $this->getTemplateFilename($template);
        $html = $this->afterRender(ob_get_contents());
        ob_end_clean();

        if ($template != $this->layout && !is_null($this->layout) && $this->templateExist($this->layout)){
            $this->data['LayoutContent'] = $html;
            $html = $this->render($this->layout);
        }
        header('Content-Type: text/html; charset=UTF-8');
        return $html;
    }

Обратите внимание, что это не просто функция, а метод класса, если соберетесь использовать его логику.

2. Использование php_mysql/php_mysqli вместо PDO
На сегодняшний день расширение php_mysql/php_mysqli морально устарело и лично я рекомендую переходить на объектную модель взаимодействия с БД.

$m=mysql_query("SELECT * FROM MyTable WHERE foo='".$_REQUEST['foo']."' AND bar='".$_REQUEST['bar']."'");
$mm=mysql_fetch_array($m);

Код гораздо более уязвим, нежели вот такой

$stmt = $dbcon->prepare('SELECT * FROM MyTable WHERE foo=:foo AND bar=:bar');
$stmt->execute([
	'foo' => $_REQUEST['foo'],
	'bar' => $_REQUEST['bar']
]);
if ($stmt->rowCount()>0){
	$allRecords = $stmt->fetchAll(\PDO::FETCH_ASSOC);
}

3. echo
На самом деле вам не нужен echo. PHP сам по себе является шаблонизатором. Для этого у него есть королкий тег <?= ?>
А также весьма удобная запись для циклов и условий
Сравните

echo '<table>';
if (is_array($myvar) && count($myvar) > 0) {
    foreach ($myvar as $values) {
        if (is_array($values) and count($values) > 0) {
            echo '<tr>';
            foreach ($values as $value){
                echo '<td>'.$value.'</td>';
            }
            echo '</tr>';
        }
    }
}
echo '</table>';

и

<table>
<? if (is_array($myvar) && count($myvar) > 0): ?>
    <? foreach ($myvar as $values): ?>
        <? if (is_array($values) and count($values) > 0) : ?>
            <tr>
                <? foreach ($values as $value): ?>
                    <td><?= $value ?></td>
                <? endforeach; ?>
            </tr>
        <? endif; ?>
    <? endforeach; ?>
<? endif; ?>
</table>

Попробуйте реструктуризировать ваш код. Для этого не надо зразу гнаться за шаблонизаторами и MVC.
Просто вынесите всю сборку данных в начало вашего скрипта. А HTML с подстановкой данных изолируйте в конце.
И многие ошибки сразу станут заметнее.

Войдите на сайт