Как ограничить кол-во записей в таблицу и обновление записей в бд из xml на странице через Ajax?

Question

[Alfieros]

Доброго времени суток!

Задача в состоит в следующем:
1) Нужно ограничить парсинг новостей в N-количестве(последних записей) в базу.
2) Пока не знаю как сделать "switchCase" для замены ссылки xml, при клике на странице RU/ENG к примеру. Чтобы сменить язык ленты.
3) Кнопка обновления статей из БД через Ajax, чтобы не обновлять страницу.

Вот что накалякал сам, в силу своих знаний. Буду рад любой помощи, даже оптимизации кода.

PHP

<?
//.... Подключение к бд 

$xmlURL = "https://bash.im/rss/"; //левая ссылка для примера

$sxml = simplexml_load_file($xmlURL);

foreach($sxml->channel->item as $item) {
  $id = stripcslashes($item);
  $title = stripslashes($item->title);
  $link = stripslashes($item->link);
  $image = stripslashes($item->image);
  $description = stripslashes($item->description);
  $date = stripcslashes($item->pubDate);

  $query = mysqli_query($db, "SELECT COUNT(*) FROM news WHERE id='$title'");
  $item = mysqli_fetch_row($query);
  $total = $item[0];

  if ("$total" == 0) {
    $sql = mysqli_query($db, "INSERT INTO news (id, title, link, image, description, created_at) VALUES('$id', '$title', '$link', '$image', '$description', '$date')");
  } else if ("$total" > 0) {
    $sql2 = mysqli_query($db, "UPDATE news (id, title, link, image, description, created_at) VALUES('$id', '$title', '$link', '$image', '$description', '$date')");
  }
}

?>

HTML

<div class="rss-feed">
<?php
  $sql = mysqli_query($db, 'SELECT `title`, `link`, `description`, `created_at` FROM `news` ORDER BY `created_at` desc LIMIT 5');
  while ($result = mysqli_fetch_array($sql)) {
    echo '<div class="item">';
    echo '<h2><a href='.$result['link'].'>'.$result['title'].'</a></h2>';
    echo '<div class="description">'.$result['description'].'</div>';
    echo '<p class="datetime">'.$result['created_at'].'</p>';
    echo '</div>';
  }
?>
</div>

Answer 1

[AUser0]

Ну для начала: $id = stripslashes($item) - это вообще ни к селу, ни к городу.
Тогда уж $id = mysqli_real_escape_string($db, stripslashes($item->title)) хотя бы.
А лучше $id = mysqli_real_escape_string($db, stripslashes($item->guid)). И изменить длину столбца `id` в базе.
И все входные данные пропустить через mysqi_real_escape_string($db, ...);

Далее, меняем на $query = mysqli_query($db, "SELECT COUNT(*) FROM news WHERE id='{$id}'");
И ещё, зачем делать лишний UPDATE записи, которая уже находится в базе, и по логике - полностью идентична новым данным?

А при обращении из AJAX (которого тут не приведено) отдавать только последние XX записей, вот и всё, нет проблем.

Comments

[Alfieros]

В том и проблема, что в реальной ссылке RSS нет тега , там нет ни одного id или атрибута по котором можно делать сверку на уникальность, только title или link. Все статьи отличаются только заголовками и урлами.

[AUser0]

Alfieros, тогда $id = mysqli_real_escape_string($db, stripslashes($item->title)), как минимум.

[Alfieros]

AUser0

Далее, меняем на $query = mysqli_query($db, "SELECT COUNT(*) FROM news WHERE id='{$id}'");

После данной процедуры всё ломается, и в базе появляются дубли всех записей при каждом обновлении страницы. ID в моё случае это не уникальный идентификатор статьи, а просто порядковый номер в базе.

[Alfieros]

AUser0, спасибо что указали на ошибку, про экранирование не подумал. Лишнее удалил.

[Alfieros]

