Как получить JSON-массив с кодом статуса после запроса к БД?

Question

[Pestrich]

Есть кнопка при нажатии на которую отправляется запрос к БД (записываются данные в таблицу). Нужно чтобы в случае успешного завершения запроса возвращался JSON-массив с кодом статуса «1» и текстом сообщения «Товар добавлен». В случае ошибки - возвращается код статуса «0» и текстом сообщения ошибки. Текст сообщения из JSON нужно показать рядом с кнопкой в виде подсказки.
JS код:

$(document).ready(function() {
	$('#buy').on('click', myAJAX);
});

function myAJAX() {
	$.ajax({
		url : "add_to_cart.php",
		type : "GET",
		data : { "count" : $('#col').val() },
		success : function(data) {
			console.log(data);
		}
	});
}

Add_to_cart.php:

<?php
	/* Подключение к БД */
	require_once("db.php"); 

	if (mysqli_connect_errno()) {
	echo "Подключение невозможно: ".mysqli_connect_error();
	} else {
		echo "БД подключена <br />";
	}

	$count = $_GET['count'];
	$user_ip = $_SERVER['REMOTE_ADDR'];
	$date = date('Y-m-d H:i:s');

	/* Заполнение БД */
	$sql = "INSERT INTO `orders` (`order_id`, `order_date`, `ip`, `product_count`) VALUES (NULL, '$date', '$user_ip', '$count')";
	if (mysqli_query($conn, $sql)) {
	    echo "New record created successfully";
	    $arr = array('status code' => 1, 'message' => 'Item added to cart');
		echo json_encode($arr);
	} else {
	    echo "Error: " . $sql . "<br>" . mysqli_error($conn);
	    $arr = array('status code' => 0, 'message' => 'Error');
		echo json_encode($arr);
	}

В консоли браузера приходят нужные массивы, но я почему-то думаю что это неверно, а также не знаю как вывести подсказку. Как это все правильно реализовать?

Answer 1

[DanKud]

Добавьте динамический блок возле кнопки для отображения статуса. Например добавьте возле кнопки <span class="status"></span> и допишите в AJAX-запрос:

.....
function myAJAX() {
    $.ajax({
        url : "add_to_cart.php",
        type : "GET",
        data : { "count" : $('#col').val() },
        dataType: 'json', /* сразу же даем понять в запросе, что принимаем JSON, чтобы не декодировать отдельно */
        success : function(data) {
            console.log(data);
            $('.status').html(data.message); /* добавляем текст статуса в элемент .status */
        }
    });
}

Ну и на счет PHP-кода он у вас конечно во многих местах крайне невалидный. Перепишите для начала хотя бы так:

require_once("db.php"); 

/* во-первых для чего вы везде делаете вывод данных через echo если у вас это будет
просто скрипт, который должен отдавать статус добавления товара в JSON-формате */

header('Content-Type: application/json'); /* даем понять в заголовках, что тип данных у нас JSON */

if (mysqli_connect_errno()) {
    $arr = array('message' => 'Error');
    echo json_encode($arr);
    exit(); /* отдаем ошибку подключения к базе данных и завершаем выполнение скрипта */   
}

/* принимать для записи в базу данных значения из $_GET без дополнительной фильтрации небезопасно
проще говоря так делать нельзя, хоть это конечно и будет работать */
$count = (is_numeric($_GET['count'])) ? intval($_GET['count']) : false; /* проверяем если нам отправили не число, а строку, то возвращаем false, если число то обязательно округляем до целого */
if (!$count) { /* соответственно если нам отправили некорректное значением то отдаем ошибку и завершаем выполнение скрипта */  
    $arr('message' => 'Error');
    echo json_encode($arr);
    exit();
}
$user_ip = $_SERVER['REMOTE_ADDR'];
$date = date('Y-m-d H:i:s');

$sql = "INSERT INTO `orders` (`order_id`, `order_date`, `ip`, `product_count`) VALUES (NULL, '$date', '$user_ip', '$count')";
/* не используйте в массиве значения ключей с пробелами: status code => status_code */
if (mysqli_query($conn, $sql)) {
    $arr = array('status_code' => 1, 'message' => 'Item added to cart');
} else {
    $arr = array('status_code' => 0, 'message' => 'Error');
}
echo json_encode($arr);

Comments

[Pestrich]

Благодарю за помощь и за указание на мои ошибки.
Я начинающий, поэтому echo запросы делал для себя, чтобы быть уверенным, что все точно работает или сразу увидеть в чем ошибка.
А $_GET использую, потому что по условию нужно выводить количество в адресной строке.

[DanKud]

Для тестирования кусков кода конечно вопросов нет, если вы будете использовать вывод ошибок. Но если вы уже в конечном результате хотите вывести JSON, то больше никакого вывода, кроме него, быть не должно. На счет $_GET я не имел ввиду, что надо использовать, что-то другое, просто когда вы принимаете данные в $_GET или $_POST и вам нужно эти данные либо добавить в базу, либо вывести на экран, нужно обязательно их фильтровать, чтобы не было уязвимостей. Этот момент нужно запоминать и не забывать про него с самых первых шагов в веб-программировании :)
Да и даже в вашем случае нужно отфильтровать значение просто для того, что в базу вместо количества товара не записалась просто какая-нибудь левая строка и скрипт в дальнейшем не выдавал ошибку при обработке этой записи.
А если хотите почитать для чего делать фильтрацию от уязвимостей, то можете погуглите SQL Injection и XSS.

[Pestrich]

Спасибо за информацию, я почитал про фильтрацию данных и понял о чем Вы говорили. В своей работе я сделал проверку чтобы количество товаров не было отрицательным, но совсем не подумал о том, что можно вводить различные буквы, не говоря уже о злоумышленниках и все это может привести к последствиям в реальных проектах. Мне нужно больше учиться =)

Answer 2

[morricone85]

перед echo json_encode($arr);
выводить ничего нельзя, так что, echo "New record created successfully" и т.п. нужно убрать;

в $.ajax добавить - dataType: 'json'

Comments

[Pestrich]

Именно команды "echo" нельзя выводить?
Тогда оставляю фрагмент ниже?

$arr = array('status code' => 1, 'message' => 'Item added to cart');
echo json_encode($arr);

Все верно получилось?

Не понимаю как из этого можно сообщение на экран вывести в виде подсказки.

[morricone85]

все echo нужно убрать

[Pestrich]

Убрал все остальные echo, получилось:

Правильно?
Не могли бы вы, пожалуйста, объяснить почему нельзя использовать другие echo перед echo json_encode(), и как теперь вывести это сообщение на экран?