2

Критика в пользу подключения к DB :

 $db_host = "localhost";
 $db_user = "root";
 $db_pass = "";
 $db_name = "new_db";
 define ("IMG" , "img");

 $link = mysqli_connect($db_host , $db_user , $db_pass , $db_name) OR DIE ("Не могу подключится");
 if (mysqli_connect_errno()) {
 printf("Ошибка соединения: %s\n", mysqli_connect_error());
 exit;
 }
 $query = mysqli_select_db($link , $db_name) OR DIE ("Не моу выбрать базу, либо она отсутствует");

$result = mysqli_query($link ,  "SELECT `title` , `text` , `img` FROM `news` ");

while($row = $result->fetch_assoc())
{?>
    <div class="post">
        <h2><?= $row['title'];?></h2>
             <img src="IMG/<?= $row['img'];?>" title="" />
            <p><?= $row['text'];?></p>
    </div>
<?}?>

что в коде с ошибками , как бы вы улучшили или усовершенствовали ? код выводит из DB без ошибок , всё работает но:

8
  • И в чем вопрос? Что идет после "но:"? 9 мая 2016 в 4:33
  • допустим , как отделить вью от контроллера ? по отдельности не работает , хоть include хоть include_once
    – user33274
    9 мая 2016 в 4:35
  • Чего?! А, понял. Не надо употреблять слова, значения которых ты не понимаешь - это мешает понять, что тебе нужно. Если ты не умеешь пользоваться include, то так и надо писать - мол, проблема с подключением файла в РНР скрипт. 9 мая 2016 в 4:38
  • да вроде я правильно сказал - я могу подробнее объяснить что надо , дело в том что если я делаю файл name.php и в него записываю все $var и делаю include в нужном месте , просто перестаёт работать и всё , а так же само подключение так же в другом файле и так же include и тоже перестаёт рабатать
    – user33274
    9 мая 2016 в 4:42
  • Переписывай свой вопрос, показывай, как делаешь include. 9 мая 2016 в 4:44

1 ответ 1

3

Поправить в этом коде следует две вещи:

  1. Сделать нормальную обработку ошибок.
  2. Убрать дублирующийся код.

Таким образом, код подключения будет такой:

<?php
$db_host = "localhost";
$db_user = "root";
$db_pass = "";
$db_name = "new_db";

mysqli_report(MYSQLI_REPORT_ERROR | MYSQLI_REPORT_STRICT);
$link = mysqli_connect($db_host , $db_user , $db_pass , $db_name);

этот код можно сохранить в файле, и инклюдить его везде, где нужно соединение с БД

2
  • А как убрать сам while с index.php и оставить только {?> <?}?> вот это ?
    – user33274
    9 мая 2016 в 4:46
  • Повторяю, если у тебя проблема с include, то лучше задать новый, отдельный вопрос, который озаглавить "Как правильно использовать include", и описать в нем свои проблемы. 9 мая 2016 в 4:48

Ваш ответ

Нажимая на кнопку «Отправить ответ», вы соглашаетесь с нашими пользовательским соглашением, политикой конфиденциальности и политикой о куки