Что переделать в коде?

Question

[starter2]

Есть php который принимает post запрос и выводит на экран + записывает в sql
запрос шлет андроид приложение вместе с "некой" инфой из телефона, которая не статична (меняется) и новые данные пришедшие пост запросом должны снова занестись в таблицу + на экран
код php файла:

<?php 
include 'DataBaseController.php';
if (isset($_POST['msgs'])) {
	$msgs = $_POST['msgs'];
	if (strpos($msgs, "|") > -1) {
		$msgs_array = explode("|", $msgs);
		for ($i = 0; $i < count($msgs_array); $i++) {
			if ($msgs_array[$i] != '') {
				query("INSERT INTO main (`id`, `msg`) VALUES (NULL, '$msgs_array[$i]')");
			}
		}
	} else {
		if ($msgs != '') {
			query("INSERT INTO main (`id`, `msg`) VALUES (NULL, '$msgs')");
		}
	}
	echo "123".$msgs;
} else {
		$query = query("SELECT * FROM main ORDER BY ID DESC");
		while ($row = mysqli_fetch_assoc($query)) 
echo $row['id']." | ".$row['msg']."<br>";
}		
?>

Проблема вот в чем: первый запрос обрабатывает, все записывает в табл и на экран выводит, а НОВУЮ инфу не принимает
в чем ошибка? нужно зациклировать или что сделать?

Answer 1

[GTRxShock]

инъекции? какие инъекции? :)

Comments

[starter2]

какие иньекции? почти ничего не смыслю в этом

[Никита]

starter2: sql иньекции )

[starter2]

Никита: эх ребят можно прямее, написал же что ничего не понимаю в этом, можно ли что то поменять в коде чтобы все работало??

[vyrkmod]

Вот тут про инъекции. А новая инфа только после нового запроса, это на клиенте решать надо.

[starter2]

vyrkmod: клиент отлично работал еще с двумя серверами и сбоев не было, только с этим и код этот новый

[starter2]

vyrkmod: а и еще таблица отображается по этому коду сверху вниз - визу номер 1id, вверху 267 :) на предыдущих было наоборот

[GTRxShock]

starter2 а может ну его, это программирование? :)

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

[vyrkmod]

starter2: то есть "ORDER BY ID DESC" вам ничего не говорит? Ну тогда либо как один лысый товарищ завещал "учиться, учиться и учиться", либо

а может ну его, это программирование? :)

Answer 2

[Владимир]

У вас в поле id (которое наверняка уникальное), всё время пытается записаться NULL:

query("INSERT INTO main (`id`, `msg`) VALUES (NULL, '$msgs')");
....
query("INSERT INTO main (`id`, `msg`) VALUES (NULL, '$msgs')");

здеь вся проблема. Сделайте это поле автоинкрементным и ничего туда не пишите в запросе, оно само запишется.