AUser0, каким образом можно ограничить добавление в БД, только 10 последних статей. И заменять не всю таблицу, а только 1-2 строки в зависимости от кол-ва новых записей?

[AUser0]

Alfieros, ну так разберитесь, у вас COUNT(*) проверяет наличие записи таким `id`, или не проверяет?
Если не проверяет, и это порядковый номер - тогда меняйте на "SELECT COUNT(*) FROM news WHERE title='{$title}'". Из INSERT-а убирайте `id`, его должна генерировать сама база. А что бы хранилось только 10 записей - удаляйте одну запись с наименьшим `id`: "DELETE FROM `news` ORDER BY `id` ASC LIMIT 1".

[AUser0]

Alfieros, $total - это у вас количество SQL-записей с `title`='$title', так? Тогда меняем:

$query = mysqli_query($db, "SELECT COUNT(*) FROM news WHERE title='{$title}'");
  $item = mysqli_fetch_row($query);
  if ($item[0] == 0) {
    mysqli_query($db, "INSERT INTO news (title, link, image, description, created_at) VALUES('{$title}', '{$link}', '{$image}', '{$description}', '{$date}')");
  }

  $query = mysqli_query($db, "SELECT COUNT(*) FROM news");
  $item = mysqli_fetch_row($query);
  if ($item[0] > 10) {
    mysqli_query($db, "DELETE FROM news ORDER BY `id` ASC LIMIT ".($item[0] - 10));
  }

[Alfieros]

AUser0, Да. Спасибо за помощь, именно то что нужно. Кст, никак нельзя сделать, чтобы при обновлении данных id сохраняли первичную нумерацию от 0 до 10? Сейчас при каждом обновлении id растёт. Или ничего страшного? Просто как в дальнейшем работать с этими данными, если нужно будет к примеру на странице вывести 1 статью с ID 10 к примеру?

[AUser0]

Alfieros, если сделать `id` UNSIGNED BIGINT (далеко за 18 с 18-ю нолями), то он оооочень не скоро кончится.
Если именно `id`=10, то:
SELECT * FROM `news` WHERE `id`=10
А если только 10-ую запись (одну) из всех имеющихся:
SELECT * FROM `news` ORDER BY `id` ASC LIMIT 10, 1

[Alfieros]

AUser0, не, как сделать выборку по ID я знаю. Вопрос в том, чтобы в самой базе нумерация не росла, а оставалась всегда от 0 до 10?

[Alfieros]

если сделать `id` UNSIGNED BIGINT (далеко за 18 с 18-ю нолями), то он оооочень не скоро кончится.

В первую очередь об этом подумал, так же и сделал. Но это как-то не по феншую что ли, если в базе всего 10 записей хранится.

[AUser0]

Alfieros, кажется должно сработать:

mysqli_query($db, "SET @pos := 0");
mysqli_query($db, "UPDATE `news` SET `id`=(SELECT @pos:=@pos+1) ORDER BY `id` ASC");

Делать после всех добавлений/удалений в таблице.

[Alfieros]

AUser0, спасибо сработало.

[AUser0]

Alfieros, жуйкверный AJAX с data: {update: 'yes'} ты и сам напишешь, умеешь же. ;-)
Разумеется success: function(response) {$('div.rss-feed').innerHTML = response;}

А серверная часть - проще некуда. В том же скрипте, после инициализации подключения к MySQL:

if (isset($_REQUEST['update']))
{
  $sql = mysqli_query($db, 'SELECT `title`, `link`, `description`, `created_at` FROM `news` ORDER BY `created_at` desc LIMIT 10');
  while ($result = mysqli_fetch_array($sql)) {
    echo '<div class="item">';
    echo '<h2><a href='.$result['link'].'>'.$result['title'].'</a></h2>';
    echo '<div class="description">'.$result['description'].'</div>';
    echo '<p class="datetime">'.$result['created_at'].'</p>';
    echo '</div>';
  }
  exit();
}